[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