src/java.base/macosx/classes/sun/nio/ch/KQueueSelectorImpl.java
author alanb
Thu, 05 Apr 2018 15:01:57 +0100
changeset 49526 cad4c844902a
parent 49493 814bd31f8da0
child 49527 5aa40f834b50
permissions -rw-r--r--
8200583: (se) Selector clean-up, part 4 Reviewed-by: bpb, 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;
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
    30
import java.nio.channels.Selector;
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
    31
import java.nio.channels.spi.SelectorProvider;
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    32
import java.util.ArrayDeque;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    33
import java.util.Deque;
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
    34
import java.util.HashMap;
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    35
import java.util.Map;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    36
import java.util.concurrent.TimeUnit;
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    37
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    38
import static sun.nio.ch.KQueue.EVFILT_READ;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    39
import static sun.nio.ch.KQueue.EVFILT_WRITE;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    40
import static sun.nio.ch.KQueue.EV_ADD;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    41
import static sun.nio.ch.KQueue.EV_DELETE;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    42
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    43
/**
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    44
 * KQueue based Selector implementation for macOS
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    45
 */
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    46
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    47
class KQueueSelectorImpl extends SelectorImpl {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    48
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    49
    // maximum number of events to poll in one call to kqueue
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    50
    private static final int MAX_KEVENTS = 256;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    51
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    52
    // kqueue file descriptor
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    53
    private final int kqfd;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    54
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    55
    // address of poll array (event list) when polling for pending events
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    56
    private final long pollArrayAddress;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    57
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    58
    // file descriptors used for interrupt
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
    59
    private final int fd0;
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
    60
    private final int fd1;
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    61
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    62
    // maps file descriptor to selection key, synchronize on selector
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    63
    private final Map<Integer, SelectionKeyImpl> fdToKey = new HashMap<>();
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    64
49526
cad4c844902a 8200583: (se) Selector clean-up, part 4
alanb
parents: 49493
diff changeset
    65
    // pending new registrations/updates, queued by setEventOps
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    66
    private final Object updateLock = new Object();
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    67
    private final Deque<SelectionKeyImpl> updateKeys = new ArrayDeque<>();
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    68
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    69
    // interrupt triggering and clearing
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
    70
    private final Object interruptLock = new Object();
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
    71
    private boolean interruptTriggered;
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    72
12438
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
    73
    // 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
    74
    // descriptor is polled by more than one filter
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    75
    private int pollCount;
12438
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
    76
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
    77
    KQueueSelectorImpl(SelectorProvider sp) throws IOException {
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    78
        super(sp);
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    79
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    80
        this.kqfd = KQueue.create();
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    81
        this.pollArrayAddress = KQueue.allocatePollArray(MAX_KEVENTS);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    82
32856
c654cdf1d685 8138819: (se) File descriptor leak when Selector.open fails
igerasim
parents: 25859
diff changeset
    83
        try {
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    84
            long fds = IOUtil.makePipe(false);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    85
            this.fd0 = (int) (fds >>> 32);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    86
            this.fd1 = (int) fds;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    87
        } catch (IOException ioe) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    88
            KQueue.freePollArray(pollArrayAddress);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    89
            FileDispatcherImpl.closeIntFD(kqfd);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    90
            throw ioe;
32856
c654cdf1d685 8138819: (se) File descriptor leak when Selector.open fails
igerasim
parents: 25859
diff changeset
    91
        }
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    92
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    93
        // register one end of the socket pair for wakeups
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    94
        KQueue.register(kqfd, fd0, EVFILT_READ, EV_ADD);
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    95
    }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
    96
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
    97
    private void ensureOpen() {
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    98
        if (!isOpen())
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
    99
            throw new ClosedSelectorException();
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   100
    }
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   101
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   102
    @Override
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   103
    protected int doSelect(long timeout) throws IOException {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   104
        assert Thread.holdsLock(this);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   105
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   106
        long to = Math.min(timeout, Integer.MAX_VALUE);  // max kqueue timeout
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   107
        boolean blocking = (to != 0);
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   108
        boolean timedPoll = (to > 0);
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   109
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   110
        int numEntries;
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   111
        processUpdateQueue();
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   112
        processDeregisterQueue();
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   113
        try {
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   114
            begin(blocking);
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   115
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   116
            do {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   117
                long startTime = timedPoll ? System.nanoTime() : 0;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   118
                numEntries = KQueue.poll(kqfd, pollArrayAddress, MAX_KEVENTS, to);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   119
                if (numEntries == IOStatus.INTERRUPTED && timedPoll) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   120
                    // timed poll interrupted so need to adjust timeout
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   121
                    long adjust = System.nanoTime() - startTime;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   122
                    to -= TimeUnit.MILLISECONDS.convert(adjust, TimeUnit.NANOSECONDS);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   123
                    if (to <= 0) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   124
                        // timeout expired so no retry
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   125
                        numEntries = 0;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   126
                    }
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   127
                }
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   128
            } while (numEntries == IOStatus.INTERRUPTED);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   129
            assert IOStatus.check(numEntries);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   130
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   131
        } finally {
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   132
            end(blocking);
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   133
        }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   134
        processDeregisterQueue();
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   135
        return updateSelectedKeys(numEntries);
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   136
    }
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
    /**
49526
cad4c844902a 8200583: (se) Selector clean-up, part 4
alanb
parents: 49493
diff changeset
   139
     * Process changes to the interest ops.
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   140
     */
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   141
    private void processUpdateQueue() {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   142
        assert Thread.holdsLock(this);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   143
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   144
        synchronized (updateLock) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   145
            SelectionKeyImpl ski;
49526
cad4c844902a 8200583: (se) Selector clean-up, part 4
alanb
parents: 49493
diff changeset
   146
            while ((ski = updateKeys.pollFirst()) != null) {
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   147
                if (ski.isValid()) {
49526
cad4c844902a 8200583: (se) Selector clean-up, part 4
alanb
parents: 49493
diff changeset
   148
                    int fd = ski.getFDVal();
cad4c844902a 8200583: (se) Selector clean-up, part 4
alanb
parents: 49493
diff changeset
   149
                    // add to fdToKey if needed
cad4c844902a 8200583: (se) Selector clean-up, part 4
alanb
parents: 49493
diff changeset
   150
                    SelectionKeyImpl previous = fdToKey.putIfAbsent(fd, ski);
cad4c844902a 8200583: (se) Selector clean-up, part 4
alanb
parents: 49493
diff changeset
   151
                    assert (previous == null) || (previous == ski);
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   152
49526
cad4c844902a 8200583: (se) Selector clean-up, part 4
alanb
parents: 49493
diff changeset
   153
                    int newEvents = ski.translateInterestOps();
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   154
                    int registeredEvents = ski.registeredEvents();
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   155
                    if (newEvents != registeredEvents) {
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   156
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   157
                        // add or delete interest in read events
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   158
                        if ((registeredEvents & Net.POLLIN) != 0) {
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   159
                            if ((newEvents & Net.POLLIN) == 0) {
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   160
                                KQueue.register(kqfd, fd, EVFILT_READ, EV_DELETE);
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   161
                            }
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   162
                        } else if ((newEvents & Net.POLLIN) != 0) {
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   163
                            KQueue.register(kqfd, fd, EVFILT_READ, EV_ADD);
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   164
                        }
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   165
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   166
                        // add or delete interest in write events
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   167
                        if ((registeredEvents & Net.POLLOUT) != 0) {
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   168
                            if ((newEvents & Net.POLLOUT) == 0) {
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   169
                                KQueue.register(kqfd, fd, EVFILT_WRITE, EV_DELETE);
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   170
                            }
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   171
                        } else if ((newEvents & Net.POLLOUT) != 0) {
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   172
                            KQueue.register(kqfd, fd, EVFILT_WRITE, EV_ADD);
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   173
                        }
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   174
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   175
                        ski.registeredEvents(newEvents);
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   176
                    }
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   177
                }
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   178
            }
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   179
        }
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   180
    }
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   181
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   182
    /**
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   183
     * Update the keys of file descriptors that were polled and add them to
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   184
     * the selected-key set.
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   185
     * 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
   186
     */
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   187
    private int updateSelectedKeys(int numEntries) throws IOException {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   188
        assert Thread.holdsLock(this);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   189
        assert Thread.holdsLock(nioSelectedKeys());
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   190
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   191
        int numKeysUpdated = 0;
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   192
        boolean interrupted = false;
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   193
12438
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
   194
        // A file descriptor may be registered with kqueue with more than one
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   195
        // 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
   196
        // count is incremented here and compared against the SelectionKey's
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   197
        // "lastPolled" field. This ensures that the ready ops is updated rather
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   198
        // than replaced when a file descriptor is polled by both the read and
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   199
        // write filter.
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   200
        pollCount++;
12438
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
   201
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   202
        for (int i = 0; i < numEntries; i++) {
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   203
            long kevent = KQueue.getEvent(pollArrayAddress, i);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   204
            int fd = KQueue.getDescriptor(kevent);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   205
            if (fd == fd0) {
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   206
                interrupted = true;
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   207
            } else {
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   208
                SelectionKeyImpl ski = fdToKey.get(fd);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   209
                if (ski != null) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   210
                    int rOps = 0;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   211
                    short filter = KQueue.getFilter(kevent);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   212
                    if (filter == EVFILT_READ) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   213
                        rOps |= Net.POLLIN;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   214
                    } else if (filter == EVFILT_WRITE) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   215
                        rOps |= Net.POLLOUT;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   216
                    }
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   217
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   218
                    if (selectedKeys.contains(ski)) {
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   219
                        // file descriptor may be polled more than once per poll
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   220
                        if (ski.lastPolled != pollCount) {
49526
cad4c844902a 8200583: (se) Selector clean-up, part 4
alanb
parents: 49493
diff changeset
   221
                            if (ski.translateAndSetReadyOps(rOps)) {
12438
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
   222
                                numKeysUpdated++;
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   223
                                ski.lastPolled = pollCount;
12438
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
   224
                            }
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
   225
                        } else {
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
   226
                            // ready ops have already been set on this update
49526
cad4c844902a 8200583: (se) Selector clean-up, part 4
alanb
parents: 49493
diff changeset
   227
                            ski.translateAndUpdateReadyOps(rOps);
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   228
                        }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   229
                    } else {
49526
cad4c844902a 8200583: (se) Selector clean-up, part 4
alanb
parents: 49493
diff changeset
   230
                        ski.translateAndSetReadyOps(rOps);
12438
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
   231
                        if ((ski.nioReadyOps() & ski.nioInterestOps()) != 0) {
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   232
                            selectedKeys.add(ski);
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   233
                            numKeysUpdated++;
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   234
                            ski.lastPolled = pollCount;
12047
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
                }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   238
            }
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
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   241
        if (interrupted) {
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   242
            clearInterrupt();
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   243
        }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   244
        return numKeysUpdated;
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   245
    }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   246
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   247
    @Override
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   248
    protected void implClose() throws IOException {
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   249
        assert !isOpen();
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   250
        assert Thread.holdsLock(this);
12438
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
   251
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   252
        // prevent further wakeup
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   253
        synchronized (interruptLock) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   254
            interruptTriggered = true;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   255
        }
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   256
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   257
        FileDispatcherImpl.closeIntFD(kqfd);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   258
        KQueue.freePollArray(pollArrayAddress);
12438
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
   259
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   260
        FileDispatcherImpl.closeIntFD(fd0);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   261
        FileDispatcherImpl.closeIntFD(fd1);
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   262
    }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   263
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   264
    @Override
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   265
    protected void implDereg(SelectionKeyImpl ski) throws IOException {
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   266
        assert !ski.isValid();
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   267
        assert Thread.holdsLock(this);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   268
49526
cad4c844902a 8200583: (se) Selector clean-up, part 4
alanb
parents: 49493
diff changeset
   269
        int fd = ski.getFDVal();
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   270
        int registeredEvents = ski.registeredEvents();
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   271
        if (fdToKey.remove(fd) != null) {
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   272
            if (registeredEvents != 0) {
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   273
                if ((registeredEvents & Net.POLLIN) != 0)
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   274
                    KQueue.register(kqfd, fd, EVFILT_READ, EV_DELETE);
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   275
                if ((registeredEvents & Net.POLLOUT) != 0)
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   276
                    KQueue.register(kqfd, fd, EVFILT_WRITE, EV_DELETE);
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   277
                ski.registeredEvents(0);
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   278
            }
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   279
        } else {
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   280
            assert registeredEvents == 0;
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   281
        }
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   282
    }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   283
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   284
    @Override
49526
cad4c844902a 8200583: (se) Selector clean-up, part 4
alanb
parents: 49493
diff changeset
   285
    public void setEventOps(SelectionKeyImpl ski) {
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   286
        ensureOpen();
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   287
        synchronized (updateLock) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   288
            updateKeys.addLast(ski);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   289
        }
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   290
    }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   291
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   292
    @Override
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   293
    public Selector wakeup() {
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   294
        synchronized (interruptLock) {
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   295
            if (!interruptTriggered) {
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   296
                try {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   297
                    IOUtil.write1(fd1, (byte)0);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   298
                } catch (IOException ioe) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   299
                    throw new InternalError(ioe);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   300
                }
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   301
                interruptTriggered = true;
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   302
            }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   303
        }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   304
        return this;
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   305
    }
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   306
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   307
    private void clearInterrupt() throws IOException {
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   308
        synchronized (interruptLock) {
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   309
            IOUtil.drain(fd0);
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   310
            interruptTriggered = false;
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   311
        }
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   312
    }
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   313
}