linux-kernel - RE: [PATCH] Documentation: block: cmdline-partition.txt fixes and additions

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: <C3050A4DBA34F345975765E43127F10F8B7720F8@dggemm511-mbs.china.huawei.com>
Date:   Mon, 7 May 2018 02:43:24 +0000
From:   Caizhiyong <caizhiyong@...ilicon.com>
To:     Randy Dunlap <rdunlap@...radead.org>,
        "linux-block@...r.kernel.org" <linux-block@...r.kernel.org>,
        axboe <axboe@...nel.dk>
CC:     LKML <linux-kernel@...r.kernel.org>,
        Andrew Morton <akpm@...ux-foundation.org>,
        "linux-doc@...r.kernel.org" <linux-doc@...r.kernel.org>,
        Jonathan Corbet <corbet@....net>,
        Jiangheng <jiangheng@...ilicon.com>
Subject: RE: [PATCH] Documentation: block: cmdline-partition.txt fixes and
 additions

> -----Original Message-----
> From: Randy Dunlap [mailto:rdunlap@...radead.org]
> Sent: Monday, May 07, 2018 2:50 AM
> To: linux-block@...r.kernel.org; axboe <axboe@...nel.dk>
> Cc: LKML <linux-kernel@...r.kernel.org>; Caizhiyong
> <caizhiyong@...ilicon.com>; Andrew Morton <akpm@...ux-foundation.org>;
> linux-doc@...r.kernel.org; Jonathan Corbet <corbet@....net>
> Subject: [PATCH] Documentation: block: cmdline-partition.txt fixes and
> additions
> 
> From: Randy Dunlap <rdunlap@...radead.org>
> 
> Make the description of the kernel command line option "blkdevparts"
> a bit more flowing and readable.
> 
> Fix a few typos.
> Add the optional <size> and <offset> suffixes.
> Note that size can be "-" to indicate all of the remaining space.
> 
> Signed-off-by: Randy Dunlap <rdunlap@...radead.org>
> Cc: Cai Zhiyong <caizhiyong@...wei.com>
> ---
> Should the "ro" and "lk" flags options be described?

thank you for fix typos.
The "ro" and "lk" feature are not implemented in block devices. could you implement them?

>  Documentation/block/cmdline-partition.txt |   21 +++++++++++++-------
>  1 file changed, 14 insertions(+), 7 deletions(-)
> 
> --- linux-next-20180504.orig/Documentation/block/cmdline-partition.txt
> +++ linux-next-20180504/Documentation/block/cmdline-partition.txt
> @@ -1,7 +1,9 @@
>  Embedded device command line partition parsing
> 
> ==========================================================
> ===========
> 
> -Support for reading the block device partition table from the command line.
> +The "blkdevparts" command line option adds support for reading the
> +block device partition table from the kernel command line.
> +
>  It is typically used for fixed block (eMMC) embedded devices.
>  It has no MBR, so saves storage space. Bootloader can be easily accessed
>  by absolute address of data on the block device.
> @@ -14,22 +16,27 @@ blkdevparts=<blkdev-def>[;<blkdev-def>]
>      <partdef> := <size>[@<offset>](part-name)
> 
>  <blkdev-id>
> -    block device disk name, embedded device used fixed block device,
> -    it's disk name also fixed. such as: mmcblk0, mmcblk1, mmcblk0boot0.
> +    block device disk name. Embedded device uses fixed block device.
> +    Its disk name is also fixed, such as: mmcblk0, mmcblk1, mmcblk0boot0.
> 
>  <size>
>      partition size, in bytes, such as: 512, 1m, 1G.
> +    size may contain an optional suffix of (upper or lower case):
> +      K, M, G, T, P, E.
> +    "-" is used to denote all remaining space.
> 
>  <offset>
>      partition start address, in bytes.
> +    offset may contain an optional suffix of (upper or lower case):
> +      K, M, G, T, P, E.
> 
>  (part-name)
> -    partition name, kernel send uevent with "PARTNAME". application can
> create
> -    a link to block device partition with the name "PARTNAME".
> -    user space application can access partition by partition name.
> +    partition name. Kernel sends uevent with "PARTNAME". Application can
> +    create a link to block device partition with the name "PARTNAME".
> +    User space application can access partition by partition name.
> 
>  Example:
> -    eMMC disk name is "mmcblk0" and "mmcblk0boot0"
> +    eMMC disk names are "mmcblk0" and "mmcblk0boot0".
> 
>    bootargs:
>      'blkdevparts=mmcblk0:1G(data0),1G(data1),-;mmcblk0boot0:1m(boot),-
> (kernel)'