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)
Tue, Nov 5, 7:21 PM
Unknown Object (File)
Sep 27 2024, 4:04 PM
Unknown Object (File)
Sep 24 2024, 6:38 PM
Unknown Object (File)
Sep 23 2024, 5:18 PM
Unknown Object (File)
Sep 23 2024, 5:12 PM
Unknown Object (File)
Sep 23 2024, 1:16 AM
Unknown Object (File)
Sep 22 2024, 6:11 AM
Unknown Object (File)
Sep 15 2024, 5:51 PM
View All 28 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...