Message ID | 1438938514-10304-1-git-send-email-pablo.de.lara.guarch@intel.com (mailing list archive) |
---|---|
State | Superseded, archived |
Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id B23735957; Fri, 7 Aug 2015 11:08:38 +0200 (CEST) Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id 0548D37AA for <dev@dpdk.org>; Fri, 7 Aug 2015 11:08:36 +0200 (CEST) Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by orsmga103.jf.intel.com with ESMTP; 07 Aug 2015 02:08:36 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.15,629,1432623600"; d="scan'208";a="537648728" Received: from irvmail001.ir.intel.com ([163.33.26.43]) by FMSMGA003.fm.intel.com with ESMTP; 07 Aug 2015 02:08:35 -0700 Received: from sivswdev02.ir.intel.com (sivswdev02.ir.intel.com [10.237.217.46]) by irvmail001.ir.intel.com (8.14.3/8.13.6/MailSET/Hub) with ESMTP id t7798Ywq025194 for <dev@dpdk.org>; Fri, 7 Aug 2015 10:08:34 +0100 Received: from sivswdev02.ir.intel.com (localhost [127.0.0.1]) by sivswdev02.ir.intel.com with ESMTP id t7798YYg010338 for <dev@dpdk.org>; Fri, 7 Aug 2015 10:08:34 +0100 Received: (from pdelarax@localhost) by sivswdev02.ir.intel.com with id t7798YPO010334 for dev@dpdk.org; Fri, 7 Aug 2015 10:08:34 +0100 From: Pablo de Lara <pablo.de.lara.guarch@intel.com> To: dev@dpdk.org Date: Fri, 7 Aug 2015 10:08:34 +0100 Message-Id: <1438938514-10304-1-git-send-email-pablo.de.lara.guarch@intel.com> X-Mailer: git-send-email 1.7.4.1 Subject: [dpdk-dev] [PATCH] examples/l3fwd: fix compilation issue when using exact-match X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK <dev.dpdk.org> List-Unsubscribe: <http://dpdk.org/ml/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://dpdk.org/ml/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <http://dpdk.org/ml/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
Commit Message
De Lara Guarch, Pablo
Aug. 7, 2015, 9:08 a.m. UTC
L3fwd was trying to use an inexistent function "simple_ipv6_fwd_4pkts",
instead it should be "simple_ipv6_fwd_8pkts".
Fixes: 80fcb4d4 ("examples/l3fwd: increase lookup burst size to 8")
Signed-off-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
---
examples/l3fwd/main.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Comments
On 07/08/2015 10:08, Pablo de Lara wrote: > L3fwd was trying to use an inexistent function "simple_ipv6_fwd_4pkts", > instead it should be "simple_ipv6_fwd_8pkts". > > Fixes: 80fcb4d4 ("examples/l3fwd: increase lookup burst size to 8") > > Signed-off-by: Pablo de Lara <pablo.de.lara.guarch@intel.com> > --- > Acked-by: Sergio Gonzalez Monroy <sergio.gonzalez.monroy@intel.com>
Hi Pablo, 2015-08-07 10:08, Pablo de Lara: > L3fwd was trying to use an inexistent function "simple_ipv6_fwd_4pkts", > instead it should be "simple_ipv6_fwd_8pkts". > > Fixes: 80fcb4d4 ("examples/l3fwd: increase lookup burst size to 8") There are 3 things wrong here. 1/ We must absolutely avoid compile-time paths: #if (ENABLE_MULTI_BUFFER_OPTIMIZE == 1) #if (APP_LOOKUP_METHOD == APP_LOOKUP_EXACT_MATCH) It makes test coverage too hard to track. 2/ When replacing a function, grepping it is a must have. -simple_ipv6_fwd_4pkts(struct rte_mbuf* m[4], uint8_t portid, struct lcore_conf *qconf) +simple_ipv6_fwd_8pkts(struct rte_mbuf *m[8], uint8_t portid, struct lcore_conf *qconf) So this change would be straight forward: > - simple_ipv6_fwd_4pkts(&pkts_burst[j], > + simple_ipv6_fwd_8pkts(&pkts_burst[j], 3/ The above commit makes also this wrong replacement: - simple_ipv4_fwd_4pkts(&pkts_burst[j], + simple_ipv8_fwd_4pkts(&pkts_burst[j], It is still not fixed. Please send a v2 for this last typo. Thanks
Hi Thomas, > -----Original Message----- > From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com] > Sent: Sunday, August 09, 2015 10:54 AM > To: De Lara Guarch, Pablo > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH] examples/l3fwd: fix compilation issue when > using exact-match > > Hi Pablo, > > 2015-08-07 10:08, Pablo de Lara: > > L3fwd was trying to use an inexistent function "simple_ipv6_fwd_4pkts", > > instead it should be "simple_ipv6_fwd_8pkts". > > > > Fixes: 80fcb4d4 ("examples/l3fwd: increase lookup burst size to 8") > > There are 3 things wrong here. > > 1/ We must absolutely avoid compile-time paths: > #if (ENABLE_MULTI_BUFFER_OPTIMIZE == 1) > #if (APP_LOOKUP_METHOD == APP_LOOKUP_EXACT_MATCH) > It makes test coverage too hard to track. > > 2/ When replacing a function, grepping it is a must have. > -simple_ipv6_fwd_4pkts(struct rte_mbuf* m[4], uint8_t portid, struct > lcore_conf *qconf) > +simple_ipv6_fwd_8pkts(struct rte_mbuf *m[8], uint8_t portid, struct > lcore_conf *qconf) > So this change would be straight forward: > > - > simple_ipv6_fwd_4pkts(&pkts_burst[j], > > + > simple_ipv6_fwd_8pkts(&pkts_burst[j], > > 3/ The above commit makes also this wrong replacement: > - simple_ipv4_fwd_4pkts(&pkts_burst[j], > + simple_ipv8_fwd_4pkts(&pkts_burst[j], > It is still not fixed. > > Please send a v2 for this last typo. Thanks Thanks for spotting it, v2 on the way! Pablo
diff --git a/examples/l3fwd/main.c b/examples/l3fwd/main.c index 9351322..350c1cb 100644 --- a/examples/l3fwd/main.c +++ b/examples/l3fwd/main.c @@ -1714,7 +1714,7 @@ main_loop(__attribute__((unused)) void *dummy) portid, qconf); } else if (ol_flag & PKT_RX_IPV6_HDR) { #endif /* RTE_NEXT_ABI */ - simple_ipv6_fwd_4pkts(&pkts_burst[j], + simple_ipv6_fwd_8pkts(&pkts_burst[j], portid, qconf); } else { l3fwd_simple_forward(pkts_burst[j],