devargs cleanup
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v5 Latest10 patches 3 repliesIn review
- v410 patches 4 repliesSuperseded
- v310 patches 7 repliesSuperseded
Earlier versions · 2
| Version | Posted | Patches | Replies | Status |
|---|---|---|---|---|
| v2 | 18 patches | 18 replies | Superseded | |
| v1 | 6 patches | 0 replies | Superseded |
Changes in v5 from the cover letter
* Merged iterator patches
* Fixed rte_devargs_add documentation in its own patch.
* Updated deprecation notice when relevant.
Patches in v5 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/10 | devargs: introduce iteratorlib/ lib/ lib/ | +49 | 0 replies | |
| 2/10 | bus/vdev: do not reference devargs listdrivers/ drivers/ | +4 | 0 replies | |
| 3/10 | bus/pci: do not reference devargs listdrivers/ drivers/ | +2 | 0 replies | |
| 4/10 | net/vdev_netvsc: do not reference devargs listdrivers/ | +1 | 0 replies | |
| 5/10 | test: remove devargs unit testsMAINTAINERS test/ test/ | +0 | 1 reply | |
| 6/10 | devargs: make devargs list privatedoc/ lib/ lib/ +1 more | +3 | 0 replies | |
| 7/10 | devargs: make parsing variadicdrivers/ drivers/ lib/ +2 more | +25 | 0 replies | |
| 8/10 | devargs: update devargs add documentationlib/ | +16 | 0 replies | |
| 9/10 | devargs: use proper namespace prefixdrivers/ drivers/ drivers/ +4 more | +33 | 0 replies | |
| 10/10 | devargs: rename legacy APIapp/ doc/ drivers/ +8 more | +81 | 1 reply |
Reviews trailers per patch
All 10 patches have a Reviewed-by or Acked-by.
| Patch | Thomas Monjalon | Jianfeng Tan |
|---|---|---|
| 1/10 | A-b | |
| 2/10 | A-b | A-b |
| 3/10 | A-b | |
| 4/10 | A-b | |
| 5/10 | A-b | |
| 6/10 | A-b | |
| 7/10 | A-b | |
| 8/10 | A-b | |
| 9/10 | A-b | |
| 10/10 | A-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH v5 00/10] devargs cleanup
Cleanup the rte_devargs API.
This is a continuous process.
The previous version of this patchset was dependent on changes on the PCI bus and the rte_bus library. Not all these changes were integrated. This patchset has thus been limited to elements that are available right now, without dependencies.
The devargs list is made private. All devargs users are meant to use them using the proper API. Devargs will allocate dynamic memory. With device hotplug, this memory is usually mishandled and lost by libraries not following the evolutions of rte_devargs.
v4:
* Rebased on master.
v5:
* Merged iterator patches
* Fixed rte_devargs_add documentation in its own patch.
* Updated deprecation notice when relevant.
Gaetan Rivet (10):
devargs: introduce iterator
bus/vdev: do not reference devargs list
bus/pci: do not reference devargs list
net/vdev_netvsc: do not reference devargs list
test: remove devargs unit tests
devargs: make devargs list private
devargs: make parsing variadic
devargs: update devargs add documentation
devargs: use proper namespace prefix
devargs: rename legacy API
MAINTAINERS | 1 - app/test-pmd/cmdline.c | 2 +- doc/guides/rel_notes/deprecation.rst | 5 -- drivers/bus/pci/Makefile | 1 + drivers/bus/pci/pci_common.c | 6 +- drivers/bus/vdev/Makefile | 1 + drivers/bus/vdev/vdev.c | 13 +-- drivers/net/failsafe/failsafe_args.c | 2 +- drivers/net/failsafe/failsafe_eal.c | 2 +- drivers/net/vdev_netvsc/vdev_netvsc.c | 4 +- examples/bond/Makefile | 1 + examples/bond/main.c | 2 +- lib/librte_eal/common/eal_common_dev.c | 39 ++------- lib/librte_eal/common/eal_common_devargs.c | 53 ++++++++++-- lib/librte_eal/common/eal_common_options.c | 2 +- lib/librte_eal/common/include/rte_devargs.h | 130 ++++++++++++++++++++++------ lib/librte_eal/rte_eal_version.map | 11 ++- lib/librte_ether/rte_ethdev.c | 44 +++++----- test/test/Makefile | 1 - test/test/commands.c | 2 +- test/test/test_devargs.c | 103 ---------------------- 21 files changed, 199 insertions(+), 226 deletions(-) delete mode 100644 test/test/test_devargs.c
Signature · 1 line
2.11.0