Page MenuHomeFreeBSD

[unix(4) cleanup 3/8] Rename unp_pcb_lock2().
ClosedPublic

Authored by markj on Sep 2 2020, 4:55 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jan 24, 6:30 PM
Unknown Object (File)
Thu, Jan 23, 6:00 AM
Unknown Object (File)
Tue, Jan 21, 9:41 PM
Unknown Object (File)
Thu, Jan 9, 10:50 PM
Unknown Object (File)
Dec 8 2024, 1:52 AM
Unknown Object (File)
Dec 5 2024, 10:48 AM
Unknown Object (File)
Nov 27 2024, 2:20 AM
Unknown Object (File)
Nov 1 2024, 5:10 AM
Subscribers

Details

Summary

I think unp_pcb_lock_pair() is a better name. Also make it handle the
case where the two sockets are the same instead of forcing callers to do
it. No functional change intended.

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 33335
Build 30646: arc lint + arc unit

Event Timeline

markj requested review of this revision.Sep 2 2020, 4:55 PM
markj created this revision.
kevans added inline comments.
sys/kern/uipc_usrreq.c
752–753

Might not be a bad idea to (perhaps later) introduce a unp_pcb_unlock_pair for these cases, if for nothing but the symmetry of lock_pair/unlock_pair to make it easier to verify correctness.

sys/kern/uipc_usrreq.c
752–753

Yes, I think it's a good idea.

Add unp_pcb_unlock_pair().

This revision is now accepted and ready to land.Sep 2 2020, 6:15 PM
This revision was automatically updated to reflect the committed changes.