[ovs-dev] [PATCH] netdev-linux: do not remove ingress qdisc when disable policing.

Ben Pfaff blp at ovn.org
Fri Nov 3 21:32:47 UTC 2017


On Fri, Oct 13, 2017 at 02:01:07PM +0800, Guoshuai Li wrote:
> rate limiting may be implemented in other ways (such as nova),
> this time need to disable rate limiting.
> I think it should not remove tc qdisk when ingress_policing_burst
> is disabled.
> 
> Signed-off-by: Guoshuai Li <ligs at dtdream.com>
> Co-authored-by: huweihua <huwh at dtdream.com>

After this patch is applied, it looks to me like removing policing
becomes a complete no-op.  That cannot be right; it means that policing,
once enabled, cannot be disabled.

Can you help me understand better?

Thanks,

Ben.


More information about the dev mailing list