Skip to contentSkip to conversation

usertools/dpdk-devbind.py: execute "./usertools/dpdk-devbind.py --status-dev misc" command to get CBDMA device info failed.

dpdk-dev Applied [email protected] 5 participants 7 messages Nov 24, 2020 – Nov 25, 2020

lore
Share & tools
lore

[email protected]

author Acked-by Bruce Richardson
#0 lore raw

https://bugs.dpdk.org/show_bug.cgi?id=582

            Bug ID: 582
           Summary: [Regression Test] usertools/dpdk-devbind.py: execute
                    "./usertools/dpdk-devbind.py --status-dev misc"
                    command to get CBDMA device info failed.
           Product: DPDK
           Version: 20.11
          Hardware: All
                OS: All
            Status: UNCONFIRMED
          Severity: minor
          Priority: Normal
         Component: other
          Assignee: [email protected]
          Reporter: [email protected]

Target Milestone: ---

DPDK version:
61bd270c4adc7dff1f9c8af1f2c72618860d4395 refs/heads/main
Other software versions: name/version for QEMU, OVS, etc. Repeat as required.
OS: 5.3.0-59-generic/Ubuntu 18.04.5 LTS
Compiler: gcc version 7.5.0 (Ubuntu 7.5.0-3ubuntu1~18.04) Hardware platform: Intel(R) Xeon(R) Platinum 8280M CPU @ 2.70GHz. NIC hardware: Ethernet Controller XL710 for 40GbE QSFP+ 1583.
NIC firmware: i40e_2.13.10/8.00 0x80008b8a 1.2766.0
Test Setup
Steps to reproduce
List the steps to reproduce the issue.

# tar -xvf dpdk.tar.gz
# cd dpdk
# ./usertools/dpdk-devbind.py --status-dev misc
Show the output from the previous commands.

# root@dpdk-Perf-dut222:/home/autoregression/dts/dts/dep/dpdk# ./usertools/dpdk-devbind.py --status-dev misc
usage: dpdk-devbind.py [-h] [-s]

                       [--status-dev
{net,baseband,crypto,event,mempool,compress}]
                       [-b DRIVER | -u] [--force]
                       [DEVICE [DEVICE ...]]

dpdk-devbind.py: error: argument --status-dev: invalid choice: 'misc' (choose from 'net', 'baseband', 'crypto', 'event', 'mempool', 'compress')

Expected Result
Explain what is the expected result in text or as an example output:

# root@dpdk-Perf-dut222:~# dpdk-devbind.py --status-dev miscMisc (rawdev) devices using kernel driver
=========================================
0000:00:04.0 'Sky Lake-E CBDMA Registers 2021' if= drv=ioatdma unused=igb_uio,vfio-pci
0000:00:04.1 'Sky Lake-E CBDMA Registers 2021' if= drv=ioatdma unused=igb_uio,vfio-pci
0000:00:04.2 'Sky Lake-E CBDMA Registers 2021' if= drv=ioatdma unused=igb_uio,vfio-pci
0000:00:04.3 'Sky Lake-E CBDMA Registers 2021' if= drv=ioatdma unused=igb_uio,vfio-pci
0000:00:04.4 'Sky Lake-E CBDMA Registers 2021' if= drv=ioatdma unused=igb_uio,vfio-pci
0000:00:04.5 'Sky Lake-E CBDMA Registers 2021' if= drv=ioatdma unused=igb_uio,vfio-pci
0000:00:04.6 'Sky Lake-E CBDMA Registers 2021' if= drv=ioatdma unused=igb_uio,vfio-pci
0000:00:04.7 'Sky Lake-E CBDMA Registers 2021' if= drv=ioatdma unused=igb_uio,vfio-pci
0000:80:04.0 'Sky Lake-E CBDMA Registers 2021' if= drv=ioatdma unused=igb_uio,vfio-pci
0000:80:04.1 'Sky Lake-E CBDMA Registers 2021' if= drv=ioatdma unused=igb_uio,vfio-pci
0000:80:04.2 'Sky Lake-E CBDMA Registers 2021' if= drv=ioatdma unused=igb_uio,vfio-pci
0000:80:04.3 'Sky Lake-E CBDMA Registers 2021' if= drv=ioatdma unused=igb_uio,vfio-pci
0000:80:04.4 'Sky Lake-E CBDMA Registers 2021' if= drv=ioatdma unused=igb_uio,vfio-pci
0000:80:04.5 'Sky Lake-E CBDMA Registers 2021' if= drv=ioatdma unused=igb_uio,vfio-pci
0000:80:04.6 'Sky Lake-E CBDMA Registers 2021' if= drv=ioatdma unused=igb_uio,vfio-pci
0000:80:04.7 'Sky Lake-E CBDMA Registers 2021' if= drv=ioatdma unused=igb_uio,vfio-pci
Regression
Is this issue a regression: (Y)

Version the regression was introduced: Specify git id if known.

Bad commit:

commit 81255f27c65c7f42f50ee8dfb0feeb40c36c4c61 (HEAD)
Author: Stephen Hemminger <[email protected]>
Date: Tue Nov 3 22:48:41 2020 -0800

usertools: replace optparse with argparse

The optparse module is deprecated and replaced with new argparse. Using the python standard argument parser instead of C library style getopt gives a number of advantages such as checking for conflicting arguments, restricting choices, and automatically generating help messages.

Some of the help messages are now less wordy.

The code now enforces the rule that only one of the pmdinfo formats can be specified: raw or json.

Signature · 2 lines
You are receiving this mail because:
You are the assignee for the bug.
#1 lore raw

PATCH 1/1 usertools/dpdk-devbind: fix missing device classes

The "misc" and "regex" device classes were missing from the list used to check arguments, preventing them from being used with "--status-dev" flag to list only devices of those types.

When adding them to the list, the list is also sorted alphabetically for consistency.

Fixes 81255f27c65c ("usertools: replace optparse with argparse")

Bugzilla ID: 582

---
 usertools/dpdk-devbind.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
usertools/dpdk-devbind.py +2 -1
index 054ad2e1c..c2ede3d4d 100755
@@ -634,7 +634,8 @@ def parse_args():
     parser.add_argument(
         '--status-dev',
         help="Print the status of given device group.",
-        choices=['net', 'baseband', 'crypto', 'event', 'mempool', 'compress'])
+        choices=['baseband', 'compress', 'crypto', 'event',
+                'mempool', 'misc', 'net', 'regex'])
     bind_group = parser.add_mutually_exclusive_group()
     bind_group.add_argument(
         '-b',
Signature · 1 line
2.27.0
Acked-by on v1 1/1
#2 lore raw

Re: [dpdk-dev] [PATCH] usertools/dpdk-devbind: fix missing device classes

Quoted 7 lines · On Tue, 24 Nov 2020 09:46:56 +0000 Bruce Richardson…

The "misc" and "regex" device classes were missing from the list used to check arguments, preventing them from being used with "--status-dev" flag to list only devices of those types.

When adding them to the list, the list is also sorted alphabetically for consistency.

Quoted 7 lines
Fixes 81255f27c65c ("usertools: replace optparse with argparse")

Bugzilla ID: 582

Almost the same patch I was sending right now...

Tested-by
#3 lore raw

Re: [dpdk-dev] [PATCH] usertools/dpdk-devbind: fix missing device classes

Best Regards
Jiang yu

Quoted 30 lines

-----Original Message-----
From: dev [mailto:[email protected]] On Behalf Of Stephen Hemminger
Sent: Wednesday, November 25, 2020 1:53 AM
To: Richardson, Bruce <[email protected]>
Cc: [email protected]; Ling, WeiX <[email protected]>
Subject: Re: [dpdk-dev] [PATCH] usertools/dpdk-devbind: fix missing device classes

Quoted 7 lines · On Tue, 24 Nov 2020 09:46:56 +0000 Bruce Richardson…

The "misc" and "regex" device classes were missing from the list used to check arguments, preventing them from being used with "--status-dev" flag to list only devices of those types.

When adding them to the list, the list is also sorted alphabetically for consistency.

Quoted 7 lines
Fixes 81255f27c65c ("usertools: replace optparse with argparse")

Bugzilla ID: 582

Almost the same patch I was sending right now...

#6 lore raw

[dpdk-dev] [Bug 582] [dpdk-20.11] usertools/dpdk-devbind.py "misc" option no longer available

In reply to [email protected]

https://bugs.dpdk.org/show_bug.cgi?id=582

jiang,yu ([email protected]) changed:

What |Removed |Added
----------------------------------------------------------------------------

         Resolution|---                         |FIXED
             Status|UNCONFIRMED                 |RESOLVED

--- Comment #2 from jiang,yu ([email protected]) ---
Verified patchset http://patches.dpdk.org/patch/84498/ on baseline 61bd270c4
version: 20.11-rc5.
Tested pass on fvl40G / Ubuntu 18.04.5 LTS
https://mails.dpdk.org/archives/dev/2020-November/193443.html

Signature · 2 lines
You are receiving this mail because:
You are the assignee for the bug.
Tested-by Yu Jiang
#4 lore raw

Re: [dpdk-dev] [PATCH] usertools/dpdk-devbind: fix missing device classes

In reply to Jiang, YuX

25/11/2020 03:23, Jiang, YuX:

Quoted 1 line

Thanks for the test.
Next time, please reply below with this exact line (name):

Quoted 31 lines
Quoted 30 lines

-----Original Message-----
From: dev [mailto:[email protected]] On Behalf Of Stephen Hemminger
Sent: Wednesday, November 25, 2020 1:53 AM
To: Richardson, Bruce <[email protected]>
Cc: [email protected]; Ling, WeiX <[email protected]>
Subject: Re: [dpdk-dev] [PATCH] usertools/dpdk-devbind: fix missing device classes

Quoted 7 lines · On Tue, 24 Nov 2020 09:46:56 +0000 Bruce Richardson…

The "misc" and "regex" device classes were missing from the list used to check arguments, preventing them from being used with "--status-dev" flag to list only devices of those types.

When adding them to the list, the list is also sorted alphabetically for consistency.

Quoted 7 lines
Fixes 81255f27c65c ("usertools: replace optparse with argparse")

Bugzilla ID: 582

Almost the same patch I was sending right now...

#5 lore raw

Re: [dpdk-dev] [PATCH] usertools/dpdk-devbind: fix missing device classes

In reply to Stephen Hemminger

24/11/2020 18:53, Stephen Hemminger:

Quoted 13 lines
Quoted 10 lines · On Tue, 24 Nov 2020 09:46:56 +0000 Bruce Richardson…

The "misc" and "regex" device classes were missing from the list used to check arguments, preventing them from being used with "--status-dev" flag to list only devices of those types.

When adding them to the list, the list is also sorted alphabetically for consistency.

Fixes 81255f27c65c ("usertools: replace optparse with argparse")
Quoted 8 lines
Quoted 4 lines

Bugzilla ID: 582

Almost the same patch I was sending right now...

Applied, thanks