On Fri, 24 May 2024, Julien Grall wrote:
> Hi Henry,
>
> On 23/05/2024 08:40, Henry Wang wrote:
> > With the XEN_DOMCTL_dt_overlay DOMCTL added, users should be able to
> > attach/detach devices from the provided DT overlay to domains.
> > Support this by introducing a new set of "xl dt-overlay" commands and
> > related documentation, i.e. "xl dt-overlay {attach,detach}". Slightly
> > rework the command option parsing logic.
> >
> > Signed-off-by: Henry Wang <xin.wa...@amd.com>
> > Reviewed-by: Jason Andryuk <jason.andr...@amd.com>
Reviewed-by: Stefano Stabellini <sstabell...@kernel.org>
> > ---
> > v4:
> > - Add Jason's Reviewed-by tag.
> > v3:
> > - Introduce new API libxl_dt_overlay_domain() and co., instead of
> > reusing existing API libxl_dt_overlay().
> > - Add in-code comments for the LIBXL_DT_OVERLAY_* macros.
> > - Use find_domain() to avoid getting domain_id from strtol().
> > v2:
> > - New patch.
> > ---
> > tools/include/libxl.h | 10 +++++++
> > tools/include/xenctrl.h | 3 +++
> > tools/libs/ctrl/xc_dt_overlay.c | 31 +++++++++++++++++++++
> > tools/libs/light/libxl_dt_overlay.c | 28 +++++++++++++++++++
> > tools/xl/xl_cmdtable.c | 4 +--
> > tools/xl/xl_vmcontrol.c | 42 ++++++++++++++++++++---------
> > 6 files changed, 104 insertions(+), 14 deletions(-)
> >
> > diff --git a/tools/include/libxl.h b/tools/include/libxl.h
> > index 62cb07dea6..6cc6d6bf6a 100644
> > --- a/tools/include/libxl.h
> > +++ b/tools/include/libxl.h
>
> I think you also need to introduce LIBXL_HAVE_...
Added
I have removed the LIBXL_DT_OVERLAY_DOMAIN_DETACH and the relate
mentions. I kept Jasons' ack.
