[libvirt PATCH 1/5] conf: define a new "maximum" CPU mode
Daniel P. Berrangé
berrange at redhat.com
Wed Feb 10 10:39:22 UTC 2021
On Tue, Feb 09, 2021 at 03:30:47PM +0100, Pavel Hrdina wrote:
> On Tue, Feb 09, 2021 at 01:58:57PM +0000, Daniel P. Berrangé wrote:
> > For hardware virtualization this is functionally identical to the
> > existing host-passthrough mode so the same caveats apply.
> >
> > For emulated guest this exposes the maximum featureset supported by
> > the emulator. Note that despite being emulated this is not guaranteed
> > to be migration safe, especially if different emulator software versions
> > are used on each host.
> >
> > Signed-off-by: Daniel P. Berrangé <berrange at redhat.com>
> > ---
> > docs/formatdomain.rst | 24 ++++++++++++++++++++++++
> > src/conf/cpu_conf.c | 13 +++++++++----
> > src/conf/cpu_conf.h | 1 +
> > src/cpu/cpu.c | 1 +
> > src/qemu/qemu_capabilities.c | 1 +
> > src/qemu/qemu_command.c | 1 +
> > src/qemu/qemu_domain.c | 1 +
> > src/qemu/qemu_validate.c | 1 +
> > 8 files changed, 39 insertions(+), 4 deletions(-)
>
> [...]
>
> > diff --git a/src/conf/cpu_conf.c b/src/conf/cpu_conf.c
> > index f98b0a0eb3..eb4bfbbcfa 100644
> > --- a/src/conf/cpu_conf.c
> > +++ b/src/conf/cpu_conf.c
> > @@ -402,10 +403,11 @@ virCPUDefParseXML(xmlXPathContextPtr ctxt,
> > if ((migratable = virXMLPropString(ctxt->node, "migratable"))) {
> > int val;
> >
> > - if (def->mode != VIR_CPU_MODE_HOST_PASSTHROUGH) {
> > + if (def->mode != VIR_CPU_MODE_HOST_PASSTHROUGH &&
> > + def->mode != VIR_CPU_MODE_MAXIMUM) {
> > virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
> > _("Attribute migratable is only allowed for "
> > - "host-passthrough CPU"));
> > + "'host-passthrough' / 'maximum' CPU mode"));
>
> Since you are modifying the error message I would reformat it to single
> line.
That'd would make the line waaaaay longer than normal practice. I know
we don't need to strictly stick to 80 chars, but at the same time we
don't want things massively long.
> Reviewed-by: Pavel Hrdina <phrdina at redhat.com>
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 :|
More information about the libvir-list
mailing list