Page MenuHomeFreeBSD
Differential D27770

fusefs: delete some dead code
ClosedPublic
Actions

Authored by asomers on Dec 24 2020, 8:26 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jul 10, 3:08 AM
Unknown Object (File)
Fri, Jun 26, 1:24 PM
Unknown Object (File)
Thu, Jun 25, 7:06 PM
Unknown Object (File)
Jun 16 2026, 10:24 PM
Unknown Object (File)
Jun 16 2026, 1:53 AM
Unknown Object (File)
Jun 12 2026, 4:04 PM
Unknown Object (File)
Jun 5 2026, 8:01 PM
Unknown Object (File)
Jun 2 2026, 9:15 AM
View All Files
Subscribers
imp

Details

Reviewers
cem
emaste
Commits
rG43f57e98d032: fusefs: delete some dead code
rG4f4111d2c5ab: fusefs: delete some dead code
Summary

The original fusefs GSoC project seems to have envisioned exchanging two
types of messages with FUSE servers. Perhaps vectored and non-vectored?
But in practice only one type has ever been used. Delete the other type.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 35683
Build 32572: arc lint + arc unit

Event Timeline

asomers created this revision.Dec 24 2020, 8:26 PM
Herald added a subscriber: imp. · View Herald TranscriptDec 24 2020, 8:26 PM
asomers requested review of this revision.Dec 24 2020, 8:26 PM
Harbormaster completed remote builds in B35683: Diff 81146.Dec 24 2020, 8:26 PM
cem accepted this revision.Dec 24 2020, 9:14 PM
cem added inline comments.
sys/fs/fuse/fuse_ipc.h
133

Are these bufsize and bufdata fields unused now?

This revision is now accepted and ready to land.Dec 24 2020, 9:14 PM
asomers added inline comments.Dec 24 2020, 9:37 PM
sys/fs/fuse/fuse_ipc.h
133

Uh yes, yes they are. Good catch.

asomers updated this revision to Diff 81155.Dec 24 2020, 9:37 PM
  • Delete more dead code
This revision now requires review to proceed.Dec 24 2020, 9:37 PM
Harbormaster completed remote builds in B35686: Diff 81155.Dec 24 2020, 9:37 PM
asomers updated this revision to Diff 81156.Dec 24 2020, 9:38 PM

Remove files that shouldn't have been part of this review

Harbormaster completed remote builds in B35687: Diff 81156.Dec 24 2020, 9:38 PM
cem accepted this revision.Dec 24 2020, 10:03 PM

Awesome, thanks!

This revision is now accepted and ready to land.Dec 24 2020, 10:03 PM
asomers mentioned this in D27769: fusefs: fix the tests for a wider range of maxphys.Dec 26 2020, 5:34 PM
Closed by commit rG4f4111d2c5ab: fusefs: delete some dead code (authored by asomers). · Explain WhyDec 28 2020, 7:06 PM
This revision was automatically updated to reflect the committed changes.
asomers added a commit: rG4f4111d2c5ab: fusefs: delete some dead code.
asomers added a commit: rG43f57e98d032: fusefs: delete some dead code.Jul 27 2021, 5:15 PM

Revision Contents
Changeset List

PathSize
sys/
fs/
fuse/
61 lines
2 lines
22 lines

Diff 81146

View Options

sys/fs/fuse/fuse_device.c

Loading...
View Options

sys/fs/fuse/fuse_ipc.h

Loading...
View Options

sys/fs/fuse/fuse_ipc.c

Loading...