From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from 1wt.eu (ded1.1wt.eu [163.172.96.212]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 5260853BE for ; Tue, 24 Jun 2025 10:00:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=163.172.96.212 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1750759221; cv=none; b=u+LdNRnLqNMcgxlCtw7TV/ZdhnwP8nYRBjNbg/t3if9zV4W8rnVaHGeIEENVxcVQma9CctDwAhVEABtTc6lIj8H82+3pr2OJqjf5S7fPfqYfgVZBimjRXsZf8iPtJDq2cXChpmQbj4+bmi9KP/x75fHFL5U627OI0ZM+LisbI1w= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1750759221; c=relaxed/simple; bh=aCNHC/uxRIZNbS49b3qmH3bmDQDeZHLLs6ZfW6WxAMs=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=G/NOIkjddH2fwfRTuobius9VVitBBhbf6Ii8fwPoXR0xXlMQe+M7tCEKxSobSaZJWtgUdtBpS3lR0O19H1hVRQMOTngamNu9d1mzCe2G/CBObUeR1p6JIkCkNlyyKfcsSLe8bfmY4STvEhhPe+X903J0nOxHaWt68z275MqZyqQ= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=1wt.eu; spf=pass smtp.mailfrom=1wt.eu; arc=none smtp.client-ip=163.172.96.212 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=1wt.eu Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=1wt.eu Received: (from willy@localhost) by pcw.home.local (8.15.2/8.15.2/Submit) id 55OA087C031994; Tue, 24 Jun 2025 12:00:08 +0200 Date: Tue, 24 Jun 2025 12:00:08 +0200 From: Willy Tarreau To: Thomas =?iso-8859-1?Q?Wei=DFschuh?= Cc: linux-kernel@vger.kernel.org, Masahiro Yamada , Arnd Bergmann Subject: Re: [RFC PATCH 2/3] tools/nolibc: add a new target "headers_all_archs" to loop over all archs Message-ID: <20250624100008.GE30919@1wt.eu> References: <20250620103705.10208-1-w@1wt.eu> <20250620103705.10208-3-w@1wt.eu> <4c48147d-aebc-4a2c-a60f-eba2e90584ed@t-8ch.de> <20250624062002.GD30919@1wt.eu> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) On Tue, Jun 24, 2025 at 09:46:00AM +0200, Thomas Weißschuh wrote: > On 2025-06-24 08:20:02+0200, Willy Tarreau wrote: > > On Mon, Jun 23, 2025 at 11:56:29PM +0200, Thomas Weißschuh wrote: > > > On 2025-06-20 12:37:04+0200, Willy Tarreau wrote: > > > > This target allows to install the nolibc headers for all supported > > > > architectures at once, just like it is in the development tree. This > > > > is a first step to support full multi-architecture support. > > > > > > > > Signed-off-by: Willy Tarreau > > > > --- > > > > tools/include/nolibc/Makefile | 10 ++++++++++ > > > > 1 file changed, 10 insertions(+) > > > > > > > > diff --git a/tools/include/nolibc/Makefile b/tools/include/nolibc/Makefile > > > > index 9197c79b267a4..8de6ac5cec425 100644 > > > > --- a/tools/include/nolibc/Makefile > > > > +++ b/tools/include/nolibc/Makefile > > > > @@ -23,6 +23,8 @@ else > > > > Q=@ > > > > endif > > > > > > > > +nolibc_supported_archs := aarch64 arm loongarch m68k mips powerpc riscv s390 sparc x86 > > > > + > > > > nolibc_arch := $(patsubst arm64,aarch64,$(ARCH)) > > > > arch_file := arch-$(nolibc_arch).h > > > > all_files := \ > > > > @@ -83,6 +85,7 @@ help: > > > > @echo " all call \"headers\"" > > > > @echo " clean clean the sysroot" > > > > @echo " headers prepare a sysroot in \$${OUTPUT}sysroot" > > > > + @echo " headers_all_archs prepare a multi-arch sysroot in \$${OUTPUT}sysroot" > > > > @echo " headers_standalone like \"headers\", and also install kernel headers" > > > > @echo " help this help" > > > > @echo "" > > > > @@ -110,6 +113,13 @@ headers_standalone: headers > > > > $(Q)$(MAKE) -C $(srctree) headers > > > > $(Q)$(MAKE) -C $(srctree) headers_install INSTALL_HDR_PATH=$(OUTPUT)sysroot > > > > > > > > +# installs headers for all archs at once. > > > > +headers_all_archs: > > > > + $(Q)mkdir -p "$(OUTPUT)sysroot" > > > > + $(Q)mkdir -p "$(OUTPUT)sysroot/include" > > > > + $(Q)cp --parents $(all_files) arch.h "$(OUTPUT)sysroot/include/" > > > > + $(Q)cp $(addsuffix .h,$(addprefix arch-,$(nolibc_supported_archs))) "$(OUTPUT)sysroot/include/" > > > > > > IMO we could always just install all architecture headers. > > > It's not much code after all. > > > If it is a problem for a user they can either just delete the > > > superfluous architectures or do 'mv arch-$foo.h arch.h; rm arch-*.h'. > > > > I wanted to do that first, then thought that maybe some would like > > to only install the nolibc headers because they already have the > > UAPI headers from another source (local libc, distro packages, > > toolchain etc). Even for us during nolibc development, not having > > to iterate through all archs to reinstall everything is a huge time > > saver. > > > > However, I had another idea that floated in my mind, which is that > > given that we're only saving a few small arch-* files by not > > installing all archs all the time, maybe we should replace the > > "headers" target to always install nolibc headers for all archs > > like above, and keep the uapi headers install separate (only one > > or all). This would remove the need for the target above whose > > role is a bit ambiguous. What do you think ? > > That is exactly what I tried to express :-) Ah OK, I understood that you wanted to install all UAPI in the target above :-) OK that's fine then, will do that. Willy