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 23C4C226D0C; Fri, 7 Mar 2025 16:14:09 +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=1741364050; cv=none; b=NHKftQLq1MAYFseurN+HXmXFO7LexTuKSSSMh6AvjQs7IvPzJ8Ve2X6PILXnIwwV29bO5YfeqVdHGvC5egk6f4pz2d4rMhi+H3uAfZc+qCUb1OqEv7CRRw+g2ZxY2NS4mK2j97CyrwJnSAtaiR0fqEYa1aLedOImN51BPkvA7Ng= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741364050; c=relaxed/simple; bh=92cx1cBCDTShMfwReN9yU9enqFO6kybEdjhN1J7Q8K4=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=BQrpfDkmNb79w7gCDKfqEQeacwwpEOHhLTTfLiJAKr+Z45jD6Bbh63+JP3O71Snfl8e5tzyAuq3kt4m8iEsTBTVlgj2jVAKn0UwWWWVHVqjK6slr8jOUjKc3xmFPe4ji7s7aNpiLubcFu2lufGGPIMm9rsiQD3Ej8gxWVhQNtIA= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 Received: by smtp.kernel.org (Postfix) with ESMTPSA id A8C2AC4CED1; Fri, 7 Mar 2025 16:14:07 +0000 (UTC) Date: Fri, 7 Mar 2025 11:14:07 -0500 From: Steven Rostedt To: Kris Van Hees Cc: Masahiro Yamada , linux-kernel@vger.kernel.org, linux-kbuild@vger.kernel.org, linux-modules@vger.kernel.org, linux-trace-kernel@vger.kernel.org, Nick Alcock , Alan Maguire , Sam James , Luis Chamberlain , Masami Hiramatsu , Nick Desaulniers , Jiri Olsa , Elena Zannoni , Daniel Gomez , Jack Vogel Subject: Re: [PATCH] kbuild: exclude .rodata.(cst|str)* when building ranges Message-ID: <20250307111407.0073ff75@gandalf.local.home> In-Reply-To: <20250305192808.2826336-1-kris.van.hees@oracle.com> References: <20250305192808.2826336-1-kris.van.hees@oracle.com> X-Mailer: Claws Mail 3.20.0git84 (GTK+ 2.24.33; x86_64-pc-linux-gnu) Precedence: bulk X-Mailing-List: linux-trace-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Wed, 5 Mar 2025 14:27:58 -0500 Kris Van Hees wrote: > The .rodata.(cst|str)* sections are often resized during the final > linking and since these sections do not cover actual symbols there is > no need to include them in the modules.builtin.ranges data. > > When these sections were included in processing and resizing occurred, > modules were reported with ranges that extended beyond their true end, > causing subsequent symbols (in address order) to be associated with > the wrong module. If that's the case, shouldn't this be considered a fix, and include a fixes tag as well as Cc stable? -- Steve > > Signed-off-by: Kris Van Hees > Reviewed-by: Jack Vogel > ---