Re: [PATCH v2 2/3] x86/mem_sharing: add fork_complete field to mem_sharing_domain

[Tamas K Lengyel <tamas@xxxxxxxxxxxxx>]

On Tue, Mar 29, 2022 at 11:51 AM Jan Beulich <jbeulich@xxxxxxxx> wrote:
>
> On 29.03.2022 16:03, Tamas K Lengyel wrote:
> > The fork's physmap should only be populated with select special pages during
> > the setup of the fork. The rest of the fork's physmap should only be 
> > populated
> > as needed after the fork is complete. Add a field to specify when the fork 
> > is
> > complete so fork_page() can determine whether it's time to start adding 
> > entries
> > to the physmap.
> >
> > Signed-off-by: Tamas K Lengyel <tamas.lengyel@xxxxxxxxx>
>
> Acked-by: Jan Beulich <jbeulich@xxxxxxxx>

Thanks! After further consideration I'll drop this and the empty_p2m
patch from the series as there is a corner-case with PAE-mode guests
which doesn't play nice with enforcing a start with an empty p2m.
Working around that issue leads to a cascading changeset which
ultimately isn't a worthwhile effort.

Tamas