From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8550627F732; Thu, 26 Feb 2026 18:33:03 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772130783; cv=none; b=h5oCd0tvLSujtIwf7ZkBdSKYhHHdymFIPiY/nCsWF+MsMaOTbpegoanehAf5NWQw3+7e0x/GMDv1qZiH2e85DrA0edBeex6AnsHsm50qpmxhlrp3qiIk6ya8zL+MSC+FXVWDXfiC//YriKpruCuACx4J5S7K0v9UGBORhNklUFE= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772130783; c=relaxed/simple; bh=VSeTKWpqHQ/xQAnBpFUGW6l3RlpfyjkXqHosrfzhgZY=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=ivTbPzHHt+bs+815TIjRz0hfkbxar/eoxFMnASXRD/VrpLzCCGZm/IY0N+gfbMWp5BJiyG+XHDrYjxjqRgoXr85zOrovUUsWNAQzg6/58uleawc+B4Xr6tFSqskzier/BwI90AH6fUVihG55EUImzQ4RkC2iO4Kkz8V5xhyHKS0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=CQQM3soB; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="CQQM3soB" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 8FAD8C116C6; Thu, 26 Feb 2026 18:33:02 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1772130783; bh=VSeTKWpqHQ/xQAnBpFUGW6l3RlpfyjkXqHosrfzhgZY=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=CQQM3soBA3HATli5radoLC4J9n6+UpiXxlyOkuORCYj5pdddFoQskRZYORrR3G3JB GfAIKWgDkBvzdNVJ7WF+2lgjUKPp/q/dbE0YAW3ixvADk6DBkJE4zj/VOoJLbUhLNR s8/GIAC3Y2wl6RpxDvFYviVFlyG+21IkIxLxhejxuFcR5BvWZnLYxq8wg1bvmFhBkD UvqC4y+M7fVoqqdWKZAFTKH4cuqZB+nXtmy8+BSD4RQiTjIRWET8tdk8UAv8mYfpw1 dRbWfxIfqGBCJKsl6rRAkVfd9Bx9nJSHP60Jx38MXvbg8H7VXXj6jdwAOCxG0RY+ly VtFehVk+deMBg== Date: Thu, 26 Feb 2026 10:33:00 -0800 From: Namhyung Kim To: Nathan Chancellor Cc: Leo Yan , Kees Cook , Nicolas Schier , Nick Desaulniers , Bill Wendling , Justin Stitt , Arnaldo Carvalho de Melo , Ian Rogers , James Clark , linux-kbuild@vger.kernel.org, linux-kernel@vger.kernel.org, llvm@lists.linux.dev Subject: Re: [PATCH RESEND v2] tools build: Use -fzero-init-padding-bits=all Message-ID: References: <20260224-tools_build_fix_zero_init-v2-1-b1acc817a01e@arm.com> <20260224171956.GA639152@ax162> <202602241310.C3641B97@keescook> <20260225092210.GC4184494@e132581.arm.com> <20260225192505.GC2755225@ax162> 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=utf-8 Content-Disposition: inline In-Reply-To: <20260225192505.GC2755225@ax162> Hello, On Wed, Feb 25, 2026 at 12:25:05PM -0700, Nathan Chancellor wrote: > On Wed, Feb 25, 2026 at 09:22:10AM +0000, Leo Yan wrote: > > On Tue, Feb 24, 2026 at 01:11:19PM -0800, Kees Cook wrote: > > > On Tue, Feb 24, 2026 at 10:19:56AM -0700, Nathan Chancellor wrote: > > > > Kbuild does not maintain/touch tools/. This should go via another tree > > > > like perf or something. It does not look like > > > > tools/scripts/Makefile.include has a clear owner, perf and bpf tend to > > > > be the ones who touch it the most. > > > > This is a circular deadlock. Namhyung (the perf maintainer) advised me > > to send patch to the linux-kbuild [1], for fixing an union init issue > > found recently. > > The tools/ build system is not Kbuild, so I do not want to take patches > for it, sorry. Issues from patches I take become my responsibility to > deal with and I am not at all familiar with the tools build system > because I am not a consumer of it. Nicolas may feel differently but I am > going to assume not based on his level of expertise with tools/ [1]. I > am not trying to deadlock you though. > > Namhyung, I think this patch can reasonably go via the perf tree since > it will be a primary consumer of it. Sorry for the trouble. I'm fine to take it to the perf tree once everyone involved is happy about it. Let me CC the bpf list in the original patch. Thanks, Namhyung > > > > You could claim it! ;) > > Heh, Kbuild and ClangBuiltLinux give me more than enough to do around > here ;) > > > > Regardless, I like to see cc-option available here, as I doubt this will > > > be the last conditional option for tool builds. (Actually, are there > > > other conditional options that could use this today in the tools > > > Makefiles?) > > > > Some subprojects in tools have their own conditional options. > > > > This patch is ambitious that it changes the global Makefile.include file > > so it can propagate the '-fzero-init-padding-bits=all' option to > > projects that include it. Why do we need to do this globally? This is > > because Perf needs to build several subprojects (libperf and bpftool). > > This seems like a great reason for it to go via the perf tree as noted > above. > > [1]: https://lore.kernel.org/aNhGp7NDCCrtwJqm@levanger/ > > Cheers, > Nathan