src/java.base/macosx/classes/sun/nio/ch/KQueueSelectorImpl.java
author alanb
Wed, 20 Nov 2019 08:35:53 +0000
changeset 59146 455612b3161a
parent 50602 ed8de3d0cd28
permissions -rw-r--r--
8231259: (dc) DatagramChannel::disconnect re-binds socket to the wildcard address (macOS) Reviewed-by: dfuchs, chegar
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
     1
/*
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
     2
 * Copyright (c) 2011, 2018, Oracle and/or its affiliates. All rights reserved.
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
     4
 *
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
12438
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
     7
 * published by the Free Software Foundation.  Oracle designates this
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
     8
 * particular file as subject to the "Classpath" exception as provided
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
     9
 * by Oracle in the LICENSE file that accompanied this code.
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    10
 *
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    11
 * This code is distributed in the hope that it will be useful, but WITHOUT
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    12
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    13
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    14
 * version 2 for more details (a copy is included in the LICENSE file that
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    15
 * accompanied this code).
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    16
 *
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    17
 * You should have received a copy of the GNU General Public License version
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    18
 * 2 along with this work; if not, write to the Free Software Foundation,
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    19
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    20
 *
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    21
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    22
 * or visit www.oracle.com if you need additional information or have any
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    23
 * questions.
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    24
 */
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    25
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    26
package sun.nio.ch;
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    27
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    28
import java.io.IOException;
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
    29
import java.nio.channels.ClosedSelectorException;
50602
ed8de3d0cd28 8199433: (se) select(Consumer<SelectionKey> action) as alternative to selected-key set
alanb
parents: 49527
diff changeset
    30
import java.nio.channels.SelectionKey;
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
    31
import java.nio.channels.Selector;
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
    32
import java.nio.channels.spi.SelectorProvider;
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    33
import java.util.ArrayDeque;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    34
import java.util.Deque;
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
    35
import java.util.HashMap;
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    36
import java.util.Map;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    37
import java.util.concurrent.TimeUnit;
50602
ed8de3d0cd28 8199433: (se) select(Consumer<SelectionKey> action) as alternative to selected-key set
alanb
parents: 49527
diff changeset
    38
import java.util.function.Consumer;
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    39
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    40
import static sun.nio.ch.KQueue.EVFILT_READ;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    41
import static sun.nio.ch.KQueue.EVFILT_WRITE;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    42
import static sun.nio.ch.KQueue.EV_ADD;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    43
import static sun.nio.ch.KQueue.EV_DELETE;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    44
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    45
/**
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    46
 * KQueue based Selector implementation for macOS
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    47
 */
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    48
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    49
class KQueueSelectorImpl extends SelectorImpl {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    50
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    51
    // maximum number of events to poll in one call to kqueue
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    52
    private static final int MAX_KEVENTS = 256;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    53
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    54
    // kqueue file descriptor
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    55
    private final int kqfd;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    56
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    57
    // address of poll array (event list) when polling for pending events
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    58
    private final long pollArrayAddress;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    59
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    60
    // file descriptors used for interrupt
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
    61
    private final int fd0;
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
    62
    private final int fd1;
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    63
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    64
    // maps file descriptor to selection key, synchronize on selector
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    65
    private final Map<Integer, SelectionKeyImpl> fdToKey = new HashMap<>();
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    66
49526
cad4c844902a 8200583: (se) Selector clean-up, part 4
alanb
parents: 49493
diff changeset
    67
    // pending new registrations/updates, queued by setEventOps
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    68
    private final Object updateLock = new Object();
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    69
    private final Deque<SelectionKeyImpl> updateKeys = new ArrayDeque<>();
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    70
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    71
    // interrupt triggering and clearing
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
    72
    private final Object interruptLock = new Object();
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
    73
    private boolean interruptTriggered;
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    74
12438
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
    75
    // used by updateSelectedKeys to handle cases where the same file
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
    76
    // descriptor is polled by more than one filter
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    77
    private int pollCount;
12438
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
    78
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
    79
    KQueueSelectorImpl(SelectorProvider sp) throws IOException {
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    80
        super(sp);
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    81
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    82
        this.kqfd = KQueue.create();
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    83
        this.pollArrayAddress = KQueue.allocatePollArray(MAX_KEVENTS);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    84
32856
c654cdf1d685 8138819: (se) File descriptor leak when Selector.open fails
igerasim
parents: 25859
diff changeset
    85
        try {
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    86
            long fds = IOUtil.makePipe(false);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    87
            this.fd0 = (int) (fds >>> 32);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    88
            this.fd1 = (int) fds;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    89
        } catch (IOException ioe) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    90
            KQueue.freePollArray(pollArrayAddress);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    91
            FileDispatcherImpl.closeIntFD(kqfd);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    92
            throw ioe;
32856
c654cdf1d685 8138819: (se) File descriptor leak when Selector.open fails
igerasim
parents: 25859
diff changeset
    93
        }
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    94
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    95
        // register one end of the socket pair for wakeups
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    96
        KQueue.register(kqfd, fd0, EVFILT_READ, EV_ADD);
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    97
    }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    98
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
    99
    private void ensureOpen() {
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   100
        if (!isOpen())
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   101
            throw new ClosedSelectorException();
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   102
    }
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   103
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   104
    @Override
50602
ed8de3d0cd28 8199433: (se) select(Consumer<SelectionKey> action) as alternative to selected-key set
alanb
parents: 49527
diff changeset
   105
    protected int doSelect(Consumer<SelectionKey> action, long timeout)
ed8de3d0cd28 8199433: (se) select(Consumer<SelectionKey> action) as alternative to selected-key set
alanb
parents: 49527
diff changeset
   106
        throws IOException
ed8de3d0cd28 8199433: (se) select(Consumer<SelectionKey> action) as alternative to selected-key set
alanb
parents: 49527
diff changeset
   107
    {
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   108
        assert Thread.holdsLock(this);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   109
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   110
        long to = Math.min(timeout, Integer.MAX_VALUE);  // max kqueue timeout
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   111
        boolean blocking = (to != 0);
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   112
        boolean timedPoll = (to > 0);
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   113
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   114
        int numEntries;
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   115
        processUpdateQueue();
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   116
        processDeregisterQueue();
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   117
        try {
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   118
            begin(blocking);
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   119
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   120
            do {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   121
                long startTime = timedPoll ? System.nanoTime() : 0;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   122
                numEntries = KQueue.poll(kqfd, pollArrayAddress, MAX_KEVENTS, to);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   123
                if (numEntries == IOStatus.INTERRUPTED && timedPoll) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   124
                    // timed poll interrupted so need to adjust timeout
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   125
                    long adjust = System.nanoTime() - startTime;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   126
                    to -= TimeUnit.MILLISECONDS.convert(adjust, TimeUnit.NANOSECONDS);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   127
                    if (to <= 0) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   128
                        // timeout expired so no retry
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   129
                        numEntries = 0;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   130
                    }
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   131
                }
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   132
            } while (numEntries == IOStatus.INTERRUPTED);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   133
            assert IOStatus.check(numEntries);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   134
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   135
        } finally {
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   136
            end(blocking);
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   137
        }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   138
        processDeregisterQueue();
50602
ed8de3d0cd28 8199433: (se) select(Consumer<SelectionKey> action) as alternative to selected-key set
alanb
parents: 49527
diff changeset
   139
        return processEvents(numEntries, action);
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   140
    }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   141
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   142
    /**
49526
cad4c844902a 8200583: (se) Selector clean-up, part 4
alanb
parents: 49493
diff changeset
   143
     * Process changes to the interest ops.
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   144
     */
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   145
    private void processUpdateQueue() {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   146
        assert Thread.holdsLock(this);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   147
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   148
        synchronized (updateLock) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   149
            SelectionKeyImpl ski;
49526
cad4c844902a 8200583: (se) Selector clean-up, part 4
alanb
parents: 49493
diff changeset
   150
            while ((ski = updateKeys.pollFirst()) != null) {
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   151
                if (ski.isValid()) {
49526
cad4c844902a 8200583: (se) Selector clean-up, part 4
alanb
parents: 49493
diff changeset
   152
                    int fd = ski.getFDVal();
cad4c844902a 8200583: (se) Selector clean-up, part 4
alanb
parents: 49493
diff changeset
   153
                    // add to fdToKey if needed
cad4c844902a 8200583: (se) Selector clean-up, part 4
alanb
parents: 49493
diff changeset
   154
                    SelectionKeyImpl previous = fdToKey.putIfAbsent(fd, ski);
cad4c844902a 8200583: (se) Selector clean-up, part 4
alanb
parents: 49493
diff changeset
   155
                    assert (previous == null) || (previous == ski);
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   156
49526
cad4c844902a 8200583: (se) Selector clean-up, part 4
alanb
parents: 49493
diff changeset
   157
                    int newEvents = ski.translateInterestOps();
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   158
                    int registeredEvents = ski.registeredEvents();
59146
455612b3161a 8231259: (dc) DatagramChannel::disconnect re-binds socket to the wildcard address (macOS)
alanb
parents: 50602
diff changeset
   159
455612b3161a 8231259: (dc) DatagramChannel::disconnect re-binds socket to the wildcard address (macOS)
alanb
parents: 50602
diff changeset
   160
                    // DatagramChannelImpl::disconnect has reset socket
455612b3161a 8231259: (dc) DatagramChannel::disconnect re-binds socket to the wildcard address (macOS)
alanb
parents: 50602
diff changeset
   161
                    if (ski.getAndClearReset() && registeredEvents != 0) {
455612b3161a 8231259: (dc) DatagramChannel::disconnect re-binds socket to the wildcard address (macOS)
alanb
parents: 50602
diff changeset
   162
                        KQueue.register(kqfd, fd, EVFILT_READ, EV_DELETE);
455612b3161a 8231259: (dc) DatagramChannel::disconnect re-binds socket to the wildcard address (macOS)
alanb
parents: 50602
diff changeset
   163
                        registeredEvents = 0;
455612b3161a 8231259: (dc) DatagramChannel::disconnect re-binds socket to the wildcard address (macOS)
alanb
parents: 50602
diff changeset
   164
                    }
455612b3161a 8231259: (dc) DatagramChannel::disconnect re-binds socket to the wildcard address (macOS)
alanb
parents: 50602
diff changeset
   165
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   166
                    if (newEvents != registeredEvents) {
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   167
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   168
                        // add or delete interest in read events
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   169
                        if ((registeredEvents & Net.POLLIN) != 0) {
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   170
                            if ((newEvents & Net.POLLIN) == 0) {
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   171
                                KQueue.register(kqfd, fd, EVFILT_READ, EV_DELETE);
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   172
                            }
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   173
                        } else if ((newEvents & Net.POLLIN) != 0) {
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   174
                            KQueue.register(kqfd, fd, EVFILT_READ, EV_ADD);
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   175
                        }
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   176
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   177
                        // add or delete interest in write events
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   178
                        if ((registeredEvents & Net.POLLOUT) != 0) {
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   179
                            if ((newEvents & Net.POLLOUT) == 0) {
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   180
                                KQueue.register(kqfd, fd, EVFILT_WRITE, EV_DELETE);
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   181
                            }
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   182
                        } else if ((newEvents & Net.POLLOUT) != 0) {
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   183
                            KQueue.register(kqfd, fd, EVFILT_WRITE, EV_ADD);
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   184
                        }
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   185
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   186
                        ski.registeredEvents(newEvents);
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   187
                    }
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   188
                }
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   189
            }
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   190
        }
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   191
    }
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   192
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   193
    /**
50602
ed8de3d0cd28 8199433: (se) select(Consumer<SelectionKey> action) as alternative to selected-key set
alanb
parents: 49527
diff changeset
   194
     * Process the polled events.
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   195
     * If the interrupt fd has been selected, drain it and clear the interrupt.
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   196
     */
50602
ed8de3d0cd28 8199433: (se) select(Consumer<SelectionKey> action) as alternative to selected-key set
alanb
parents: 49527
diff changeset
   197
    private int processEvents(int numEntries, Consumer<SelectionKey> action)
ed8de3d0cd28 8199433: (se) select(Consumer<SelectionKey> action) as alternative to selected-key set
alanb
parents: 49527
diff changeset
   198
        throws IOException
ed8de3d0cd28 8199433: (se) select(Consumer<SelectionKey> action) as alternative to selected-key set
alanb
parents: 49527
diff changeset
   199
    {
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   200
        assert Thread.holdsLock(this);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   201
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   202
        int numKeysUpdated = 0;
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   203
        boolean interrupted = false;
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   204
12438
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
   205
        // A file descriptor may be registered with kqueue with more than one
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   206
        // filter and so there may be more than one event for a fd. The poll
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   207
        // count is incremented here and compared against the SelectionKey's
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   208
        // "lastPolled" field. This ensures that the ready ops is updated rather
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   209
        // than replaced when a file descriptor is polled by both the read and
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   210
        // write filter.
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   211
        pollCount++;
12438
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
   212
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   213
        for (int i = 0; i < numEntries; i++) {
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   214
            long kevent = KQueue.getEvent(pollArrayAddress, i);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   215
            int fd = KQueue.getDescriptor(kevent);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   216
            if (fd == fd0) {
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   217
                interrupted = true;
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   218
            } else {
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   219
                SelectionKeyImpl ski = fdToKey.get(fd);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   220
                if (ski != null) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   221
                    int rOps = 0;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   222
                    short filter = KQueue.getFilter(kevent);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   223
                    if (filter == EVFILT_READ) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   224
                        rOps |= Net.POLLIN;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   225
                    } else if (filter == EVFILT_WRITE) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   226
                        rOps |= Net.POLLOUT;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   227
                    }
50602
ed8de3d0cd28 8199433: (se) select(Consumer<SelectionKey> action) as alternative to selected-key set
alanb
parents: 49527
diff changeset
   228
                    int updated = processReadyEvents(rOps, ski, action);
ed8de3d0cd28 8199433: (se) select(Consumer<SelectionKey> action) as alternative to selected-key set
alanb
parents: 49527
diff changeset
   229
                    if (updated > 0 && ski.lastPolled != pollCount) {
ed8de3d0cd28 8199433: (se) select(Consumer<SelectionKey> action) as alternative to selected-key set
alanb
parents: 49527
diff changeset
   230
                        numKeysUpdated++;
ed8de3d0cd28 8199433: (se) select(Consumer<SelectionKey> action) as alternative to selected-key set
alanb
parents: 49527
diff changeset
   231
                        ski.lastPolled = pollCount;
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   232
                    }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   233
                }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   234
            }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   235
        }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   236
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   237
        if (interrupted) {
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   238
            clearInterrupt();
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   239
        }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   240
        return numKeysUpdated;
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   241
    }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   242
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   243
    @Override
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   244
    protected void implClose() throws IOException {
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   245
        assert !isOpen();
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   246
        assert Thread.holdsLock(this);
12438
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
   247
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   248
        // prevent further wakeup
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   249
        synchronized (interruptLock) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   250
            interruptTriggered = true;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   251
        }
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   252
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   253
        FileDispatcherImpl.closeIntFD(kqfd);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   254
        KQueue.freePollArray(pollArrayAddress);
12438
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
   255
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   256
        FileDispatcherImpl.closeIntFD(fd0);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   257
        FileDispatcherImpl.closeIntFD(fd1);
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   258
    }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   259
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   260
    @Override
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   261
    protected void implDereg(SelectionKeyImpl ski) throws IOException {
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   262
        assert !ski.isValid();
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   263
        assert Thread.holdsLock(this);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   264
49526
cad4c844902a 8200583: (se) Selector clean-up, part 4
alanb
parents: 49493
diff changeset
   265
        int fd = ski.getFDVal();
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   266
        int registeredEvents = ski.registeredEvents();
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   267
        if (fdToKey.remove(fd) != null) {
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   268
            if (registeredEvents != 0) {
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   269
                if ((registeredEvents & Net.POLLIN) != 0)
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   270
                    KQueue.register(kqfd, fd, EVFILT_READ, EV_DELETE);
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   271
                if ((registeredEvents & Net.POLLOUT) != 0)
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   272
                    KQueue.register(kqfd, fd, EVFILT_WRITE, EV_DELETE);
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   273
                ski.registeredEvents(0);
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   274
            }
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   275
        } else {
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   276
            assert registeredEvents == 0;
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   277
        }
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   278
    }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   279
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   280
    @Override
49526
cad4c844902a 8200583: (se) Selector clean-up, part 4
alanb
parents: 49493
diff changeset
   281
    public void setEventOps(SelectionKeyImpl ski) {
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   282
        ensureOpen();
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   283
        synchronized (updateLock) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   284
            updateKeys.addLast(ski);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   285
        }
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   286
    }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   287
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   288
    @Override
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   289
    public Selector wakeup() {
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   290
        synchronized (interruptLock) {
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   291
            if (!interruptTriggered) {
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   292
                try {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   293
                    IOUtil.write1(fd1, (byte)0);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   294
                } catch (IOException ioe) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   295
                    throw new InternalError(ioe);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   296
                }
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   297
                interruptTriggered = true;
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   298
            }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   299
        }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   300
        return this;
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   301
    }
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   302
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   303
    private void clearInterrupt() throws IOException {
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   304
        synchronized (interruptLock) {
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   305
            IOUtil.drain(fd0);
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   306
            interruptTriggered = false;
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   307
        }
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   308
    }
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   309
}