[PATCH] devargs: Fix rte_devargs_parse uninitialized calls

2022-02-10 Thread Gaetan Rivet
("devargs: parse global device syntax") Signed-off-by: Gaetan Rivet --- lib/eal/common/eal_common_devargs.c | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/eal/common/eal_common_devargs.c b/lib/eal/common/eal_common_devargs.c index 8c7650cf6c..184fe676aa 100644 --- a/lib/

[dpdk-dev] [PATCH] bus/pci: fix comment explaining device naming

2020-11-16 Thread Gaetan Rivet
The original triple negative was hard to read and the attempt to improve the formulation was commendable, unfortunately the new comment is the inverse of correct. Fixes: a65a34a85ebf ("eal: replace usage of blacklist/whitelist in enums") Cc: step...@networkplumber.org Signed-off-by: Ga

[dpdk-dev] [PATCH v2] bus/dpaa: optimize device name parsing

2020-11-09 Thread Gaetan Rivet
uses snprintf without a real need for it. Both can be avoided, which improves the parsing performance. The function is also simpler and shorter. Signed-off-by: Gaetan Rivet --- drivers/bus/dpaa/dpaa_bus.c | 68 +++-- 1 file changed, 28 insertions(+), 40 deletions

[dpdk-dev] [PATCH] bus/dpaa: optimize device name parsing

2020-11-09 Thread Gaetan Rivet
uses snprintf without a real need for it. Both can be avoided, which improves the parsing performance. The function is also simpler and shorter. Signed-off-by: Gaetan Rivet --- drivers/bus/dpaa/dpaa_bus.c | 69 +++-- 1 file changed, 27 insertions(+), 42 deletions

[dpdk-dev] [PATCH v2] net/failsafe: check eth dev stop status

2020-10-15 Thread Gaetan Rivet
From: Ivan Ilchenko rte_eth_dev_stop() return value was changed from void to int, so this patch modify usage of this function across net/failsafe according to new return type. Signed-off-by: Ivan Ilchenko Signed-off-by: Andrew Rybchenko --- Hello Andrew, Thanks for doing this. I had some com

[dpdk-dev] [PATCH] net/failsafe: fix state synchro cleanup

2020-10-12 Thread Gaetan Rivet
state repeatedly. Consider all devices when doing a removal round, but limit burst control and stats saving to active devices. Fixes: 598fb8aec6f6 ("net/failsafe: support device removal") Cc: sta...@dpdk.org Signed-off-by: Gaetan Rivet --- drivers/net/failsafe/failsafe_et

[dpdk-dev] [PATCH v1 1/2] pci: fix allowing underflow when parsing PCI id

2020-05-13 Thread Gaetan Rivet
ed. Fixes: af75078fece3 ("first public release") Cc: sta...@dpdk.org Signed-off-by: Gaetan Rivet --- lib/librte_pci/rte_pci.c | 12 1 file changed, 12 insertions(+) diff --git a/lib/librte_pci/rte_pci.c b/lib/librte_pci/rte_pci.c index d1ab6b414..e4ecdc32f 100644 --- a/lib/

[dpdk-dev] [PATCH v1 2/2] pci: explain how empty strings are rejected in DBDF

2020-05-13 Thread Gaetan Rivet
] != ':'` afterward will return -EINVAL. Enforcing consistency between parsers by reading the code is not helped by this property being implicit. Add a comment to explain. Signed-off-by: Gaetan Rivet --- lib/librte_pci/rte_pci.c | 4 1 file changed, 4 insertions(+) diff -

[dpdk-dev] [PATCH v1 0/2] pci: a comment and a minor fix

2020-05-13 Thread Gaetan Rivet
Add a clarifying comment and fix the PCI addr parser behavior for some edge-case. Gaetan Rivet (2): pci: fix allowing underflow when parsing PCI id pci: explain how empty strings are rejected in DBDF lib/librte_pci/rte_pci.c | 16 1 file changed, 16 insertions

[dpdk-dev] [PATCH v1] doc: fix typos and errors in abi policy doc

2020-05-13 Thread Gaetan Rivet
: * Long sentences that would be better cut short. * Comma abuse. * 'May' used where 'can' seems more fitting. I'm not a native speaker though, so grain of salt applies. Fixes: fdf7471cccb8 ("doc: introduce major ABI versions") Cc: Ray Kinsella cc: Neil

[dpdk-dev] [PATCH v1] doc: remove deprecation notice about old devargs changes

2020-05-13 Thread Gaetan Rivet
When modifying the rte_devargs implementation, a deprecation notice was done for v18.11, regarding internal rte_devargs structure and exposed functions. Most of the changes were part of v18.11, but the notice was not removed. Signed-off-by: Gaetan Rivet --- doc/guides/rel_notes/deprecation.rst

[dpdk-dev] [PATCH v1] kvargs: fix strcmp helper documentation

2020-05-13 Thread Gaetan Rivet
Minor error, unless was used instead of unlike. Cc: sta...@dpdk.org Fixes: a3b85476c51e ("kvargs: add generic string matching callback") Signed-off-by: Gaetan Rivet --- lib/librte_kvargs/rte_kvargs.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/lib

[dpdk-dev] [PATCH v2] net/ring: fix eth_dev device pointer on allocation

2020-05-06 Thread Gaetan Rivet
pointer properly in do_eth_dev_ring_create(). Cc: sta...@dpdk.org Fixes: fbe90cdd776c ("ethdev: add probing finish function") Signed-off-by: Gaetan Rivet --- drivers/net/ring/rte_eth_ring.c | 29 +++-- 1 file changed, 15 insertions(+), 14 deletions(-)

[dpdk-dev] [PATCH v1 3/3] net/failsafe: fix default service proxy state

2020-05-05 Thread Gaetan Rivet
ix fd leak") Signed-off-by: Gaetan Rivet Cc: wangyunj...@huawei.com Cc: Ali Alnubani --- drivers/net/failsafe/failsafe.c | 1 + drivers/net/failsafe/failsafe_private.h | 8 2 files changed, 9 insertions(+) diff --git a/drivers/net/failsafe/failsafe.c b/drivers/net/failsafe/

[dpdk-dev] [PATCH v1 2/3] net/ring: fix eth_dev device pointer on allocation

2020-05-05 Thread Gaetan Rivet
nd write the pointer properly in do_eth_dev_ring_create(). Cc: sta...@dpdk.org Fixes: fbe90cdd776c ("ethdev: add probing finish function") Cc: ferruh.yi...@intel.com Cc: tho...@monjalon.net Signed-off-by: Gaetan Rivet --- drivers/net/ring/rte_eth_ring.c | 36 ++---

[dpdk-dev] [PATCH v1 0/3] failsafe & ring fixes

2020-05-05 Thread Gaetan Rivet
Some issues seen on next-net. Gaetan Rivet (3): net/failsafe: avoid crash on malformed eth_dev net/ring: fix eth_dev device pointer on allocation net/failsafe: fix default service proxy state drivers/net/failsafe/failsafe.c | 1 + drivers/net/failsafe/failsafe_ether.c | 5

[dpdk-dev] [PATCH v1 1/3] net/failsafe: avoid crash on malformed eth_dev

2020-05-05 Thread Gaetan Rivet
. Instead, warn the user of the error repeatedly. The failsafe PMD can warn of the issue and continue. It will repeatedly attempt to initialize the failed port and complain about it, which should result in the same developer focus but with less crashing. Signed-off-by: Gaetan Rivet --- drivers/net

[dpdk-dev] [PATCH v1] ethdev: add rte_device to port_id function

2020-04-17 Thread Gaetan Rivet
PMDs that could be simplified with it. Cc: Chas Williams Cc: Liron Himi Cc: Andrew Rybchenko Cc: Thomas Monjalon Signed-off-by: Gaetan Rivet --- Simplifies a recurring usage pattern. This patch was integrated on top of a small bonding fixes patchset: https://mails.dpdk.org/archives/dev

[dpdk-dev] [PATCH v1 2/2] bonding: fix PCI address comparison on non-pci ports

2020-04-17 Thread Gaetan Rivet
ned-off-by: Gaetan Rivet --- drivers/net/bonding/rte_eth_bond_args.c | 58 +++-- 1 file changed, 25 insertions(+), 33 deletions(-) diff --git a/drivers/net/bonding/rte_eth_bond_args.c b/drivers/net/bonding/rte_eth_bond_args.c index 35616fb8b..8c5f90dc6 100644 --- a/drivers/n

[dpdk-dev] [PATCH v1 1/2] bonding: fix port id validity check on parsing

2020-04-17 Thread Gaetan Rivet
: new link bonding library") Cc: sta...@dpdk.org Cc: Chas Williams Signed-off-by: Gaetan Rivet --- drivers/net/bonding/rte_eth_bond_args.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/net/bonding/rte_eth_bond_args.c b/drivers/net/bonding/rte_eth_bond_ar

[dpdk-dev] [PATCH v1 0/2] bonding: fix port id check and PCI addr cmp

2020-04-17 Thread Gaetan Rivet
rly at the time: the comment about lacking the proper rte_bus_pci impl was already not correct anymore when the check on the kdrv was removed. The necessary functions were already available I think. Gaetan Rivet (2): bonding: fix port id validity check on parsing bonding: fix PCI address com

[dpdk-dev] [PATCH v1 2/2] doc/failsafe: fix incorrect statement regarding number of sub-device

2020-02-21 Thread Gaetan Rivet
Supporting more than 2 sub-device is a remnant from earlier fail-safe design, that was never actually published. Fixes: a46f8d584eb8 ("net/failsafe: add fail-safe PMD") Cc: sta...@dpdk.org Signed-off-by: Gaetan Rivet --- doc/guides/nics/fail_safe.rst | 2 +- 1 file changed, 1 inser

[dpdk-dev] [PATCH v1 0/2] failsafe: improve documentation

2020-02-21 Thread Gaetan Rivet
The fail-safe documentation is subpar. It can be improved (1), and an error is present from the beginning (2). Gaetan Rivet (2): doc/failsafe: improve fail-safe documentation doc/failsafe: fix incorrect statement regarding number of sub-device doc/guides/nics/fail_safe.rst | 109

[dpdk-dev] [PATCH v1 1/2] doc/failsafe: improve fail-safe documentation

2020-02-21 Thread Gaetan Rivet
Reading the fail-safe doc with a few years added, a few phrasing choices are ambiguous or confusing. Signed-off-by: Gaetan Rivet --- doc/guides/nics/fail_safe.rst | 107 +- 1 file changed, 66 insertions(+), 41 deletions(-) diff --git a/doc/guides/nics

Re: [dpdk-dev] [PATCH v2] eal: fix to set the rte_device ptr's device args before hotplug

2020-02-14 Thread Gaetan Rivet
ff-by: Somnath Kotur --- v1->v2: Incorporated suggestions from Gaetan Rivet drivers/bus/pci/linux/pci.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/bus/pci/linux/pci.c b/drivers/bus/pci/linux/pci.c index 740a2cd..71b0a30 100644 --- a/drivers/bus/pci/linux/pci.c +++ b/driv

Re: [dpdk-dev] [PATCH] eal: fix to set the rte_device ptr's device args before hotplug

2020-02-11 Thread Gaetan Rivet
[...] (gdb) p dev2 $5 = (struct rte_pci_device *) 0x54de5e0 (gdb) p /x *dev2 $6 = {next = {tqe_next = 0x5307460, tqe_prev = 0x5539f80}, device = {next = {tqe_next = 0x0, tqe_prev = 0x0}, name = 0x54e4f00, driver = 0x3a6b7f0, bus = 0x3a59a00, numa_node = 0x0, devargs = 0x0}, addr = {domain =

Re: [dpdk-dev] [PATCH] eal: fix to set the rte_device ptr's device args before hotplug

2020-02-10 Thread Gaetan Rivet
On 10/02/2020 15:34, Gaetan Rivet wrote: Hi Somnath, Reformating the mails, to keep with the inner-posting (mixing top-posting and inner-posting makes it hard to follow). See the end of the mail. [...] Any updates on this ? My thoughts on this are just as I'd suspected / sugg

Re: [dpdk-dev] [PATCH] eal: fix to set the rte_device ptr's device args before hotplug

2020-02-10 Thread Gaetan Rivet
Hi Somnath, Reformating the mails, to keep with the inner-posting (mixing top-posting and inner-posting makes it hard to follow). See the end of the mail. On Wed, Feb 5, 2020 at 2:47 PM Gaetan Rivet wrote: On 05/02/2020 09:52, Somnath Kotur wrote: Hello Gaetan, On Tue, Feb 4, 2020 at 3

Re: [dpdk-dev] [PATCH] eal: fix to set the rte_device ptr's device args before hotplug

2020-02-05 Thread Gaetan Rivet
On 05/02/2020 09:52, Somnath Kotur wrote: Hello Gaetan, On Tue, Feb 4, 2020 at 3:19 PM Gaetan Rivet wrote: On 04/02/2020 10:15, Somnath Kotur wrote: As per the comments in this code section, "since there is a matching device, it is now its responsibility to manage the devargs we&#x

Re: [dpdk-dev] [PATCH v7] eal: add manual probing option

2020-02-04 Thread Gaetan Rivet
On 04/02/2020 16:06, Thomas Monjalon wrote: 04/02/2020 13:43, Gaetan Rivet: On 04/02/2020 12:07, Thomas Monjalon wrote: 04/02/2020 11:03, Gaetan Rivet: On 03/02/2020 23:21, Thomas Monjalon wrote: 03/02/2020 06:16, Pavan Nikhilesh Bhagavatula: @David Marchand @tho...@monjalon.net Ping? Are

Re: [dpdk-dev] [PATCH v7] eal: add manual probing option

2020-02-04 Thread Gaetan Rivet
On 04/02/2020 12:07, Thomas Monjalon wrote: 04/02/2020 11:03, Gaetan Rivet: On 03/02/2020 23:21, Thomas Monjalon wrote: 03/02/2020 06:16, Pavan Nikhilesh Bhagavatula: @David Marchand @tho...@monjalon.net Ping? Are there any more changes required for this patch? It's been in queue

Re: [dpdk-dev] [PATCH v7] eal: add manual probing option

2020-02-04 Thread Gaetan Rivet
On 03/02/2020 23:21, Thomas Monjalon wrote: 03/02/2020 06:16, Pavan Nikhilesh Bhagavatula: @David Marchand @tho...@monjalon.net Ping? Are there any more changes required for this patch? It's been in queue since last October. Sorry we have not decided whether it is a good idea or not. All c

Re: [dpdk-dev] [PATCH] eal: fix to set the rte_device ptr's device args before hotplug

2020-02-04 Thread Gaetan Rivet
On 04/02/2020 10:15, Somnath Kotur wrote: As per the comments in this code section, "since there is a matching device, it is now its responsibility to manage the devargs we've just inserted." But the matching device ptr's devargs is still uninitialized or not pointing to the newest dev_args that

Re: [dpdk-dev] [RFC PATCH 1/5] graph: introduce graph subsystem

2020-02-03 Thread Gaetan Rivet
On 02/02/2020 12:21, Jerin Jacob wrote: On Sun, Feb 2, 2020 at 4:08 PM Stephen Hemminger wrote: On Fri, 31 Jan 2020 22:31:57 +0530 wrote: + +#define set_err(err, where, fmt, ...) do { \ + graph_err(fmt, ##__VA_ARGS__);

[dpdk-dev] [PATCH v7] eal: add manual probing option

2020-01-23 Thread Gaetan Rivet
--vdev), will be probed using the hotplug API, in the order they are declared. This has the effect of offering a way for users to control probe order of their devices, for drivers requiring it. Signed-off-by: Gaetan Rivet Acked-by : Vamsi Attunuru Tested-by: Vamsi Attunuru Reviewed-by: Jerin

Re: [dpdk-dev] [PATCH v5] eal: add manual probing option

2020-01-23 Thread Gaetan Rivet
nyway in the meantime, not sure whether there is anything to change. Thanks for the rebase, I will follow-up if there are remarks blocking the integration. -Original Message- From: dev On Behalf Of Gaetan Rivet Sent: Friday, October 25, 2019 9:17 PM To: dev@dpdk.org Cc: Gaetan Rivet ;

[dpdk-dev] [PATCH] maintainers: update for failsafe and pci lib

2020-01-08 Thread Gaetan Rivet
My email address has changed, gaetan.ri...@6wind.com is no longer valid. Signed-off-by: Gaetan Rivet --- Sorry, I did not take the time to change it while I controlled it. MAINTAINERS | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index

[dpdk-dev] [PATCH v5] eal: add manual probing option

2019-10-25 Thread Gaetan Rivet
--vdev), will be probed using the hotplug API, in the order they are declared. This has the effect of offering a way for users to control probe order of their devices, for drivers requiring it. Signed-off-by: Gaetan Rivet --- I haven't heard many opinions on the matter, please shout if yo

[dpdk-dev] [PATCH v4] eal: add manual probing option

2019-10-25 Thread Gaetan Rivet
--vdev), will be probed using the hotplug API, in the order they are declared. This has the effect of offering a way for users to control probe order of their devices, for drivers requiring it. Signed-off-by: Gaetan Rivet --- I haven't heard many opinions on the matter, please shout if yo

[dpdk-dev] [PATCH v3] eal: add manual probing option

2019-10-04 Thread Gaetan Rivet
--vdev), will be probed using the hotplug API, in the order they are declared. This has the effect of offering a way for users to control probe order of their devices, for drivers requiring it. Signed-off-by: Gaetan Rivet --- I haven't heard many opinions on the matter, please shout if yo

[dpdk-dev] [PATCH v2] eal: add manual probing option

2019-10-03 Thread Gaetan Rivet
--vdev), will be probed using the hotplug API, in the order they are declared. This has the effect of offering a way for users to control probe order of their devices, for drivers requiring it. Signed-off-by: Gaetan Rivet --- I haven't heard many opinions on the matter, please shout if yo

[dpdk-dev] [PATCH v1] eal: add manual probing option

2019-09-30 Thread Gaetan Rivet
--vdev), will be probed using the hotplug API, in the order they are declared. This has the effect of offering a way for users to control probe order of their devices, for drivers requiring it. Signed-off-by: Gaetan Rivet --- I haven't heard many opinions on the matter, please shout if yo

[dpdk-dev] [PATCH v1 8/8] option: return an error when register fail

2018-12-20 Thread Gaetan Rivet
Make rte_option_register return a negative value when an error occur. Signed-off-by: Gaetan Rivet --- lib/librte_eal/common/include/rte_option.h | 6 +- lib/librte_eal/common/rte_option.c | 8 +--- 2 files changed, 10 insertions(+), 4 deletions(-) diff --git a/lib/librte_eal

[dpdk-dev] [PATCH v1 5/8] option: check against common option on register

2018-12-20 Thread Gaetan Rivet
Not only check against other registered options, but also common EAL options. This will mitigate user confusion. Signed-off-by: Gaetan Rivet --- lib/librte_eal/common/rte_option.c | 14 ++ 1 file changed, 14 insertions(+) diff --git a/lib/librte_eal/common/rte_option.c b/lib

[dpdk-dev] [PATCH v1 6/8] option: fix log level of error in register

2018-12-20 Thread Gaetan Rivet
INFO is not correct when logging an error. Signed-off-by: Gaetan Rivet --- lib/librte_eal/common/rte_option.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/librte_eal/common/rte_option.c b/lib/librte_eal/common/rte_option.c index ae8a0e2ca..b2c93b220 100644 --- a/lib

[dpdk-dev] [PATCH v1 7/8] option: improve library documentation

2018-12-20 Thread Gaetan Rivet
Use doxygen to describe the main structure and describe a little more why it exists. Signed-off-by: Gaetan Rivet --- lib/librte_eal/common/include/rte_option.h | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/lib/librte_eal/common/include/rte_option.h b/lib

[dpdk-dev] [PATCH v1 4/8] option: rename name field

2018-12-20 Thread Gaetan Rivet
option->opt_* is redundant. The field should also be constant. Signed-off-by: Gaetan Rivet --- lib/librte_eal/common/include/rte_option.h | 2 +- lib/librte_eal/common/rte_option.c | 8 lib/librte_telemetry/rte_telemetry.c | 2 +- 3 files changed, 6 insertions(+)

[dpdk-dev] [PATCH v1 3/8] option: add usage string

2018-12-20 Thread Gaetan Rivet
Add a usage string field in rte_option, allowing to display help to the user and describe which options are currently available. Signed-off-by: Gaetan Rivet --- lib/librte_eal/common/eal_common_options.c | 1 + lib/librte_eal/common/eal_private.h| 7 +++ lib/librte_eal/common

[dpdk-dev] [PATCH v1 1/8] option: use bare option string as name

2018-12-20 Thread Gaetan Rivet
Current options name can be passed with arbitrary format. Force the use of "--" prefix and thus POSIX long options format. This restricts the ability to introduce surprising options and will help future additional checks. Signed-off-by: Gaetan Rivet --- lib/librte_eal/common/rte_opti

[dpdk-dev] [PATCH v1 2/8] option: do not use static iterator

2018-12-20 Thread Gaetan Rivet
This is rather weird. Someone should have caught that during review. Signed-off-by: Gaetan Rivet --- lib/librte_eal/common/rte_option.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/lib/librte_eal/common/rte_option.c b/lib/librte_eal/common/rte_option.c index

[dpdk-dev] [PATCH v1 0/8] Clean up rte_option

2018-12-20 Thread Gaetan Rivet
?) - fix doc, improve API a little... Gaetan Rivet (8): option: use bare option string as name option: do not use static iterator option: add usage string option: rename name field option: check against common option on register option: fix log level of error in register option: improve

[dpdk-dev] [PATCH] mk: use misleading indentation warning when available

2018-12-18 Thread Gaetan Rivet
-by: Gaetan Rivet --- I completely agree that we should change the coding style and force all if()s to have brackets. In the meantime, this patch might help alleviate the issue. mk/toolchain/gcc/rte.vars.mk | 4 1 file changed, 4 insertions(+) diff --git a/mk/toolchain/gcc/rte.vars.mk b/mk

[dpdk-dev] [PATCH] eal/option: fix option register duplicate detection

2018-12-17 Thread Gaetan Rivet
Missing brackets around the if means that the loop will end at its first iteration. Cc: sta...@dpdk.org Fixes: 2395332798d0 ("eal: add option register infrastructure") Signed-off-by: Gaetan Rivet --- lib/librte_eal/common/rte_option.c | 3 ++- 1 file changed, 2 insertions(+),

[dpdk-dev] [PATCH v1] devargs: fix variadic parsing memory leak

2018-10-17 Thread Gaetan Rivet
rte_devargs_parsef will leak memory each time it is called. The device string must be freed. Fixes: a23bc2c4e01b ("devargs: add non-variadic parsing function") CC: sta...@dpdk.org Signed-off-by: Gaetan Rivet --- lib/librte_eal/common/eal_common_devargs.c | 6 +- 1 file changed, 5

[dpdk-dev] [PATCH v2] testpmd: sfp eeprom display

2018-10-05 Thread Gaetan Rivet
The interactive command show port sfp_eeprom will dump the content of the SFP EEPROM for the selected port. Dumping SFP eeprom of all ports at once is not supported. Signed-off-by: Gaetan Rivet --- app/test-pmd/cmdline.c | 9 +++-- app/test-pmd/config.c | 32

[dpdk-dev] [PATCH v1] devargs: remove comment regarding rte-log

2018-09-19 Thread Gaetan Rivet
rte_log() is available in the context of this compilation unit, do not deter from using it. Signed-off-by: Gaetan Rivet --- lib/librte_eal/common/eal_common_devargs.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/lib/librte_eal/common/eal_common_devargs.c b/lib/librte_eal/common

[dpdk-dev] [PATCH v2 13/13] eal: add generic dev parameter

2018-09-19 Thread Gaetan Rivet
Add the --dev parameter to the EAL. This new parameter takes a generic device declaration as argument. It uses the new devargs parsing API. Signed-off-by: Gaetan Rivet --- lib/librte_eal/common/eal_common_devargs.c | 4 +++ lib/librte_eal/common/eal_common_options.c | 36

[dpdk-dev] [PATCH v2 11/13] bus/pci: process declarative PCI devargs

2018-09-19 Thread Gaetan Rivet
Introduce the facility to process future PCI parameters. Once the matching between PCI devices and devargs has been done, it is possible to process each devargs. New parameters would have the PCI device handle to work with when parsing the device (bus specific) parameters. Signed-off-by: Gaetan

[dpdk-dev] [PATCH v2 12/13] ethdev: process declarative eth devargs

2018-09-19 Thread Gaetan Rivet
Process the eth parameters of a devargs. For each parameters that have a setter implemented, the relevant field in rte_eth_dev field is written. Currently only "name" is implemented. Signed-off-by: Gaetan Rivet --- lib/librte_ethdev/ethdev_private.h | 5 +++ lib/librte_ethdev/rte_c

[dpdk-dev] [PATCH v2 10/13] bus/vdev: pre-process declarative vdev devargs

2018-09-19 Thread Gaetan Rivet
The new devargs format does not recognize a particular device name. Each bus uses its specific format. Process each devargs privately prior to attempting a bus scan. Prepare them if they are using the new devargs format. Signed-off-by: Gaetan Rivet --- drivers/bus/vdev/vdev.c | 10

[dpdk-dev] [PATCH v2 09/13] bus/pci: pre-process declarative PCI devargs

2018-09-19 Thread Gaetan Rivet
The new devargs format does not recognize a particular device name. Each bus uses its specific format. Instead of introducing a new bus API, process those devargs privately for the moment. Prepare them for matching during scan against the bus devices. Signed-off-by: Gaetan Rivet --- drivers

[dpdk-dev] [PATCH v2 08/13] app/testpmd: add show device command

2018-09-19 Thread Gaetan Rivet
/class=eth,name=net_ring0 These devices may not be otherwise useful, some buses will spawn devices to keep track of their assets without having a driver to use them. Signed-off-by: Gaetan Rivet Acked-by: Bernard Iremonger --- app/test-pmd/cmdline.c | 54

[dpdk-dev] [PATCH v2 06/13] ethdev: register ether layer as a class

2018-09-19 Thread Gaetan Rivet
Implement the operators of an rte_class for the ethdev abstraction layer. Register the layer as such. Signed-off-by: Gaetan Rivet Acked-by: Andrew Rybchenko --- lib/librte_ethdev/Makefile| 3 +- lib/librte_ethdev/meson.build | 1 + lib/librte_ethdev/rte_class_eth.c | 79

[dpdk-dev] [PATCH v2 07/13] ethdev: add device matching field name

2018-09-19 Thread Gaetan Rivet
The eth device class can now parse a field name, matching the eth_dev name with one passed as "class=eth,name=xx" Signed-off-by: Gaetan Rivet Acked-by: Andrew Rybchenko --- lib/librte_ethdev/rte_class_eth.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/lib/lib

[dpdk-dev] [PATCH v2 05/13] ethdev: add private generic device iterator

2018-09-19 Thread Gaetan Rivet
This iterator can be customized with a comparison function that will trigger a stopping condition. It can be leveraged to write several different iterators that have similar but non-identical purposes. It is private to librte_ethdev. Signed-off-by: Gaetan Rivet --- lib/librte_ethdev/Makefile

[dpdk-dev] [PATCH v2 04/13] bus/vdev: add device matching field driver

2018-09-19 Thread Gaetan Rivet
The vdev bus parses a field "driver", matching a vdev driver name with one passed as follows: "bus=vdev,driver=" Signed-off-by: Gaetan Rivet --- drivers/bus/vdev/vdev_params.c | 15 +-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --gi

[dpdk-dev] [PATCH v2 03/13] bus/vdev: implement device iteration

2018-09-19 Thread Gaetan Rivet
Signed-off-by: Gaetan Rivet --- drivers/bus/vdev/Makefile | 3 +- drivers/bus/vdev/meson.build| 5 +++- drivers/bus/vdev/vdev.c | 10 --- drivers/bus/vdev/vdev_params.c | 51 + drivers/bus/vdev/vdev_private.h | 26 + 5

[dpdk-dev] [PATCH v2 02/13] bus/pci: add device matching field id

2018-09-19 Thread Gaetan Rivet
The PCI bus can now parse a matching field "id" as follows: "bus=pci,id=:00:00.0" or "bus=pci,id=00:00.0" Signed-off-by: Gaetan Rivet --- drivers/bus/pci/pci_params.c | 29 +++-- 1 file changed, 27 insertions(+)

[dpdk-dev] [PATCH v2 01/13] bus/pci: implement device iteration and comparison

2018-09-19 Thread Gaetan Rivet
Signed-off-by: Gaetan Rivet --- drivers/bus/pci/Makefile | 3 +- drivers/bus/pci/meson.build | 6 +++- drivers/bus/pci/pci_common.c | 3 +- drivers/bus/pci/pci_params.c | 53 drivers/bus/pci/private.h| 25 + 5 files changed, 86

[dpdk-dev] [PATCH v2 00/13] Implement new devargs framework

2018-09-19 Thread Gaetan Rivet
ly two PMDs, for now. The new devargs format is also made available through the new --dev parameter. Next work is to generalize use of new API for eth_dev creation, compatibility layer for -w, -b and --vdev with --dev, and devargs unit test. -- v2: fixed Andrew's and Stephen's remarks,

[dpdk-dev] [PATCH v1] testpmd: eeprom display

2018-09-18 Thread Gaetan Rivet
The interactive command show port eeprom will dump the content of the EEPROM for the selected port. Dumping eeprom of all ports at once is not supported. Signed-off-by: Gaetan Rivet --- app/test-pmd/cmdline.c | 9 +++-- app/test-pmd/config.c | 32 app

[dpdk-dev] [PATCH v1] e1000: support dev reset

2018-09-18 Thread Gaetan Rivet
Add support for passive device reset on IGB ports. Signed-off-by: Gaetan Rivet --- drivers/net/e1000/igb_ethdev.c | 29 + 1 file changed, 29 insertions(+) diff --git a/drivers/net/e1000/igb_ethdev.c b/drivers/net/e1000/igb_ethdev.c index 64dfe6836..4dbf46b28 100644

[dpdk-dev] [PATCH v2] eal: add strscpy function

2018-09-11 Thread Gaetan Rivet
input. The function strscpy has been discussed for some time already and has been made available in the linux kernel[1]. Propose this new function as a safe alternative. [1]: http://git.kernel.org/linus/30c44659f4a3 Signed-off-by: Gaetan Rivet Acked-by: Juhamatti Kuusisaari --- v2: use

[dpdk-dev] [PATCH v1] eal: add strscpy function

2018-09-11 Thread Gaetan Rivet
Signed-off-by: Gaetan Rivet --- I agree with the original email, here is a proposed implementation. I have added the function as part of 18.11 API proper, because this API is definitely not meant to change. This is not meant to be enforced on existing code, or even on new code. But I think it is

[dpdk-dev] [PATCH v1 13/13] eal: add generic dev parameter

2018-08-30 Thread Gaetan Rivet
Add the --dev parameter to the EAL. This new parameter takes a generic device declaration as argument. It uses the new devargs parsing API. Signed-off-by: Gaetan Rivet --- lib/librte_eal/common/eal_common_devargs.c | 4 +++ lib/librte_eal/common/eal_common_options.c | 36

[dpdk-dev] [PATCH v1 11/13] bus/pci: process declarative PCI devargs

2018-08-30 Thread Gaetan Rivet
Introduce the facility to process future PCI parameters. Once the matching between PCI devices and devargs has been done, it is possible to process each devargs. New parameters would have the PCI device handle to work with when parsing the device (bus specific) parameters. Signed-off-by: Gaetan

[dpdk-dev] [PATCH v1 10/13] bus/vdev: pre-process declarative vdev devargs

2018-08-30 Thread Gaetan Rivet
The new devargs format does not recognize a particular device name. Each bus uses its specific format. Process each devargs privately prior to attempting a bus scan. Prepare them if they are using the new devargs format. Signed-off-by: Gaetan Rivet --- drivers/bus/vdev/vdev.c | 10

[dpdk-dev] [PATCH v1 09/13] bus/pci: pre-process declarative PCI devargs

2018-08-30 Thread Gaetan Rivet
The new devargs format does not recognize a particular device name. Each bus uses its specific format. Instead of introducing a new bus API, process those devargs privately for the moment. Prepare them for matching during scan against the bus devices. Signed-off-by: Gaetan Rivet --- drivers

[dpdk-dev] [PATCH v1 12/13] ethdev: process declarative eth devargs

2018-08-30 Thread Gaetan Rivet
Process the class-specific arguments in a devargs. This processing takes the form of setting the proper eth_dev fields when relevant. Signed-off-by: Gaetan Rivet --- lib/librte_ethdev/eth_private.h | 5 +++ lib/librte_ethdev/rte_class_eth.c | 62 +++ lib

[dpdk-dev] [PATCH v1 07/13] ethdev: add device matching field name

2018-08-30 Thread Gaetan Rivet
The eth device class can now parse a field name, matching the eth_dev name with one passed as "class=eth,name=xx" Signed-off-by: Gaetan Rivet --- lib/librte_ethdev/rte_class_eth.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/lib/librte_ethdev/rte_class_et

[dpdk-dev] [PATCH v1 08/13] app/testpmd: add show device command

2018-08-30 Thread Gaetan Rivet
/class=eth,name=net_ring0 These devices may not be otherwise useful, some buses will spawn devices to keep track of their assets without having a driver to use them. Signed-off-by: Gaetan Rivet Acked-by: Bernard Iremonger --- app/test-pmd/cmdline.c | 54

[dpdk-dev] [PATCH v1 04/13] bus/vdev: add device matching field driver

2018-08-30 Thread Gaetan Rivet
The vdev bus parses a field "driver", matching a vdev driver name with one passed as follows: "bus=vdev,driver=" Signed-off-by: Gaetan Rivet --- drivers/bus/vdev/vdev_params.c | 15 +-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --gi

[dpdk-dev] [PATCH v1 05/13] ethdev: add private generic device iterator

2018-08-30 Thread Gaetan Rivet
This iterator can be customized with a comparison function that will trigger a stopping condition. It can be leveraged to write several different iterators that have similar but non-identical purposes. It is private to librte_ethdev. Signed-off-by: Gaetan Rivet --- lib/librte_ethdev/Makefile

[dpdk-dev] [PATCH v1 06/13] ethdev: register ether layer as a class

2018-08-30 Thread Gaetan Rivet
Signed-off-by: Gaetan Rivet --- lib/librte_ethdev/Makefile| 3 +- lib/librte_ethdev/meson.build | 1 + lib/librte_ethdev/rte_class_eth.c | 79 +++ 3 files changed, 82 insertions(+), 1 deletion(-) create mode 100644 lib/librte_ethdev/rte_class_eth.c

[dpdk-dev] [PATCH v1 02/13] bus/pci: add device matching field id

2018-08-30 Thread Gaetan Rivet
The PCI bus can now parse a matching field "id" as follows: "bus=pci,id=:00:00.0" or "bus=pci,id=00:00.0" Signed-off-by: Gaetan Rivet --- drivers/bus/pci/pci_params.c | 29 +++-- 1 file changed, 27 insertions(+)

[dpdk-dev] [PATCH v1 03/13] bus/vdev: implement device iteration

2018-08-30 Thread Gaetan Rivet
Signed-off-by: Gaetan Rivet --- drivers/bus/vdev/Makefile | 3 +- drivers/bus/vdev/meson.build| 5 +++- drivers/bus/vdev/vdev.c | 10 --- drivers/bus/vdev/vdev_params.c | 51 + drivers/bus/vdev/vdev_private.h | 26 + 5

[dpdk-dev] [PATCH v1 01/13] bus/pci: implement device iteration and comparison

2018-08-30 Thread Gaetan Rivet
Signed-off-by: Gaetan Rivet --- drivers/bus/pci/Makefile | 3 +- drivers/bus/pci/meson.build | 6 +++- drivers/bus/pci/pci_common.c | 3 +- drivers/bus/pci/pci_params.c | 53 drivers/bus/pci/private.h| 25 + 5 files changed, 86

[dpdk-dev] [PATCH v1 00/13] Implement new devargs framework

2018-08-30 Thread Gaetan Rivet
ly two PMDs, for now. The new devargs format is also made available through the new --dev parameter. Next work is to generalize use of new API for eth_dev creation, compatibility layer for -w, -b and --vdev with --dev, and devargs unit test. Gaetan Rivet (13): bus/pci: implement device iteration

[dpdk-dev] [PATCH v11 25/25] eal: add generic dev parameter

2018-07-11 Thread Gaetan Rivet
Add the --dev parameter to the EAL. This new parameter takes a generic device declaration as argument. It uses the new devargs parsing API. Signed-off-by: Gaetan Rivet --- lib/librte_eal/common/eal_common_devargs.c | 4 +++ lib/librte_eal/common/eal_common_options.c | 36

[dpdk-dev] [PATCH v11 24/25] ethdev: process declarative eth devargs

2018-07-11 Thread Gaetan Rivet
Process the class-specific arguments in a devargs. This processing takes the form of setting the proper eth_dev fields when relevant. Signed-off-by: Gaetan Rivet --- lib/librte_ethdev/eth_private.h | 5 +++ lib/librte_ethdev/rte_class_eth.c | 62 +++ lib

[dpdk-dev] [PATCH v11 19/25] ethdev: add device matching field name

2018-07-11 Thread Gaetan Rivet
The eth device class can now parse a field name, matching the eth_dev name with one passed as "class=eth,name=xx" Signed-off-by: Gaetan Rivet --- lib/librte_ethdev/rte_class_eth.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/lib/librte_ethdev/rte_class_et

[dpdk-dev] [PATCH v11 20/25] app/testpmd: add show device command

2018-07-11 Thread Gaetan Rivet
/class=eth,name=net_ring0 These devices may not be otherwise useful, some buses will spawn devices to keep track of their assets without having a driver to use them. Signed-off-by: Gaetan Rivet Acked-by: Bernard Iremonger --- app/test-pmd/cmdline.c | 54

[dpdk-dev] [PATCH v11 23/25] bus/pci: process declarative PCI devargs

2018-07-11 Thread Gaetan Rivet
Introduce the facility to process future PCI parameters. Once the matching between PCI devices and devargs has been done, it is possible to process each devargs. New parameters would have the PCI device handle to work with when parsing the device (bus specific) parameters. Signed-off-by: Gaetan

[dpdk-dev] [PATCH v11 22/25] bus/vdev: pre-process declarative vdev devargs

2018-07-11 Thread Gaetan Rivet
The new devargs format does not recognize a particular device name. Each bus uses its specific format. Process each devargs privately prior to attempting a bus scan. Prepare them if they are using the new devargs format. Signed-off-by: Gaetan Rivet --- drivers/bus/vdev/vdev.c | 10

[dpdk-dev] [PATCH v11 21/25] bus/pci: pre-process declarative PCI devargs

2018-07-11 Thread Gaetan Rivet
The new devargs format does not recognize a particular device name. Each bus uses its specific format. Instead of introducing a new bus API, process those devargs privately for the moment. Prepare them for matching during scan against the bus devices. Signed-off-by: Gaetan Rivet --- drivers

[dpdk-dev] [PATCH v11 18/25] ethdev: register ether layer as a class

2018-07-11 Thread Gaetan Rivet
Signed-off-by: Gaetan Rivet --- lib/librte_ethdev/Makefile| 3 +- lib/librte_ethdev/meson.build | 1 + lib/librte_ethdev/rte_class_eth.c | 79 +++ 3 files changed, 82 insertions(+), 1 deletion(-) create mode 100644 lib/librte_ethdev/rte_class_eth.c

[dpdk-dev] [PATCH v11 15/25] bus/vdev: implement device iteration

2018-07-11 Thread Gaetan Rivet
Signed-off-by: Gaetan Rivet --- drivers/bus/vdev/Makefile | 3 +- drivers/bus/vdev/meson.build| 5 +++- drivers/bus/vdev/vdev.c | 10 --- drivers/bus/vdev/vdev_params.c | 51 + drivers/bus/vdev/vdev_private.h | 26 + 5

[dpdk-dev] [PATCH v11 14/25] bus/pci: add device matching field id

2018-07-11 Thread Gaetan Rivet
The PCI bus can now parse a matching field "id" as follows: "bus=pci,id=:00:00.0" or "bus=pci,id=00:00.0" Signed-off-by: Gaetan Rivet --- drivers/bus/pci/pci_params.c | 29 +++-- 1 file changed, 27 insertions(+)

[dpdk-dev] [PATCH v11 17/25] ethdev: add private generic device iterator

2018-07-11 Thread Gaetan Rivet
This iterator can be customized with a comparison function that will trigger a stopping condition. It can be leveraged to write several different iterators that have similar but non-identical purposes. It is private to librte_ethdev. Signed-off-by: Gaetan Rivet --- lib/librte_ethdev/Makefile

[dpdk-dev] [PATCH v11 16/25] bus/vdev: add device matching field driver

2018-07-11 Thread Gaetan Rivet
The vdev bus parses a field "driver", matching a vdev driver name with one passed as follows: "bus=vdev,driver=" Signed-off-by: Gaetan Rivet --- drivers/bus/vdev/vdev_params.c | 15 +-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --gi

  1   2   3   4   5   6   7   8   9   10   >