Re: [XEN PATCH v2 3/5] xen/arm: configure dom0less domain for enabling xenstore after boot

[Julien Grall <julien@xxxxxxx>]

Hi,

On 13/01/2022 14:15, Bertrand Marquis wrote:
> Hi Stefano,
>
> + Penny in CC for the question.
>
> > On 13 Jan 2022, at 00:58, Stefano Stabellini <sstabellini@xxxxxxxxxx> wrote:
> >
> > From: Luca Miccio <lucmiccio@xxxxxxxxx>
> >
> > If "xen,enhanced" is enabled, then add to dom0less domains:
> >
> > - the hypervisor node in device tree
> > - the xenstore event channel
> >
> > The xenstore event channel is also used for the first notification to
> > let the guest know that xenstore has become available.
> >
> > Signed-off-by: Luca Miccio <lucmiccio@xxxxxxxxx>
> > Signed-off-by: Stefano Stabellini <stefano.stabellini@xxxxxxxxxx>
> > CC: Julien Grall <julien@xxxxxxx>
> > CC: Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>
> > CC: Bertrand Marquis <bertrand.marquis@xxxxxxx>
> Reviewed-by: Bertrand Marquis <bertrand.marquis@xxxxxxx>
>
> Just one question: GUEST_GNTTAB_BASE is fixed but could it be a problem for a 
> direct map guest in the future ?
It will be an issue. I think we can re-use the same method as we do in 
dom0 (see find_gnttab_region()).
Cheers,

--
Julien Grall