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
F131486012: D27770.id81273.diff
Wed, Oct 8, 12:46 PM
Unknown Object (File)
Mon, Oct 6, 11:00 PM
Unknown Object (File)
Tue, Sep 30, 5:35 AM
Unknown Object (File)
Sun, Sep 28, 2:25 PM
Unknown Object (File)
Fri, Sep 26, 3:15 PM
Unknown Object (File)
Thu, Sep 18, 1:30 PM
Unknown Object (File)
Thu, Sep 18, 1:25 PM
Unknown Object (File)
Thu, Sep 18, 1:17 PM
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
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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
129–130

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
129–130

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/
64 lines
7 lines
33 lines

Diff 81273

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...