DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] doc/devbind: remove references to igb_uio
@ 2020-09-06  3:38 Stephen Hemminger
  2020-09-07  9:08 ` Bruce Richardson
  2020-09-24 17:53 ` [dpdk-dev] [PATCH v2] " Stephen Hemminger
  0 siblings, 2 replies; 7+ messages in thread
From: Stephen Hemminger @ 2020-09-06  3:38 UTC (permalink / raw)
  To: john.mcnamara, marko.kovacevic; +Cc: dev, Stephen Hemminger

We should be encouraging the use of vfio_pci for developers, not telling
them to use igb_uio.

Also fix typo where .py suffix is visible in document.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 doc/guides/tools/devbind.rst | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/doc/guides/tools/devbind.rst b/doc/guides/tools/devbind.rst
index a2910e103aab..52690f018648 100644
--- a/doc/guides/tools/devbind.rst
+++ b/doc/guides/tools/devbind.rst
@@ -30,11 +30,11 @@ OPTIONS
         Print the current status of all known network interfaces.
         For each device, it displays the PCI domain, bus, slot and function,
         along with a text description of the device. Depending upon whether the
-        device is being used by a kernel driver, the ``igb_uio`` driver, or no
+        device is being used by a kernel driver, the ``vfio_pci`` driver, or no
         driver, other relevant information will be displayed:
         - the Linux interface name e.g. ``if=eth0``
-        - the driver being used e.g. ``drv=igb_uio``
-        - any suitable drivers not currently using that device e.g. ``unused=igb_uio``
+        - the driver being used e.g. ``drv=vfio_pci``
+        - any suitable drivers not currently using that device e.g. ``unused=vfio_pci``
         NOTE: if this flag is passed along with a bind/unbind option, the
         status display will always occur after the other operations have taken
         place.
@@ -69,7 +69,7 @@ OPTIONS
 
 .. warning::
 
-    While any user can run the ``dpdk-devbind.py`` script to view the status of the network ports,
+    While any user can run the ``dpdk-devbind`` script to view the status of the network ports,
     binding or unbinding network ports requires root privileges.
 
 
@@ -80,9 +80,9 @@ To display current device status::
 
    dpdk-devbind --status
 
-To bind eth1 from the current driver and move to use igb_uio::
+To bind eth1 from the current driver and move to use vfio_pci::
 
-   dpdk-devbind --bind=igb_uio eth1
+   dpdk-devbind --bind=vfio_pci eth1
 
 To unbind 0000:01:00.0 from using any driver::
 
@@ -92,7 +92,7 @@ To bind 0000:02:00.0 and 0000:02:00.1 to the ixgbe kernel driver::
 
    dpdk-devbind -b ixgbe 02:00.0 02:00.1
 
-To check status of all network ports, assign one to the igb_uio driver and check status again::
+To check status of all network ports, assign one to the vfio_pci driver and check status again::
 
    # Check the status of the available devices.
    dpdk-devbind --status
@@ -105,12 +105,12 @@ To check status of all network ports, assign one to the igb_uio driver and check
    0000:0a:00.0 '82599ES 10-Gigabit' if=eth2 drv=ixgbe unused=
 
 
-   # Bind the device to igb_uio.
-   sudo dpdk-devbind -b igb_uio 0000:0a:00.0
+   # Bind the device to vfio_pci.
+   sudo dpdk-devbind -b vfio_pci 0000:0a:00.0
 
 
    # Recheck the status of the devices.
    dpdk-devbind --status
    Network devices using DPDK-compatible driver
    ============================================
-   0000:0a:00.0 '82599ES 10-Gigabit' drv=igb_uio unused=
+   0000:0a:00.0 '82599ES 10-Gigabit' drv=vfio_pci unused=
-- 
2.27.0


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [dpdk-dev] [PATCH] doc/devbind: remove references to igb_uio
  2020-09-06  3:38 [dpdk-dev] [PATCH] doc/devbind: remove references to igb_uio Stephen Hemminger
@ 2020-09-07  9:08 ` Bruce Richardson
  2020-09-07 17:22   ` Stephen Hemminger
  2020-09-24 17:53 ` [dpdk-dev] [PATCH v2] " Stephen Hemminger
  1 sibling, 1 reply; 7+ messages in thread
From: Bruce Richardson @ 2020-09-07  9:08 UTC (permalink / raw)
  To: Stephen Hemminger; +Cc: john.mcnamara, marko.kovacevic, dev

On Sat, Sep 05, 2020 at 08:38:46PM -0700, Stephen Hemminger wrote:
> We should be encouraging the use of vfio_pci for developers, not telling
> them to use igb_uio.
> 
> Also fix typo where .py suffix is visible in document.
> 
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> ---
>  doc/guides/tools/devbind.rst | 20 ++++++++++----------
>  1 file changed, 10 insertions(+), 10 deletions(-)
> 
> diff --git a/doc/guides/tools/devbind.rst b/doc/guides/tools/devbind.rst
> index a2910e103aab..52690f018648 100644
> --- a/doc/guides/tools/devbind.rst
> +++ b/doc/guides/tools/devbind.rst
> @@ -30,11 +30,11 @@ OPTIONS
>          Print the current status of all known network interfaces.
>          For each device, it displays the PCI domain, bus, slot and function,
>          along with a text description of the device. Depending upon whether the
> -        device is being used by a kernel driver, the ``igb_uio`` driver, or no
> +        device is being used by a kernel driver, the ``vfio_pci`` driver, or no
>          driver, other relevant information will be displayed:
>          - the Linux interface name e.g. ``if=eth0``
> -        - the driver being used e.g. ``drv=igb_uio``
> -        - any suitable drivers not currently using that device e.g. ``unused=igb_uio``
> +        - the driver being used e.g. ``drv=vfio_pci``
> +        - any suitable drivers not currently using that device e.g. ``unused=vfio_pci``
>          NOTE: if this flag is passed along with a bind/unbind option, the
>          status display will always occur after the other operations have taken
>          place.
> @@ -69,7 +69,7 @@ OPTIONS
>  
>  .. warning::
>  
> -    While any user can run the ``dpdk-devbind.py`` script to view the status of the network ports,
> +    While any user can run the ``dpdk-devbind`` script to view the status of the network ports,
>      binding or unbinding network ports requires root privileges.

Agree with all the replacements of igb_uio with vfio, but why remove the
.py suffix here? The script won't run without giving its full name, so this
removal seems unhelpful to the user.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [dpdk-dev] [PATCH] doc/devbind: remove references to igb_uio
  2020-09-07  9:08 ` Bruce Richardson
@ 2020-09-07 17:22   ` Stephen Hemminger
  0 siblings, 0 replies; 7+ messages in thread
From: Stephen Hemminger @ 2020-09-07 17:22 UTC (permalink / raw)
  To: Bruce Richardson; +Cc: john.mcnamara, marko.kovacevic, dev

On Mon, 7 Sep 2020 10:08:16 +0100
Bruce Richardson <bruce.richardson@intel.com> wrote:

> Agree with all the replacements of igb_uio with vfio, but why remove the
> .py suffix here? The script won't run without giving its full name, so this
> removal seems unhelpful to the user.

My misunderstanding, thought the install process was stripping the .py suffix

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [dpdk-dev] [PATCH v2] doc/devbind: remove references to igb_uio
  2020-09-06  3:38 [dpdk-dev] [PATCH] doc/devbind: remove references to igb_uio Stephen Hemminger
  2020-09-07  9:08 ` Bruce Richardson
@ 2020-09-24 17:53 ` Stephen Hemminger
  2020-10-14 16:43   ` Ferruh Yigit
  1 sibling, 1 reply; 7+ messages in thread
From: Stephen Hemminger @ 2020-09-24 17:53 UTC (permalink / raw)
  To: dev; +Cc: Stephen Hemminger

We should be encouraging the use of vfio_pci for developers, not telling
them to use igb_uio.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
v2 -- drop change about filename

 doc/guides/tools/devbind.rst | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/doc/guides/tools/devbind.rst b/doc/guides/tools/devbind.rst
index a2910e103aab..5dcaf8c207dd 100644
--- a/doc/guides/tools/devbind.rst
+++ b/doc/guides/tools/devbind.rst
@@ -30,11 +30,11 @@ OPTIONS
         Print the current status of all known network interfaces.
         For each device, it displays the PCI domain, bus, slot and function,
         along with a text description of the device. Depending upon whether the
-        device is being used by a kernel driver, the ``igb_uio`` driver, or no
+        device is being used by a kernel driver, the ``vfio_pci`` driver, or no
         driver, other relevant information will be displayed:
         - the Linux interface name e.g. ``if=eth0``
-        - the driver being used e.g. ``drv=igb_uio``
-        - any suitable drivers not currently using that device e.g. ``unused=igb_uio``
+        - the driver being used e.g. ``drv=vfio_pci``
+        - any suitable drivers not currently using that device e.g. ``unused=vfio_pci``
         NOTE: if this flag is passed along with a bind/unbind option, the
         status display will always occur after the other operations have taken
         place.
@@ -80,9 +80,9 @@ To display current device status::
 
    dpdk-devbind --status
 
-To bind eth1 from the current driver and move to use igb_uio::
+To bind eth1 from the current driver and move to use vfio_pci::
 
-   dpdk-devbind --bind=igb_uio eth1
+   dpdk-devbind --bind=vfio_pci eth1
 
 To unbind 0000:01:00.0 from using any driver::
 
@@ -92,7 +92,7 @@ To bind 0000:02:00.0 and 0000:02:00.1 to the ixgbe kernel driver::
 
    dpdk-devbind -b ixgbe 02:00.0 02:00.1
 
-To check status of all network ports, assign one to the igb_uio driver and check status again::
+To check status of all network ports, assign one to the vfio_pci driver and check status again::
 
    # Check the status of the available devices.
    dpdk-devbind --status
@@ -105,12 +105,12 @@ To check status of all network ports, assign one to the igb_uio driver and check
    0000:0a:00.0 '82599ES 10-Gigabit' if=eth2 drv=ixgbe unused=
 
 
-   # Bind the device to igb_uio.
-   sudo dpdk-devbind -b igb_uio 0000:0a:00.0
+   # Bind the device to vfio_pci.
+   sudo dpdk-devbind -b vfio_pci 0000:0a:00.0
 
 
    # Recheck the status of the devices.
    dpdk-devbind --status
    Network devices using DPDK-compatible driver
    ============================================
-   0000:0a:00.0 '82599ES 10-Gigabit' drv=igb_uio unused=
+   0000:0a:00.0 '82599ES 10-Gigabit' drv=vfio_pci unused=
-- 
2.27.0


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [dpdk-dev] [PATCH v2] doc/devbind: remove references to igb_uio
  2020-09-24 17:53 ` [dpdk-dev] [PATCH v2] " Stephen Hemminger
@ 2020-10-14 16:43   ` Ferruh Yigit
  2020-11-26 11:48     ` Bruce Richardson
  0 siblings, 1 reply; 7+ messages in thread
From: Ferruh Yigit @ 2020-10-14 16:43 UTC (permalink / raw)
  To: Stephen Hemminger, dev

On 9/24/2020 6:53 PM, Stephen Hemminger wrote:
> We should be encouraging the use of vfio_pci for developers, not telling
> them to use igb_uio.
> 
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> ---
> v2 -- drop change about filename
> 
>   doc/guides/tools/devbind.rst | 18 +++++++++---------
>   1 file changed, 9 insertions(+), 9 deletions(-)
> 
> diff --git a/doc/guides/tools/devbind.rst b/doc/guides/tools/devbind.rst
> index a2910e103aab..5dcaf8c207dd 100644
> --- a/doc/guides/tools/devbind.rst
> +++ b/doc/guides/tools/devbind.rst
> @@ -30,11 +30,11 @@ OPTIONS
>           Print the current status of all known network interfaces.
>           For each device, it displays the PCI domain, bus, slot and function,
>           along with a text description of the device. Depending upon whether the
> -        device is being used by a kernel driver, the ``igb_uio`` driver, or no
> +        device is being used by a kernel driver, the ``vfio_pci`` driver, or no
>           driver, other relevant information will be displayed:
>           - the Linux interface name e.g. ``if=eth0``
> -        - the driver being used e.g. ``drv=igb_uio``
> -        - any suitable drivers not currently using that device e.g. ``unused=igb_uio``
> +        - the driver being used e.g. ``drv=vfio_pci``
> +        - any suitable drivers not currently using that device e.g. ``unused=vfio_pci``
>           NOTE: if this flag is passed along with a bind/unbind option, the
>           status display will always occur after the other operations have taken
>           place.
> @@ -80,9 +80,9 @@ To display current device status::
>   
>      dpdk-devbind --status
>   
> -To bind eth1 from the current driver and move to use igb_uio::
> +To bind eth1 from the current driver and move to use vfio_pci::
>   
> -   dpdk-devbind --bind=igb_uio eth1
> +   dpdk-devbind --bind=vfio_pci eth1

Module name is 'vfio_pci' but driver name is 'vfio-pci', for this document I 
guess all instances should be 'vfio-pci'.

>   
>   To unbind 0000:01:00.0 from using any driver::
>   
> @@ -92,7 +92,7 @@ To bind 0000:02:00.0 and 0000:02:00.1 to the ixgbe kernel driver::
>   
>      dpdk-devbind -b ixgbe 02:00.0 02:00.1
>   
> -To check status of all network ports, assign one to the igb_uio driver and check status again::
> +To check status of all network ports, assign one to the vfio_pci driver and check status again::
>   
>      # Check the status of the available devices.
>      dpdk-devbind --status
> @@ -105,12 +105,12 @@ To check status of all network ports, assign one to the igb_uio driver and check
>      0000:0a:00.0 '82599ES 10-Gigabit' if=eth2 drv=ixgbe unused=
>   
>   
> -   # Bind the device to igb_uio.
> -   sudo dpdk-devbind -b igb_uio 0000:0a:00.0
> +   # Bind the device to vfio_pci.
> +   sudo dpdk-devbind -b vfio_pci 0000:0a:00.0
>   
>   
>      # Recheck the status of the devices.
>      dpdk-devbind --status
>      Network devices using DPDK-compatible driver
>      ============================================
> -   0000:0a:00.0 '82599ES 10-Gigabit' drv=igb_uio unused=
> +   0000:0a:00.0 '82599ES 10-Gigabit' drv=vfio_pci unused=
> 


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [dpdk-dev] [PATCH v2] doc/devbind: remove references to igb_uio
  2020-10-14 16:43   ` Ferruh Yigit
@ 2020-11-26 11:48     ` Bruce Richardson
  2020-11-26 16:00       ` David Marchand
  0 siblings, 1 reply; 7+ messages in thread
From: Bruce Richardson @ 2020-11-26 11:48 UTC (permalink / raw)
  To: Ferruh Yigit; +Cc: Stephen Hemminger, dev

On Wed, Oct 14, 2020 at 05:43:29PM +0100, Ferruh Yigit wrote:
> On 9/24/2020 6:53 PM, Stephen Hemminger wrote:
> > We should be encouraging the use of vfio_pci for developers, not telling
> > them to use igb_uio.
> > 
> > Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> > ---
> > v2 -- drop change about filename
> > 
> >   doc/guides/tools/devbind.rst | 18 +++++++++---------
> >   1 file changed, 9 insertions(+), 9 deletions(-)
> > 
> > diff --git a/doc/guides/tools/devbind.rst b/doc/guides/tools/devbind.rst
> > index a2910e103aab..5dcaf8c207dd 100644
> > --- a/doc/guides/tools/devbind.rst
> > +++ b/doc/guides/tools/devbind.rst
> > @@ -30,11 +30,11 @@ OPTIONS
> >           Print the current status of all known network interfaces.
> >           For each device, it displays the PCI domain, bus, slot and function,
> >           along with a text description of the device. Depending upon whether the
> > -        device is being used by a kernel driver, the ``igb_uio`` driver, or no
> > +        device is being used by a kernel driver, the ``vfio_pci`` driver, or no
> >           driver, other relevant information will be displayed:
> >           - the Linux interface name e.g. ``if=eth0``
> > -        - the driver being used e.g. ``drv=igb_uio``
> > -        - any suitable drivers not currently using that device e.g. ``unused=igb_uio``
> > +        - the driver being used e.g. ``drv=vfio_pci``
> > +        - any suitable drivers not currently using that device e.g. ``unused=vfio_pci``
> >           NOTE: if this flag is passed along with a bind/unbind option, the
> >           status display will always occur after the other operations have taken
> >           place.
> > @@ -80,9 +80,9 @@ To display current device status::
> >      dpdk-devbind --status
> > -To bind eth1 from the current driver and move to use igb_uio::
> > +To bind eth1 from the current driver and move to use vfio_pci::
> > -   dpdk-devbind --bind=igb_uio eth1
> > +   dpdk-devbind --bind=vfio_pci eth1
> 
> Module name is 'vfio_pci' but driver name is 'vfio-pci', for this document I
> guess all instances should be 'vfio-pci'.
> 

Yes, I believe all instances of vfio_pci should be replaced with the "-"ed
version. With that change:

Acked-by: Bruce Richardson <bruce.richardson@intel.com>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [dpdk-dev] [PATCH v2] doc/devbind: remove references to igb_uio
  2020-11-26 11:48     ` Bruce Richardson
@ 2020-11-26 16:00       ` David Marchand
  0 siblings, 0 replies; 7+ messages in thread
From: David Marchand @ 2020-11-26 16:00 UTC (permalink / raw)
  To: Bruce Richardson; +Cc: Ferruh Yigit, Stephen Hemminger, dev

On Thu, Nov 26, 2020 at 12:49 PM Bruce Richardson
<bruce.richardson@intel.com> wrote:
>
> On Wed, Oct 14, 2020 at 05:43:29PM +0100, Ferruh Yigit wrote:
> > On 9/24/2020 6:53 PM, Stephen Hemminger wrote:
> > > We should be encouraging the use of vfio_pci for developers, not telling
> > > them to use igb_uio.
> > >
> > > Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>

> > Module name is 'vfio_pci' but driver name is 'vfio-pci', for this document I
> > guess all instances should be 'vfio-pci'.
> >
>
> Yes, I believe all instances of vfio_pci should be replaced with the "-"ed
> version. With that change:
> Acked-by: Bruce Richardson <bruce.richardson@intel.com>

Indeed, passing vfio_pci fails:
$ sudo ./usertools/dpdk-devbind.py --bind=vfio_pci 0000:01:00.0
Error: bind failed for 0000:01:00.0 - Cannot open
/sys/bus/pci/drivers/vfio_pci/bind
Error: unbind failed for 0000:01:00.0 - Cannot open /sys/bus/pci/drivers//unbind

$ ./usertools/dpdk-devbind.py --status |grep 0000:01:00.0
0000:01:00.0 '82599ES 10-Gigabit SFI/SFP+ Network Connection 10fb'
drv=vfio-pci unused=ixgbe

Fixed and applied.

-- 
David Marchand


^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2020-11-26 16:01 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-06  3:38 [dpdk-dev] [PATCH] doc/devbind: remove references to igb_uio Stephen Hemminger
2020-09-07  9:08 ` Bruce Richardson
2020-09-07 17:22   ` Stephen Hemminger
2020-09-24 17:53 ` [dpdk-dev] [PATCH v2] " Stephen Hemminger
2020-10-14 16:43   ` Ferruh Yigit
2020-11-26 11:48     ` Bruce Richardson
2020-11-26 16:00       ` David Marchand

DPDK patches and discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://inbox.dpdk.org/dev/0 dev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dev dev/ https://inbox.dpdk.org/dev \
		dev@dpdk.org
	public-inbox-index dev

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dev


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git