Page MenuHomeFreeBSD
Differential D34113

pseudofs: Make the sbuf usable
AbandonedPublic
Actions

Authored by manu on Jan 31 2022, 1:49 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 14, 2:02 AM
Unknown Object (File)
Oct 16 2025, 4:14 PM
Unknown Object (File)
Oct 11 2025, 8:30 PM
Unknown Object (File)
Oct 11 2025, 8:30 PM
Unknown Object (File)
Oct 11 2025, 11:29 AM
Unknown Object (File)
Sep 19 2025, 8:39 AM
Unknown Object (File)
Sep 19 2025, 7:37 AM
Unknown Object (File)
Aug 5 2025, 6:22 AM
View All 53 Files
Subscribers
des
imp

Details

Reviewers
des
Summary

Call sbuf_finish for pfs_write to underlying pseudofs can actually use
the buffer.

MFC after: 1 week
Sponsored by: Beckhoff Automation GmbH & Co. KG

Diff Detail

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

Event Timeline

manu created this revision.Jan 31 2022, 1:49 PM
Herald added a subscriber: imp. · View Herald TranscriptJan 31 2022, 1:49 PM
manu requested review of this revision.Jan 31 2022, 1:49 PM
Harbormaster completed remote builds in B44186: Diff 102155.Jan 31 2022, 1:49 PM
des added a subscriber: des.Jan 31 2022, 2:12 PM

This is incorrect. The fill function is expected to finish the buffer. See for instance procfs_doosrel() in sys/fs/procfs/procfs_orsel.c.

des added a reviewer: des.Jan 31 2022, 2:14 PM
manu abandoned this revision.Jan 31 2022, 2:21 PM
In D34113#771377, @des wrote:

This is incorrect. The fill function is expected to finish the buffer. See for instance procfs_doosrel() in sys/fs/procfs/procfs_orsel.c.

Indeed, I've missed that.
So no need for this and I've added the sbuf_finish in D34114

Revision Contents
Changeset List

PathSize
sys/
fs/
pseudofs/
1 line

Diff 102155

View Options

sys/fs/pseudofs/pseudofs_vnops.c

Loading...