[dpdk-dev] [PATCH] virtio: Fix stats issue

2015-03-27 Thread Thomas Monjalon
2015-03-27 14:26, Ouyang, Changchun: > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > 2015-03-23 16:38, Ouyang, Changchun: > > > On 3/23/2015 3:20 PM, David Marchand wrote: > > > > On Mon, Mar 23, 2015 at 7:56 AM, Ouyang Changchun wrote: > > > > --- a/lib/librte_pmd_virtio/virt

[dpdk-dev] [PATCH] virtio: Fix stats issue

2015-03-27 Thread Ouyang, Changchun
Hi Stephen, Thomas, David, > -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Monday, March 23, 2015 6:42 PM > To: Ouyang, Changchun > Cc: dev at dpdk.org; David Marchand > Subject: Re: [dpdk-dev] [PATCH] virtio: Fix stats issue

[dpdk-dev] [PATCH] virtio: Fix stats issue

2015-03-23 Thread Ouyang, Changchun
On 3/23/2015 3:20 PM, David Marchand wrote: > Hello, > > Hello, > > On Mon, Mar 23, 2015 at 7:56 AM, Ouyang Changchun > mailto:changchun.ouyang at intel.com>> wrote: > > It need clear/reset the stats information before count in all > queues data. > > Signed-off-by: Changchun Ouyang

[dpdk-dev] [PATCH] virtio: Fix stats issue

2015-03-23 Thread Stephen Hemminger
I agree with Thomas. If some other drivers have memset they should also be fixed

[dpdk-dev] [PATCH] virtio: Fix stats issue

2015-03-23 Thread Ouyang Changchun
It need clear/reset the stats information before count in all queues data. Signed-off-by: Changchun Ouyang --- lib/librte_pmd_virtio/virtio_ethdev.c | 8 1 file changed, 8 insertions(+) diff --git a/lib/librte_pmd_virtio/virtio_ethdev.c b/lib/librte_pmd_virtio/virtio_ethdev.c index 60

[dpdk-dev] [PATCH] virtio: Fix stats issue

2015-03-23 Thread Thomas Monjalon
2015-03-23 16:38, Ouyang, Changchun: > On 3/23/2015 3:20 PM, David Marchand wrote: > > On Mon, Mar 23, 2015 at 7:56 AM, Ouyang Changchun > > mailto:changchun.ouyang at intel.com>> > > wrote: > > > > It need clear/reset the stats information before count in all > > queues data. > > > >

[dpdk-dev] [PATCH] virtio: Fix stats issue

2015-03-23 Thread David Marchand
Hello, Hello, On Mon, Mar 23, 2015 at 7:56 AM, Ouyang Changchun < changchun.ouyang at intel.com> wrote: > It need clear/reset the stats information before count in all queues data. > > Signed-off-by: Changchun Ouyang > --- > lib/librte_pmd_virtio/virtio_ethdev.c | 8 > 1 file changed,