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

[Jonathan Corbet]

Shubham Chakraborty <chakrabortyshubham66@xxxxxxxxx> 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@xxxxxxxxxxxxx>
> Signed-off-by: Shubham Chakraborty <chakrabortyshubham66@xxxxxxxxx>
> ---
>  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.

Thanks,

jon