From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bruce Richardson Subject: Re: [PATCH v2 1/2] driver/net/mpipe: add rte_vect.h and enable CONFIG_RTE_LIBRTE_LPM Date: Tue, 9 Feb 2016 15:55:50 +0000 Message-ID: <20160209155550.GB7248@bricha3-MOBL3> References: <1450813790-11874-3-git-send-email-lsun@ezchip.com> <1452263948-22485-1-git-send-email-lsun@ezchip.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: dev@dpdk.org To: Liming Sun Return-path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id 4AD4595D0 for ; Tue, 9 Feb 2016 16:55:53 +0100 (CET) Content-Disposition: inline In-Reply-To: <1452263948-22485-1-git-send-email-lsun@ezchip.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Fri, Jan 08, 2016 at 09:39:07AM -0500, Liming Sun wrote: > rte_vect.h was missing earlier thus LPM was disabled and l3fwd is > not able to compile. This commit implements the vector api and > enable LPM in the tilegx configuration by default. It also includes > a minor optimization to use __insn_fetchadd4() instead of > rte_atomic32_xxx() in mpipe_dp_enter/mpipe_dp_exit to avoid the > unnecessary memory fence. This looks like it should be two patches to me. One patch to add the missing dependency and get lpm to work. The second patch should then contain the driver optimization. Do you agree? /Bruce PS: the commit title prefix for the first patch should probably be "eal/tile" rather than mpipe, since it's not directly affecting the mpipe driver. > > Signed-off-by: Liming Sun > Acked-by: Zhigang Lu > --- > config/defconfig_tile-tilegx-linuxapp-gcc | 2 +- > drivers/net/mpipe/mpipe_tilegx.c | 18 +++-- > lib/librte_eal/common/include/arch/tile/rte_vect.h | 93 ++++++++++++++++++++++ > 3 files changed, 107 insertions(+), 6 deletions(-) > create mode 100644 lib/librte_eal/common/include/arch/tile/rte_vect.h > [snip] >+ mbuf->next = NULL; > > PMD_DEBUG_RX("%s: RX mbuf %p, buffer %p, buf_addr %p, size %d\n", > mpipe_name(priv), mbuf, va, mbuf->buf_addr, size); > diff --git a/lib/librte_eal/common/include/arch/tile/rte_vect.h b/lib/librte_eal/common/include/arch/tile/rte_vect.h > new file mode 100644 > index 0000000..32d768a > --- /dev/null > +++ b/lib/librte_eal/common/include/arch/tile/rte_vect.h > @@ -0,0 +1,93 @@ > +/* > + * BSD LICENSE > + * > + * Copyright (C) EZchip Semiconductor Ltd. 2015. Maybe update the copyright year? > + * > + * Redistribution and use in source and binary forms, with or without > + * modification, are permitted provided that the following conditions [snip]