Re: [PATCH v2 3/4] x86/paravirt: switch mixed paravirt/alternative calls to alternative_2

[Juergen Gross]

On 18.10.23 13:34, Ingo Molnar wrote:
> * Juergen Gross <jgross@xxxxxxxx> wrote:
>
> > On 16.10.23 16:29, Peter Zijlstra wrote:
> > > On Mon, Oct 16, 2023 at 02:39:32PM +0200, Juergen Gross wrote:
> > > > Instead of stacking alternative and paravirt patching, use the new
> > > > ALT_FLAG_CALL flag to switch those mixed calls to pure alternative
> > > > handling.
> > > >
> > > > This eliminates the need to be careful regarding the sequence of
> > > > alternative and paravirt patching.
> > > >
> > > > For call depth tracking callthunks_setup() needs to be adapted to patch
> > > > calls at alternative patching sites instead of paravirt calls.
> > > >
> > > > Signed-off-by: Juergen Gross <jgross@xxxxxxxx>
> > >
> > > I cannot help but feel this would've been better as two patches, one
> > > introducing ALT_NOT_XEN and then a second with the rest.
> >
> > In case I need to respin I'll split it up.
>
> Yeah, would be nice to split this patch into two, as Peter requested.

Okay, will resend the series probably tomorrow.


Juergen

Attachment: OpenPGP_0xB0DE9DD628BF132F.asc
Description: OpenPGP public key

Attachment: OpenPGP_signature.asc
Description: OpenPGP digital signature