Re: [PATCH v2] docs: sysctl: add documentation for crypto and debug

[Jonathan Corbet <corbet@lwn.net>]

Jonathan Corbet <corbet@lwn.net> writes:

> Shubham Chakraborty <chakrabortyshubham66@gmail.com> writes:
>
>> Add documentation for the /proc/sys/crypto and /proc/sys/debug
>> directories in the admin-guide. This includes tunables for FIPS
>> mode (fips_enabled, fips_name, fips_version), exception-trace,
>> and kprobes-optimization.
>>
>> The documentation is based on source code analysis and addresses
>> stylistic feedback to keep it direct and concise.
>>
>> Tested-by: Randy Dunlap <rdunlap@infradead.org>
>> Signed-off-by: Shubham Chakraborty <chakrabortyshubham66@gmail.com>
>> ---
>>  Documentation/admin-guide/sysctl/crypto.rst | 47 +++++++++++++++++++
>>  Documentation/admin-guide/sysctl/debug.rst  | 52 +++++++++++++++++++++
>>  Documentation/admin-guide/sysctl/index.rst  |  6 ++-
>>  3 files changed, 103 insertions(+), 2 deletions(-)
>>  create mode 100644 Documentation/admin-guide/sysctl/crypto.rst
>>  create mode 100644 Documentation/admin-guide/sysctl/debug.rst
>
> A few other comments:
>
> - This is the second v2 you have posted, which can only lead to
>   confusion.  If you only change was to add Randy's Tested-by, there is
>   no need to post a new version just for that; it will be picked up.
>
> - When posting a new version, it is good to give a quick summary of the
>   changes from the previous version, below the "---" line.
>
> - Please slow down a bit, there is no need to post multiple times in one
>   day.  Give people time to look at your work.
>
> For me, there is no need for you to post a new version unless there are
> other comments that need to be addressed.

Now applied.

Thanks,

jon