authorgravatar for andrew@ziglang.orgAndrew Kelley <andrew@ziglang.org> 2021-09-07 14:58:25-04:00
committergravatar for noreply@github.comGitHub <noreply@github.com> 2021-09-07 14:58:25-04:00
logf1126e854c9dbe6a58cef66b380c0a7bb95f06d4
tree07ac208ff1de323cc8d6e9693dee0a87df989874
parenta48e5af69df17f225cc06658f8a616d54f31d090
parent107e3ba3cd4f5ecf48c5a5b821f43f00cf4ace3b
signature Signed by PGP key 4AEE18F83AFDEB23

Merge pull request #9695 from g-w1/evloop-usingns

std: update event loop for recent usingnamespace changes

4 files changed, 43 insertions(+), 37 deletions(-)

lib/std/c/netbsd.zig+3
......@@ -766,6 +766,9 @@ pub const EVFILT_TIMER = 6;
766766/// Filesystem events
767767pub const EVFILT_FS = 7;
768768
769/// User events
770pub const EVFILT_USER = 1;
771
769772/// On input, NOTE_TRIGGER causes the event to be triggered for output.
770773pub const NOTE_TRIGGER = 0x08000000;
771774
lib/std/event/loop.zig+37-37
......@@ -222,27 +222,27 @@ pub const Loop = struct {
222222 .handle = undefined,
223223 .overlapped = ResumeNode.overlapped_init,
224224 },
225 .eventfd = try os.eventfd(1, os.EFD_CLOEXEC | os.EFD_NONBLOCK),
226 .epoll_op = os.EPOLL_CTL_ADD,
225 .eventfd = try os.eventfd(1, os.linux.EFD.CLOEXEC | os.linux.EFD.NONBLOCK),
226 .epoll_op = os.linux.EPOLL.CTL_ADD,
227227 },
228228 .next = undefined,
229229 };
230230 self.available_eventfd_resume_nodes.push(eventfd_node);
231231 }
232232
233 self.os_data.epollfd = try os.epoll_create1(os.EPOLL_CLOEXEC);
233 self.os_data.epollfd = try os.epoll_create1(os.linux.EPOLL.CLOEXEC);
234234 errdefer os.close(self.os_data.epollfd);
235235
236 self.os_data.final_eventfd = try os.eventfd(0, os.EFD_CLOEXEC | os.EFD_NONBLOCK);
236 self.os_data.final_eventfd = try os.eventfd(0, os.linux.EFD.CLOEXEC | os.linux.EFD.NONBLOCK);
237237 errdefer os.close(self.os_data.final_eventfd);
238238
239 self.os_data.final_eventfd_event = os.epoll_event{
240 .events = os.EPOLLIN,
241 .data = os.epoll_data{ .ptr = @ptrToInt(&self.final_resume_node) },
239 self.os_data.final_eventfd_event = os.linux.epoll_event{
240 .events = os.linux.EPOLL.IN,
241 .data = os.linux.epoll_data{ .ptr = @ptrToInt(&self.final_resume_node) },
242242 };
243243 try os.epoll_ctl(
244244 self.os_data.epollfd,
245 os.EPOLL_CTL_ADD,
245 os.linux.EPOLL.CTL_ADD,
246246 self.os_data.final_eventfd,
247247 &self.os_data.final_eventfd_event,
248248 );
......@@ -283,8 +283,8 @@ pub const Loop = struct {
283283 // this one is for sending events
284284 .kevent = os.Kevent{
285285 .ident = i,
286 .filter = os.EVFILT_USER,
287 .flags = os.EV_CLEAR | os.EV_ADD | os.EV_DISABLE,
286 .filter = os.system.EVFILT_USER,
287 .flags = os.system.EV_CLEAR | os.system.EV_ADD | os.system.EV_DISABLE,
288288 .fflags = 0,
289289 .data = 0,
290290 .udata = @ptrToInt(&eventfd_node.data.base),
......@@ -295,24 +295,24 @@ pub const Loop = struct {
295295 self.available_eventfd_resume_nodes.push(eventfd_node);
296296 const kevent_array = @as(*const [1]os.Kevent, &eventfd_node.data.kevent);
297297 _ = try os.kevent(self.os_data.kqfd, kevent_array, empty_kevs, null);
298 eventfd_node.data.kevent.flags = os.EV_CLEAR | os.EV_ENABLE;
299 eventfd_node.data.kevent.fflags = os.NOTE_TRIGGER;
298 eventfd_node.data.kevent.flags = os.system.EV_CLEAR | os.system.EV_ENABLE;
299 eventfd_node.data.kevent.fflags = os.system.NOTE_TRIGGER;
300300 }
301301
302302 // Pre-add so that we cannot get error.SystemResources
303303 // later when we try to activate it.
304304 self.os_data.final_kevent = os.Kevent{
305305 .ident = extra_thread_count,
306 .filter = os.EVFILT_USER,
307 .flags = os.EV_ADD | os.EV_DISABLE,
306 .filter = os.system.EVFILT_USER,
307 .flags = os.system.EV_ADD | os.system.EV_DISABLE,
308308 .fflags = 0,
309309 .data = 0,
310310 .udata = @ptrToInt(&self.final_resume_node),
311311 };
312312 const final_kev_arr = @as(*const [1]os.Kevent, &self.os_data.final_kevent);
313313 _ = try os.kevent(self.os_data.kqfd, final_kev_arr, empty_kevs, null);
314 self.os_data.final_kevent.flags = os.EV_ENABLE;
315 self.os_data.final_kevent.fflags = os.NOTE_TRIGGER;
314 self.os_data.final_kevent.flags = os.system.EV_ENABLE;
315 self.os_data.final_kevent.fflags = os.system.NOTE_TRIGGER;
316316
317317 if (builtin.single_threaded) {
318318 assert(extra_thread_count == 0);
......@@ -404,19 +404,19 @@ pub const Loop = struct {
404404 /// resume_node must live longer than the anyframe that it holds a reference to.
405405 /// flags must contain EPOLLET
406406 pub fn linuxAddFd(self: *Loop, fd: i32, resume_node: *ResumeNode, flags: u32) !void {
407 assert(flags & os.EPOLLET == os.EPOLLET);
407 assert(flags & os.linux.EPOLL.ET == os.linux.EPOLL.ET);
408408 self.beginOneEvent();
409409 errdefer self.finishOneEvent();
410410 try self.linuxModFd(
411411 fd,
412 os.EPOLL_CTL_ADD,
412 os.linux.EPOLL.CTL_ADD,
413413 flags,
414414 resume_node,
415415 );
416416 }
417417
418418 pub fn linuxModFd(self: *Loop, fd: i32, op: u32, flags: u32, resume_node: *ResumeNode) !void {
419 assert(flags & os.EPOLLET == os.EPOLLET);
419 assert(flags & os.linux.EPOLL.ET == os.linux.EPOLL.ET);
420420 var ev = os.linux.epoll_event{
421421 .events = flags,
422422 .data = os.linux.epoll_data{ .ptr = @ptrToInt(resume_node) },
......@@ -425,13 +425,13 @@ pub const Loop = struct {
425425 }
426426
427427 pub fn linuxRemoveFd(self: *Loop, fd: i32) void {
428 os.epoll_ctl(self.os_data.epollfd, os.linux.EPOLL_CTL_DEL, fd, null) catch {};
428 os.epoll_ctl(self.os_data.epollfd, os.linux.EPOLL.CTL_DEL, fd, null) catch {};
429429 self.finishOneEvent();
430430 }
431431
432432 pub fn linuxWaitFd(self: *Loop, fd: i32, flags: u32) void {
433 assert(flags & os.EPOLLET == os.EPOLLET);
434 assert(flags & os.EPOLLONESHOT == os.EPOLLONESHOT);
433 assert(flags & os.linux.EPOLL.ET == os.linux.EPOLL.ET);
434 assert(flags & os.linux.EPOLL.ONESHOT == os.linux.EPOLL.ONESHOT);
435435 var resume_node = ResumeNode.Basic{
436436 .base = ResumeNode{
437437 .id = .Basic,
......@@ -457,8 +457,8 @@ pub const Loop = struct {
457457 // Fall back to a blocking poll(). Ideally this codepath is never hit, since
458458 // epoll should be just fine. But this is better than incorrect behavior.
459459 var poll_flags: i16 = 0;
460 if ((flags & os.EPOLLIN) != 0) poll_flags |= os.POLL.IN;
461 if ((flags & os.EPOLLOUT) != 0) poll_flags |= os.POLL.OUT;
460 if ((flags & os.linux.EPOLL.IN) != 0) poll_flags |= os.POLL.IN;
461 if ((flags & os.linux.EPOLL.OUT) != 0) poll_flags |= os.POLL.OUT;
462462 var pfd = [1]os.pollfd{os.pollfd{
463463 .fd = fd,
464464 .events = poll_flags,
......@@ -484,10 +484,10 @@ pub const Loop = struct {
484484 pub fn waitUntilFdReadable(self: *Loop, fd: os.fd_t) void {
485485 switch (builtin.os.tag) {
486486 .linux => {
487 self.linuxWaitFd(fd, os.EPOLLET | os.EPOLLONESHOT | os.EPOLLIN);
487 self.linuxWaitFd(fd, os.linux.EPOLL.ET | os.linux.EPOLL.ONESHOT | os.linux.EPOLL.IN);
488488 },
489489 .macos, .freebsd, .netbsd, .dragonfly, .openbsd => {
490 self.bsdWaitKev(@intCast(usize, fd), os.EVFILT_READ, os.EV_ONESHOT);
490 self.bsdWaitKev(@intCast(usize, fd), os.system.EVFILT_READ, os.system.EV_ONESHOT);
491491 },
492492 else => @compileError("Unsupported OS"),
493493 }
......@@ -496,10 +496,10 @@ pub const Loop = struct {
496496 pub fn waitUntilFdWritable(self: *Loop, fd: os.fd_t) void {
497497 switch (builtin.os.tag) {
498498 .linux => {
499 self.linuxWaitFd(fd, os.EPOLLET | os.EPOLLONESHOT | os.EPOLLOUT);
499 self.linuxWaitFd(fd, os.linux.EPOLL.ET | os.linux.EPOLL.ONESHOT | os.linux.EPOLL.OUT);
500500 },
501501 .macos, .freebsd, .netbsd, .dragonfly, .openbsd => {
502 self.bsdWaitKev(@intCast(usize, fd), os.EVFILT_WRITE, os.EV_ONESHOT);
502 self.bsdWaitKev(@intCast(usize, fd), os.system.EVFILT_WRITE, os.system.EV_ONESHOT);
503503 },
504504 else => @compileError("Unsupported OS"),
505505 }
......@@ -508,11 +508,11 @@ pub const Loop = struct {
508508 pub fn waitUntilFdWritableOrReadable(self: *Loop, fd: os.fd_t) void {
509509 switch (builtin.os.tag) {
510510 .linux => {
511 self.linuxWaitFd(fd, os.EPOLLET | os.EPOLLONESHOT | os.EPOLLOUT | os.EPOLLIN);
511 self.linuxWaitFd(fd, os.linux.EPOLL.ET | os.linux.EPOLL.ONESHOT | os.linux.EPOLL.OUT | os.linux.EPOLL.IN);
512512 },
513513 .macos, .freebsd, .netbsd, .dragonfly, .openbsd => {
514 self.bsdWaitKev(@intCast(usize, fd), os.EVFILT_READ, os.EV_ONESHOT);
515 self.bsdWaitKev(@intCast(usize, fd), os.EVFILT_WRITE, os.EV_ONESHOT);
514 self.bsdWaitKev(@intCast(usize, fd), os.system.EVFILT_READ, os.system.EV_ONESHOT);
515 self.bsdWaitKev(@intCast(usize, fd), os.system.EVFILT_WRITE, os.system.EV_ONESHOT);
516516 },
517517 else => @compileError("Unsupported OS"),
518518 }
......@@ -530,7 +530,7 @@ pub const Loop = struct {
530530
531531 defer {
532532 // If the kevent was set to be ONESHOT, it doesn't need to be deleted manually.
533 if (flags & os.EV_ONESHOT != 0) {
533 if (flags & os.system.EV_ONESHOT != 0) {
534534 self.bsdRemoveKev(ident, filter);
535535 }
536536 }
......@@ -547,7 +547,7 @@ pub const Loop = struct {
547547 var kev = [1]os.Kevent{os.Kevent{
548548 .ident = ident,
549549 .filter = filter,
550 .flags = os.EV_ADD | os.EV_ENABLE | os.EV_CLEAR | flags,
550 .flags = os.system.EV_ADD | os.system.EV_ENABLE | os.system.EV_CLEAR | flags,
551551 .fflags = 0,
552552 .data = 0,
553553 .udata = @ptrToInt(&resume_node.base),
......@@ -560,7 +560,7 @@ pub const Loop = struct {
560560 var kev = [1]os.Kevent{os.Kevent{
561561 .ident = ident,
562562 .filter = filter,
563 .flags = os.EV_DELETE,
563 .flags = os.system.EV_DELETE,
564564 .fflags = 0,
565565 .data = 0,
566566 .udata = 0,
......@@ -590,8 +590,8 @@ pub const Loop = struct {
590590 },
591591 .linux => {
592592 // the pending count is already accounted for
593 const epoll_events = os.EPOLLONESHOT | os.linux.EPOLLIN | os.linux.EPOLLOUT |
594 os.linux.EPOLLET;
593 const epoll_events = os.linux.EPOLL.ONESHOT | os.linux.EPOLL.IN | os.linux.EPOLL.OUT |
594 os.linux.EPOLL.ET;
595595 self.linuxModFd(
596596 eventfd_node.eventfd,
597597 eventfd_node.epoll_op,
......@@ -1344,7 +1344,7 @@ pub const Loop = struct {
13441344 .Stop => return,
13451345 .EventFd => {
13461346 const event_fd_node = @fieldParentPtr(ResumeNode.EventFd, "base", resume_node);
1347 event_fd_node.epoll_op = os.EPOLL_CTL_MOD;
1347 event_fd_node.epoll_op = os.linux.EPOLL.CTL_MOD;
13481348 const stack_node = @fieldParentPtr(std.atomic.Stack(ResumeNode.EventFd).Node, "data", event_fd_node);
13491349 self.available_eventfd_resume_nodes.push(stack_node);
13501350 },
test/standalone.zig+1
......@@ -5,6 +5,7 @@ pub fn addCases(cases: *tests.StandaloneContext) void {
55 cases.add("test/standalone/hello_world/hello.zig");
66 cases.addC("test/standalone/hello_world/hello_libc.zig");
77 cases.add("test/standalone/cat/main.zig");
8 cases.add("test/standalone/issue_9693/main.zig");
89 cases.add("test/standalone/guess_number/main.zig");
910 cases.add("test/standalone/main_return_error/error_u8.zig");
1011 cases.add("test/standalone/main_return_error/error_u8_non_zero.zig");
test/standalone/issue_9693/main.zig created+2
......@@ -0,0 +1,2 @@
1pub const io_mode = .evented;
2pub fn main() void {}