Re: [dpdk-dev] [PATCH 1/2] net/virtio: make control queue thread-safe

2017-11-30 Thread Wang, Xiao W
> -Original Message- > From: Stephen Hemminger [mailto:step...@networkplumber.org] > Sent: Thursday, November 30, 2017 10:59 AM > To: Wang, Xiao W > Cc: dev@dpdk.org; y...@fridaylinux.org > Subject: Re: [dpdk-dev] [PATCH 1/2] net/virtio: make control queue thread- >

Re: [dpdk-dev] [PATCH 1/2] net/virtio: make control queue thread-safe

2017-11-29 Thread Stephen Hemminger
On Fri, 24 Nov 2017 03:03:59 -0800 Xiao Wang wrote: > @@ -184,8 +186,10 @@ struct rte_virtio_xstats_name_off { > "vq->hw->cvq = %p vq = %p", > vq->vq_desc_head_idx, status, vq->hw->cvq, vq); > > - if ((vq->vq_free_cnt < ((uint32_t)pkt_num + 2)) || (pkt_num < 1))

Re: [dpdk-dev] [PATCH 1/2] net/virtio: make control queue thread-safe

2017-11-29 Thread Wang, Xiao W
> -Original Message- > From: Bie, Tiwei > Sent: Friday, November 24, 2017 1:39 PM > To: Wang, Xiao W > Cc: dev@dpdk.org; y...@fridaylinux.org > Subject: Re: [dpdk-dev] [PATCH 1/2] net/virtio: make control queue thread- > safe > > On Fri, Nov 24, 2017 at

Re: [dpdk-dev] [PATCH 1/2] net/virtio: make control queue thread-safe

2017-11-23 Thread Tiwei Bie
On Fri, Nov 24, 2017 at 03:03:59AM -0800, Xiao Wang wrote: [...] > diff --git a/drivers/net/virtio/virtio_rxtx.h > b/drivers/net/virtio/virtio_rxtx.h > index 54f1e84..24e3026 100644 > --- a/drivers/net/virtio/virtio_rxtx.h > +++ b/drivers/net/virtio/virtio_rxtx.h > @@ -84,6 +84,7 @@ struct virtnet

[dpdk-dev] [PATCH 1/2] net/virtio: make control queue thread-safe

2017-11-23 Thread Xiao Wang
The virtio_send_command function may be called from app's configuration routine, but also from an interrupt handler called when live migration is done on the backup side. So this patch makes control queue thread-safe first. Signed-off-by: Xiao Wang --- drivers/net/virtio/virtio_ethdev.c | 7