linux-kernel - Re: [PATCH] Documentation: fix multiple typos found in the admin-guide subdirectory

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <6d35bc17-223f-6667-a6b5-aa0523e64bbc@infradead.org>
Date:   Thu, 3 Dec 2020 09:57:06 -0800
From:   Randy Dunlap <rdunlap@...radead.org>
To:     Andrew Klychkov <andrew.a.klychkov@...il.com>, corbet@....net
Cc:     linux-kernel@...r.kernel.org
Subject: Re: [PATCH] Documentation: fix multiple typos found in the
 admin-guide subdirectory

Hi--

On 12/3/20 12:20 AM, Andrew Klychkov wrote:
> Fix thirty seven typos in dm-integrity.rst, dm-raid.rst, dm-zoned.rst,
> verity.rst, writecache.rst, tsx_async_abort.rst, ldm.rst, md.rst,
> bttv.rst, dvb_references.rst, frontend-cardlist.rst, gspca-cardlist.rst,
> ipu3.rst, remote-controller.rst, mm/index.rst, numaperf.rst,
> userfaultfd.rst, module-signing.rst, imx-ddr.rst,
> intel-speed-select.rst, intel_pstate.rst, ramoops.rst, abi.rst,
> kernel.rst, vm.rst
> 
> Signed-off-by: Andrew Klychkov <andrew.a.klychkov@...il.com>
> ---
>  Documentation/admin-guide/device-mapper/dm-integrity.rst | 4 ++--
>  Documentation/admin-guide/device-mapper/dm-raid.rst      | 2 +-
>  Documentation/admin-guide/device-mapper/dm-zoned.rst     | 6 +++---
>  Documentation/admin-guide/device-mapper/verity.rst       | 2 +-
>  Documentation/admin-guide/device-mapper/writecache.rst   | 4 ++--
>  Documentation/admin-guide/hw-vuln/tsx_async_abort.rst    | 2 +-
>  Documentation/admin-guide/ldm.rst                        | 2 +-
>  Documentation/admin-guide/md.rst                         | 2 +-
>  Documentation/admin-guide/media/bttv.rst                 | 2 +-
>  Documentation/admin-guide/media/dvb_references.rst       | 2 +-
>  Documentation/admin-guide/media/frontend-cardlist.rst    | 4 ++--
>  Documentation/admin-guide/media/gspca-cardlist.rst       | 2 +-
>  Documentation/admin-guide/media/ipu3.rst                 | 6 +++---
>  Documentation/admin-guide/media/remote-controller.rst    | 2 +-
>  Documentation/admin-guide/mm/index.rst                   | 4 ++--
>  Documentation/admin-guide/mm/numaperf.rst                | 2 +-
>  Documentation/admin-guide/mm/userfaultfd.rst             | 2 +-
>  Documentation/admin-guide/module-signing.rst             | 4 ++--
>  Documentation/admin-guide/perf/imx-ddr.rst               | 2 +-
>  Documentation/admin-guide/pm/intel-speed-select.rst      | 4 ++--
>  Documentation/admin-guide/pm/intel_pstate.rst            | 6 +++---
>  Documentation/admin-guide/ramoops.rst                    | 2 +-
>  Documentation/admin-guide/sysctl/abi.rst                 | 2 +-
>  Documentation/admin-guide/sysctl/kernel.rst              | 2 +-
>  Documentation/admin-guide/sysctl/vm.rst                  | 2 +-
>  25 files changed, 37 insertions(+), 37 deletions(-)
> 

> diff --git a/Documentation/admin-guide/ldm.rst b/Documentation/admin-guide/ldm.rst
> index 12c5713..2cb4146 100644
> --- a/Documentation/admin-guide/ldm.rst
> +++ b/Documentation/admin-guide/ldm.rst
> @@ -19,7 +19,7 @@ longer any primary or extended partitions.  Normal MSDOS style partitions are
>  now known as Basic Disks.
>  
>  If you wish to use Spanned, Striped, Mirrored or RAID 5 Volumes, you must use
> -Dynamic Disks.  The journalling allows Windows to make changes to these
> +Dynamic Disks.  The journaling allows Windows to make changes to these

Recognized alternative spellings are OK...

>  partitions and filesystems without the need to reboot.
>  
>  Once the LDM driver has divided up the disk, you can use the MD driver to


> diff --git a/Documentation/admin-guide/module-signing.rst b/Documentation/admin-guide/module-signing.rst
> index f8b5841..219e286 100644
> --- a/Documentation/admin-guide/module-signing.rst
> +++ b/Documentation/admin-guide/module-signing.rst
> @@ -185,7 +185,7 @@ in a keyring called ".builtin_trusted_keys" that can be seen by::
>  	[root@...eb ~]# cat /proc/keys
>  	...
>  	223c7853 I------     1 perm 1f030000     0     0 keyring   .builtin_trusted_keys: 1
> -	302d2d52 I------     1 perm 1f010000     0     0 asymmetri Fedora kernel signing key: d69a84e6bce3d216b979e9505b3e3ef9a7118079: X509.RSA a7118079 []
> +	302d2d52 I------     1 perm 1f010000     0     0 asymmetric Fedora kernel signing key: d69a84e6bce3d216b979e9505b3e3ef9a7118079: X509.RSA a7118079 []

Hm, well, no.  The size of that print field is limited to 9 characters.
Unfortunately.  Maybe that should be changed, but that's how it is now.

>  	...
>  
>  Beyond the public key generated specifically for module signing, additional

All of the others look good to me.


thanks.
-- 
~Randy
Reported-by: Randy Dunlap <rdunlap@...radead.org>