On Wed, 2019-04-17 at 10:57 -0400, Cole Robinson wrote:
[...]
> + virDomainDefParserConfig config = {
> + .features = VIR_DOMAIN_DEF_FEATURE_MEMORY_HOTPLUG |
> + VIR_DOMAIN_DEF_FEATURE_OFFLINE_VCPUPIN |
> + VIR_DOMAIN_DEF_FEATURE_INDIVIDUAL_VCPUS |
> + VIR_DOMAIN_DEF_FEATURE_USER_ALIAS |
> + VIR_DOMAIN_DEF_FEATURE_FW_AUTOSELECT |
> + VIR_DOMAIN_DEF_FEATURE_NET_MODEL_STRING,
> + };
virDomainDefParserConfig instances in other drivers usually follow
the (vir)${DRIVER}DomainDefParserConfig, so I would do the same here
and call it something like testDomainDefParserConfig.
Either way,
Reviewed-by: Andrea Bolognani <abolo...@redhat.com>
and safe for freeze.
The second patch in the series is not as urgent, nor is it as
obviously correct, so I'll probably look at it once 5.3.0 is out.
--
Andrea Bolognani / Red Hat / Virtualization
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
