Received-SPF: pass (google.com: domain of linux-kernel+bounces-23404-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Date: Thu, 11 Jan 2024 11:44:01 +0100
From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Namhyung Kim <namhyung@kernel.org>
Cc: Harshit Mogalapalli <harshit.m.mogalapalli@oracle.com>,
	stable@vger.kernel.org, patches@lists.linux.dev,
	linux-kernel@vger.kernel.org, torvalds@linux-foundation.org,
	akpm@linux-foundation.org, linux@roeck-us.net, shuah@kernel.org,
	patches@kernelci.org, lkft-triage@lists.linaro.org, pavel@denx.de,
	jonathanh@nvidia.com, f.fainelli@gmail.com,
	sudipm.mukherjee@gmail.com, srw@sladewatkins.net, rwarsow@gmx.de,
	conor@kernel.org, allen.lkml@gmail.com,
	Vegard Nossum <vegard.nossum@oracle.com>,
	Darren Kenny <darren.kenny@oracle.com>,
	Adrian Hunter <adrian.hunter@intel.com>,
	Arnaldo Carvalho de Melo <acme@redhat.com>,
	Ian Rogers <irogers@google.com>, Jiri Olsa <jolsa@kernel.org>,
	Lieven Hey <lieven.hey@kdab.com>
Subject: Re: [PATCH for-5.15] perf inject: Fix GEN_ELF_TEXT_OFFSET for jit
Message-ID: <2024011155-fragrance-perfected-2183@gregkh>
References: <2024010711-skeletal-material-15e8@gregkh>
 <20240109214955.451513-1-namhyung@kernel.org>
 <2024011043-unearned-manned-12b7@gregkh>
 <CAM9d7cj3fdaKVehz+E3Fd_Zf91hy31z19goM84qq+cc8WZe7RQ@mail.gmail.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <CAM9d7cj3fdaKVehz+E3Fd_Zf91hy31z19goM84qq+cc8WZe7RQ@mail.gmail.com>

On Wed, Jan 10, 2024 at 09:53:17AM -0800, Namhyung Kim wrote:
> Hello,
> 
> On Tue, Jan 9, 2024 at 11:59 PM Greg Kroah-Hartman
> <gregkh@linuxfoundation.org> wrote:
> >
> > On Tue, Jan 09, 2024 at 01:49:55PM -0800, Namhyung Kim wrote:
> > > From: Adrian Hunter <adrian.hunter@intel.com>
> > >
> > > When a program header was added, it moved the text section but
> > > GEN_ELF_TEXT_OFFSET was not updated.
> > >
> > > Fix by adding the program header size and aligning.
> > >
> > > Fixes: babd04386b1df8c3 ("perf jit: Include program header in ELF files")
> > > Signed-off-by: Adrian Hunter <adrian.hunter@intel.com>
> > > Tested-by: Arnaldo Carvalho de Melo <acme@redhat.com>
> > > Cc: Ian Rogers <irogers@google.com>
> > > Cc: Jiri Olsa <jolsa@kernel.org>
> > > Cc: Lieven Hey <lieven.hey@kdab.com>
> > > Link: https://lore.kernel.org/r/20221014170905.64069-7-adrian.hunter@intel.com
> > > Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
> > > [namhyung: use "linux/kernel.h" instead to avoid build failure]
> > > Signed-off-by: Namhyung Kim <namhyung@kernel.org>
> > > ---
> > >  tools/perf/util/genelf.h | 4 +++-
> > >  1 file changed, 3 insertions(+), 1 deletion(-)
> >
> > What is the git commit id of this change in Linus's tree?
> 
> Sorry I forgot to add that.  It's 89b15d00527b7825ff19130ed83478e80e3fae99

Thanks, now queued up.

greg k-h