mbox series

[v5,0/3] devargs: support path in global syntax

Message ID 20211020154739.2514499-1-xuemingl@nvidia.com (mailing list archive)
Headers
Series devargs: support path in global syntax |

Message

Xueming Li Oct. 20, 2021, 3:47 p.m. UTC 
  - Support path in global syntax.
- Fix bus name resolving
- Add devargs test cases

v1: initial version
v2:
 - add test cases to test suite
 - add more test cases, verify device name, bus name and class name
v3:
 - remove autotest_data.py
v4:
 - make devargs test depends on virtio driver
v5:
 - use api to filter out unsupported devarg test cases

Email thread: 20211005123012.264727-1-xuemingl@nvidia.com

Xueming Li (3):
  devargs: support path value for global device arguments
  devargs: make bus key parsing optional
  test/devargs: add devargs test cases

 app/test/meson.build                |   2 +
 app/test/test_devargs.c             | 214 ++++++++++++++++++++++++++++
 drivers/bus/pci/pci_params.c        |   8 +-
 lib/eal/common/eal_common_devargs.c | 119 ++++++----------
 4 files changed, 263 insertions(+), 80 deletions(-)
 create mode 100644 app/test/test_devargs.c

Comments

David Marchand Oct. 21, 2021, 9:22 a.m. UTC | #1 
On Wed, Oct 20, 2021 at 5:47 PM Xueming Li <xuemingl@nvidia.com> wrote:
>
> - Support path in global syntax.
> - Fix bus name resolving
> - Add devargs test cases
>
> v1: initial version
> v2:
>  - add test cases to test suite
>  - add more test cases, verify device name, bus name and class name
> v3:
>  - remove autotest_data.py
> v4:
>  - make devargs test depends on virtio driver
> v5:
>  - use api to filter out unsupported devarg test cases

Fixed style issues in meson (this is new in v5, please pay attention
to such details) and applied.
Thanks.
Xueming Li Oct. 21, 2021, 10:43 a.m. UTC | #2 
On Thu, 2021-10-21 at 11:22 +0200, David Marchand wrote:
> On Wed, Oct 20, 2021 at 5:47 PM Xueming Li <xuemingl@nvidia.com> wrote:
> > 
> > - Support path in global syntax.
> > - Fix bus name resolving
> > - Add devargs test cases
> > 
> > v1: initial version
> > v2:
> >  - add test cases to test suite
> >  - add more test cases, verify device name, bus name and class name
> > v3:
> >  - remove autotest_data.py
> > v4:
> >  - make devargs test depends on virtio driver
> > v5:
> >  - use api to filter out unsupported devarg test cases
> 
> Fixed style issues in meson (this is new in v5, please pay attention
> to such details) and applied.
> Thanks.
> 

Thanks David!

>