Differential D34753

fusefs: fix two bugs regarding VOP_RECLAIM of the root inode
ClosedPublic
Actions

Authored by asomers on Apr 2 2022, 7:43 PM.

Tags

None

Referenced Files

	F133014518: D34753.id104511.diff
	Wed, Oct 22, 2:06 AM

	Unknown Object (File)
	Sun, Oct 19, 7:02 PM

	Unknown Object (File)
	Thu, Oct 16, 1:22 PM

	Unknown Object (File)
	Thu, Oct 16, 1:17 PM

	Unknown Object (File)
	Mon, Oct 13, 7:07 PM

	Unknown Object (File)
	Mon, Oct 13, 6:11 PM

	Unknown Object (File)
	Mon, Oct 13, 2:26 PM

	Unknown Object (File)
	Mon, Oct 13, 2:26 PM

Subscribers

Details

Reviewers

emaste
pfg

Commits

rG4d5fb17274aa: fusefs: fix two bugs regarding VOP_RECLAIM of the root inode
rG507b75ced9fd: fusefs: fix two bugs regarding VOP_RECLAIM of the root inode
rG32273253667b: fusefs: fix two bugs regarding VOP_RECLAIM of the root inode

Summary

We never send FUSE_LOOKUP for the root inode, since its inode number is hard-coded to 1. Therefore, we should not send FUSE_FORGET for it, lest the server see its lookup count fall below 0.

During VOP_RECLAIM, if we are reclaiming the root inode, we must clear the file system's vroot pointer. Otherwise it will be left pointing at a reclaimed vnode, which will cause future VOP_LOOKUP operations to fail. Previously we only cleared that pointer during VFS_UMOUNT. I don't know of any real-world way to trigger this bug.

MFC after: 2 weeks

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 44972
Build 41860: arc lint + arc unit

Event Timeline

asomers created this revision.Apr 2 2022, 7:43 PM

Herald added a subscriber: imp. · View Herald TranscriptApr 2 2022, 7:43 PM

asomers requested review of this revision.Apr 2 2022, 7:43 PM

Harbormaster completed remote builds in B44967: Diff 104511.Apr 2 2022, 7:43 PM

Release root inode even if the server is already detached

Harbormaster completed remote builds in B44972: Diff 104521.Apr 2 2022, 9:34 PM

asomers added reviewers: emaste, pfg.Apr 2 2022, 9:34 PM

LGTM

This revision is now accepted and ready to land.Apr 4 2022, 3:29 AM

Closed by commit rG32273253667b: fusefs: fix two bugs regarding VOP_RECLAIM of the root inode (authored by asomers). · Explain WhyApr 6 2022, 10:18 PM

This revision was automatically updated to reflect the committed changes.

asomers added a commit: rG32273253667b: fusefs: fix two bugs regarding VOP_RECLAIM of the root inode.

asomers added a commit: rG507b75ced9fd: fusefs: fix two bugs regarding VOP_RECLAIM of the root inode.Apr 29 2022, 8:01 PM

asomers added a commit: rG4d5fb17274aa: fusefs: fix two bugs regarding VOP_RECLAIM of the root inode.Aug 20 2022, 12:49 AM

Revision Contents
Changeset List

Path

Size

sys/

fs/

fuse/

19 lines

tests/

sys/

fs/

fusefs/

1 line

dev_fuse_poll.cc

1 line

33 lines

2 lines

Diff 104521

sys/fs/fuse/fuse_vnops.c

Loading...

tests/sys/fs/fusefs/destroy.cc

Loading...

tests/sys/fs/fusefs/dev_fuse_poll.cc

Loading...

tests/sys/fs/fusefs/forget.cc

Loading...

tests/sys/fs/fusefs/lookup.cc

Loading...