Re: [dpdk-dev] [PATCH] config: sort PMD config options

2017-11-21 Thread Ferruh Yigit
On 11/21/2017 3:17 PM, Aleksey Baulin wrote: > Hi Ferruh, > > Not that it really mattered much, but I couldn't help but notice several > irregularities in the order of options that could have been avoided. We are on same boat, couldn't help on not noticing :) > In particular, I believe it would

Re: [dpdk-dev] [PATCH] config: sort PMD config options

2017-11-21 Thread Aleksey Baulin
Hi Ferruh, Not that it really mattered much, but I couldn't help but notice several irregularities in the order of options that could have been avoided. In particular, I believe it would make sense to put all xxx_DEBUG_xxx options after the real control options, thus making it two classes of optio

[dpdk-dev] [PATCH] config: sort PMD config options

2017-11-21 Thread Ferruh Yigit
No config option changed, added or removed. Only reshuffle PMD config options mostly to help new PMDs where to put their new config option. Ordered as physical, paravirtual and virtual groups. Alphabetical order within a group. Also tried to group vendor devices together which breaks alphabetical