From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 96350C4332F for ; Mon, 17 Oct 2022 18:02:45 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230400AbiJQSCo (ORCPT ); Mon, 17 Oct 2022 14:02:44 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:38316 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230364AbiJQSCn (ORCPT ); Mon, 17 Oct 2022 14:02:43 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [IPv6:2604:1380:4641:c500::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 2737663FFB; Mon, 17 Oct 2022 11:02:42 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id B2B0C611DA; Mon, 17 Oct 2022 18:02:41 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 97069C433D7; Mon, 17 Oct 2022 18:02:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1666029761; bh=Olcg7XSNxhY0aERanoHq/iCxYBs4krRjC8vRARVayLE=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Y6lGAjPgwllZem0PaQ5k5VM8yNUt7Km2raDS8SyVx95ngpGJG7E8cI3Gg/KGnxJjS CvFjZHiACJFrheaceYvl6nNfW1iVIhxWa0+qvh2NnGtBUdBiGguaw4WJig3DD/q9K9 xWZ3xfpIyFb7Cwd7EdyElqXXe1CE8tfRhVZ4o4Sa6KmSEUMKpC0xzZtg3VcnT1G0kc U93pGzmrxzAdN/Cx+c1UYf57J24dplKvv96qzf5z3MTLIy/gDGn3vCDXizE8C3W2qN Ro5dKOR0Sj6024n5B2JgjhcP2bjYxHozngQANUuhdo6UVz9jC6IMKI1n/gyZd8GnFk awRwmaOVFjoXQ== Date: Mon, 17 Oct 2022 11:02:39 -0700 From: Nathan Chancellor To: Jiri Olsa Cc: Greg KH , stable@vger.kernel.org, bpf@vger.kernel.org, Arnaldo Carvalho de Melo , Daniel Borkmann , Martin Rodriguez Reboredo , Andrii Nakryiko Subject: Re: [PATCH stable 5.15 0/2] kbuild: Fix compilation for latest pahole release Message-ID: References: <20220904131901.13025-1-jolsa@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: bpf@vger.kernel.org On Mon, Sep 19, 2022 at 12:14:40AM +0200, Jiri Olsa wrote: > On Tue, Sep 06, 2022 at 02:03:30PM +0200, Greg KH wrote: > > On Mon, Sep 05, 2022 at 09:04:10AM +0200, Jiri Olsa wrote: > > > On Sun, Sep 04, 2022 at 04:10:09PM +0200, Greg KH wrote: > > > > On Sun, Sep 04, 2022 at 03:18:59PM +0200, Jiri Olsa wrote: > > > > > hi, > > > > > new version of pahole (1.24) is causing compilation fail for 5.15 > > > > > stable kernel, discussed in here [1][2]. Sending fix for that plus > > > > > one dependency patch. > > > > > > > > > > Note for patch 1: > > > > > there was one extra line change in scripts/pahole-flags.sh file in > > > > > its linux tree merge commit: > > > > > > > > > > fc02cb2b37fe Merge tag 'net-next-for-5.16' of git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net-next > > > > > > > > > > not sure how/if to track that, I squashed the change in. > > > > > > > > Squashing is fine, thanks. > > > > > > > > And do we also need this for kernels older than 5.15? Like 5.10 and 5.4? > > > > > > yes, 5.10 needs similar patchset, but this for 5.15 won't apply there, > > > so I'll send it separately > > > > > > 5.4 passes compilation, but I don't think it will boot properly, still > > > need to check on that > > > > > > in any case, more patches are coming ;-) > > > > Ok, these two are now queued up, feel free to send the rest when you > > have them ready. > > hi, > as for 5.10 changes, I have them ready, pushed in here: > git://git.kernel.org/pub/scm/linux/kernel/git/jolsa/perf.git pahole_fix_5_10 > > but it looks like CONFIG_DEBUG_INFO_BTF is not being used in 5.10, > because I had to backport other similar option, that would break > the build even earlier (--skip_encoding_btf_vars), or people use > just old pahole ;-) > > I suggest we wait with this change until somebody actually wants > this fixed, AFAICS there's no report of broken 5.10 build yet Consider this your first report :) My build containers include the latest pahole, as I have noticed more issues with older pahole in newer kernels than newer pahole in older kernels, as least until now. I have tripped over this issue on both 5.19 and 5.10, as the stable-only commit b775fbf532dc ("kbuild: Add skip_encoding_btf_enum64 option to pahole") was only applied to 5.15, even though it is needed in all kernels prior to 6.0. Please consider explicitly sending the 5.10 series to stable and requesting b775fbf532dc to be applied to 5.19. Cheers, Nathan