> -----Original Message----- > From: Stephen Hemminger [mailto:step...@networkplumber.org] > Sent: Thursday, November 30, 2017 10:59 AM > To: Wang, Xiao W <xiao.w.w...@intel.com> > Cc: dev@dpdk.org; y...@fridaylinux.org > Subject: Re: [dpdk-dev] [PATCH 1/2] net/virtio: make control queue thread- > safe > > On Fri, 24 Nov 2017 03:03:59 -0800 > Xiao Wang <xiao.w.w...@intel.com> 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)) > > + if ((vq->vq_free_cnt < ((uint32_t)pkt_num + 2)) || (pkt_num < 1)) { > > You ndon't need so many paranthesis. > if (vq->vq_free_cnt < pkt_num + 2 || pkt_num < 1)
Yes, it looks better. Will change it in v2. Thanks, Xiao
