ClosedPublic
Actions

Authored by jfree on Feb 16 2023, 5:45 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Sep 29, 5:27 AM

	Unknown Object (File)
	Mon, Sep 22, 1:01 PM

	Unknown Object (File)
	Sep 15 2025, 8:36 PM

	Unknown Object (File)
	Sep 6 2025, 4:18 AM

	Unknown Object (File)
	Aug 30 2025, 12:07 AM

	Unknown Object (File)
	Aug 26 2025, 4:53 PM

	Unknown Object (File)
	Jul 23 2025, 1:27 AM

	Unknown Object (File)
	Jul 10 2025, 9:49 PM

View All 71 Files

Subscribers

imp

Details

Reviewers

brooks

Summary

The freebsd32.h header uses an fsid_t type in struct ostatfs32. Include sys/mount.h to avoid potential compilation errors in situations where it is not already included.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

jfree created this revision.Feb 16 2023, 5:45 AM

Herald added a subscriber: imp. · View Herald TranscriptFeb 16 2023, 5:45 AM

jfree requested review of this revision.Feb 16 2023, 5:45 AM

jfree added a parent revision: D38459: timerfd: Add native support for Linux's timerfd.

Seems fine.

I'm not entirely convinced struct ostatfs32 shouldn't be defined directly in sys/compat/freebsd32/freebsd32_misc.c, but that's an argument for another day.

This revision is now accepted and ready to land.Feb 16 2023, 6:51 PM

This differential is superseded by this review:
https://reviews.freebsd.org/D40177
which covers some more include self-containment issues in aarch64.

jfree closed this revision.May 20 2023, 4:37 AM

jfree removed a parent revision: D38459: timerfd: Add native support for Linux's timerfd.May 21 2023, 2:32 AM