src/java.base/macosx/classes/sun/nio/ch/KQueueSelectorImpl.java
author alanb
Fri, 30 Mar 2018 08:28:09 +0100
changeset 49493 814bd31f8da0
parent 49290 07779973cbe2
child 49526 cad4c844902a
permissions -rw-r--r--
8200257: (se) More Selector cleanup Reviewed-by: bpb
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
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    65
    // pending new registrations/updates, queued by implRegister and putEventOps
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> newKeys = new ArrayDeque<>();
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
    68
    private final Deque<SelectionKeyImpl> updateKeys = new ArrayDeque<>();
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
    69
    private final Deque<Integer> updateEvents = 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
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   105
    protected int doSelect(long timeout) throws IOException {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   106
        assert Thread.holdsLock(this);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   107
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   108
        long to = Math.min(timeout, Integer.MAX_VALUE);  // max kqueue timeout
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   109
        boolean blocking = (to != 0);
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   110
        boolean timedPoll = (to > 0);
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   111
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   112
        int numEntries;
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   113
        processUpdateQueue();
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   114
        processDeregisterQueue();
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   115
        try {
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   116
            begin(blocking);
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   117
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   118
            do {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   119
                long startTime = timedPoll ? System.nanoTime() : 0;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   120
                numEntries = KQueue.poll(kqfd, pollArrayAddress, MAX_KEVENTS, to);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   121
                if (numEntries == IOStatus.INTERRUPTED && timedPoll) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   122
                    // timed poll interrupted so need to adjust timeout
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   123
                    long adjust = System.nanoTime() - startTime;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   124
                    to -= TimeUnit.MILLISECONDS.convert(adjust, TimeUnit.NANOSECONDS);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   125
                    if (to <= 0) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   126
                        // timeout expired so no retry
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   127
                        numEntries = 0;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   128
                    }
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   129
                }
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   130
            } while (numEntries == IOStatus.INTERRUPTED);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   131
            assert IOStatus.check(numEntries);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   132
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   133
        } finally {
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   134
            end(blocking);
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   135
        }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   136
        processDeregisterQueue();
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   137
        return updateSelectedKeys(numEntries);
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   138
    }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   139
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   140
    /**
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   141
     * Process new registrations and changes to the interest ops.
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   142
     */
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   143
    private void processUpdateQueue() {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   144
        assert Thread.holdsLock(this);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   145
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   146
        synchronized (updateLock) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   147
            SelectionKeyImpl ski;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   148
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   149
            // new registrations
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   150
            while ((ski = newKeys.pollFirst()) != null) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   151
                if (ski.isValid()) {
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   152
                    int fd = ski.channel.getFDVal();
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   153
                    SelectionKeyImpl previous = fdToKey.put(fd, ski);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   154
                    assert previous == null;
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   155
                    assert ski.registeredEvents() == 0;
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   156
                }
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   157
            }
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   158
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   159
            // changes to interest ops
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   160
            assert updateKeys.size() == updateKeys.size();
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   161
            while ((ski = updateKeys.pollFirst()) != null) {
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   162
                int newEvents = updateEvents.pollFirst();
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   163
                int fd = ski.channel.getFDVal();
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   164
                if (ski.isValid() && fdToKey.containsKey(fd)) {
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   165
                    int registeredEvents = ski.registeredEvents();
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
    /**
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   194
     * Update the keys of file descriptors that were polled and add them to
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   195
     * the selected-key set.
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   196
     * 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
   197
     */
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   198
    private int updateSelectedKeys(int numEntries) throws IOException {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   199
        assert Thread.holdsLock(this);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   200
        assert Thread.holdsLock(nioSelectedKeys());
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
                    }
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   228
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   229
                    if (selectedKeys.contains(ski)) {
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   230
                        // file descriptor may be polled more than once per poll
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   231
                        if (ski.lastPolled != pollCount) {
12438
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
   232
                            if (ski.channel.translateAndSetReadyOps(rOps, ski)) {
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
   233
                                numKeysUpdated++;
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   234
                                ski.lastPolled = pollCount;
12438
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
   235
                            }
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
   236
                        } else {
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
   237
                            // ready ops have already been set on this update
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
   238
                            ski.channel.translateAndUpdateReadyOps(rOps, ski);
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
                    } else {
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   241
                        ski.channel.translateAndSetReadyOps(rOps, ski);
12438
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
   242
                        if ((ski.nioReadyOps() & ski.nioInterestOps()) != 0) {
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   243
                            selectedKeys.add(ski);
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   244
                            numKeysUpdated++;
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   245
                            ski.lastPolled = pollCount;
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   246
                        }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   247
                    }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   248
                }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   249
            }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   250
        }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   251
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   252
        if (interrupted) {
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   253
            clearInterrupt();
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   254
        }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   255
        return numKeysUpdated;
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   256
    }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   257
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   258
    @Override
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   259
    protected void implClose() throws IOException {
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   260
        assert !isOpen();
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   261
        assert Thread.holdsLock(this);
12438
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
   262
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   263
        // prevent further wakeup
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   264
        synchronized (interruptLock) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   265
            interruptTriggered = true;
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   266
        }
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   267
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   268
        FileDispatcherImpl.closeIntFD(kqfd);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   269
        KQueue.freePollArray(pollArrayAddress);
12438
c1183d4a80d4 7143744: (se) Stabilize KQueue SelectorProvider and make default on MacOSX
alanb
parents: 12047
diff changeset
   270
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   271
        FileDispatcherImpl.closeIntFD(fd0);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   272
        FileDispatcherImpl.closeIntFD(fd1);
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   273
    }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   274
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   275
    @Override
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   276
    protected void implRegister(SelectionKeyImpl ski) {
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   277
        ensureOpen();
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   278
        synchronized (updateLock) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   279
            newKeys.addLast(ski);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   280
        }
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   281
    }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   282
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   283
    @Override
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   284
    protected void implDereg(SelectionKeyImpl ski) throws IOException {
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   285
        assert !ski.isValid();
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   286
        assert Thread.holdsLock(this);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   287
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   288
        int fd = ski.channel.getFDVal();
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   289
        int registeredEvents = ski.registeredEvents();
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   290
        if (fdToKey.remove(fd) != null) {
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   291
            if (registeredEvents != 0) {
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   292
                if ((registeredEvents & Net.POLLIN) != 0)
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   293
                    KQueue.register(kqfd, fd, EVFILT_READ, EV_DELETE);
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   294
                if ((registeredEvents & Net.POLLOUT) != 0)
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   295
                    KQueue.register(kqfd, fd, EVFILT_WRITE, EV_DELETE);
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   296
                ski.registeredEvents(0);
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   297
            }
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   298
        } else {
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   299
            assert registeredEvents == 0;
49290
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
    }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   302
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   303
    @Override
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   304
    public void putEventOps(SelectionKeyImpl ski, int events) {
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   305
        ensureOpen();
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   306
        synchronized (updateLock) {
49493
814bd31f8da0 8200257: (se) More Selector cleanup
alanb
parents: 49290
diff changeset
   307
            updateEvents.addLast(events);  // events first in case adding key fails
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   308
            updateKeys.addLast(ski);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   309
        }
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   310
    }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   311
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   312
    @Override
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   313
    public Selector wakeup() {
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   314
        synchronized (interruptLock) {
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   315
            if (!interruptTriggered) {
49290
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   316
                try {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   317
                    IOUtil.write1(fd1, (byte)0);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   318
                } catch (IOException ioe) {
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   319
                    throw new InternalError(ioe);
07779973cbe2 8199791: (se) More Selector cleanup
alanb
parents: 49248
diff changeset
   320
                }
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   321
                interruptTriggered = true;
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   322
            }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   323
        }
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   324
        return this;
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   325
    }
49248
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   326
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   327
    private void clearInterrupt() throws IOException {
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   328
        synchronized (interruptLock) {
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   329
            IOUtil.drain(fd0);
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   330
            interruptTriggered = false;
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   331
        }
15a0e60c8b97 8199611: (se) Minor selector implementation clean-up
alanb
parents: 47216
diff changeset
   332
    }
12047
320a714614e9 7113349: Initial changeset for Macosx port to jdk
michaelm
parents:
diff changeset
   333
}