From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-oi0-f48.google.com (mail-oi0-f48.google.com [209.85.218.48]) by dpdk.org (Postfix) with ESMTP id 596EC7CE5 for ; Mon, 21 Aug 2017 15:36:58 +0200 (CEST) Received: by mail-oi0-f48.google.com with SMTP id j144so27015132oib.1 for ; Mon, 21 Aug 2017 06:36:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=netronome-com.20150623.gappssmtp.com; s=20150623; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=2zjxvHxpjlI9zENkwISoOOGRzJandWse79ACpj5Kkyg=; b=LL7FkKo3Cj5N8AQu1/GwkGv2w/08OB/EeeyHMVUj7A0+EKBbjU5oxmAGBykA5ydcus UACLWg+X/c9TA+5r5Ja1/tgCZMkEGXAESF+I30YdiFvKczfZlG/lnIM+Gd+RE3CQUykr YwKXb2KdPmaQAZ3+MOM0AcTs/rAWWloA+tfv2SLtt0W9/NwC8excKkDiNq+oZdFkv6zb nVIVNF24dF/Z8+Ssse9QCd+Smae57uKfohv+/BmEMV/7WWC8gxDnpruz+zCEnRASt7uI 5GC6CDUXV3gsP1LhWAFQ1nXK6hLuPXvaKshZDN2UlrCU2LpyLpAaVlBTtnNeSSvrCDP+ MRvQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=2zjxvHxpjlI9zENkwISoOOGRzJandWse79ACpj5Kkyg=; b=a7ztWCVjykbDvWbf2lx66x0YBeFzCyW3ZZfbe7Xm6BOuH2yDcWHnSTodWflLXRqhgi oi1VTz829L3wJu46/gXWUET7IA+yg2sazNsycXu5VElS0ybYI3UCYRt5nf8ohQZM962Q vVSm4rmNy7ECy2/2jLKe+cB8sX4jNb498NjOAAqqd8TJdq/j8kCMg6Fd44uqtas64uQ+ Aaw/Jy1rzFnXnBU9FQmhVhyVT4qE88vhqm3+QPhhRHSncrOMPGlFHtxYudTPJxBRKOJV k0RTubv34nNBXZCdNzv3g6DfM/jUU902kLriha9jwbSiQOMxjQWRhfoAhvcvPNK8f3hM dVlQ== X-Gm-Message-State: AHYfb5iMITpBuzgwap+MsoAIIFL1iLQ0//DhRSd8arP0Obaip4yTv8qx rFtiWb70vFFff4TMZUFa0Pr/CqQYJ24cZpuh1Q== X-Received: by 10.202.69.70 with SMTP id s67mr6789158oia.22.1503322618053; Mon, 21 Aug 2017 06:36:58 -0700 (PDT) MIME-Version: 1.0 Received: by 10.58.33.219 with HTTP; Mon, 21 Aug 2017 06:36:57 -0700 (PDT) In-Reply-To: <37f59d30-57fb-d211-3de6-6fae76628c37@intel.com> References: <1502461849-19561-1-git-send-email-alejandro.lucero@netronome.com> <37f59d30-57fb-d211-3de6-6fae76628c37@intel.com> From: Alejandro Lucero Date: Mon, 21 Aug 2017 14:36:57 +0100 Message-ID: To: Ferruh Yigit Cc: dev Content-Type: text/plain; charset="UTF-8" X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Subject: Re: [dpdk-dev] [PATCH] nfp: support new medatada api 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, 21 Aug 2017 13:36:59 -0000 On Fri, Aug 18, 2017 at 4:51 PM, Ferruh Yigit wrote: > On 8/11/2017 3:30 PM, Alejandro Lucero wrote: > > We need to support how metadata was handled and the new api, which will > > allow to work with different metadata types and data dynamically. > > What is the API mentioned? > > This is a internal NFP firmware API for handling metadata. I will mention this in next version. > > > > Although this patch just supports one type handled by the PMD. Future > uses > > regarding firmware apps will extend this support. > > > > Signed-off-by: Alejandro Lucero > > --- > > drivers/net/nfp/nfp_net.c | 37 ++++++++++++++++++++++++++++++ > +------ > > drivers/net/nfp/nfp_net_ctrl.h | 5 +++++ > > drivers/net/nfp/nfp_net_pmd.h | 2 ++ > > 3 files changed, 38 insertions(+), 6 deletions(-) > > > > diff --git a/drivers/net/nfp/nfp_net.c b/drivers/net/nfp/nfp_net.c > > index c9895a2..5ea55c7 100644 > > --- a/drivers/net/nfp/nfp_net.c > > +++ b/drivers/net/nfp/nfp_net.c > > @@ -1734,6 +1734,8 @@ static void nfp_net_read_mac(struct nfp_net_hw *hw) > > #define NFP_HASH_OFFSET ((uint8_t *)mbuf->buf_addr + > mbuf->data_off - 4) > > #define NFP_HASH_TYPE_OFFSET ((uint8_t *)mbuf->buf_addr + > mbuf->data_off - 8) > > > > +#define NFP_DESC_META_LEN(d) (d->rxd.meta_len_dd & > PCIE_DESC_RX_META_LEN_MASK) > > + > > /* > > * nfp_net_set_hash - Set mbuf hash data > > * > > @@ -1744,18 +1746,41 @@ static void nfp_net_read_mac(struct nfp_net_hw > *hw) > > nfp_net_set_hash(struct nfp_net_rxq *rxq, struct nfp_net_rx_desc *rxd, > > struct rte_mbuf *mbuf) > > { > > - uint32_t hash; > > - uint32_t hash_type; > > struct nfp_net_hw *hw = rxq->hw; > > + uint8_t *meta_offset; > > + uint32_t meta_info; > > + uint32_t hash = 0; > > + uint32_t hash_type = 0; > > > > if (!(hw->ctrl & NFP_NET_CFG_CTRL_RSS)) > > return; > > > > - if (!(rxd->rxd.flags & PCIE_DESC_RX_RSS)) > > + if (NFD_CFG_MAJOR_VERSION_of(hw->ver) <= 3) { > > It seems metadata parsing is valid for MAJOR_VERSION > 3, does it make > sense to mention from this in commit log? > That is a good idea. Thanks > > > + if (!(rxd->rxd.flags & PCIE_DESC_RX_RSS)) > > + return; > > + > > + hash = rte_be_to_cpu_32(*(uint32_t *)NFP_HASH_OFFSET); > > + hash_type = rte_be_to_cpu_32(*(uint32_t > *)NFP_HASH_TYPE_OFFSET); > > + > > + } else if (NFP_DESC_META_LEN(rxd)) { > > + meta_offset = (uint8_t *)mbuf->buf_addr + mbuf->data_off; > > It is possible to use rte_pktmbuf_mtod_offset(), result will be same tough. > > I will change it. > > + meta_offset -= NFP_DESC_META_LEN(rxd); > > + meta_info = rte_be_to_cpu_32(*(uint32_t *)meta_offset); > > + meta_offset += 4; > > + /* NFP PMD just supports metadata for hashing */ > > + switch (meta_info & NFP_NET_META_FIELD_MASK) { > > + case NFP_NET_META_HASH: > > + meta_info >>= NFP_NET_META_FIELD_SIZE; > > + hash = rte_be_to_cpu_32(*(uint32_t *)meta_offset); > > + hash_type = meta_info && NFP_NET_META_FIELD_MASK; > > META_FIELD already masked out above, right? Is this META_FIELD again or > just reused macro because of it has same value? > > meta_info is changing after it is masked out the first time. Metadata is described with at least two fields, but it can have more depending on the metadata type. So NFP_NET_META_FIELD_SIZE is used for adjusting which field is going to be processed. I guess adding this same description will help. so I will do that in next patch version. > > + break; > > + default: > > + /* Unsupported metadata can be a performance issue > */ > > + return; > > + } > > + } else { > > return; > > - > > - hash = rte_be_to_cpu_32(*(uint32_t *)NFP_HASH_OFFSET); > > - hash_type = rte_be_to_cpu_32(*(uint32_t *)NFP_HASH_TYPE_OFFSET); > > + } > > > > mbuf->hash.rss = hash; > > mbuf->ol_flags |= PKT_RX_RSS_HASH; > > diff --git a/drivers/net/nfp/nfp_net_ctrl.h b/drivers/net/nfp/nfp_net_ > ctrl.h > > index 2c50043..becfabd 100644 > > --- a/drivers/net/nfp/nfp_net_ctrl.h > > +++ b/drivers/net/nfp/nfp_net_ctrl.h > > @@ -52,6 +52,11 @@ > > /* Offset in Freelist buffer where packet starts on RX */ > > #define NFP_NET_RX_OFFSET 32 > > > > +/* Prepend field types */ > > +#define NFP_NET_META_FIELD_SIZE 4 > > +#define NFP_NET_META_HASH 1 /* next field carries hash > type */ > > +#define NFP_NET_META_MARK 2 > > META_MARK seems not used. > > Not yet for the PMD. I will remove it. > > + > > /* Hash type pre-pended when a RSS hash was computed */ > > #define NFP_NET_RSS_NONE 0 > > #define NFP_NET_RSS_IPV4 1 > > diff --git a/drivers/net/nfp/nfp_net_pmd.h > b/drivers/net/nfp/nfp_net_pmd.h > > index c6bddaa..df22fa1 100644 > > --- a/drivers/net/nfp/nfp_net_pmd.h > > +++ b/drivers/net/nfp/nfp_net_pmd.h > > @@ -283,6 +283,8 @@ struct nfp_net_txq { > > #define PCIE_DESC_RX_UDP_CSUM_OK (1 << 1) > > #define PCIE_DESC_RX_VLAN (1 << 0) > > > > +#define NFP_NET_META_FIELD_MASK (0xf) > > Can represent as following to stress the relation: > #define NFP_NET_META_FIELD_MASK ((1 << NFP_NET_META_FIELD_SIZE) - 1) > this makes sense. Thanks > > > + > > struct nfp_net_rx_desc { > > union { > > /* Freelist descriptor */ > > > >