Page MenuHomeFreeBSD

Remove "struct ucred*" argument from vtruncbuf
ClosedPublic

Authored by asomers on May 23 2019, 2:03 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Jan 16, 4:19 AM
Unknown Object (File)
Sun, Jan 12, 8:39 AM
Unknown Object (File)
Fri, Jan 3, 9:54 AM
Unknown Object (File)
Dec 10 2024, 7:19 PM
Unknown Object (File)
Dec 3 2024, 3:44 AM
Unknown Object (File)
Nov 16 2024, 6:00 PM
Unknown Object (File)
Nov 13 2024, 10:22 PM
Unknown Object (File)
Nov 10 2024, 10:28 AM
Subscribers

Details

Summary

Remove "struct ucred*" argument from vtruncbuf

vtruncbuf takes a "struct ucred*" argument. AFAICT, it's been unused ever
since that function was first added in r34611. Remove it.

Diff Detail

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

Event Timeline

LGTM, thanks!

sys/fs/fuse/fuse_io.c
392

I'd structure this as two separate commits, vtruncbuf first, fuse second; or at least update the commit message to reflect that FUSE was also changed in a similar fashion.

466

Might as well fix the line wrapping whitespace while you're here? Not a big deal. Ditto below.

sys/fs/nfsclient/nfs_clbio.c
1835

Ditto the comment in FUSE: I'd structure this as another independent commit, but if not, please mention it in the commit message as well.

This revision is now accepted and ready to land.May 23 2019, 4:24 PM
sys/fs/nfsclient/nfs_clbio.c
1835

Ok, I'll mention it in the commit message. I don't want to separate it into two commits, because that would create more unused arguments in ncl_meta_setsize and fuse_vnode_setsize.

sys/fs/nfsclient/nfs_clbio.c
1835

Briefly, yes, but then the followup commit(s) would remove those. Doing it all in one is totally reasonable, too.

This revision was automatically updated to reflect the committed changes.