Mdev test cleanup, completion and parent addr exposure

[PATCH 0/3] Mdev test cleanup, completion and parent addr exposure

Boris Fiuczynski posted 3 patches 2 years, 2 months ago

Download series mbox

Test syntax-check failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20220204153218.71177-1-fiuczy@linux.ibm.com

docs/schemas/nodedev.rng                      |  5 ++
src/conf/node_device_conf.c                   |  2 +
...52_9b13_9b13_9b13_cc23009b1326-create.argv |  5 ++
...52_9b13_9b13_9b13_cc23009b1326-create.json |  1 +
...52_9b13_9b13_9b13_cc23009b1326-define.argv |  5 ++
...52_9b13_9b13_9b13_cc23009b1326-define.json |  1 +
.../mdevctl-list-multiple.json                | 56 +++++++++++--------
.../mdevctl-list-multiple.out.xml             | 14 +++++
tests/nodedevmdevctltest.c                    | 31 +++++++++-
...v_cc000052_9b13_9b13_9b13_cc23009b1326.xml |  8 +++
10 files changed, 104 insertions(+), 24 deletions(-)
create mode 100644 tests/nodedevmdevctldata/mdev_cc000052_9b13_9b13_9b13_cc23009b1326-create.argv
create mode 100644 tests/nodedevmdevctldata/mdev_cc000052_9b13_9b13_9b13_cc23009b1326-create.json
create mode 100644 tests/nodedevmdevctldata/mdev_cc000052_9b13_9b13_9b13_cc23009b1326-define.argv
create mode 100644 tests/nodedevmdevctldata/mdev_cc000052_9b13_9b13_9b13_cc23009b1326-define.json
create mode 100644 tests/nodedevschemadata/mdev_cc000052_9b13_9b13_9b13_cc23009b1326.xml

Expand all Fold all

[PATCH 0/3] Mdev test cleanup, completion and parent addr exposure

Posted by Boris Fiuczynski 2 years, 2 months ago

Clean up the testing json. Add a test for vfio-ccw mdev.
Expose the parent address in the XML when dumping a mdev nodedev as the
address is now part of the mdev nodedev name and users should not parse
and manipulate the mdev nodedev name to retrieve it.

Boris Fiuczynski (3):
  tests: correct formating in mdevctl test
  tests: adding vfio-ccw to nodedev tests
  nodedev: add parent_addr to mdev nodedev dumpxml

 docs/schemas/nodedev.rng                      |  5 ++
 src/conf/node_device_conf.c                   |  2 +
 ...52_9b13_9b13_9b13_cc23009b1326-create.argv |  5 ++
 ...52_9b13_9b13_9b13_cc23009b1326-create.json |  1 +
 ...52_9b13_9b13_9b13_cc23009b1326-define.argv |  5 ++
 ...52_9b13_9b13_9b13_cc23009b1326-define.json |  1 +
 .../mdevctl-list-multiple.json                | 56 +++++++++++--------
 .../mdevctl-list-multiple.out.xml             | 14 +++++
 tests/nodedevmdevctltest.c                    | 31 +++++++++-
 ...v_cc000052_9b13_9b13_9b13_cc23009b1326.xml |  8 +++
 10 files changed, 104 insertions(+), 24 deletions(-)
 create mode 100644 tests/nodedevmdevctldata/mdev_cc000052_9b13_9b13_9b13_cc23009b1326-create.argv
 create mode 100644 tests/nodedevmdevctldata/mdev_cc000052_9b13_9b13_9b13_cc23009b1326-create.json
 create mode 100644 tests/nodedevmdevctldata/mdev_cc000052_9b13_9b13_9b13_cc23009b1326-define.argv
 create mode 100644 tests/nodedevmdevctldata/mdev_cc000052_9b13_9b13_9b13_cc23009b1326-define.json
 create mode 100644 tests/nodedevschemadata/mdev_cc000052_9b13_9b13_9b13_cc23009b1326.xml

-- 
2.33.1

Re: [PATCH 0/3] Mdev test cleanup, completion and parent addr exposure

Posted by Michal Prívozník 2 years, 2 months ago

On 2/4/22 16:32, Boris Fiuczynski wrote:
> Clean up the testing json. Add a test for vfio-ccw mdev.
> Expose the parent address in the XML when dumping a mdev nodedev as the
> address is now part of the mdev nodedev name and users should not parse
> and manipulate the mdev nodedev name to retrieve it.
> 
> Boris Fiuczynski (3):
>   tests: correct formating in mdevctl test
>   tests: adding vfio-ccw to nodedev tests
>   nodedev: add parent_addr to mdev nodedev dumpxml
> 
>  docs/schemas/nodedev.rng                      |  5 ++
>  src/conf/node_device_conf.c                   |  2 +
>  ...52_9b13_9b13_9b13_cc23009b1326-create.argv |  5 ++
>  ...52_9b13_9b13_9b13_cc23009b1326-create.json |  1 +
>  ...52_9b13_9b13_9b13_cc23009b1326-define.argv |  5 ++
>  ...52_9b13_9b13_9b13_cc23009b1326-define.json |  1 +
>  .../mdevctl-list-multiple.json                | 56 +++++++++++--------
>  .../mdevctl-list-multiple.out.xml             | 14 +++++
>  tests/nodedevmdevctltest.c                    | 31 +++++++++-
>  ...v_cc000052_9b13_9b13_9b13_cc23009b1326.xml |  8 +++
>  10 files changed, 104 insertions(+), 24 deletions(-)
>  create mode 100644 tests/nodedevmdevctldata/mdev_cc000052_9b13_9b13_9b13_cc23009b1326-create.argv
>  create mode 100644 tests/nodedevmdevctldata/mdev_cc000052_9b13_9b13_9b13_cc23009b1326-create.json
>  create mode 100644 tests/nodedevmdevctldata/mdev_cc000052_9b13_9b13_9b13_cc23009b1326-define.argv
>  create mode 100644 tests/nodedevmdevctldata/mdev_cc000052_9b13_9b13_9b13_cc23009b1326-define.json
>  create mode 100644 tests/nodedevschemadata/mdev_cc000052_9b13_9b13_9b13_cc23009b1326.xml
> 

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

Michal