Re: [PATCH RESEND 16/62] brd: remove "ramdisk_size" command line parameter

Le 13/09/2025 à 02:37, Askar Safin a écrit :
> [Vous ne recevez pas souvent de courriers de [email protected]. Découvrez pourquoi ceci est important à https://aka.ms/LearnAboutSenderIdentification ]
> 
> It was used mostly for initrd. It could be used only if
> brd is built-in. Use "brd.rd_size" instead

For me it would make more sense to remove ramdisk_start and ramdisk_size 
at the same time.


> 
> Signed-off-by: Askar Safin <[email protected]>
> ---
>   .../admin-guide/kernel-parameters.txt         |  3 ---
>   Documentation/arch/m68k/kernel-options.rst    | 20 ++-----------------
>   arch/arm/configs/s3c6400_defconfig            |  2 +-
>   drivers/block/brd.c                           | 10 ----------
>   4 files changed, 3 insertions(+), 32 deletions(-)
> 
> diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt
> index ad52e3d26014..e862a7b1d2ec 100644
> --- a/Documentation/admin-guide/kernel-parameters.txt
> +++ b/Documentation/admin-guide/kernel-parameters.txt
> @@ -5279,9 +5279,6 @@
>          raid=           [HW,RAID]
>                          See Documentation/admin-guide/md.rst.
> 
> -       ramdisk_size=   [RAM] Sizes of RAM disks in kilobytes
> -                       See Documentation/admin-guide/blockdev/ramdisk.rst.
> -
>          random.trust_cpu=off
>                          [KNL,EARLY] Disable trusting the use of the CPU's
>                          random number generator (if available) to
> diff --git a/Documentation/arch/m68k/kernel-options.rst b/Documentation/arch/m68k/kernel-options.rst
> index 2008a20b4329..f6469ebeb2c7 100644
> --- a/Documentation/arch/m68k/kernel-options.rst
> +++ b/Documentation/arch/m68k/kernel-options.rst
> @@ -215,27 +215,11 @@ Devices possible for Atari:
>              seconds.
> 
> 
> -2.6) ramdisk_size=
> -------------------
> -
> -:Syntax: ramdisk_size=<size>
> -
> -This option instructs the kernel to set up a ramdisk of the given
> -size in KBytes. Do not use this option if the ramdisk contents are
> -passed by bootstrap! In this case, the size is selected automatically
> -and should not be overwritten.
> -
> -The only application is for root filesystems on floppy disks, that
> -should be loaded into memory. To do that, select the corresponding
> -size of the disk as ramdisk size, and set the root device to the disk
> -drive (with "root=").
> -
> -
> -2.7) swap=
> +2.5) swap=
> 
>     I can't find any sign of this option in 2.2.6.
> 
> -2.8) buff=
> +2.6) buff=
>   -----------
> 
>     I can't find any sign of this option in 2.2.6.
> diff --git a/arch/arm/configs/s3c6400_defconfig b/arch/arm/configs/s3c6400_defconfig
> index a37e6ac40825..23635d5b9322 100644
> --- a/arch/arm/configs/s3c6400_defconfig
> +++ b/arch/arm/configs/s3c6400_defconfig
> @@ -4,7 +4,7 @@ CONFIG_ARCH_MULTI_V6=y
>   # CONFIG_ARCH_MULTI_V7 is not set
>   CONFIG_ARCH_S3C64XX=y
>   CONFIG_MACH_WLF_CRAGG_6410=y
> -CONFIG_CMDLINE="console=ttySAC0,115200 root=/dev/ram init=/linuxrc initrd=0x51000000,6M ramdisk_size=6144"
> +CONFIG_CMDLINE="console=ttySAC0,115200 root=/dev/ram init=/linuxrc initrd=0x51000000,6M"
>   CONFIG_VFP=y
>   CONFIG_MODULES=y
>   CONFIG_MODULE_UNLOAD=y
> diff --git a/drivers/block/brd.c b/drivers/block/brd.c
> index 72f02d2b8a99..05c4325904d2 100644
> --- a/drivers/block/brd.c
> +++ b/drivers/block/brd.c
> @@ -222,16 +222,6 @@ MODULE_LICENSE("GPL");
>   MODULE_ALIAS_BLOCKDEV_MAJOR(RAMDISK_MAJOR);
>   MODULE_ALIAS("rd");
> 
> -#ifndef MODULE
> -/* Legacy boot options - nonmodular */
> -static int __init ramdisk_size(char *str)
> -{
> -       rd_size = simple_strtol(str, NULL, 0);
> -       return 1;
> -}
> -__setup("ramdisk_size=", ramdisk_size);
> -#endif
> -
>   /*
>    * The device scheme is derived from loop.c. Keep them in synch where possible
>    * (should share code eventually).
> --
> 2.47.2
> 
>