Commit 5efcc2e9e7

Andrew Kelley <andrew@ziglang.org>
2024-07-12 23:20:20
build runner: refactor fs watch logic for OS abstraction
Makes the build runner compile successfully for non-linux targets; printing an error if you ask for --watch rather than making build scripts fail to compile.
1 parent f77b43d
Changed files (2)
lib
compiler
std
Build
lib/compiler/build_runner.zig
@@ -10,6 +10,7 @@ const File = std.fs.File;
 const Step = std.Build.Step;
 const Watch = std.Build.Watch;
 const Allocator = std.mem.Allocator;
+const fatal = std.zig.fatal;
 
 pub const root = @import("@build");
 pub const dependencies = @import("@dependencies");
@@ -371,18 +372,7 @@ pub fn main() !void {
         else => return err,
     };
 
-    var w = Watch.init;
-    if (watch) {
-        w.fan_fd = try std.posix.fanotify_init(.{
-            .CLASS = .NOTIF,
-            .CLOEXEC = true,
-            .NONBLOCK = true,
-            .REPORT_NAME = true,
-            .REPORT_DIR_FID = true,
-            .REPORT_FID = true,
-            .REPORT_TARGET_FID = true,
-        }, 0);
-    }
+    var w = if (watch) try Watch.init() else undefined;
 
     try run.thread_pool.init(thread_pool_options);
     defer run.thread_pool.deinit();
@@ -403,127 +393,53 @@ pub fn main() !void {
         };
         if (!watch) return cleanExit();
 
-        // Add missing marks and note persisted ones.
-        for (run.step_stack.keys()) |step| {
-            for (step.inputs.table.keys(), step.inputs.table.values()) |path, *files| {
-                const reaction_set = rs: {
-                    const gop = try w.dir_table.getOrPut(gpa, path);
-                    if (!gop.found_existing) {
-                        const dir_handle = try Watch.getDirHandle(gpa, path);
-                        // `dir_handle` may already be present in the table in
-                        // the case that we have multiple Cache.Path instances
-                        // that compare inequal but ultimately point to the same
-                        // directory on the file system.
-                        // In such case, we must revert adding this directory, but keep
-                        // the additions to the step set.
-                        const dh_gop = try w.handle_table.getOrPut(gpa, dir_handle);
-                        if (dh_gop.found_existing) {
-                            _ = w.dir_table.pop();
-                        } else {
-                            assert(dh_gop.index == gop.index);
-                            dh_gop.value_ptr.* = .{};
-                            std.posix.fanotify_mark(w.fan_fd, .{
-                                .ADD = true,
-                                .ONLYDIR = true,
-                            }, Watch.fan_mask, path.root_dir.handle.fd, path.subPathOrDot()) catch |err| {
-                                fatal("unable to watch {}: {s}", .{ path, @errorName(err) });
-                            };
-                        }
-                        break :rs dh_gop.value_ptr;
-                    }
-                    break :rs &w.handle_table.values()[gop.index];
-                };
-                for (files.items) |basename| {
-                    const gop = try reaction_set.getOrPut(gpa, basename);
-                    if (!gop.found_existing) gop.value_ptr.* = .{};
-                    try gop.value_ptr.put(gpa, step, w.generation);
-                }
-            }
+        switch (builtin.os.tag) {
+            .linux => {},
+            else => fatal("--watch not yet implemented for {s}", .{@tagName(builtin.os.tag)}),
         }
 
-        {
-            // Remove marks for files that are no longer inputs.
-            var i: usize = 0;
-            while (i < w.handle_table.entries.len) {
-                {
-                    const reaction_set = &w.handle_table.values()[i];
-                    var step_set_i: usize = 0;
-                    while (step_set_i < reaction_set.entries.len) {
-                        const step_set = &reaction_set.values()[step_set_i];
-                        var dirent_i: usize = 0;
-                        while (dirent_i < step_set.entries.len) {
-                            const generations = step_set.values();
-                            if (generations[dirent_i] == w.generation) {
-                                dirent_i += 1;
-                                continue;
-                            }
-                            step_set.swapRemoveAt(dirent_i);
-                        }
-                        if (step_set.entries.len > 0) {
-                            step_set_i += 1;
-                            continue;
-                        }
-                        reaction_set.swapRemoveAt(step_set_i);
-                    }
-                    if (reaction_set.entries.len > 0) {
-                        i += 1;
-                        continue;
-                    }
-                }
-
-                const path = w.dir_table.keys()[i];
-
-                std.posix.fanotify_mark(w.fan_fd, .{
-                    .REMOVE = true,
-                    .ONLYDIR = true,
-                }, Watch.fan_mask, path.root_dir.handle.fd, path.subPathOrDot()) catch |err| switch (err) {
-                    error.FileNotFound => {}, // Expected, harmless.
-                    else => |e| std.log.warn("unable to unwatch '{}': {s}", .{ path, @errorName(e) }),
-                };
-
-                w.dir_table.swapRemoveAt(i);
-                w.handle_table.swapRemoveAt(i);
-            }
-            w.generation +%= 1;
-        }
+        try w.update(gpa, run.step_stack.keys());
 
         // Wait until a file system notification arrives. Read all such events
         // until the buffer is empty. Then wait for a debounce interval, resetting
         // if any more events come in. After the debounce interval has passed,
         // trigger a rebuild on all steps with modified inputs, as well as their
         // recursive dependants.
-        var poll_fds: [1]std.posix.pollfd = .{
-            .{
-                .fd = w.fan_fd,
-                .events = std.posix.POLL.IN,
-                .revents = undefined,
-            },
-        };
         var caption_buf: [std.Progress.Node.max_name_len]u8 = undefined;
         const caption = std.fmt.bufPrint(&caption_buf, "Watching {d} Directories", .{
             w.dir_table.entries.len,
         }) catch &caption_buf;
         var debouncing_node = main_progress_node.start(caption, 0);
-        var debouncing = false;
-        while (true) {
-            const timeout: i32 = if (debouncing) debounce_interval_ms else -1;
-            const events_len = try std.posix.poll(&poll_fds, timeout);
-            if (events_len == 0) {
+        var debounce_timeout: Watch.Timeout = .none;
+        while (true) switch (try w.wait(gpa, debounce_timeout)) {
+            .timeout => {
                 debouncing_node.end();
-                Watch.markFailedStepsDirty(gpa, run.step_stack.keys());
+                markFailedStepsDirty(gpa, run.step_stack.keys());
                 continue :rebuild;
-            }
-            if (try w.markDirtySteps(gpa)) {
-                if (!debouncing) {
-                    debouncing = true;
-                    debouncing_node.end();
-                    debouncing_node = main_progress_node.start("Debouncing (Change Detected)", 0);
-                }
-            }
-        }
+            },
+            .dirty => if (debounce_timeout == .none) {
+                debounce_timeout = .{ .ms = debounce_interval_ms };
+                debouncing_node.end();
+                debouncing_node = main_progress_node.start("Debouncing (Change Detected)", 0);
+            },
+            .clean => {},
+        };
     }
 }
 
+fn markFailedStepsDirty(gpa: Allocator, all_steps: []const *Step) void {
+    for (all_steps) |step| switch (step.state) {
+        .dependency_failure, .failure, .skipped => step.recursiveReset(gpa),
+        else => continue,
+    };
+    // Now that all dirty steps have been found, the remaining steps that
+    // succeeded from last run shall be marked "cached".
+    for (all_steps) |step| switch (step.state) {
+        .success => step.result_cached = true,
+        else => continue,
+    };
+}
+
 const Run = struct {
     max_rss: u64,
     max_rss_is_default: bool,
@@ -1430,11 +1346,6 @@ fn fatalWithHint(comptime f: []const u8, args: anytype) noreturn {
     process.exit(1);
 }
 
-fn fatal(comptime f: []const u8, args: anytype) noreturn {
-    std.debug.print(f ++ "\n", args);
-    process.exit(1);
-}
-
 fn validateSystemLibraryOptions(b: *std.Build) void {
     var bad = false;
     for (b.graph.system_library_options.keys(), b.graph.system_library_options.values()) |k, v| {
lib/std/Build/Watch.zig
@@ -1,41 +1,21 @@
+const builtin = @import("builtin");
 const std = @import("../std.zig");
 const Watch = @This();
 const Step = std.Build.Step;
 const Allocator = std.mem.Allocator;
 const assert = std.debug.assert;
+const fatal = std.zig.fatal;
 
 dir_table: DirTable,
-/// Keyed differently but indexes correspond 1:1 with `dir_table`.
-handle_table: HandleTable,
-fan_fd: std.posix.fd_t,
+os: Os,
 generation: Generation,
 
-pub const fan_mask: std.os.linux.fanotify.MarkMask = .{
-    .CLOSE_WRITE = true,
-    .CREATE = true,
-    .DELETE = true,
-    .DELETE_SELF = true,
-    .EVENT_ON_CHILD = true,
-    .MOVED_FROM = true,
-    .MOVED_TO = true,
-    .MOVE_SELF = true,
-    .ONDIR = true,
-};
-
-pub const init: Watch = .{
-    .dir_table = .{},
-    .handle_table = .{},
-    .fan_fd = -1,
-    .generation = 0,
-};
-
 /// Key is the directory to watch which contains one or more files we are
 /// interested in noticing changes to.
 ///
 /// Value is generation.
 const DirTable = std.ArrayHashMapUnmanaged(Cache.Path, void, Cache.Path.TableAdapter, false);
 
-const HandleTable = std.ArrayHashMapUnmanaged(LinuxFileHandle, ReactionSet, LinuxFileHandle.Adapter, false);
 /// Special key of "." means any changes in this directory trigger the steps.
 const ReactionSet = std.StringArrayHashMapUnmanaged(StepSet);
 const StepSet = std.AutoArrayHashMapUnmanaged(*Step, Generation);
@@ -45,6 +25,255 @@ const Generation = u8;
 const Hash = std.hash.Wyhash;
 const Cache = std.Build.Cache;
 
+const Os = switch (builtin.os.tag) {
+    .linux => struct {
+        const posix = std.posix;
+
+        /// Keyed differently but indexes correspond 1:1 with `dir_table`.
+        handle_table: HandleTable,
+        poll_fds: [1]posix.pollfd,
+
+        const HandleTable = std.ArrayHashMapUnmanaged(FileHandle, ReactionSet, FileHandle.Adapter, false);
+
+        const fan_mask: std.os.linux.fanotify.MarkMask = .{
+            .CLOSE_WRITE = true,
+            .CREATE = true,
+            .DELETE = true,
+            .DELETE_SELF = true,
+            .EVENT_ON_CHILD = true,
+            .MOVED_FROM = true,
+            .MOVED_TO = true,
+            .MOVE_SELF = true,
+            .ONDIR = true,
+        };
+
+        const FileHandle = struct {
+            handle: *align(1) std.os.linux.file_handle,
+
+            fn clone(lfh: FileHandle, gpa: Allocator) Allocator.Error!FileHandle {
+                const bytes = lfh.slice();
+                const new_ptr = try gpa.alignedAlloc(
+                    u8,
+                    @alignOf(std.os.linux.file_handle),
+                    @sizeOf(std.os.linux.file_handle) + bytes.len,
+                );
+                const new_header: *std.os.linux.file_handle = @ptrCast(new_ptr);
+                new_header.* = lfh.handle.*;
+                const new: FileHandle = .{ .handle = new_header };
+                @memcpy(new.slice(), lfh.slice());
+                return new;
+            }
+
+            fn destroy(lfh: FileHandle, gpa: Allocator) void {
+                const ptr: [*]u8 = @ptrCast(lfh.handle);
+                const allocated_slice = ptr[0 .. @sizeOf(std.os.linux.file_handle) + lfh.handle.handle_bytes];
+                return gpa.free(allocated_slice);
+            }
+
+            fn slice(lfh: FileHandle) []u8 {
+                const ptr: [*]u8 = &lfh.handle.f_handle;
+                return ptr[0..lfh.handle.handle_bytes];
+            }
+
+            const Adapter = struct {
+                pub fn hash(self: Adapter, a: FileHandle) u32 {
+                    _ = self;
+                    const unsigned_type: u32 = @bitCast(a.handle.handle_type);
+                    return @truncate(Hash.hash(unsigned_type, a.slice()));
+                }
+                pub fn eql(self: Adapter, a: FileHandle, b: FileHandle, b_index: usize) bool {
+                    _ = self;
+                    _ = b_index;
+                    return a.handle.handle_type == b.handle.handle_type and std.mem.eql(u8, a.slice(), b.slice());
+                }
+            };
+        };
+
+        fn getDirHandle(gpa: Allocator, path: std.Build.Cache.Path) !FileHandle {
+            var file_handle_buffer: [@sizeOf(std.os.linux.file_handle) + 128]u8 align(@alignOf(std.os.linux.file_handle)) = undefined;
+            var mount_id: i32 = undefined;
+            var buf: [std.fs.max_path_bytes]u8 = undefined;
+            const adjusted_path = if (path.sub_path.len == 0) "./" else std.fmt.bufPrint(&buf, "{s}/", .{
+                path.sub_path,
+            }) catch return error.NameTooLong;
+            const stack_ptr: *std.os.linux.file_handle = @ptrCast(&file_handle_buffer);
+            stack_ptr.handle_bytes = file_handle_buffer.len - @sizeOf(std.os.linux.file_handle);
+            try posix.name_to_handle_at(path.root_dir.handle.fd, adjusted_path, stack_ptr, &mount_id, std.os.linux.AT.HANDLE_FID);
+            const stack_lfh: FileHandle = .{ .handle = stack_ptr };
+            return stack_lfh.clone(gpa);
+        }
+
+        fn markDirtySteps(w: *Watch, gpa: Allocator) !bool {
+            const fan_fd = w.os.getFanFd();
+            const fanotify = std.os.linux.fanotify;
+            const M = fanotify.event_metadata;
+            var events_buf: [256 + 4096]u8 = undefined;
+            var any_dirty = false;
+            while (true) {
+                var len = posix.read(fan_fd, &events_buf) catch |err| switch (err) {
+                    error.WouldBlock => return any_dirty,
+                    else => |e| return e,
+                };
+                var meta: [*]align(1) M = @ptrCast(&events_buf);
+                while (len >= @sizeOf(M) and meta[0].event_len >= @sizeOf(M) and meta[0].event_len <= len) : ({
+                    len -= meta[0].event_len;
+                    meta = @ptrCast(@as([*]u8, @ptrCast(meta)) + meta[0].event_len);
+                }) {
+                    assert(meta[0].vers == M.VERSION);
+                    if (meta[0].mask.Q_OVERFLOW) {
+                        any_dirty = true;
+                        std.log.warn("file system watch queue overflowed; falling back to fstat", .{});
+                        markAllFilesDirty(w, gpa);
+                        return true;
+                    }
+                    const fid: *align(1) fanotify.event_info_fid = @ptrCast(meta + 1);
+                    switch (fid.hdr.info_type) {
+                        .DFID_NAME => {
+                            const file_handle: *align(1) std.os.linux.file_handle = @ptrCast(&fid.handle);
+                            const file_name_z: [*:0]u8 = @ptrCast((&file_handle.f_handle).ptr + file_handle.handle_bytes);
+                            const file_name = std.mem.span(file_name_z);
+                            const lfh: FileHandle = .{ .handle = file_handle };
+                            if (w.os.handle_table.getPtr(lfh)) |reaction_set| {
+                                if (reaction_set.getPtr(".")) |glob_set|
+                                    any_dirty = markStepSetDirty(gpa, glob_set, any_dirty);
+                                if (reaction_set.getPtr(file_name)) |step_set|
+                                    any_dirty = markStepSetDirty(gpa, step_set, any_dirty);
+                            }
+                        },
+                        else => |t| std.log.warn("unexpected fanotify event '{s}'", .{@tagName(t)}),
+                    }
+                }
+            }
+        }
+
+        fn getFanFd(os: *const @This()) posix.fd_t {
+            return os.poll_fds[0].fd;
+        }
+
+        fn update(w: *Watch, gpa: Allocator, steps: []const *Step) !void {
+            const fan_fd = w.os.getFanFd();
+            // Add missing marks and note persisted ones.
+            for (steps) |step| {
+                for (step.inputs.table.keys(), step.inputs.table.values()) |path, *files| {
+                    const reaction_set = rs: {
+                        const gop = try w.dir_table.getOrPut(gpa, path);
+                        if (!gop.found_existing) {
+                            const dir_handle = try Os.getDirHandle(gpa, path);
+                            // `dir_handle` may already be present in the table in
+                            // the case that we have multiple Cache.Path instances
+                            // that compare inequal but ultimately point to the same
+                            // directory on the file system.
+                            // In such case, we must revert adding this directory, but keep
+                            // the additions to the step set.
+                            const dh_gop = try w.os.handle_table.getOrPut(gpa, dir_handle);
+                            if (dh_gop.found_existing) {
+                                _ = w.dir_table.pop();
+                            } else {
+                                assert(dh_gop.index == gop.index);
+                                dh_gop.value_ptr.* = .{};
+                                posix.fanotify_mark(fan_fd, .{
+                                    .ADD = true,
+                                    .ONLYDIR = true,
+                                }, fan_mask, path.root_dir.handle.fd, path.subPathOrDot()) catch |err| {
+                                    fatal("unable to watch {}: {s}", .{ path, @errorName(err) });
+                                };
+                            }
+                            break :rs dh_gop.value_ptr;
+                        }
+                        break :rs &w.os.handle_table.values()[gop.index];
+                    };
+                    for (files.items) |basename| {
+                        const gop = try reaction_set.getOrPut(gpa, basename);
+                        if (!gop.found_existing) gop.value_ptr.* = .{};
+                        try gop.value_ptr.put(gpa, step, w.generation);
+                    }
+                }
+            }
+
+            {
+                // Remove marks for files that are no longer inputs.
+                var i: usize = 0;
+                while (i < w.os.handle_table.entries.len) {
+                    {
+                        const reaction_set = &w.os.handle_table.values()[i];
+                        var step_set_i: usize = 0;
+                        while (step_set_i < reaction_set.entries.len) {
+                            const step_set = &reaction_set.values()[step_set_i];
+                            var dirent_i: usize = 0;
+                            while (dirent_i < step_set.entries.len) {
+                                const generations = step_set.values();
+                                if (generations[dirent_i] == w.generation) {
+                                    dirent_i += 1;
+                                    continue;
+                                }
+                                step_set.swapRemoveAt(dirent_i);
+                            }
+                            if (step_set.entries.len > 0) {
+                                step_set_i += 1;
+                                continue;
+                            }
+                            reaction_set.swapRemoveAt(step_set_i);
+                        }
+                        if (reaction_set.entries.len > 0) {
+                            i += 1;
+                            continue;
+                        }
+                    }
+
+                    const path = w.dir_table.keys()[i];
+
+                    posix.fanotify_mark(fan_fd, .{
+                        .REMOVE = true,
+                        .ONLYDIR = true,
+                    }, fan_mask, path.root_dir.handle.fd, path.subPathOrDot()) catch |err| switch (err) {
+                        error.FileNotFound => {}, // Expected, harmless.
+                        else => |e| std.log.warn("unable to unwatch '{}': {s}", .{ path, @errorName(e) }),
+                    };
+
+                    w.dir_table.swapRemoveAt(i);
+                    w.os.handle_table.swapRemoveAt(i);
+                }
+                w.generation +%= 1;
+            }
+        }
+    },
+    else => void,
+};
+
+pub fn init() !Watch {
+    switch (builtin.os.tag) {
+        .linux => {
+            const fan_fd = try std.posix.fanotify_init(.{
+                .CLASS = .NOTIF,
+                .CLOEXEC = true,
+                .NONBLOCK = true,
+                .REPORT_NAME = true,
+                .REPORT_DIR_FID = true,
+                .REPORT_FID = true,
+                .REPORT_TARGET_FID = true,
+            }, 0);
+            return .{
+                .dir_table = .{},
+                .os = switch (builtin.os.tag) {
+                    .linux => .{
+                        .handle_table = .{},
+                        .poll_fds = .{
+                            .{
+                                .fd = fan_fd,
+                                .events = std.posix.POLL.IN,
+                                .revents = undefined,
+                            },
+                        },
+                    },
+                    else => {},
+                },
+                .generation = 0,
+            };
+        },
+        else => @panic("unimplemented"),
+    }
+}
+
 pub const Match = struct {
     /// Relative to the watched directory, the file path that triggers this
     /// match.
@@ -68,119 +297,8 @@ pub const Match = struct {
     };
 };
 
-pub const LinuxFileHandle = struct {
-    handle: *align(1) std.os.linux.file_handle,
-
-    pub fn clone(lfh: LinuxFileHandle, gpa: Allocator) Allocator.Error!LinuxFileHandle {
-        const bytes = lfh.slice();
-        const new_ptr = try gpa.alignedAlloc(
-            u8,
-            @alignOf(std.os.linux.file_handle),
-            @sizeOf(std.os.linux.file_handle) + bytes.len,
-        );
-        const new_header: *std.os.linux.file_handle = @ptrCast(new_ptr);
-        new_header.* = lfh.handle.*;
-        const new: LinuxFileHandle = .{ .handle = new_header };
-        @memcpy(new.slice(), lfh.slice());
-        return new;
-    }
-
-    pub fn destroy(lfh: LinuxFileHandle, gpa: Allocator) void {
-        const ptr: [*]u8 = @ptrCast(lfh.handle);
-        const allocated_slice = ptr[0 .. @sizeOf(std.os.linux.file_handle) + lfh.handle.handle_bytes];
-        return gpa.free(allocated_slice);
-    }
-
-    pub fn slice(lfh: LinuxFileHandle) []u8 {
-        const ptr: [*]u8 = &lfh.handle.f_handle;
-        return ptr[0..lfh.handle.handle_bytes];
-    }
-
-    pub const Adapter = struct {
-        pub fn hash(self: Adapter, a: LinuxFileHandle) u32 {
-            _ = self;
-            const unsigned_type: u32 = @bitCast(a.handle.handle_type);
-            return @truncate(Hash.hash(unsigned_type, a.slice()));
-        }
-        pub fn eql(self: Adapter, a: LinuxFileHandle, b: LinuxFileHandle, b_index: usize) bool {
-            _ = self;
-            _ = b_index;
-            return a.handle.handle_type == b.handle.handle_type and std.mem.eql(u8, a.slice(), b.slice());
-        }
-    };
-};
-
-pub fn getDirHandle(gpa: Allocator, path: std.Build.Cache.Path) !LinuxFileHandle {
-    var file_handle_buffer: [@sizeOf(std.os.linux.file_handle) + 128]u8 align(@alignOf(std.os.linux.file_handle)) = undefined;
-    var mount_id: i32 = undefined;
-    var buf: [std.fs.max_path_bytes]u8 = undefined;
-    const adjusted_path = if (path.sub_path.len == 0) "./" else std.fmt.bufPrint(&buf, "{s}/", .{
-        path.sub_path,
-    }) catch return error.NameTooLong;
-    const stack_ptr: *std.os.linux.file_handle = @ptrCast(&file_handle_buffer);
-    stack_ptr.handle_bytes = file_handle_buffer.len - @sizeOf(std.os.linux.file_handle);
-    try std.posix.name_to_handle_at(path.root_dir.handle.fd, adjusted_path, stack_ptr, &mount_id, std.os.linux.AT.HANDLE_FID);
-    const stack_lfh: LinuxFileHandle = .{ .handle = stack_ptr };
-    return stack_lfh.clone(gpa);
-}
-
-pub fn markDirtySteps(w: *Watch, gpa: Allocator) !bool {
-    const fanotify = std.os.linux.fanotify;
-    const M = fanotify.event_metadata;
-    var events_buf: [256 + 4096]u8 = undefined;
-    var any_dirty = false;
-    while (true) {
-        var len = std.posix.read(w.fan_fd, &events_buf) catch |err| switch (err) {
-            error.WouldBlock => return any_dirty,
-            else => |e| return e,
-        };
-        var meta: [*]align(1) M = @ptrCast(&events_buf);
-        while (len >= @sizeOf(M) and meta[0].event_len >= @sizeOf(M) and meta[0].event_len <= len) : ({
-            len -= meta[0].event_len;
-            meta = @ptrCast(@as([*]u8, @ptrCast(meta)) + meta[0].event_len);
-        }) {
-            assert(meta[0].vers == M.VERSION);
-            if (meta[0].mask.Q_OVERFLOW) {
-                any_dirty = true;
-                std.log.warn("file system watch queue overflowed; falling back to fstat", .{});
-                markAllFilesDirty(w, gpa);
-                return true;
-            }
-            const fid: *align(1) fanotify.event_info_fid = @ptrCast(meta + 1);
-            switch (fid.hdr.info_type) {
-                .DFID_NAME => {
-                    const file_handle: *align(1) std.os.linux.file_handle = @ptrCast(&fid.handle);
-                    const file_name_z: [*:0]u8 = @ptrCast((&file_handle.f_handle).ptr + file_handle.handle_bytes);
-                    const file_name = std.mem.span(file_name_z);
-                    const lfh: Watch.LinuxFileHandle = .{ .handle = file_handle };
-                    if (w.handle_table.getPtr(lfh)) |reaction_set| {
-                        if (reaction_set.getPtr(".")) |glob_set|
-                            any_dirty = markStepSetDirty(gpa, glob_set, any_dirty);
-                        if (reaction_set.getPtr(file_name)) |step_set|
-                            any_dirty = markStepSetDirty(gpa, step_set, any_dirty);
-                    }
-                },
-                else => |t| std.log.warn("unexpected fanotify event '{s}'", .{@tagName(t)}),
-            }
-        }
-    }
-}
-
-pub fn markFailedStepsDirty(gpa: Allocator, all_steps: []const *Step) void {
-    for (all_steps) |step| switch (step.state) {
-        .dependency_failure, .failure, .skipped => step.recursiveReset(gpa),
-        else => continue,
-    };
-    // Now that all dirty steps have been found, the remaining steps that
-    // succeeded from last run shall be marked "cached".
-    for (all_steps) |step| switch (step.state) {
-        .success => step.result_cached = true,
-        else => continue,
-    };
-}
-
 fn markAllFilesDirty(w: *Watch, gpa: Allocator) void {
-    for (w.handle_table.values()) |reaction_set| {
+    for (w.os.handle_table.values()) |reaction_set| {
         for (reaction_set.values()) |step_set| {
             for (step_set.keys()) |step| {
                 step.recursiveReset(gpa);
@@ -199,3 +317,47 @@ fn markStepSetDirty(gpa: Allocator, step_set: *StepSet, any_dirty: bool) bool {
     }
     return any_dirty or this_any_dirty;
 }
+
+pub fn update(w: *Watch, gpa: Allocator, steps: []const *Step) !void {
+    switch (builtin.os.tag) {
+        .linux => return Os.update(w, gpa, steps),
+        else => @compileError("unimplemented"),
+    }
+}
+
+pub const Timeout = union(enum) {
+    none,
+    ms: u16,
+
+    pub fn to_i32_ms(t: Timeout) i32 {
+        return switch (t) {
+            .none => -1,
+            .ms => |ms| ms,
+        };
+    }
+};
+
+pub const WaitResult = enum {
+    timeout,
+    /// File system watching triggered on files that were marked as inputs to at least one Step.
+    /// Relevant steps have been marked dirty.
+    dirty,
+    /// File system watching triggered but none of the events were relevant to
+    /// what we are listening to. There is nothing to do.
+    clean,
+};
+
+pub fn wait(w: *Watch, gpa: Allocator, timeout: Timeout) !WaitResult {
+    switch (builtin.os.tag) {
+        .linux => {
+            const events_len = try std.posix.poll(&w.os.poll_fds, timeout.to_i32_ms());
+            return if (events_len == 0)
+                .timeout
+            else if (try Os.markDirtySteps(w, gpa))
+                .dirty
+            else
+                .clean;
+        },
+        else => @compileError("unimplemented"),
+    }
+}