From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from pegase2.c-s.fr (pegase2.c-s.fr [93.17.235.10]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 0101030C35A; Mon, 15 Sep 2025 12:21:12 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=93.17.235.10 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1757938874; cv=none; b=n1HsfQjcgyuTD5e21sqFRi+TaeHO7gurdpAPKSucOE88iCi4ZkR5UANI9eZhZnEDdbpjgHsBzhZcoH4JWN+EE3trbe5mwGt2E+LMzNefnQlWyncMR1Zb3/UPnRV+eEjBN+zLC9a9XI1pqCIw5AOHPFsAtr5iDLMr3ioHRUzQP9U= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1757938874; c=relaxed/simple; bh=9RLcS91I7/JfJmj5gROsGNlF/zDbJRvo+2amvvvj6yQ=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=TZykKVC7pyy/mtCA1vR/XeLkskmICdXY19JpPrfPvCxndFS44alE5iIUJ4F532oWvlsA4WuBwV4zI8geUSt6dNvu6T9GrDT/dhkkcCBBVl9rofLH8ppvpAvkcE5pfqExzTh/P8Q8g9s2W9uIlci+63ND7ILUaB00dcJvoAIgEEk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=csgroup.eu; spf=pass smtp.mailfrom=csgroup.eu; arc=none smtp.client-ip=93.17.235.10 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=csgroup.eu Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=csgroup.eu Received: from localhost (mailhub4.si.c-s.fr [172.26.127.67]) by localhost (Postfix) with ESMTP id 4cQNnR0pmTz9sxp; Mon, 15 Sep 2025 13:57:31 +0200 (CEST) X-Virus-Scanned: amavisd-new at c-s.fr Received: from pegase2.c-s.fr ([172.26.127.65]) by localhost (pegase2.c-s.fr [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id CofbtF7b-0VV; Mon, 15 Sep 2025 13:57:31 +0200 (CEST) Received: from messagerie.si.c-s.fr (messagerie.si.c-s.fr [192.168.25.192]) by pegase2.c-s.fr (Postfix) with ESMTP id 4cQNnQ5DNGz9sxl; Mon, 15 Sep 2025 13:57:30 +0200 (CEST) Received: from localhost (localhost [127.0.0.1]) by messagerie.si.c-s.fr (Postfix) with ESMTP id 981DC8B766; Mon, 15 Sep 2025 13:57:30 +0200 (CEST) X-Virus-Scanned: amavisd-new at c-s.fr Received: from messagerie.si.c-s.fr ([127.0.0.1]) by localhost (messagerie.si.c-s.fr [127.0.0.1]) (amavisd-new, port 10023) with ESMTP id aYicW_kKuipv; Mon, 15 Sep 2025 13:57:30 +0200 (CEST) Received: from [10.25.207.160] (unknown [10.25.207.160]) by messagerie.si.c-s.fr (Postfix) with ESMTP id 227808B763; Mon, 15 Sep 2025 13:57:30 +0200 (CEST) Message-ID: <99563c3d-7322-4164-81f3-0d28e91ed653@csgroup.eu> Date: Mon, 15 Sep 2025 13:57:29 +0200 Precedence: bulk X-Mailing-List: linux-efi@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH RESEND 06/62] arm: init: remove special logic for setting brd.rd_size To: Askar Safin , linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Linus Torvalds , Greg Kroah-Hartman , Christian Brauner , Al Viro , Jan Kara , Christoph Hellwig , Jens Axboe , Andy Shevchenko , Aleksa Sarai , =?UTF-8?Q?Thomas_Wei=C3=9Fschuh?= , Julian Stecklina , Gao Xiang , Art Nikpal , Andrew Morton , Eric Curtin , Alexander Graf , Rob Landley , Lennart Poettering , linux-arch@vger.kernel.org, linux-alpha@vger.kernel.org, linux-snps-arc@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-csky@vger.kernel.org, linux-hexagon@vger.kernel.org, loongarch@lists.linux.dev, linux-m68k@lists.linux-m68k.org, linux-mips@vger.kernel.org, linux-openrisc@vger.kernel.org, linux-parisc@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-riscv@lists.infradead.org, linux-s390@vger.kernel.org, linux-sh@vger.kernel.org, sparclinux@vger.kernel.org, linux-um@lists.infradead.org, x86@kernel.org, Ingo Molnar , linux-block@vger.kernel.org, initramfs@vger.kernel.org, linux-api@vger.kernel.org, linux-doc@vger.kernel.org, linux-efi@vger.kernel.org, linux-ext4@vger.kernel.org, "Theodore Y . Ts'o" , linux-acpi@vger.kernel.org, Michal Simek , devicetree@vger.kernel.org, Luis Chamberlain , Kees Cook , Thorsten Blum , Heiko Carstens , patches@lists.linux.dev References: <20250913003842.41944-1-safinaskar@gmail.com> <20250913003842.41944-7-safinaskar@gmail.com> From: Christophe Leroy Content-Language: fr-FR In-Reply-To: <20250913003842.41944-7-safinaskar@gmail.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Le 13/09/2025 à 02:37, Askar Safin a écrit : > [Vous ne recevez pas souvent de courriers de safinaskar@gmail.com. Découvrez pourquoi ceci est important à https://aka.ms/LearnAboutSenderIdentification ] > > There is no any reason for having special mechanism > for setting ramdisk size. > > Also this allows us to change rd_size variable to static Can you squash patches 6 to 9 all together ? > > Signed-off-by: Askar Safin > --- > arch/arm/kernel/atags_parse.c | 12 ------------ > drivers/block/brd.c | 8 ++++---- > include/linux/initrd.h | 3 --- > 3 files changed, 4 insertions(+), 19 deletions(-) > > diff --git a/arch/arm/kernel/atags_parse.c b/arch/arm/kernel/atags_parse.c > index a3f0a4f84e04..615d9e83c9b5 100644 > --- a/arch/arm/kernel/atags_parse.c > +++ b/arch/arm/kernel/atags_parse.c > @@ -87,18 +87,6 @@ static int __init parse_tag_videotext(const struct tag *tag) > __tagtable(ATAG_VIDEOTEXT, parse_tag_videotext); > #endif > > -#ifdef CONFIG_BLK_DEV_RAM > -static int __init parse_tag_ramdisk(const struct tag *tag) > -{ > - if (tag->u.ramdisk.size) > - rd_size = tag->u.ramdisk.size; > - > - return 0; > -} > - > -__tagtable(ATAG_RAMDISK, parse_tag_ramdisk); > -#endif > - > static int __init parse_tag_serialnr(const struct tag *tag) > { > system_serial_low = tag->u.serialnr.low; > diff --git a/drivers/block/brd.c b/drivers/block/brd.c > index 0c2eabe14af3..72f02d2b8a99 100644 > --- a/drivers/block/brd.c > +++ b/drivers/block/brd.c > @@ -27,6 +27,10 @@ > > #include > > +static unsigned long rd_size = CONFIG_BLK_DEV_RAM_SIZE; > +module_param(rd_size, ulong, 0444); > +MODULE_PARM_DESC(rd_size, "Size of each RAM disk in kbytes."); > + > /* > * Each block ramdisk device has a xarray brd_pages of pages that stores > * the pages containing the block device's contents. > @@ -209,10 +213,6 @@ static int rd_nr = CONFIG_BLK_DEV_RAM_COUNT; > module_param(rd_nr, int, 0444); > MODULE_PARM_DESC(rd_nr, "Maximum number of brd devices"); > > -unsigned long rd_size = CONFIG_BLK_DEV_RAM_SIZE; > -module_param(rd_size, ulong, 0444); > -MODULE_PARM_DESC(rd_size, "Size of each RAM disk in kbytes."); > - > static int max_part = 1; > module_param(max_part, int, 0444); > MODULE_PARM_DESC(max_part, "Num Minors to reserve between devices"); > diff --git a/include/linux/initrd.h b/include/linux/initrd.h > index 6320a9cb6686..b42235c21444 100644 > --- a/include/linux/initrd.h > +++ b/include/linux/initrd.h > @@ -5,9 +5,6 @@ > > #define INITRD_MINOR 250 /* shouldn't collide with /dev/ram* too soon ... */ > > -/* size of a single RAM disk */ > -extern unsigned long rd_size; > - > /* 1 if it is not an error if initrd_start < memory_start */ > extern int initrd_below_start_ok; > > -- > 2.47.2 > >