[dpdk-dev] [PATCH] virtio: fix used ring address calculation

2015-09-25 Thread Xie, Huawei
On 9/25/2015 5:01 AM, Stephen Hemminger wrote: > On Thu, 24 Sep 2015 18:35:37 + > "Xie, Huawei" wrote: > >> On 9/25/2015 12:36 AM, Stephen Hemminger wrote: >>> On Thu, 24 Sep 2015 07:30:41 + >>> "Xie, Huawei" wrote: >>> On 9/21/2015 11:39 AM, Xie, Huawei wrote: vring_size calcul

[dpdk-dev] [PATCH] virtio: fix used ring address calculation

2015-09-25 Thread Stephen Hemminger
On Fri, 25 Sep 2015 15:46:34 + "Xie, Huawei" wrote: > On 9/25/2015 5:01 AM, Stephen Hemminger wrote: > > On Thu, 24 Sep 2015 18:35:37 + > > "Xie, Huawei" wrote: > > > >> On 9/25/2015 12:36 AM, Stephen Hemminger wrote: > >>> On Thu, 24 Sep 2015 07:30:41 + > >>> "Xie, Huawei" wrote: >

[dpdk-dev] [PATCH] virtio: fix used ring address calculation

2015-09-24 Thread Xie, Huawei
On 9/25/2015 12:36 AM, Stephen Hemminger wrote: > On Thu, 24 Sep 2015 07:30:41 + > "Xie, Huawei" wrote: > >> On 9/21/2015 11:39 AM, Xie, Huawei wrote: >> vring_size calculation should consider both used_event_idx at the tail >> of avail ring and avail_event_idx at the tail of used ring. >> Wil

[dpdk-dev] [PATCH] virtio: fix used ring address calculation

2015-09-24 Thread Stephen Hemminger
On Thu, 24 Sep 2015 18:35:37 + "Xie, Huawei" wrote: > On 9/25/2015 12:36 AM, Stephen Hemminger wrote: > > On Thu, 24 Sep 2015 07:30:41 + > > "Xie, Huawei" wrote: > > > >> On 9/21/2015 11:39 AM, Xie, Huawei wrote: > >> vring_size calculation should consider both used_event_idx at the tail

[dpdk-dev] [PATCH] virtio: fix used ring address calculation

2015-09-24 Thread Stephen Hemminger
On Thu, 24 Sep 2015 07:30:41 + "Xie, Huawei" wrote: > On 9/21/2015 11:39 AM, Xie, Huawei wrote: > vring_size calculation should consider both used_event_idx at the tail > of avail ring and avail_event_idx at the tail of used ring. > Will merge those two fixes and send a new patch. > > used ev

[dpdk-dev] [PATCH] virtio: fix used ring address calculation

2015-09-24 Thread Xie, Huawei
On 9/21/2015 11:39 AM, Xie, Huawei wrote: vring_size calculation should consider both used_event_idx at the tail of avail ring and avail_event_idx at the tail of used ring. Will merge those two fixes and send a new patch. > used event idx is put at the end of available ring. It isn't taken into >

[dpdk-dev] [PATCH] virtio: fix used ring address calculation

2015-09-21 Thread Huawei Xie
used event idx is put at the end of available ring. It isn't taken into account when we calculate the address of used ring. Fortunately, it doesn't introduce the bug with fixed queue number 256 and 4KB alignment. Signed-off-by: hxie5 --- drivers/net/virtio/virtio_ring.h | 2 +- 1 file changed, 1

[dpdk-dev] [PATCH] virtio: fix used ring address calculation

2015-09-21 Thread Tan, Jianfeng
-Original Message- From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Huawei Xie Sent: Monday, September 21, 2015 11:39 AM To: dev at dpdk.org Subject: [dpdk-dev] [PATCH] virtio: fix used ring address calculation used event idx is put at the end of available ring. It isn't taken