Page MenuHomeFreeBSD
Differential D35880

Convert runtime param checks to KASSERTs for fo_fspacectl
ClosedPublic
Actions

Authored by khng on Jul 22 2022, 6:11 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sep 23 2024, 5:41 AM
Unknown Object (File)
Sep 21 2024, 3:57 PM
Unknown Object (File)
Sep 21 2024, 3:57 PM
Unknown Object (File)
Sep 21 2024, 3:57 PM
Unknown Object (File)
Sep 21 2024, 3:47 PM
Unknown Object (File)
Sep 20 2024, 6:41 AM
Unknown Object (File)
Sep 17 2024, 2:39 AM
Unknown Object (File)
Sep 14 2024, 4:18 AM
View All 37 Files
Subscribers
imp

Details

Reviewers
kib
markj
Commits
rG8c9aa94b42bf: Convert runtime param checks to KASSERTs for fo_fspacectl

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

khng created this revision.Jul 22 2022, 6:11 PM
Herald added a subscriber: imp. · View Herald TranscriptJul 22 2022, 6:11 PM
khng requested review of this revision.Jul 22 2022, 6:11 PM
Harbormaster completed remote builds in B46512: Diff 108415.Jul 22 2022, 6:11 PM
markj accepted this revision.Jul 22 2022, 11:27 PM
markj added inline comments.
sys/kern/vfs_vnops.c
3611

There is also SPACECTL_F_SUPPORTED.

3613

IMHO, it's nicer to be consistent and use the same set of assertions in both functions. (The third one uses local vars above, but not here.)

This revision is now accepted and ready to land.Jul 22 2022, 11:27 PM
Closed by commit rG8c9aa94b42bf: Convert runtime param checks to KASSERTs for fo_fspacectl (authored by khng). · Explain WhyJul 23 2022, 7:17 PM
This revision was automatically updated to reflect the committed changes.
khng added a commit: rG8c9aa94b42bf: Convert runtime param checks to KASSERTs for fo_fspacectl.

Revision Contents
Changeset List

PathSize
sys/
kern/
10 lines
8 lines

Diff 108439

View Options

sys/kern/uipc_shm.c

Loading...
View Options

sys/kern/vfs_vnops.c

Loading...