Differential D56982

linux: Fix some problems with header pollution
ClosedPublic
Actions

Authored by markj on May 13 2026, 12:41 AM.

Tags

None

Referenced Files

	F171197792: D56982.diff
	Wed, Sep 9, 11:00 AM

	Unknown Object (File)
	Sat, Sep 5, 2:55 PM

	Unknown Object (File)
	Fri, Sep 4, 3:58 PM

	Unknown Object (File)
	Fri, Sep 4, 3:25 AM

	Unknown Object (File)
	Sun, Aug 23, 10:28 AM

	Unknown Object (File)
	Thu, Aug 13, 12:07 AM

	Unknown Object (File)
	Thu, Aug 13, 12:06 AM

	Unknown Object (File)
	Wed, Aug 12, 7:27 PM

Subscribers

Details

Reviewers

kib
dchagin
olce
andrew
manu
emaste
imp

Commits

rG7f8d45bb5f66: linux: Fix some problems with header pollution
rGb99954433d42: linux: Fix some problems with header pollution

Summary

Avoid including sys/proc.h in linux_vdso_gtod.c. It's not needed, but the implicit inclusion of sys/param.h via sys/ucred.h->bsm/audit.h was bringing in some required definitions.
Include a couple of required headers: sys/time.h (for struct bintime), and limits.h (for INT_MAX).
Move some helpers from linux.h, which depend on sys/param.h for NODEV, to the one CU where they're actually used.

No functional change intended.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 73063
Build 69946: arc lint + arc unit

Event Timeline

markj created this revision.May 13 2026, 12:41 AM

Herald added a reviewer: andrew. · View Herald TranscriptMay 13 2026, 12:41 AM

Herald added a reviewer: andrew. · View Herald Transcript

Herald added a reviewer: manu. · View Herald Transcript

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

markj requested review of this revision.May 13 2026, 12:41 AM

Harbormaster completed remote builds in B73063: Diff 177723.May 13 2026, 12:41 AM

kib accepted this revision.May 13 2026, 12:51 AM

This revision is now accepted and ready to land.May 13 2026, 12:51 AM

emaste accepted this revision.May 13 2026, 12:54 AM

Does this let us take anything out of badfiles.inc?

In D56982#1305794, @imp wrote:

Does this let us take anything out of badfiles.inc?

No, this is a step towards landing D56993, which fixes a problem that caused me a bunch of pain a while ago.

Closed by commit rGb99954433d42: linux: Fix some problems with header pollution (authored by markj). · Explain WhyMay 15 2026, 1:11 AM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rGb99954433d42: linux: Fix some problems with header pollution.

markj added a commit: rG7f8d45bb5f66: linux: Fix some problems with header pollution.May 29 2026, 10:10 PM

Revision Contents
Changeset List

Path

Size

sys/

amd64/

linux/

linux_vdso_gtod.c

4 lines

linux32/

linux32_vdso_gtod.c

4 lines

arm64/

linux/

linux_vdso_gtod.c

4 lines

compat/

linux/

50 lines

46 lines

i386/

linux/

linux_vdso_gtod.c

4 lines

Diff 177723

sys/amd64/linux/linux_vdso_gtod.c

Loading...

sys/amd64/linux32/linux32_vdso_gtod.c

Loading...

sys/arm64/linux/linux_vdso_gtod.c

Loading...

sys/compat/linux/linux.h

Loading...

sys/compat/linux/linux_stats.c

Loading...

sys/i386/linux/linux_vdso_gtod.c

Loading...