Page MenuHomeFreeBSD

Abbreviate softdep lock names.
ClosedPublic

Authored by markj on Oct 15 2019, 4:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Dec 3, 4:13 PM
Unknown Object (File)
Thu, Nov 20, 9:50 PM
Unknown Object (File)
Thu, Nov 20, 9:50 PM
Unknown Object (File)
Thu, Nov 20, 9:50 PM
Unknown Object (File)
Thu, Nov 20, 9:41 PM
Unknown Object (File)
Sun, Nov 16, 3:19 AM
Unknown Object (File)
Sat, Nov 15, 12:39 PM
Unknown Object (File)
Sat, Nov 15, 8:13 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

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 27050
Build 25337: arc lint + arc unit

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.