ViewVC Help
View File | Revision Log | Show Annotations | Download File
/cvs/libev/ev_port.c
(Generate patch)

Comparing libev/ev_port.c (file contents):
Revision 1.21 by root, Fri Oct 15 22:44:41 2010 UTC vs.
Revision 1.26 by root, Wed Feb 16 08:02:50 2011 UTC

1/* 1/*
2 * libev solaris event port backend 2 * libev solaris event port backend
3 * 3 *
4 * Copyright (c) 2007,2008,2009 Marc Alexander Lehmann <libev@schmorp.de> 4 * Copyright (c) 2007,2008,2009,2010,2011 Marc Alexander Lehmann <libev@schmorp.de>
5 * All rights reserved. 5 * All rights reserved.
6 * 6 *
7 * Redistribution and use in source and binary forms, with or without modifica- 7 * Redistribution and use in source and binary forms, with or without modifica-
8 * tion, are permitted provided that the following conditions are met: 8 * tion, are permitted provided that the following conditions are met:
9 * 9 *
10 * 1. Redistributions of source code must retain the above copyright notice, 10 * 1. Redistributions of source code must retain the above copyright notice,
11 * this list of conditions and the following disclaimer. 11 * this list of conditions and the following disclaimer.
12 * 12 *
13 * 2. Redistributions in binary form must reproduce the above copyright 13 * 2. Redistributions in binary form must reproduce the above copyright
14 * notice, this list of conditions and the following disclaimer in the 14 * notice, this list of conditions and the following disclaimer in the
15 * documentation and/or other materials provided with the distribution. 15 * documentation and/or other materials provided with the distribution.
16 * 16 *
17 * THIS SOFTWARE IS PROVIDED BY THE AUTHOR ``AS IS'' AND ANY EXPRESS OR IMPLIED 17 * THIS SOFTWARE IS PROVIDED BY THE AUTHOR ``AS IS'' AND ANY EXPRESS OR IMPLIED
18 * WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MER- 18 * WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MER-
19 * CHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO 19 * CHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO
20 * EVENT SHALL THE AUTHOR BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPE- 20 * EVENT SHALL THE AUTHOR BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPE-
21 * CIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, 21 * CIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
… …
33 * version of this file under the BSD license, indicate your decision 33 * version of this file under the BSD license, indicate your decision
34 * by deleting the provisions above and replace them with the notice 34 * by deleting the provisions above and replace them with the notice
35 * and other provisions required by the GPL. If you do not delete the 35 * and other provisions required by the GPL. If you do not delete the
36 * provisions above, a recipient may use your version of this file under 36 * provisions above, a recipient may use your version of this file under
37 * either the BSD or the GPL. 37 * either the BSD or the GPL.
38 */
39
40/* useful reading:
41 *
42 * http://bugs.opensolaris.org/view_bug.do?bug_id=6268715 (random results)
43 * http://bugs.opensolaris.org/view_bug.do?bug_id=6455223 (just totally broken)
44 * http://bugs.opensolaris.org/view_bug.do?bug_id=6873782 (manpage ETIME)
45 * http://bugs.opensolaris.org/view_bug.do?bug_id=6874410 (implementation ETIME)
46 * http://www.mail-archive.com/networking-discuss@opensolaris.org/msg11898.html ETIME vs. nget
47 * http://src.opensolaris.org/source/xref/onnv/onnv-gate/usr/src/lib/libc/port/gen/event_port.c (libc)
48 * http://cvs.opensolaris.org/source/xref/onnv/onnv-gate/usr/src/uts/common/fs/portfs/port.c#1325 (kernel)
38 */ 49 */
39 50
40#include <sys/types.h> 51#include <sys/types.h>
41#include <sys/time.h> 52#include <sys/time.h>
42#include <poll.h> 53#include <poll.h>
… …
83{ 94{
84 int res, i; 95 int res, i;
85 struct timespec ts; 96 struct timespec ts;
86 uint_t nget = 1; 97 uint_t nget = 1;
87 98
99 /* we initialise this to something we will skip in the loop, as */
100 /* port_getn can return with nget unchanged, but no indication */
101 /* whether it was the original value or has been updated :/ */
102 port_events [0].portev_source = 0;
103
88 EV_RELEASE_CB; 104 EV_RELEASE_CB;
89 EV_TS_SET (ts, timeout); 105 EV_TS_SET (ts, timeout);
90 res = port_getn (backend_fd, port_events, port_eventmax, &nget, &ts); 106 res = port_getn (backend_fd, port_events, port_eventmax, &nget, &ts);
91 EV_ACQUIRE_CB; 107 EV_ACQUIRE_CB;
92 108
93 if (res == -1) 109 /* port_getn may or may not set nget on error */
94 { 110 /* so we rely on port_events [0].portev_source not being updated */
95 if (errno != EINTR && errno != ETIME) 111 if (res == -1 && errno != ETIME && errno != EINTR)
96 ev_syserr ("(libev) port_getn (see http://bugs.opensolaris.org/view_bug.do?bug_id=6268715, try LIBEV_FLAGS=3 env variable)"); 112 ev_syserr ("(libev) port_getn (see http://bugs.opensolaris.org/view_bug.do?bug_id=6268715, try LIBEV_FLAGS=3 env variable)");
97
98 return;
99 }
100 113
101 for (i = 0; i < nget; ++i) 114 for (i = 0; i < nget; ++i)
102 { 115 {
103 if (port_events [i].portev_source == PORT_SOURCE_FD) 116 if (port_events [i].portev_source == PORT_SOURCE_FD)
104 { 117 {
… …
109 fd, 122 fd,
110 (port_events [i].portev_events & (POLLOUT | POLLERR | POLLHUP) ? EV_WRITE : 0) 123 (port_events [i].portev_events & (POLLOUT | POLLERR | POLLHUP) ? EV_WRITE : 0)
111 | (port_events [i].portev_events & (POLLIN | POLLERR | POLLHUP) ? EV_READ : 0) 124 | (port_events [i].portev_events & (POLLIN | POLLERR | POLLHUP) ? EV_READ : 0)
112 ); 125 );
113 126
114 port_associate_and_check (EV_A_ fd, anfds [fd].events); 127 fd_change (EV_A_ fd, EV__IOFDSET);
115 } 128 }
116 } 129 }
117 130
118 if (expect_false (nget == port_eventmax)) 131 if (expect_false (nget == port_eventmax))
119 { 132 {
… …
127port_init (EV_P_ int flags) 140port_init (EV_P_ int flags)
128{ 141{
129 /* Initialize the kernel queue */ 142 /* Initialize the kernel queue */
130 if ((backend_fd = port_create ()) < 0) 143 if ((backend_fd = port_create ()) < 0)
131 return 0; 144 return 0;
145
146 assert (("libev: PORT_SOURCE_FD must not be zero", PORT_SOURCE_FD));
132 147
133 fcntl (backend_fd, F_SETFD, FD_CLOEXEC); /* not sure if necessary, hopefully doesn't hurt */ 148 fcntl (backend_fd, F_SETFD, FD_CLOEXEC); /* not sure if necessary, hopefully doesn't hurt */
134 149
135 backend_fudge = 1e-3; /* needed to compensate for port_getn returning early */ 150 backend_fudge = 1e-3; /* needed to compensate for port_getn returning early */
136 backend_modify = port_modify; 151 backend_modify = port_modify;

Diff Legend

– Removed lines
+ Added lines
< Changed lines
> Changed lines