Page MenuHomeFreeBSD

fusefs: improve warnings about buggy FUSE servers
ClosedPublic

Authored by asomers on Jun 15 2021, 9:41 PM.
Tags
None
Referenced Files
Unknown Object (File)
Jan 15 2024, 2:42 PM
Unknown Object (File)
Dec 14 2023, 10:15 PM
Unknown Object (File)
Nov 9 2023, 7:38 PM
Unknown Object (File)
Nov 5 2023, 4:19 AM
Unknown Object (File)
Oct 13 2023, 3:29 AM
Unknown Object (File)
Sep 2 2023, 9:21 AM
Unknown Object (File)
Jun 29 2023, 2:58 PM
Unknown Object (File)
Jun 19 2023, 8:23 PM
Subscribers

Details

Summary

The fusefs driver will print warning messages about FUSE servers that
commit protocol violations. Previously it would print those warnings on
every violation, but that could spam the console. Now it will print
each warning no more than once per lifetime of the mount. There is also
now a dtrace probe for each violation.

MFC after: 2 weeks
Sponsored by: Axcient

Diff Detail

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

Event Timeline

This revision is now accepted and ready to land.Jun 16 2021, 6:01 PM

I like this idea.

I'd like us to apply the same concept to the Linuxulator, for similar types of warnings (although in Linuxulator the spammy warnings are for unimplemented kernel functionality).

sys/fs/fuse/fuse_internal.c
917–918

This change seems to be un-style(9)ing

sys/fs/fuse/fuse_ipc.h
236–240

might we end up with enough distinct warnings to run out of flag bits?
(just add a second flags field if/when that happens?)

asomers added inline comments.
sys/fs/fuse/fuse_internal.c
917–918

Ok, I'll revert this line.

sys/fs/fuse/fuse_ipc.h
236–240

Yep, but the flags field never leaves this module. So we can add more fields or reassign bits at will.

emaste added inline comments.
sys/fs/fuse/fuse_ipc.h
236–240

Ok, fair enough.

This revision was automatically updated to reflect the committed changes.
asomers marked 2 inline comments as done.