Page MenuHomeFreeBSD
Differential D2937

Use the presence of a susp_clean method to test for suspendability.
ClosedPublic
Actions

Authored by markj on Jun 29 2015, 2:29 AM.
Tags
None
Referenced Files
F103255369: D2937.diff
Fri, Nov 22, 4:54 PM
Unknown Object (File)
Tue, Nov 19, 7:10 PM
Unknown Object (File)
Tue, Nov 19, 7:07 PM
Unknown Object (File)
Fri, Nov 15, 10:10 AM
Unknown Object (File)
Wed, Nov 13, 3:47 AM
Unknown Object (File)
Tue, Nov 5, 12:17 PM
Unknown Object (File)
Thu, Oct 31, 6:24 AM
Unknown Object (File)
Mon, Oct 28, 4:07 AM
View All 66 Files
Subscribers
None

Details

Reviewers
kib
mjg
Commits
rS285182: Check suspendability on the mountpoint returned by VOP_GETWRITEMOUNT.
Summary

This change removes the MNTK_SUSPENDABLE flag and instead tests the mount
point returned by VOP_GETWRITEMOUNT. This removes the need for
passthrough filesystems like nullfs and unionfs to inherit mount flags
from lower layers.

Test Plan

Verified that vn_start_write_locked() was or was not being called under the same conditions as before, using ufs, tmpfs, nullfs on tmpfs, and devfs.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

markj updated this revision to Diff 6531.Jun 29 2015, 2:29 AM
markj retitled this revision from to Use the presence of a susp_clean method to test for suspendability..
markj edited the test plan for this revision. (Show Details)
markj updated this object.
markj edited the test plan for this revision. (Show Details)Jun 29 2015, 2:46 AM
markj added reviewers: kib, mjg.
mjg accepted this revision.Jun 29 2015, 3:49 AM
mjg edited edge metadata.

I would possibly rename vn_suspendable_mp to vn_suspendable, but looks good enough to me as it is.

This revision is now accepted and ready to land.Jun 29 2015, 3:49 AM
kib accepted this revision.Jun 29 2015, 5:41 AM
kib edited edge metadata.
kib added inline comments.
sys/fs/nullfs/null_vfsops.c
202 ↗(On Diff #6531)

This is weird reformatting.

markj added a comment.Jun 29 2015, 6:41 PM
In D2937#57266, @mjg wrote:

I would possibly rename vn_suspendable_mp to vn_suspendable, but looks good enough to me as it is.

I'll fix that in the commit, thanks.

sys/fs/nullfs/null_vfsops.c
202 ↗(On Diff #6531)

It's a bit easier to read when the RHS is on a single line, but I agree that it looks weird. I'll fix it before committing.

Closed by commit rS285182: Check suspendability on the mountpoint returned by VOP_GETWRITEMOUNT. (authored by markj). · Explain WhyJul 5 2015, 10:37 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
fs/
nullfs/
2 lines
tmpfs/
12 lines
unionfs/
3 lines
kern/
48 lines
sys/
2 lines
ufs/
ffs/
3 lines

Diff 6713

View Options

head/sys/fs/nullfs/null_vfsops.c

Loading...
View Options

head/sys/fs/tmpfs/tmpfs_vfsops.c

Loading...
View Options

head/sys/fs/unionfs/union_vfsops.c

Loading...
View Options

head/sys/kern/vfs_vnops.c

Loading...
View Options

head/sys/sys/mount.h

Loading...
View Options

head/sys/ufs/ffs/ffs_vfsops.c

Loading...