[pve-devel] [PATCH 3/6] setup_tc_rate_limit : delete class and filter before update

[Alexandre DERUMIER]

>>This expression makes no sense to me? Maybe we just need 
>>
>>return if !$rate; 

pve-manager can send rate=0 to api2, is it filtered somewhere ?
(just to be sure, in perl, is 0 != undef ? )
----- Mail original ----- 

De: "Dietmar Maurer" <dietmar at proxmox.com> 
À: "Alexandre Derumier" <aderumier at odiso.com>, pve-devel at pve.proxmox.com 
Envoyé: Mardi 5 Mars 2013 11:52:23 
Objet: RE: [pve-devel] [PATCH 3/6] setup_tc_rate_limit : delete class and filter before update 

> system("/sbin/tc qdisc del dev $iface root >/dev/null 2>&1"); 
> 
> + return if (!$rate || ($rate && $rate == 0)); 

This expression makes no sense to me? Maybe we just need 

return if !$rate;