Re: [PATCH 4.19 12/87] arm/ftrace: Fix BE text poking

From: Greg Kroah-Hartman
Date: Thu Mar 05 2020 - 09:54:04 EST


On Thu, Mar 05, 2020 at 02:49:57PM +0100, Pavel Machek wrote:
> Hi!
>
> > From: Peter Zijlstra <peterz@xxxxxxxxxxxxx>
> >
> > [ Upstream commit be993e44badc448add6a18d6f12b20615692c4c3 ]
> >
> > The __patch_text() function already applies __opcode_to_mem_*(), so
> > when __opcode_to_mem_*() is not the identity (BE*), it is applied
> > twice, wrecking the instruction.
> >
> > Fixes: 42e51f187f86 ("arm/ftrace: Use __patch_text()")
>
> I don't see 42e51f187f86 anywhere. Mainline contains
>
> commit 5a735583b764750726621b0396d03e4782911b77
> Author: Peter Zijlstra <peterz@xxxxxxxxxxxxx>
> Date: Tue Oct 15 21:07:35 2019 +0200
>
> arm/ftrace: Use __patch_text()
>
> But that one is not present in 4.19, so perhaps we should not need
> this?

Good catch, I'll go drop this from everywhere.

I think Sasha has now fixed up his scripts to handle things when the
Fixes: tag does not point to a valid one.

thanks,

greg k-h