Differential D33957

fusefs: Address -Wunused-but-set-variable warnings
ClosedPublic
Actions

Authored by markj on Jan 19 2022, 10:49 PM.

Tags

None

Referenced Files

	F154348150: D33957.id101664.diff
	Tue, Apr 28, 12:28 AM

	Unknown Object (File)
	Wed, Apr 15, 2:33 AM

	Unknown Object (File)
	Wed, Apr 15, 2:28 AM

	Unknown Object (File)
	Tue, Apr 7, 7:04 AM

	Unknown Object (File)
	Mon, Mar 30, 2:27 AM

	Unknown Object (File)
	Sun, Mar 29, 7:09 PM

	Unknown Object (File)
	Mar 28 2026, 7:51 AM

	Unknown Object (File)
	Mar 27 2026, 6:51 AM

Subscribers

Details

Reviewers

Commits

rG3d8562348c9c: fusefs: Address -Wunused-but-set-variable warnings

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

markj created this revision.Jan 19 2022, 10:49 PM

Herald added a subscriber: imp. · View Herald TranscriptJan 19 2022, 10:49 PM

markj requested review of this revision.Jan 19 2022, 10:49 PM

Harbormaster completed remote builds in B43997: Diff 101661.Jan 19 2022, 10:50 PM

markj added inline comments.Jan 19 2022, 10:51 PM

sys/fs/fuse/fuse_io.c
525	The need for this went away with commit 84879e46c2bea842c3ad7b1b8b26aeb81946f5a8.
sys/fs/fuse/fuse_node.c
454	I'm not sure if this is an actual bug, though invalidating more than necessary probably isn't a correctness issue.

asomers requested changes to this revision.Jan 19 2022, 11:15 PM

asomers added inline comments.

sys/fs/fuse/fuse_node.c
454	It's deliberate. If we get here, that means the FUSE server changed the file's size even though we had it cached. If it changed the size, then it probably changed other stuff too. That's why it's best to invalidate the cache for the whole file.

This revision now requires changes to proceed.Jan 19 2022, 11:15 PM

Drop left_lbn.

Harbormaster completed remote builds in B43999: Diff 101664.Jan 19 2022, 11:17 PM

asomers accepted this revision.Jan 20 2022, 12:07 AM

This revision is now accepted and ready to land.Jan 20 2022, 12:07 AM

Closed by commit rG3d8562348c9c: fusefs: Address -Wunused-but-set-variable warnings (authored by markj). · Explain WhyJan 20 2022, 2:59 PM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rG3d8562348c9c: fusefs: Address -Wunused-but-set-variable warnings.

Revision Contents
Changeset List

Path

Size

sys/

fs/

fuse/

fuse_internal.c

8 lines

6 lines

3 lines

7 lines

Diff 101683

sys/fs/fuse/fuse_internal.c

Loading...

sys/fs/fuse/fuse_io.c

Loading...

sys/fs/fuse/fuse_node.c

Loading...

sys/fs/fuse/fuse_vnops.c

Loading...