On Fri, 2017-07-21 at 10:07 +0200, Andrea Bolognani wrote:
> That is the main reason to fill all the values in right away.
Since
> there apparently was a period, where a default would be used, but not
> recorded, it needs some trickery unfortunately.
>
> In such case you basically standardize, that the now-filled-in default
> model (which can't ever change) if it was not provided by the user is to
> be dropped from the migratable XML.
>
> Once you start assign a new default model, that then needs to be
> explicitly sent over, so that the migration will not be successufl
> unless the destination is able to use the new model.
>
> This means basically, that a missing model name becomes assigned to a
> particular value.
That makes sense.
Doesn't it also mean that we don't really need to record
whether the user set the model name explicitly or not? We
can just skip formatting it if it's spapr-pci-host-bridge
and all versions of libvirt, past or future, will handle
that correctly.
Wait, that's exactly how this patch behaves, except it's
not immediately apparent because the check for the model
name happens inside virDomainControllerIsPCIHostBridge()
rather than being explicit.
So maybe I'm still missing something?
--
Andrea Bolognani / Red Hat / Virtualization