[dpdk-dev] [PATCH v2] doc: update overview

2016-04-07 Thread Thomas Monjalon
2016-04-07 01:34, Lu, Wenzhuo:
> From: Thomas Monjalon [mailto:thomas.monjalon at 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_*.



[dpdk-dev] [PATCH v2] doc: update overview

2016-04-07 Thread Lu, Wenzhuo
Hi Thomas,

> -Original Message-
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> Sent: Thursday, April 7, 2016 3:38 PM
> To: Lu, Wenzhuo
> Cc: dev at dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v2] doc: update overview
> 
> 2016-04-07 01:34, Lu, Wenzhuo:
> > From: Thomas Monjalon [mailto:thomas.monjalon at 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.



[dpdk-dev] [PATCH v2] doc: update overview

2016-04-07 Thread Lu, Wenzhuo
Hi Thomas,

> -Original Message-
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> Sent: Wednesday, April 6, 2016 4:27 PM
> To: Lu, Wenzhuo
> Cc: dev at 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 docX  
> >X
> > design doc
> > perf doc
> 
> Do you plan to complete the docs for these drivers?
Sure, we'll try the enrich the docs.


[dpdk-dev] [PATCH v2] doc: update overview

2016-04-06 Thread Thomas Monjalon
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 

Applied, thanks



[dpdk-dev] [PATCH v2] doc: update overview

2016-04-06 Thread Thomas Monjalon
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 docX
>  X
> design doc
> perf doc

Do you plan to complete the docs for these drivers?


[dpdk-dev] [PATCH v2] doc: update overview

2016-04-05 Thread 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 
---
 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 eventX 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 modeX   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 modeX 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 filterX 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 filterX 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 filterX X X X
+   flexible filter  X
hash filter  X X X X
-   flow directorX X
-   flow control X   X X
-   rate limitation
-   traffic mirroringX 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 checksumX   X   X
-   inner L4 checksumX   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 directorX X X X
+   flow control X X X X X   X X
+   rate limitation  X X
+   traffic mirroringX 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 checksumX   X