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)
Thu, Apr 25, 10:19 AM
Unknown Object (File)
Wed, Apr 17, 4:50 AM
Unknown Object (File)
Tue, Apr 16, 1:22 AM
Unknown Object (File)
Apr 13 2024, 4:23 AM
Unknown Object (File)
Apr 10 2024, 8:14 PM
Unknown Object (File)
Mar 19 2024, 10:05 PM
Unknown Object (File)
Mar 19 2024, 2:19 PM
Unknown Object (File)
Mar 7 2024, 3:28 PM
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

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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
745 ↗(On Diff #76542)

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
745 ↗(On Diff #76542)

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.