On Wed, May 29, 2024 at 02:17:29PM +0300, Kirill A. Shutemov wrote:
> > That jmp 1f becomes redundant now as it simply jumps 1 line below.
> >
>
> Nothing changed wrt this jump. It dates back to initial kexec
> implementation.
>
> See 5234f5eb04ab ("[PATCH] kexec: x86_64 kexec implementation").
>
> But I don't see functional need in it.
>
> Anyway, it is outside of the scope of the patch.
Yap, Kirill did what Nikolay should've done - git archeology. Please
don't forget to do that next time.
And back in the day they didn't comment non-obvious things because
commenting is for losers. :-\
So that unconditional forward jump either flushes branch prediction on
some old uarch or something else weird, uarch-special.
I doubt we can remove it just like that.
Lemme add Andy - he should know.
--
Regards/Gruss,
Boris.
https://people.kernel.org/tglx/notes-about-netiquette
