Received-SPF: pass (google.com: domain of linux-kernel+bounces-22574-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
Precedence: bulk
MIME-Version: 1.0
References: <2024010711-skeletal-material-15e8@gregkh> <20240109214955.451513-1-namhyung@kernel.org>
 <2024011043-unearned-manned-12b7@gregkh>
In-Reply-To: <2024011043-unearned-manned-12b7@gregkh>
From: Namhyung Kim <namhyung@kernel.org>
Date: Wed, 10 Jan 2024 09:53:17 -0800
Message-ID: <CAM9d7cj3fdaKVehz+E3Fd_Zf91hy31z19goM84qq+cc8WZe7RQ@mail.gmail.com>
Subject: Re: [PATCH for-5.15] perf inject: Fix GEN_ELF_TEXT_OFFSET for jit
To: Greg Kroah-Hartman <gregkh@linuxfoundation.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>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Hello,

On Tue, Jan 9, 2024 at 11:59=E2=80=AFPM 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@in=
tel.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,
Namhyung