Re: [PATCH v2 1/2] driver/net/mpipe: add rte_vect.h and enable CONFIG_RTE_LIBRTE_LPM
From: Liming Sun <hidden>
Date: 2016-02-10 14:52:16
Thanks Bruce. I split the changes as suggested and resubmitted as below. http://dpdk.org/dev/patchwork/patch/10454/ http://dpdk.org/dev/patchwork/patch/10455/ - Liming -----Original Message----- From: Bruce Richardson [mailto:bruce.richardson@intel.com] Sent: Tuesday, February 09, 2016 10:56 AM To: Liming Sun Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH v2 1/2] driver/net/mpipe: add rte_vect.h and enable CONFIG_RTE_LIBRTE_LPM 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 <redacted> Acked-by: Zhigang Lu <redacted> --- 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]
quoted hunk ↗ jump to hunk
+ 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]