Page MenuHomeFreeBSD

Abbreviate softdep lock names.
ClosedPublic

Authored by markj on Oct 15 2019, 4:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Oct 16, 11:36 AM
Unknown Object (File)
Tue, Oct 14, 2:36 PM
Unknown Object (File)
Tue, Oct 14, 2:36 PM
Unknown Object (File)
Tue, Oct 14, 2:36 PM
Unknown Object (File)
Tue, Oct 14, 12:57 AM
Unknown Object (File)
Sat, Oct 11, 2:36 AM
Unknown Object (File)
Sat, Sep 27, 12:57 AM
Unknown Object (File)
Sep 19 2025, 3:31 AM
Subscribers

Details

Summary

The softdep lock names are unusually long and tend to stick out in lock
profiling reports. Abbreviate them to keep consistent with our
conventional style.

Test Plan

This is how it looks without the change:

Count indv cuml rcnt     nsec Lock                   Caller                                                                                   
-------------------------------------------------------------------------------                                                               
 8676  31%  31% 0.00    10307 pmap pv list           _rw_wlock_cookie+0xbc     
 7961  28%  59% 0.00    54853 vm object              _rw_wlock_cookie+0xbc                                                                    
 6112  22%  80% 0.00    20783 tcpinp                 _rw_wlock_cookie+0xbc     
 5519  19% 100% 0.00     6301 Per-Filesystem Softdep Lock _rw_wlock_cookie+0xbc                                                               
   73   0% 100% 0.00     2857 bufobj interlock       _rw_wlock_cookie+0xbc                                                                    
    2   0% 100% 0.00      300 tidhash                _rw_wlock_cookie+0xbc                                                                    
-------------------------------------------------------------------------------

Diff Detail

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

Event Timeline

markj edited the test plan for this revision. (Show Details)
markj added a reviewer: mckusick.

This request is entirely reasonable.

This revision is now accepted and ready to land.Oct 17 2019, 11:31 PM
This revision was automatically updated to reflect the committed changes.