Received: by 2002:a05:6a10:9848:0:0:0:0 with SMTP id x8csp1359188pxf; Fri, 2 Apr 2021 08:23:47 -0700 (PDT) X-Google-Smtp-Source: ABdhPJz/707Qant3umplb5CHKVLuFRGwB+zbUBkXkg7UILPrltMpZmkToMOR3/z9ZHu3JIBcwGiX X-Received: by 2002:a17:906:bfcc:: with SMTP id us12mr14808421ejb.266.1617377027221; Fri, 02 Apr 2021 08:23:47 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1617377027; cv=none; d=google.com; s=arc-20160816; b=btVF84kEm+qDi10DF24omPdAs1x/3oJ5GeGEtxq+vBXRlMMTAVEs+lDt5m881Si2Rq CencxilgY6ImGQtUNrlR7ovxZBVvZZL7IGh79Je0s1CDMmpL1H7FcytavloZm/Dz9sfY uLEuNJb8wEI5iocVk5csg0xxu5YxF7YvLyBiUUPbzzKum2Fg8UwW8kkBtMCsPh51LKjx f8a8LvKh6eWICcy7B8YjH4A0Tq/UZ2cjESVDsE3cPKpKTMzw0CNjxnkemRb1FLHqA7s1 bwCk11gzo72Vk8jjqznxp/Csx3w1TyAIs9zh3E+DbD5XGhm/+9zJ7MJVGvY0cZKiNXUc 37qg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:content-language :in-reply-to:mime-version:user-agent:date:message-id:from:references :cc:to:subject; bh=OozgyM5EiDf2KT1zC1zVblWj7E+h53k7xY/iHzKT2e4=; b=ak8A9H4sc9ur49fyGkS09Ofhncl5rHsdMNw5xZ15K/46GAk+OXKZ3/9JwdRgQ3WqG9 Oy3oDlGgq/EyFXhziqU+v/u+iqq8hg5SWZKKlolmSS0I/IKKwC110mUyWzvJyib6Avw+ sPWugXfdY2CVw8OrE5wIJkPOK6FOR06c8y0Q9tnoPSAlbi9oVh1JkgvI2bFUpCN6PdCm QxuivYS0neT9u2krZGuM6Vnd58faVZINXr6wYnWyWP0pk5VKZhuBO8G8xej/yNOCBsSU Y0VRv0N9MjjFF/py0Kxnaq03IbSrln9Ro1FveJjbkSfPCTAlH8NHH/P4DVpgHDmGZ2uM 46+Q== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id jx25si6431355ejc.382.2021.04.02.08.23.24; Fri, 02 Apr 2021 08:23:47 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235291AbhDBPUw (ORCPT + 99 others); Fri, 2 Apr 2021 11:20:52 -0400 Received: from pegase1.c-s.fr ([93.17.236.30]:1202 "EHLO pegase1.c-s.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229553AbhDBPUv (ORCPT ); Fri, 2 Apr 2021 11:20:51 -0400 Received: from localhost (mailhub1-int [192.168.12.234]) by localhost (Postfix) with ESMTP id 4FBkJt3fjkz9txcv; Fri, 2 Apr 2021 17:20:46 +0200 (CEST) X-Virus-Scanned: Debian amavisd-new at c-s.fr Received: from pegase1.c-s.fr ([192.168.12.234]) by localhost (pegase1.c-s.fr [192.168.12.234]) (amavisd-new, port 10024) with ESMTP id suidXNMlvCF2; Fri, 2 Apr 2021 17:20:46 +0200 (CEST) Received: from messagerie.si.c-s.fr (messagerie.si.c-s.fr [192.168.25.192]) by pegase1.c-s.fr (Postfix) with ESMTP id 4FBkJq5Yryz9v2lx; Fri, 2 Apr 2021 17:20:43 +0200 (CEST) Received: from localhost (localhost [127.0.0.1]) by messagerie.si.c-s.fr (Postfix) with ESMTP id 9C4ED8BB79; Fri, 2 Apr 2021 17:20:45 +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 LvleWMf3wBya; Fri, 2 Apr 2021 17:20:45 +0200 (CEST) Received: from [192.168.4.90] (unknown [192.168.4.90]) by messagerie.si.c-s.fr (Postfix) with ESMTP id 86FC38BB7D; Fri, 2 Apr 2021 17:20:44 +0200 (CEST) Subject: Re: [PATCH v3 00/17] Implement GENERIC_CMDLINE To: Rob Herring Cc: Will Deacon , Daniel Walker , Daniel Gimpelevich , "open list:GENERIC INCLUDE/ASM HEADER FILES" , devicetree@vger.kernel.org, linuxppc-dev , "linux-kernel@vger.kernel.org" , linux-arm-kernel , microblaze , "open list:MIPS" , nios2 , Openrisc , linux-hexagon@vger.kernel.org, linux-riscv , X86 ML , linux-xtensa@linux-xtensa.org, SH-Linux , sparclinux References: From: Christophe Leroy Message-ID: Date: Fri, 2 Apr 2021 17:20:45 +0200 User-Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:78.0) Gecko/20100101 Thunderbird/78.9.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: fr Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Le 26/03/2021 à 16:04, Rob Herring a écrit : > On Fri, Mar 26, 2021 at 7:44 AM Christophe Leroy > wrote: >> >> The purpose of this series is to improve and enhance the >> handling of kernel boot arguments. >> >> It is first focussed on powerpc but also extends the capability >> for other arches. >> >> This is based on suggestion from Daniel Walker >> >> Main changes in V3: >> - Also accept destination equal to source in cmdline_build() by setting a tmp buffer in __initdata. Powerpc provides different source and destination and call __cmdline_build() directly. >> - Taken comments received from Will and Rob >> - Converted all architectures (Only tested on powerpc) >> >> Christophe Leroy (17): >> cmdline: Add generic function to build command line. >> drivers: of: use cmdline building function >> cmdline: Gives architectures opportunity to use generically defined >> boot cmdline manipulation >> powerpc: Convert to GENERIC_CMDLINE >> arm: Convert to GENERIC_CMDLINE >> arm64: Convert to GENERIC_CMDLINE >> hexagon: Convert to GENERIC_CMDLINE >> microblaze: Convert to GENERIC_CMDLINE >> nios2: Convert to GENERIC_CMDLINE >> openrisc: Convert to GENERIC_CMDLINE >> riscv: Convert to GENERIC_CMDLINE >> sh: Convert to GENERIC_CMDLINE >> sparc: Convert to GENERIC_CMDLINE >> xtensa: Convert to GENERIC_CMDLINE >> x86: Convert to GENERIC_CMDLINE >> mips: Convert to GENERIC_CMDLINE >> cmdline: Remove CONFIG_CMDLINE_EXTEND >> >> arch/arm/Kconfig | 38 +------------- >> arch/arm/kernel/atags_parse.c | 15 ++---- >> arch/arm64/Kconfig | 33 +----------- >> arch/arm64/kernel/idreg-override.c | 9 ++-- >> arch/hexagon/Kconfig | 11 +--- >> arch/hexagon/kernel/setup.c | 10 +--- >> arch/microblaze/Kconfig | 24 +-------- >> arch/microblaze/configs/mmu_defconfig | 2 +- >> arch/microblaze/kernel/head.S | 4 +- >> arch/mips/Kconfig | 1 + >> arch/mips/Kconfig.debug | 44 ---------------- >> arch/mips/configs/ar7_defconfig | 1 - >> arch/mips/configs/bcm47xx_defconfig | 1 - >> arch/mips/configs/bcm63xx_defconfig | 1 - >> arch/mips/configs/bmips_be_defconfig | 1 - >> arch/mips/configs/bmips_stb_defconfig | 1 - >> arch/mips/configs/capcella_defconfig | 1 - >> arch/mips/configs/ci20_defconfig | 1 - >> arch/mips/configs/cu1000-neo_defconfig | 1 - >> arch/mips/configs/cu1830-neo_defconfig | 1 - >> arch/mips/configs/e55_defconfig | 1 - >> arch/mips/configs/generic_defconfig | 1 - >> arch/mips/configs/gpr_defconfig | 1 - >> arch/mips/configs/loongson3_defconfig | 1 - >> arch/mips/configs/mpc30x_defconfig | 1 - >> arch/mips/configs/rt305x_defconfig | 1 - >> arch/mips/configs/tb0219_defconfig | 1 - >> arch/mips/configs/tb0226_defconfig | 1 - >> arch/mips/configs/tb0287_defconfig | 1 - >> arch/mips/configs/workpad_defconfig | 1 - >> arch/mips/configs/xway_defconfig | 1 - >> arch/mips/kernel/relocate.c | 4 +- >> arch/mips/kernel/setup.c | 40 +-------------- >> arch/mips/pic32/pic32mzda/early_console.c | 2 +- >> arch/mips/pic32/pic32mzda/init.c | 2 - >> arch/nios2/Kconfig | 24 +-------- >> arch/nios2/kernel/setup.c | 13 ++--- >> arch/openrisc/Kconfig | 10 +--- >> arch/powerpc/Kconfig | 37 +------------ >> arch/powerpc/kernel/prom_init.c | 17 +++--- >> arch/riscv/Kconfig | 44 +--------------- >> arch/riscv/kernel/setup.c | 5 +- >> arch/sh/Kconfig | 30 +---------- >> arch/sh/configs/ap325rxa_defconfig | 2 +- >> arch/sh/configs/dreamcast_defconfig | 2 +- >> arch/sh/configs/ecovec24-romimage_defconfig | 2 +- >> arch/sh/configs/ecovec24_defconfig | 2 +- >> arch/sh/configs/edosk7760_defconfig | 2 +- >> arch/sh/configs/espt_defconfig | 2 +- >> arch/sh/configs/j2_defconfig | 2 +- >> arch/sh/configs/kfr2r09-romimage_defconfig | 2 +- >> arch/sh/configs/kfr2r09_defconfig | 2 +- >> arch/sh/configs/lboxre2_defconfig | 2 +- >> arch/sh/configs/microdev_defconfig | 2 +- >> arch/sh/configs/migor_defconfig | 2 +- >> arch/sh/configs/polaris_defconfig | 2 +- >> arch/sh/configs/r7780mp_defconfig | 2 +- >> arch/sh/configs/r7785rp_defconfig | 2 +- >> arch/sh/configs/rsk7201_defconfig | 2 +- >> arch/sh/configs/rsk7203_defconfig | 2 +- >> arch/sh/configs/rts7751r2d1_defconfig | 2 +- >> arch/sh/configs/rts7751r2dplus_defconfig | 2 +- >> arch/sh/configs/sdk7780_defconfig | 2 +- >> arch/sh/configs/sdk7786_defconfig | 2 +- >> arch/sh/configs/se7206_defconfig | 2 +- >> arch/sh/configs/se7343_defconfig | 2 +- >> arch/sh/configs/se7712_defconfig | 2 +- >> arch/sh/configs/se7721_defconfig | 2 +- >> arch/sh/configs/se7724_defconfig | 2 +- >> arch/sh/configs/se7751_defconfig | 2 +- >> arch/sh/configs/se7780_defconfig | 2 +- >> arch/sh/configs/sh03_defconfig | 2 +- >> arch/sh/configs/sh2007_defconfig | 2 +- >> arch/sh/configs/sh7757lcr_defconfig | 2 +- >> arch/sh/configs/sh7763rdp_defconfig | 2 +- >> arch/sh/configs/shmin_defconfig | 2 +- >> arch/sh/configs/shx3_defconfig | 2 +- >> arch/sh/configs/titan_defconfig | 2 +- >> arch/sh/configs/ul2_defconfig | 2 +- >> arch/sh/kernel/setup.c | 11 +--- >> arch/sparc/Kconfig | 18 +------ >> arch/sparc/prom/bootstr_64.c | 2 +- >> arch/x86/Kconfig | 45 +--------------- >> arch/x86/kernel/setup.c | 17 +----- >> arch/xtensa/Kconfig | 15 +----- >> arch/xtensa/configs/audio_kc705_defconfig | 1 - >> arch/xtensa/configs/common_defconfig | 1 - >> arch/xtensa/configs/generic_kc705_defconfig | 1 - >> arch/xtensa/configs/iss_defconfig | 1 - >> arch/xtensa/configs/nommu_kc705_defconfig | 1 - >> arch/xtensa/configs/smp_lx200_defconfig | 1 - >> arch/xtensa/configs/virt_defconfig | 1 - >> arch/xtensa/configs/xip_kc705_defconfig | 1 - >> arch/xtensa/kernel/setup.c | 10 +--- >> drivers/firmware/efi/libstub/x86-stub.c | 26 +++++----- > > You missed efi-stub.c which has CMDLINE_EXTEND. > I think I completely missed EFI. Reworked in V4.