Skip to content

IoUring: use typed Flags and Features for IoUring #25394

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 18 commits into
base: master
Choose a base branch
from
+3,708 −2,707
Draft

IoUring: use typed Flags and Features for IoUring #25394

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
6890119
IoUring: use typed Flags and Features for IoUring
bernardassan Sep 28, 2025
5bb26d5
replace some more fn flags with Typed Flags
bernardassan Sep 29, 2025
3956da7
move Flags that were mistakenly tagged as constants
bernardassan Sep 30, 2025
c7fdd4b
Replace STATX_* with StatxMask & StatxAttr
bernardassan Sep 30, 2025
7e0b7a0
Improve organization of fn and structs in IoUring
bernardassan Oct 1, 2025
8a2897f
Replace AT,W,SHUT,SOCK with a packed struct Flag type
bernardassan Oct 1, 2025
754f77e
update some syscall APIs to use the new flags
bernardassan Oct 2, 2025
2ec6ea5
Restore deprecated contants using new Flag types
bernardassan Oct 5, 2025
13fd0cd
Replace MSG with Packed Struct Flags
bernardassan Oct 6, 2025
35461ca
Get test passing for all the newly introduced flags
bernardassan Oct 7, 2025
44afd58
Add So and Sol typed flags
bernardassan Oct 9, 2025
0471009
Replace EPOLL struct with an EpollOp enum and Epoll packed struct type
bernardassan Oct 9, 2025
c2636b5
fix error of setting some fields in Epoll to true by default
bernardassan Oct 10, 2025
7535acd
Remove io_uring_sqe.zig from CMakeLists
bernardassan Oct 10, 2025
1acc634
Use lower case identifiers for IoUring flags and enums
bernardassan Oct 10, 2025
aa4f14a
replace direct set of some flags with link_next and set_flags calls
bernardassan Oct 10, 2025
aa150cb
Remove io_uring bit and pieces from linux.zig
bernardassan Oct 11, 2025
0213381
fix use of At in rmdir syscall
bernardassan Oct 11, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -475,7 +475,6 @@ set(ZIG_STAGE2_SOURCES
lib/std/os/linux.zig
lib/std/os/linux.zig
lib/std/os/linux/IoUring.zig
lib/std/os/linux/io_uring_sqe.zig
lib/std/os/linux/x86_64.zig
lib/std/os/linux/x86_64.zig
lib/std/os/windows.zig
Expand Down
2 changes: 1 addition & 1 deletion lib/std/fs.zig
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -223,7 +223,7 @@ pub fn cwd() Dir {
} else if (native_os == .wasi) {
return .{ .fd = std.options.wasiCwd() };
} else {
return .{ .fd = posix.AT.FDCWD };
return .{ .fd = posix.AT.fdcwd };
}
}

Expand Down
10 changes: 8 additions & 2 deletions lib/std/fs/Dir.zig
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2720,8 +2720,14 @@ pub fn statFile(self: Dir, sub_path: []const u8) StatFileError!Stat {
const rc = linux.statx(
self.fd,
&sub_path_c,
linux.AT.NO_AUTOMOUNT,
linux.STATX_TYPE | linux.STATX_MODE | linux.STATX_ATIME | linux.STATX_MTIME | linux.STATX_CTIME,
.{ .no_automount = true },
.{
.type = true,
.mode = true,
.atime = true,
.mtime = true,
.ctime = true,
},
&stx,
);

Expand Down
10 changes: 8 additions & 2 deletions lib/std/fs/File.zig
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -564,8 +564,14 @@ pub fn stat(self: File) StatError!Stat {
const rc = linux.statx(
self.handle,
"",
linux.AT.EMPTY_PATH,
linux.STATX_TYPE | linux.STATX_MODE | linux.STATX_ATIME | linux.STATX_MTIME | linux.STATX_CTIME,
.{ .empty_path = true },
.{
.type = true,
.mode = true,
.atime = true,
.mtime = true,
.ctime = true,
},
&stx,
);

Expand Down
Loading
Loading