[dpdk-dev] [PATCH] virtio: use volatile to get used->idx in the loop

2016-06-14 Thread Yuanhan Liu
On Thu, Jun 02, 2016 at 08:54:38AM +, Xie, Huawei wrote: > On 6/2/2016 4:52 PM, Yuanhan Liu wrote: > > On Thu, Jun 02, 2016 at 08:39:36AM +, Xie, Huawei wrote: > >> On 6/1/2016 2:03 PM, Yuanhan Liu wrote: > >>> On Wed, Jun 01, 2016 at 05:40:08AM +, Xie, Huawei wrote: > On 5/30/2016

[dpdk-dev] [PATCH] virtio: use volatile to get used->idx in the loop

2016-06-02 Thread Yuanhan Liu
On Thu, Jun 02, 2016 at 08:39:36AM +, Xie, Huawei wrote: > On 6/1/2016 2:03 PM, Yuanhan Liu wrote: > > On Wed, Jun 01, 2016 at 05:40:08AM +, Xie, Huawei wrote: > >> On 5/30/2016 4:20 PM, Yuanhan Liu wrote: > >>> On Wed, May 25, 2016 at 12:16:41AM +0800, Huawei Xie wrote: > There is no

[dpdk-dev] [PATCH] virtio: use volatile to get used->idx in the loop

2016-06-02 Thread Xie, Huawei
On 6/2/2016 4:52 PM, Yuanhan Liu wrote: > On Thu, Jun 02, 2016 at 08:39:36AM +, Xie, Huawei wrote: >> On 6/1/2016 2:03 PM, Yuanhan Liu wrote: >>> On Wed, Jun 01, 2016 at 05:40:08AM +, Xie, Huawei wrote: On 5/30/2016 4:20 PM, Yuanhan Liu wrote: > On Wed, May 25, 2016 at 12:16:41AM +

[dpdk-dev] [PATCH] virtio: use volatile to get used->idx in the loop

2016-06-02 Thread Xie, Huawei
On 6/1/2016 2:03 PM, Yuanhan Liu wrote: > On Wed, Jun 01, 2016 at 05:40:08AM +, Xie, Huawei wrote: >> On 5/30/2016 4:20 PM, Yuanhan Liu wrote: >>> On Wed, May 25, 2016 at 12:16:41AM +0800, Huawei Xie wrote: There is no external function call or any barrier in the loop, the used->idx w

[dpdk-dev] [PATCH] virtio: use volatile to get used->idx in the loop

2016-06-01 Thread Yuanhan Liu
On Wed, Jun 01, 2016 at 05:40:08AM +, Xie, Huawei wrote: > On 5/30/2016 4:20 PM, Yuanhan Liu wrote: > > On Wed, May 25, 2016 at 12:16:41AM +0800, Huawei Xie wrote: > >> There is no external function call or any barrier in the loop, > >> the used->idx would only be retrieved once. > >> > >> Sign

[dpdk-dev] [PATCH] virtio: use volatile to get used->idx in the loop

2016-06-01 Thread Xie, Huawei
On 5/30/2016 4:20 PM, Yuanhan Liu wrote: > On Wed, May 25, 2016 at 12:16:41AM +0800, Huawei Xie wrote: >> There is no external function call or any barrier in the loop, >> the used->idx would only be retrieved once. >> >> Signed-off-by: Huawei Xie >> --- >> drivers/net/virtio/virtio_ethdev.c | 3

[dpdk-dev] [PATCH] virtio: use volatile to get used->idx in the loop

2016-05-30 Thread Yuanhan Liu
On Wed, May 25, 2016 at 12:16:41AM +0800, Huawei Xie wrote: > There is no external function call or any barrier in the loop, > the used->idx would only be retrieved once. > > Signed-off-by: Huawei Xie > --- > drivers/net/virtio/virtio_ethdev.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deleti

[dpdk-dev] [PATCH] virtio: use volatile to get used->idx in the loop

2016-05-25 Thread Xie, Huawei
On 5/25/2016 6:01 PM, Richardson, Bruce wrote: > On Wed, May 25, 2016 at 12:50:02PM +0300, Michael S. Tsirkin wrote: >> On Wed, May 25, 2016 at 10:47:30AM +0100, Bruce Richardson wrote: >>> On Wed, May 25, 2016 at 11:34:24AM +0300, Michael S. Tsirkin wrote: On Wed, May 25, 2016 at 08:25:20AM +

[dpdk-dev] [PATCH] virtio: use volatile to get used->idx in the loop

2016-05-25 Thread Michael S. Tsirkin
On Wed, May 25, 2016 at 10:47:30AM +0100, Bruce Richardson wrote: > On Wed, May 25, 2016 at 11:34:24AM +0300, Michael S. Tsirkin wrote: > > On Wed, May 25, 2016 at 08:25:20AM +, Xie, Huawei wrote: > > > On 5/25/2016 4:12 PM, Xie, Huawei wrote: > > > > There is no external function call or any b

[dpdk-dev] [PATCH] virtio: use volatile to get used->idx in the loop

2016-05-25 Thread Michael S. Tsirkin
On Wed, May 25, 2016 at 08:25:20AM +, Xie, Huawei wrote: > On 5/25/2016 4:12 PM, Xie, Huawei wrote: > > There is no external function call or any barrier in the loop, > > the used->idx would only be retrieved once. > > > > Signed-off-by: Huawei Xie > > --- > > drivers/net/virtio/virtio_ethdev

[dpdk-dev] [PATCH] virtio: use volatile to get used->idx in the loop

2016-05-25 Thread Bruce Richardson
On Wed, May 25, 2016 at 12:50:02PM +0300, Michael S. Tsirkin wrote: > On Wed, May 25, 2016 at 10:47:30AM +0100, Bruce Richardson wrote: > > On Wed, May 25, 2016 at 11:34:24AM +0300, Michael S. Tsirkin wrote: > > > On Wed, May 25, 2016 at 08:25:20AM +, Xie, Huawei wrote: > > > > On 5/25/2016 4:1

[dpdk-dev] [PATCH] virtio: use volatile to get used->idx in the loop

2016-05-25 Thread Bruce Richardson
On Wed, May 25, 2016 at 11:34:24AM +0300, Michael S. Tsirkin wrote: > On Wed, May 25, 2016 at 08:25:20AM +, Xie, Huawei wrote: > > On 5/25/2016 4:12 PM, Xie, Huawei wrote: > > > There is no external function call or any barrier in the loop, > > > the used->idx would only be retrieved once. > >

[dpdk-dev] [PATCH] virtio: use volatile to get used->idx in the loop

2016-05-25 Thread Xie, Huawei
On 5/25/2016 4:12 PM, Xie, Huawei wrote: > There is no external function call or any barrier in the loop, > the used->idx would only be retrieved once. > > Signed-off-by: Huawei Xie > --- > drivers/net/virtio/virtio_ethdev.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git

[dpdk-dev] [PATCH] virtio: use volatile to get used->idx in the loop

2016-05-25 Thread Huawei Xie
There is no external function call or any barrier in the loop, the used->idx would only be retrieved once. Signed-off-by: Huawei Xie --- drivers/net/virtio/virtio_ethdev.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/net/virtio/virtio_ethdev.c b/drivers/net/virt