[libvirt] [PATCH] domcaps: Remove function initializing domain caps as unsupported
Cole Robinson
crobinso at redhat.com
Thu Nov 21 22:19:30 UTC 2019
On 11/21/19 4:23 AM, Peter Krempa wrote:
> Commit 5751a0b6b1968bb2354b2ac21cc5938b93009590 added a helper function
> called virDomainCapsFeaturesInitUnsupported which initialized all domain
> capability features as unsupported.
>
> When adding a new feature this would initialize it as unsupported also
> for hypervisor drivers which the original author possibly didn't intend
> to modify. To prevent accidental wrong value being reported in such case
> revert back to initializing individual features in the hypervisor
> drivers themselves.
>
> This is not a straight revert as additonal patches modified how we store
> the capabilities.
>
> Signed-off-by: Peter Krempa <pkrempa at redhat.com>
> ---
> CCing all people in the original conversation:
> CC: Cole Robinson <crobinso at redhat.com>
> CC: Ján Tomko <jtomko at redhat.com>
> CC: Michal Prívozník <mprivozn at redhat.com>
>
> src/bhyve/bhyve_capabilities.c | 4 +++-
> src/conf/domain_capabilities.c | 15 ---------------
> src/conf/domain_capabilities.h | 2 --
> src/libvirt_private.syms | 1 -
> src/libxl/libxl_capabilities.c | 5 +++--
> src/qemu/qemu_capabilities.c | 3 ++-
> 6 files changed, 8 insertions(+), 22 deletions(-)
Reviewed-by: Cole Robinson <crobinso at redhat.com>
Thank you for agreeing to change this
- Cole
More information about the libvir-list
mailing list