Re: [PATCH] network: add debug when bandwidth settings are not applied

[Laine Stump <laine@xxxxxxxxx>]

On 9/13/19 12:05 PM, Daniel P. Berrangé wrote:
> To aid in troubleshooting add some debug messages wrt
> bandwidth settings and networks.
>
> Signed-off-by: Daniel P. Berrangé <berrange@xxxxxxxxxx>


Reviewed-by: Laine Stump <laine@xxxxxxxxx>


(although it sounds like you already found the problem you were looking 
for :-)


> ---
>   src/network/bridge_driver.c | 9 +++++++--
>   1 file changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/src/network/bridge_driver.c b/src/network/bridge_driver.c
> index 6a97bb17e2..c54be96407 100644
> --- a/src/network/bridge_driver.c
> +++ b/src/network/bridge_driver.c
> @@ -5192,9 +5192,14 @@ networkCheckBandwidth(virNetworkObjPtr obj,
>           return -1;
>       }
>   
> +    if (!netBand || !netBand->in) {
> +        VIR_DEBUG("No network bandwidth controls present");
> +        /* no QoS required, claim success */
> +        return 1;
> +    }
>       if (((!ifaceBand || !ifaceBand->in || !ifaceBand->in->floor) &&
> -         (!oldBandwidth || !oldBandwidth->in || !oldBandwidth->in->floor)) ||
> -        !netBand || !netBand->in) {
> +         (!oldBandwidth || !oldBandwidth->in || !oldBandwidth->in->floor))) {
> +        VIR_DEBUG("No old/new interface bandwidth floor");
>           /* no QoS required, claim success */
>           return 1;
>       }


--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list