[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