[dpdk-dev] [PATCH v5 3/4] ethdev: redesign link speed config API
Marc
marcdevel at gmail.com
Tue Feb 2 23:30:59 CET 2016
On 2 February 2016 at 03:20, Stephen Hemminger <stephen at networkplumber.org>
wrote:
> On Thu, 28 Jan 2016 17:33:20 +0000
> Harish Patil <harish.patil at qlogic.com> wrote:
>
> > * Added utility MACROs ETH_SPEED_NUM_XXX with the numeric
> > values of all supported link speeds, in Mbps.
>
> I would prefer that there were no speed value macros.
> Linux used to have these, but people kept adding new hardware speeds
> and it soon gets out of date.
>
I see what you mean, but I am not sure I agree. Link speeds are generally a
reduced amount of items (~20). Though it is true it can eventually grow,
but at small rate. Having numeric constants all over the source seems less
readable and less maintainable (e.g. less "grepable"/"sedable") to me.
>
> If you are going to redo it, then just increase speed to 64 bit, and allow
> any non-zero value.
>
Value is now 32 bits, which I think is enough for future rates in mbps.
Since these constants were there, and before doing something to have to
revert it, can someone else give his/her opinion on this?
If there is consensus, I've no problem on removing it for v8
Thanks
marc
More information about the dev
mailing list