[dpdk-dev] [PATCH 02/11] devices-args: introduce rte_devargs in eal

2014-03-01 Thread Olivier MATZ
Hi Stephen, On 02/28/2014 10:39 PM, Stephen Hemminger wrote: > Minor nit. please align case with switch(). > The DPDK in general follows Linux coding style. > In fact, I have run the code through checkpatch to find style issues. Thank you for reporting this. Following you comment, I launched chec

[dpdk-dev] [PATCH 02/11] devices-args: introduce rte_devargs in eal

2014-02-28 Thread Olivier Matz
This commit introduces a new API for storing device arguments given by the user. It only adds the framework and the test. The modification of EAL to use this new module is done in next commit. The final goals: - unify pci-blacklist, pci-whitelist, and virtual devices arguments in one file - all

[dpdk-dev] [PATCH 02/11] devices-args: introduce rte_devargs in eal

2014-02-28 Thread Stephen Hemminger
On Fri, 28 Feb 2014 18:25:41 +0100 Olivier Matz wrote: > switch (devargs->type) { > + case RTE_DEVTYPE_WHITELISTED_PCI: > + case RTE_DEVTYPE_BLACKLISTED_PCI: > + /* try to parse pci identifier */ > + if (eal_parse_pci_BDF(buf,