DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] doc: update overview
@ 2016-04-01  8:18 Wenzhuo Lu
  2016-04-01 21:33 ` Thomas Monjalon
  2016-04-05  2:54 ` [dpdk-dev] [PATCH v2] " Wenzhuo Lu
  0 siblings, 2 replies; 9+ messages in thread
From: Wenzhuo Lu @ 2016-04-01  8:18 UTC (permalink / raw)
  To: dev; +Cc: Wenzhuo Lu

Update the overview.rst for e1000, igb, ixgbe.

Signed-off-by: Wenzhuo Lu <wenzhuo.lu@intel.com>
---
 doc/guides/nics/overview.rst | 94 ++++++++++++++++++++++----------------------
 1 file changed, 47 insertions(+), 47 deletions(-)

diff --git a/doc/guides/nics/overview.rst b/doc/guides/nics/overview.rst
index 542479a..bc5cde5 100644
--- a/doc/guides/nics/overview.rst
+++ b/doc/guides/nics/overview.rst
@@ -86,62 +86,62 @@ Most of these differences are summarized below.
                                           e   e       e   e   e                     e
                                           c   c       c   c   c                     c
    ==================== = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = =
-   link status                  X       X X                                   X X
-   link status event                    X X                                     X
+   link status                  X X     X X     X   X X X X                   X X
+   link status event              X     X X     X   X X                         X
    queue status event                                                           X
-   Rx interrupt                         X X X X
-   queue start/stop             X   X   X X X X                               X
-   MTU update                   X   X
-   jumbo frame                  X   X   X X X X
-   scattered Rx                 X   X   X X X X                               X
-   LRO
-   TSO                          X   X   X X X X
-   promiscuous mode             X       X X X X                               X
-   allmulticast mode            X       X X X X                               X
-   unicast MAC filter                   X X X X
+   Rx interrupt                   X   X X X X X X X X X X X
+   queue start/stop             X   X   X X X X     X X                       X
+   MTU update                   X X X           X   X X X X
+   jumbo frame                  X X X   X X X X X   X X X X
+   scattered Rx                 X X X   X X X X X X X X X X                   X
+   LRO                                              X X X X
+   TSO                          X   X   X X X X X X X X X X
+   promiscuous mode             X   X   X X X X X X X X                       X
+   allmulticast mode            X   X   X X X X X X X X X X                   X
+   unicast MAC filter               X   X X X X X X X X X X
    multicast MAC filter                 X X X X
-   RSS hash                     X   X   X X X X
-   RSS key update                   X   X X X X
-   RSS reta update                  X   X X X X
-   VMDq                                 X X
-   SR-IOV                           X   X X
-   DCB                                  X X
-   VLAN filter                          X X X X
-   ethertype filter                     X X
-   n-tuple filter
-   SYN filter
-   tunnel filter                        X X
-   flexible filter
+   RSS hash                     X   X   X X X X X   X X X X
+   RSS key update                   X   X X X X X   X X X X
+   RSS reta update                  X   X X X X X   X X X X
+   VMDq                                 X X     X   X X
+   SR-IOV                           X   X X     X   X X
+   DCB                                  X X     X   X X
+   VLAN filter                    X     X X X X X X X X X X
+   ethertype filter                     X X     X   X X
+   n-tuple filter                               X   X X
+   SYN filter                                   X   X X
+   tunnel filter                        X X         X X
+   flexible filter                              X
    hash filter                          X X X X
-   flow director                        X X
-   flow control                 X       X X
-   rate limitation
-   traffic mirroring                    X X
-   CRC offload                  X   X   X   X
-   VLAN offload                 X   X   X   X
-   QinQ offload                         X   X
-   L3 checksum offload          X   X   X   X
-   L4 checksum offload          X   X   X   X
-   inner L3 checksum                X   X   X
-   inner L4 checksum                X   X   X
-   packet type parsing          X       X   X
-   timesync                             X X
-   basic stats                  X   X   X X X X                               X X
-   extended stats                   X   X X X X
+   flow director                        X X         X X
+   flow control                 X X     X X     X   X X
+   rate limitation                                  X X
+   traffic mirroring                    X X         X X
+   CRC offload                  X X X   X   X   X X X   X
+   VLAN offload                 X X X   X   X   X X X   X
+   QinQ offload                   X     X   X   X X X   X
+   L3 checksum offload          X X X   X   X   X X X   X
+   L4 checksum offload          X X X   X   X   X X X   X
+   inner L3 checksum                X   X   X       X   X
+   inner L4 checksum                X   X   X       X   X
+   packet type parsing          X       X   X   X X X   X
+   timesync                             X X     X   X X
+   basic stats                  X X X   X X X X X X X X X X                   X X
+   extended stats                   X   X X X X X X X X X X
    stats per queue              X                                             X
-   EEPROM dump
-   registers dump
-   multiprocess aware                   X X X X
-   BSD nic_uio                  X       X X X X
-   Linux UIO                    X   X   X X X X
-   Linux VFIO                   X       X X X X
+   EEPROM dump                                  X   X X
+   registers dump                               X X X X X X
+   multiprocess aware                   X X X X     X X X X
+   BSD nic_uio                  X X     X X X X X X X X X X
+   Linux UIO                    X X X   X X X X X X X X X X
+   Linux VFIO                   X X     X X X X X X X X X X
    other kdrv                                                                 X
    ARMv7
    ARMv8
    Power8
    TILE-Gx
-   x86-32                       X   X   X X X X                                 X
-   x86-64                       X   X   X X X X                               X X
+   x86-32                       X X X   X X X X X X X X X X                     X
+   x86-64                       X X X   X X X X X X X X X X                   X X
    usage doc                    X                                             X
    design doc
    perf doc
-- 
1.9.3

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

* Re: [dpdk-dev] [PATCH] doc: update overview
  2016-04-01  8:18 [dpdk-dev] [PATCH] doc: update overview Wenzhuo Lu
@ 2016-04-01 21:33 ` Thomas Monjalon
  2016-04-05  0:50   ` Lu, Wenzhuo
  2016-04-05  2:54 ` [dpdk-dev] [PATCH v2] " Wenzhuo Lu
  1 sibling, 1 reply; 9+ messages in thread
From: Thomas Monjalon @ 2016-04-01 21:33 UTC (permalink / raw)
  To: Wenzhuo Lu; +Cc: dev

2016-04-01 16:18, Wenzhuo Lu:
> Update the overview.rst for e1000, igb, ixgbe.
> 
> Signed-off-by: Wenzhuo Lu <wenzhuo.lu@intel.com>

Please double check the patch.
You are filling some features for ena and enic.

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

* Re: [dpdk-dev] [PATCH] doc: update overview
  2016-04-01 21:33 ` Thomas Monjalon
@ 2016-04-05  0:50   ` Lu, Wenzhuo
  0 siblings, 0 replies; 9+ messages in thread
From: Lu, Wenzhuo @ 2016-04-05  0:50 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: dev

Hi Thomas,

> -----Original Message-----
> From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com]
> Sent: Saturday, April 2, 2016 5:33 AM
> To: Lu, Wenzhuo
> Cc: dev@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH] doc: update overview
> 
> 2016-04-01 16:18, Wenzhuo Lu:
> > Update the overview.rst for e1000, igb, ixgbe.
> >
> > Signed-off-by: Wenzhuo Lu <wenzhuo.lu@intel.com>
> 
> Please double check the patch.
> You are filling some features for ena and enic.
Sorry, I'll check it and send a V2.

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

* [dpdk-dev] [PATCH v2] doc: update overview
  2016-04-01  8:18 [dpdk-dev] [PATCH] doc: update overview Wenzhuo Lu
  2016-04-01 21:33 ` Thomas Monjalon
@ 2016-04-05  2:54 ` Wenzhuo Lu
  2016-04-06  8:25   ` Thomas Monjalon
  2016-04-06  8:26   ` Thomas Monjalon
  1 sibling, 2 replies; 9+ messages in thread
From: Wenzhuo Lu @ 2016-04-05  2:54 UTC (permalink / raw)
  To: dev; +Cc: Wenzhuo Lu

Update the overview.rst for e1000, igb, ixgbe.

v2:
- Some "X"s are put in the wrong place, correct it.

Signed-off-by: Wenzhuo Lu <wenzhuo.lu@intel.com>
---
 doc/guides/nics/overview.rst | 94 ++++++++++++++++++++++----------------------
 1 file changed, 47 insertions(+), 47 deletions(-)

diff --git a/doc/guides/nics/overview.rst b/doc/guides/nics/overview.rst
index 542479a..6c53826 100644
--- a/doc/guides/nics/overview.rst
+++ b/doc/guides/nics/overview.rst
@@ -86,62 +86,62 @@ Most of these differences are summarized below.
                                           e   e       e   e   e                     e
                                           c   c       c   c   c                     c
    ==================== = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = =
-   link status                  X       X X                                   X X
-   link status event                    X X                                     X
+   link status                  X X     X X     X   X X X X                   X X
+   link status event              X     X X     X   X X                         X
    queue status event                                                           X
-   Rx interrupt                         X X X X
-   queue start/stop             X   X   X X X X                               X
-   MTU update                   X   X
-   jumbo frame                  X   X   X X X X
-   scattered Rx                 X   X   X X X X                               X
-   LRO
-   TSO                          X   X   X X X X
-   promiscuous mode             X       X X X X                               X
-   allmulticast mode            X       X X X X                               X
-   unicast MAC filter                   X X X X
+   Rx interrupt                   X     X X X X X X X X X X
+   queue start/stop             X   X   X X X X     X X                       X
+   MTU update                   X X X           X   X X X X
+   jumbo frame                  X X X   X X X X X   X X X X
+   scattered Rx                 X X X   X X X X X X X X X X                   X
+   LRO                                              X X X X
+   TSO                          X   X   X X X X X X X X X X
+   promiscuous mode             X X     X X X X X X X X                       X
+   allmulticast mode            X X     X X X X X X X X X X                   X
+   unicast MAC filter             X     X X X X X X X X X X
    multicast MAC filter                 X X X X
-   RSS hash                     X   X   X X X X
-   RSS key update                   X   X X X X
-   RSS reta update                  X   X X X X
-   VMDq                                 X X
-   SR-IOV                           X   X X
-   DCB                                  X X
-   VLAN filter                          X X X X
-   ethertype filter                     X X
-   n-tuple filter
-   SYN filter
-   tunnel filter                        X X
-   flexible filter
+   RSS hash                     X   X   X X X X X   X X X X
+   RSS key update                   X   X X X X X   X X X X
+   RSS reta update                  X   X X X X X   X X X X
+   VMDq                                 X X     X   X X
+   SR-IOV                           X   X X     X   X X
+   DCB                                  X X     X   X X
+   VLAN filter                    X     X X X X X X X X X X
+   ethertype filter                     X X     X   X X
+   n-tuple filter                               X   X X
+   SYN filter                                   X   X X
+   tunnel filter                        X X         X X
+   flexible filter                              X
    hash filter                          X X X X
-   flow director                        X X
-   flow control                 X       X X
-   rate limitation
-   traffic mirroring                    X X
-   CRC offload                  X   X   X   X
-   VLAN offload                 X   X   X   X
-   QinQ offload                         X   X
-   L3 checksum offload          X   X   X   X
-   L4 checksum offload          X   X   X   X
-   inner L3 checksum                X   X   X
-   inner L4 checksum                X   X   X
-   packet type parsing          X       X   X
-   timesync                             X X
-   basic stats                  X   X   X X X X                               X X
-   extended stats                   X   X X X X
+   flow director                        X X         X X
+   flow control                 X X     X X     X   X X
+   rate limitation                                  X X
+   traffic mirroring                    X X         X X
+   CRC offload                  X X X   X   X   X X X   X
+   VLAN offload                 X X X   X   X   X X X   X
+   QinQ offload                   X     X   X   X X X   X
+   L3 checksum offload          X X X   X   X   X X X   X
+   L4 checksum offload          X X X   X   X   X X X   X
+   inner L3 checksum                X   X   X       X   X
+   inner L4 checksum                X   X   X       X   X
+   packet type parsing          X       X   X   X X X   X
+   timesync                             X X     X   X X
+   basic stats                  X X X   X X X X X X X X X X                   X X
+   extended stats                   X   X X X X X X X X X X
    stats per queue              X                                             X
-   EEPROM dump
-   registers dump
-   multiprocess aware                   X X X X
-   BSD nic_uio                  X       X X X X
-   Linux UIO                    X   X   X X X X
-   Linux VFIO                   X       X X X X
+   EEPROM dump                                  X   X X
+   registers dump                               X X X X X X
+   multiprocess aware                   X X X X     X X X X
+   BSD nic_uio                  X X     X X X X X X X X X X
+   Linux UIO                    X X X   X X X X X X X X X X
+   Linux VFIO                   X X     X X X X X X X X X X
    other kdrv                                                                 X
    ARMv7
    ARMv8
    Power8
    TILE-Gx
-   x86-32                       X   X   X X X X                                 X
-   x86-64                       X   X   X X X X                               X X
+   x86-32                       X X X   X X X X X X X X X X                     X
+   x86-64                       X X X   X X X X X X X X X X                   X X
    usage doc                    X                                             X
    design doc
    perf doc
-- 
1.9.3

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

* Re: [dpdk-dev] [PATCH v2] doc: update overview
  2016-04-05  2:54 ` [dpdk-dev] [PATCH v2] " Wenzhuo Lu
@ 2016-04-06  8:25   ` Thomas Monjalon
  2016-04-06  8:26   ` Thomas Monjalon
  1 sibling, 0 replies; 9+ messages in thread
From: Thomas Monjalon @ 2016-04-06  8:25 UTC (permalink / raw)
  To: Wenzhuo Lu; +Cc: dev

2016-04-05 10:54, Wenzhuo Lu:
> Update the overview.rst for e1000, igb, ixgbe.
> 
> v2:
> - Some "X"s are put in the wrong place, correct it.
> 
> Signed-off-by: Wenzhuo Lu <wenzhuo.lu@intel.com>

Applied, thanks

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

* Re: [dpdk-dev] [PATCH v2] doc: update overview
  2016-04-05  2:54 ` [dpdk-dev] [PATCH v2] " Wenzhuo Lu
  2016-04-06  8:25   ` Thomas Monjalon
@ 2016-04-06  8:26   ` Thomas Monjalon
  2016-04-07  1:34     ` Lu, Wenzhuo
  1 sibling, 1 reply; 9+ messages in thread
From: Thomas Monjalon @ 2016-04-06  8:26 UTC (permalink / raw)
  To: Wenzhuo Lu; +Cc: dev

2016-04-05 10:54, Wenzhuo Lu:
> Update the overview.rst for e1000, igb, ixgbe.
[...]
>     stats per queue              X                                             X

You have not filled stats per queue. Is it because of an issue
with queue_stats_mapping?

[...]
>     usage doc                    X                                             X
>     design doc
>     perf doc

Do you plan to complete the docs for these drivers?

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

* Re: [dpdk-dev] [PATCH v2] doc: update overview
  2016-04-06  8:26   ` Thomas Monjalon
@ 2016-04-07  1:34     ` Lu, Wenzhuo
  2016-04-07  7:38       ` Thomas Monjalon
  0 siblings, 1 reply; 9+ messages in thread
From: Lu, Wenzhuo @ 2016-04-07  1:34 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: dev

Hi Thomas,

> -----Original Message-----
> From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com]
> Sent: Wednesday, April 6, 2016 4:27 PM
> To: Lu, Wenzhuo
> Cc: dev@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v2] doc: update overview
> 
> 2016-04-05 10:54, Wenzhuo Lu:
> > Update the overview.rst for e1000, igb, ixgbe.
> [...]
> >     stats per queue              X                                             X
> 
> You have not filled stats per queue. Is it because of an issue with
> queue_stats_mapping?
I didn't know it means this function. I'll create a patch to update it. Thanks.

> 
> [...]
> >     usage doc                    X                                             X
> >     design doc
> >     perf doc
> 
> Do you plan to complete the docs for these drivers?
Sure, we'll try the enrich the docs.

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

* Re: [dpdk-dev] [PATCH v2] doc: update overview
  2016-04-07  1:34     ` Lu, Wenzhuo
@ 2016-04-07  7:38       ` Thomas Monjalon
  2016-04-07  8:17         ` Lu, Wenzhuo
  0 siblings, 1 reply; 9+ messages in thread
From: Thomas Monjalon @ 2016-04-07  7:38 UTC (permalink / raw)
  To: Lu, Wenzhuo; +Cc: dev

2016-04-07 01:34, Lu, Wenzhuo:
> From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com]
> > >     stats per queue              X                                             X
> > 
> > You have not filled stats per queue. Is it because of an issue with
> > queue_stats_mapping?
> I didn't know it means this function. I'll create a patch to update it. Thanks.

Excuse me, I'm confusing you.
The "stats per queue" features means the function rte_eth_stats_get()
fills the arrays q_*.

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

* Re: [dpdk-dev] [PATCH v2] doc: update overview
  2016-04-07  7:38       ` Thomas Monjalon
@ 2016-04-07  8:17         ` Lu, Wenzhuo
  0 siblings, 0 replies; 9+ messages in thread
From: Lu, Wenzhuo @ 2016-04-07  8:17 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: dev

Hi Thomas,

> -----Original Message-----
> From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com]
> Sent: Thursday, April 7, 2016 3:38 PM
> To: Lu, Wenzhuo
> Cc: dev@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v2] doc: update overview
> 
> 2016-04-07 01:34, Lu, Wenzhuo:
> > From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com]
> > > >     stats per queue              X                                             X
> > >
> > > You have not filled stats per queue. Is it because of an issue with
> > > queue_stats_mapping?
> > I didn't know it means this function. I'll create a patch to update it. Thanks.
> 
> Excuse me, I'm confusing you.
> The "stats per queue" features means the function rte_eth_stats_get() fills the
> arrays q_*.
NP. I missed this part, because I assumed it is a specific API like rte_eth_queue_stats_get. Did not notice the stats are included in rte_eth_stats_get.
I've sent another patch for this mistake.

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

end of thread, other threads:[~2016-04-07  8:17 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-04-01  8:18 [dpdk-dev] [PATCH] doc: update overview Wenzhuo Lu
2016-04-01 21:33 ` Thomas Monjalon
2016-04-05  0:50   ` Lu, Wenzhuo
2016-04-05  2:54 ` [dpdk-dev] [PATCH v2] " Wenzhuo Lu
2016-04-06  8:25   ` Thomas Monjalon
2016-04-06  8:26   ` Thomas Monjalon
2016-04-07  1:34     ` Lu, Wenzhuo
2016-04-07  7:38       ` Thomas Monjalon
2016-04-07  8:17         ` Lu, Wenzhuo

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).