doc: merge virtio vector features into virtio ones

Message ID 20200420075310.6059-1-maxime.coquelin@redhat.com (mailing list archive)
State Accepted, archived
Delegated to: Maxime Coquelin
Headers
Series doc: merge virtio vector features into virtio ones |

Checks

Context Check Description
ci/checkpatch warning coding style issues
ci/travis-robot success Travis build: passed
ci/Intel-compilation success Compilation OK

Commit Message

Maxime Coquelin April 20, 2020, 7:53 a.m. UTC
  The goal is to make the table more readable.

Mark as partially supported features that are supported in
the genric Virtio driver but not in the vectorized ones.

Suggested-by: Thomas Monjalon <thomas@monjalon.net>
Signed-off-by: Maxime Coquelin <maxime.coquelin@redhat.com>
---
 doc/guides/nics/features/virtio.ini     |  4 ++--
 doc/guides/nics/features/virtio_vec.ini | 25 -------------------------
 2 files changed, 2 insertions(+), 27 deletions(-)
 delete mode 100644 doc/guides/nics/features/virtio_vec.ini
  

Comments

Thomas Monjalon April 24, 2020, 9:28 a.m. UTC | #1
20/04/2020 09:53, Maxime Coquelin:
> The goal is to make the table more readable.
> 
> Mark as partially supported features that are supported in
> the genric Virtio driver but not in the vectorized ones.
> 
> Suggested-by: Thomas Monjalon <thomas@monjalon.net>
> Signed-off-by: Maxime Coquelin <maxime.coquelin@redhat.com>

Looks good, thanks.
No ack from me, I don't know virtio PMD enough :)
  
Maxime Coquelin April 28, 2020, 4:05 p.m. UTC | #2
On 4/20/20 9:53 AM, Maxime Coquelin wrote:
> The goal is to make the table more readable.
> 
> Mark as partially supported features that are supported in
> the genric Virtio driver but not in the vectorized ones.
> 
> Suggested-by: Thomas Monjalon <thomas@monjalon.net>
> Signed-off-by: Maxime Coquelin <maxime.coquelin@redhat.com>
> ---
>  doc/guides/nics/features/virtio.ini     |  4 ++--
>  doc/guides/nics/features/virtio_vec.ini | 25 -------------------------
>  2 files changed, 2 insertions(+), 27 deletions(-)
>  delete mode 100644 doc/guides/nics/features/virtio_vec.ini

Applied to dpdk-next-virtio/master

Thanks,
Maxime
  

Patch

diff --git a/doc/guides/nics/features/virtio.ini b/doc/guides/nics/features/virtio.ini
index a16b817214..bbf9730318 100644
--- a/doc/guides/nics/features/virtio.ini
+++ b/doc/guides/nics/features/virtio.ini
@@ -9,7 +9,7 @@  Link status          = Y
 Link status event    = Y
 Rx interrupt         = Y
 Queue start/stop     = Y
-Scattered Rx         = Y
+Scattered Rx         = P
 Promiscuous mode     = Y
 Allmulticast mode    = Y
 Unicast MAC filter   = Y
@@ -17,7 +17,7 @@  Multicast MAC filter = Y
 VLAN filter          = Y
 Basic stats          = Y
 Stats per queue      = Y
-Extended stats       = Y
+Extended stats       = P
 Multiprocess aware   = Y
 BSD nic_uio          = Y
 Linux UIO            = Y
diff --git a/doc/guides/nics/features/virtio_vec.ini b/doc/guides/nics/features/virtio_vec.ini
deleted file mode 100644
index e60fe36aeb..0000000000
--- a/doc/guides/nics/features/virtio_vec.ini
+++ /dev/null
@@ -1,25 +0,0 @@ 
-;
-; Supported features of the 'virtio_vec' network poll mode driver.
-;
-; Refer to default.ini for the full list of available PMD features.
-;
-[Features]
-Speed capabilities   = P
-Link status          = Y
-Link status event    = Y
-Rx interrupt         = Y
-Queue start/stop     = Y
-Promiscuous mode     = Y
-Allmulticast mode    = Y
-Unicast MAC filter   = Y
-Multicast MAC filter = Y
-VLAN filter          = Y
-Basic stats          = Y
-Stats per queue      = Y
-BSD nic_uio          = Y
-Linux UIO            = Y
-Linux VFIO           = Y
-ARMv7                = Y
-ARMv8                = Y
-x86-32               = Y
-x86-64               = Y