Re: [PATCH V2 4/7] net/mlx4_en: Set max rate-limit for a TC

On Mon, 2012-03-26 at 11:07 +0200, Amir Vadai wrote:

Set max rate-limit using sysfs file /sys/class/net/<interface>/ratelimit/tc<n>
Values in Mbps units.

For example, to set ratelimit of 5G to TC3 issue:
echo 5000 > /sys/class/net/eth2/ratelimit/tc3

This looks the wrong way round; wouldn't a tc be an object with a
ratelimit as attribute?

Signed-off-by: Amir Vadai <redacted>
---

We used sysfs since max bw isn't part of the ETS / DCBX NL support, and we're
open to other suggestions to add generic support for max bw, e.g add call to
the DCBX NL API.

[...]

netlink interfaces are generally easily extensible and it doesn't make
sense to me to augment such an interface through sysfs.  Perhaps you're
concerned that netlink extensions won't be supported in older kernel
versions running your OOT driver?  That's unfortunate, but let's not
standardise an ugly interface based on a temporary problem like that.

Ben.

-- 
Ben Hutchings, Staff Engineer, Solarflare
Not speaking for my employer; that's the marketing department's job.
They asked us to note that Solarflare product names are trademarked.