Page MenuHomeFreeBSD
Differential D22874

make tmpfs.h friendly with _KERNEL undefined
ClosedPublic
Actions

Authored by dougm on Dec 19 2019, 4:16 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 2, 11:07 AM
Unknown Object (File)
Nov 26 2025, 7:17 AM
Unknown Object (File)
Nov 26 2025, 6:03 AM
Unknown Object (File)
Nov 23 2025, 7:48 PM
Unknown Object (File)
Nov 23 2025, 10:25 AM
Unknown Object (File)
Nov 22 2025, 2:53 PM
Unknown Object (File)
Nov 21 2025, 12:17 AM
Unknown Object (File)
Nov 18 2025, 1:43 AM
View All Files
Subscribers
imp
ler

Details

Reviewers
alc
kib
Commits
rS355913: Including <sys/tmpfs.h> into non-kernel software leads to a
Summary

Including <sys/tmpfs.h> into non-kernel software leads to a compilation error because, without _KERNEL defined, the macro TMPFS_VALIDATE_DIR is invoked, but never defined. User-level software that includes sys/tmpfs.h must define _KERNEL to make the definition of TMPFS_VALIDATE_DIR visible.

This change puts all the inline functions that, directly or indirectly, invoke MPASS into the scope of the _KERNEL block, allowing many user-space includers of <sys/tmpfs.h> to stop defining _KERNEL.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

dougm created this revision.Dec 19 2019, 4:16 AM
kib added inline comments.Dec 19 2019, 2:26 PM
sys/fs/tmpfs/tmpfs.h
407

The practice of including headers from headers is discouraged. It should be handled by consumer.

538

This is fine.

dougm updated this revision to Diff 65813.Dec 19 2019, 4:20 PM
dougm edited the summary of this revision. (Show Details)

Remove #include <sys/lock.h> from tmpfs.h.

Leave #include <sys/queue.h> and #include <sys/tree.h> in tmpfs.h.

dougm marked 2 inline comments as done.Dec 19 2019, 4:21 PM
kib accepted this revision.Dec 19 2019, 4:31 PM
This revision is now accepted and ready to land.Dec 19 2019, 4:31 PM
alc accepted this revision.Dec 19 2019, 4:36 PM
Closed by commit rS355913: Including <sys/tmpfs.h> into non-kernel software leads to a (authored by dougm). · Explain WhyDec 19 2019, 4:40 PM
This revision was automatically updated to reflect the committed changes.
dougm added a commit: rS355913: Including <sys/tmpfs.h> into non-kernel software leads to a.
Herald added a subscriber: imp. · View Herald TranscriptDec 19 2019, 4:40 PM

Revision Contents
Changeset List

PathSize
sys/
fs/
tmpfs/
4 lines

Diff 65804

View Options

sys/fs/tmpfs/tmpfs.h

Loading...