Differential D50761

linux: Add inotify support
ClosedPublic
Actions

Authored by markj on Jun 9 2025, 8:16 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Jul 8, 5:03 PM

	Unknown Object (File)
	Sun, Jul 6, 10:14 PM

	Unknown Object (File)
	Fri, Jul 4, 8:00 PM

	Unknown Object (File)
	Fri, Jun 27, 9:10 PM

	Unknown Object (File)
	Fri, Jun 27, 2:39 AM

	Unknown Object (File)
	Thu, Jun 26, 8:17 PM

	Unknown Object (File)
	Mon, Jun 23, 11:08 AM

	Unknown Object (File)
	Sat, Jun 21, 12:43 AM

View All 18 Files

Subscribers

Details

Reviewers

andrew
manu
brooks

Group Reviewers

Linux Emulation

Commits

rG3965de642c29: linux: Add inotify support

Summary

Based on D50315.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

markj created this revision.Jun 9 2025, 8:16 PM

Herald added a reviewer: andrew. · View Herald TranscriptJun 9 2025, 8:16 PM

Herald added a reviewer: andrew. · View Herald Transcript

Herald added a reviewer: manu. · View Herald Transcript

Herald added a reviewer: brooks. · View Herald Transcript

Herald added subscribers: olce, emaste, imp. · View Herald Transcript

markj requested review of this revision.Jun 9 2025, 8:16 PM

Harbormaster completed remote builds in B64730: Diff 156730.Jun 9 2025, 8:16 PM

markj added a reviewer: Linux Emulation.Jun 9 2025, 8:16 PM

markj edited the summary of this revision. (Show Details)

Why don't all the syscalls.master have an inotify_init1?

In D50761#1158798, @brooks wrote:

Why don't all the syscalls.master have an inotify_init1?

They do. For amd64 it's syscall 294, already has the right definition so not modified in this diff. On arm64 it's syscall 26.

In D50761#1158810, @markj wrote:

In D50761#1158798, @brooks wrote:

Why don't all the syscalls.master have an inotify_init1?

They do. For amd64 it's syscall 294, already has the right definition so not modified in this diff. On arm64 it's syscall 26.

Ah, that makes sense.

brooks accepted this revision.Jun 10 2025, 7:30 PM

This revision is now accepted and ready to land.Jun 10 2025, 7:30 PM

Closed by commit rG3965de642c29: linux: Add inotify support (authored by markj). · Explain WhyFri, Jul 4, 2:57 PM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rG3965de642c29: linux: Add inotify support.

Revision Contents
Changeset List

Path

Size

sys/

amd64/

linux/

syscalls.master

11 lines

linux32/

syscalls.master

15 lines

arm64/

linux/

syscalls.master

11 lines

compat/

linux/

4 lines

32 lines

121 lines

i386/

linux/

syscalls.master

15 lines

kern/

4 lines

sys/

4 lines

x86/

linux/

linux_dummy_x86.c

2 lines

Diff 157948

sys/amd64/linux/syscalls.master

Loading...

sys/amd64/linux32/syscalls.master

Loading...

sys/arm64/linux/syscalls.master

Loading...

sys/compat/linux/linux_dummy.c

Loading...

sys/compat/linux/linux_file.h

Loading...

sys/compat/linux/linux_file.c

Loading...

sys/i386/linux/syscalls.master

Loading...

sys/kern/vfs_inotify.c

Loading...

sys/sys/inotify.h

Loading...

sys/x86/linux/linux_dummy_x86.c

Loading...