[libvirt PATCH 3/5] qemu: call networkPlugBandwidth() for all types of network

Michal Privoznik mprivozn at redhat.com
Wed Feb 12 08:20:55 UTC 2020


On 2/10/20 5:10 PM, Pavel Mores wrote:
> To fix the actual bug, it was necessary to make networkPlugBandwidth() be
> called also for 'bridge'-type networks implemented using macvtap's 'bridge'
> mode (previously it was only called for those implemented on top of an
> existing bridge).
> 
> However, it seems beneficial to call it for other network types as well,
> at least because it removes an inconsistency in types of bandwidth
> configuration changes permissible in inactive and active domain configs.
> 
> Signed-off-by: Pavel Mores <pmores at redhat.com>
> ---
>   src/network/bridge_driver.c | 7 +++----
>   1 file changed, 3 insertions(+), 4 deletions(-)
> 

I'd mention in the commit message that this is safe to do, because if no 
QoS is set, then the function is NOP. Or something among those lines. It 
might help us in the future to pin a point in time when this used to work.

Reviewed-by: Michal Privoznik <mprivozn at redhat.com>

Michal




More information about the libvir-list mailing list