[dpdk-dev] [PATCH v6 1/3] net/virtio: make control queue thread-safe
Wang, Xiao W
xiao.w.wang at intel.com
Mon Jan 8 16:25:29 CET 2018
Hi,
> -----Original Message-----
> From: Yuanhan Liu [mailto:yliu at fridaylinux.org]
> Sent: Monday, January 8, 2018 9:07 PM
> To: Wang, Xiao W <xiao.w.wang at intel.com>
> Cc: Bie, Tiwei <tiwei.bie at intel.com>; dev at dpdk.org;
> stephen at networkplumber.org
> Subject: Re: [PATCH v6 1/3] net/virtio: make control queue thread-safe
>
> On Sun, Jan 07, 2018 at 04:05:11AM -0800, Xiao Wang wrote:
> > diff --git a/drivers/net/virtio/virtio_rxtx.h b/drivers/net/virtio/virtio_rxtx.h
> > index 54f1e849b..71b5798b0 100644
> > --- a/drivers/net/virtio/virtio_rxtx.h
> > +++ b/drivers/net/virtio/virtio_rxtx.h
> > @@ -84,6 +84,7 @@ struct virtnet_ctl {
> > rte_iova_t virtio_net_hdr_mem; /**< hdr for each xmit packet */
> > uint16_t port_id; /**< Device port identifier. */
> > const struct rte_memzone *mz; /**< mem zone to populate CTL ring.
> */
> > + rte_spinlock_t sl; /**< spinlock for control queue. */
>
> It's weird to name it "sl". The typical naming is just "lock".
>
> --yliu
I'm open to the naming method, but you can see that:
struct rte_mempool_ops_table also has a "sl" field.
BRs,
Xiao
More information about the dev
mailing list