Re: CVS commit: src

[Ryota Ozaki <ozaki-r%netbsd.org@localhost>]

On Sat, Mar 17, 2018 at 2:04 AM, matthew green <mrg%eterna.com.au@localhost> wrote:
> "Ryota Ozaki" writes:
>> Module Name:  src
>> Committed By: ozaki-r
>> Date:         Fri Mar 16 04:37:55 UTC 2018
>>
>> Modified Files:
>>       src/share/man/man4: ddb.4
>>       src/sys/ddb: db_command.c
>>       src/sys/sys: lockdebug.h
>>
>> Log Message:
>> Add a new command, show lockstat, which shows statistics of locks
>>
>> Currently the command shows the number of allocated locks.
>>
>> The command is useful only if LOCKDEBUG is enabled.
>
> would you mind renaming this?  "lockstat" makes me think of
> lockstat(8), not what this does.  "lockinfo"?  "lockstate"?

I'm not sure that we have to worry about the similarity between
a userland command and a ddb command.

Well, how about "lockstats"?  It's different from lockstat(8) and
still appropriate (because stats means statistics).

  ozaki-r