On Thu, Jun 27, 2024 at 11:06:50AM +0200, Stefano Garzarella wrote: > On Fri, Jun 21, 2024 at 03:29:06PM GMT, Roy Hopkins wrote: > > Adds an IGVM loader to QEMU which processes a given IGVM file and > > applies the directives within the file to the current guest > > configuration. > > > > The IGVM loader can be used to configure both confidential and > > non-confidential guests. For confidential guests, the > > ConfidentialGuestSupport object for the system is used to encrypt > > memory, apply the initial CPU state and perform other confidential guest > > operations. > > > > The loader is configured via a new IgvmCfg QOM object which allows the > > user to provide a path to the IGVM file to process. > > > > Signed-off-by: Roy Hopkins <roy.hopk...@suse.com> > > --- > > qapi/qom.json | 16 + > > backends/igvm.h | 37 ++ > > include/sysemu/igvm-cfg.h | 54 +++ > > backends/igvm-cfg.c | 66 ++++ > > backends/igvm.c | 791 ++++++++++++++++++++++++++++++++++++++ > > backends/meson.build | 2 + > > 6 files changed, 966 insertions(+) > > create mode 100644 backends/igvm.h > > create mode 100644 include/sysemu/igvm-cfg.h > > create mode 100644 backends/igvm-cfg.c > > create mode 100644 backends/igvm.c > > > > diff --git a/qapi/qom.json b/qapi/qom.json > > index 8bd299265e..e586707c4c 100644 > > --- a/qapi/qom.json > > +++ b/qapi/qom.json > > @@ -874,6 +874,18 @@ > > 'base': 'RngProperties', > > 'data': { '*filename': 'str' } } > > > > +## > > +# @IgvmCfgProperties: > > +# > > +# Properties common to objects that handle IGVM files. > > +# > > +# @file: IGVM file to use to configure guest (default: none) > > +# > > +# Since: 9.1 > > +## > > +{ 'struct': 'IgvmCfgProperties', > > + 'data': { '*file': 'str' } } > > 'if': 'CONFIG_IGVM' > > I recently did a similar modification to QAPIs and Markus suggested to add > the if here as well, see > https://lore.kernel.org/qemu-devel/87zfs2z7jo....@pond.sub.org/
Yes, it avoids the code generator emitting an otherwise unusd struct when CONFIG_IGVM is unset. With regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|