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

[Stefano Stabellini <sstabellini@xxxxxxxxxx>]

On Sun, 23 Jan 2022, Julien Grall wrote:
> 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()).

Good idea. I prototyped it and it works fine.  I am not going to add the
patch to this series because it needs Penny's but I can easily provide a
patch to her for it.