From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by dpdk.org (Postfix) with ESMTP id 332354C95 for ; Mon, 10 Sep 2018 08:08:39 +0200 (CEST) X-Amp-Result: UNSCANNABLE X-Amp-File-Uploaded: False Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga106.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 09 Sep 2018 23:08:39 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.53,354,1531810800"; d="scan'208";a="89085950" Received: from btwcube1.sh.intel.com (HELO debian) ([10.67.104.194]) by orsmga001.jf.intel.com with ESMTP; 09 Sep 2018 23:08:37 -0700 Date: Mon, 10 Sep 2018 14:07:38 +0800 From: Tiwei Bie To: "Gavin Hu (Arm Technology China)" Cc: Jens Freimann , "dev@dpdk.org" , "maxime.coquelin@redhat.com" Message-ID: <20180910060738.GA9902@debian> References: <20180906181947.20646-1-jfreimann@redhat.com> <20180906181947.20646-3-jfreimann@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) Subject: Re: [dpdk-dev] [PATCH v5 02/11] net/virtio: add virtio 1.1 defines X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 10 Sep 2018 06:08:40 -0000 On Mon, Sep 10, 2018 at 05:22:43AM +0000, Gavin Hu (Arm Technology China) wrote: > The following 1.1 new defines should be added in this patch or the parent patch. > VIRTIO_F_IO_BARRIER > VIRTIO_F_SR_IOV I don't think we should introduce these defines in this patch or this series. This series should focus on packed ring. And the title of this patch should be something about packed ring instead of virtio 1.1. Thanks > > > -----Original Message----- > > From: dev On Behalf Of Jens Freimann > > Sent: Friday, September 7, 2018 2:20 AM > > To: dev@dpdk.org > > Cc: tiwei.bie@intel.com; maxime.coquelin@redhat.com > > Subject: [dpdk-dev] [PATCH v5 02/11] net/virtio: add virtio 1.1 defines > > > > Signed-off-by: Jens Freimann > > --- > > drivers/net/virtio/virtio_ring.h | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/drivers/net/virtio/virtio_ring.h b/drivers/net/virtio/virtio_ring.h > > index cea4d441e..e2c597434 100644 > > --- a/drivers/net/virtio/virtio_ring.h > > +++ b/drivers/net/virtio/virtio_ring.h > > @@ -15,7 +15,11 @@ > > #define VRING_DESC_F_WRITE 2 > > /* This means the buffer contains a list of buffer descriptors. */ > > #define VRING_DESC_F_INDIRECT 4 > > +/* This flag means the descriptor was made available by the driver */ > > > > +#define VRING_DESC_F_AVAIL(b) ((uint16_t)(b) << 7) > > +/* This flag means the descriptor was used by the device */ > > +#define VRING_DESC_F_USED(b) ((uint16_t)(b) << 15) > > /* The Host uses this in used->flags to advise the Guest: don't kick me > > * when you add a buffer. It's unreliable, so it's simply an > > * optimization. Guest will still kick if it's out of buffers. */ > > -- > > 2.17.1 > > IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.