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)
Sat, May 25, 2:16 AM
Unknown Object (File)
Thu, May 23, 9:09 PM
Unknown Object (File)
May 18 2024, 8:30 PM
Unknown Object (File)
May 5 2024, 12:33 PM
Unknown Object (File)
May 2 2024, 12:56 PM
Unknown Object (File)
Apr 18 2024, 2:30 PM
Unknown Object (File)
Apr 18 2024, 11:10 AM
Unknown Object (File)
Apr 18 2024, 10:11 AM
View All 30 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...