From: "Chao Zhu" <chaozhu@linux.vnet.ibm.com>
To: "'Gowrishankar Muthukrishnan'"
<gowrishankar.m@linux.vnet.ibm.com>, <dev@dpdk.org>
Cc: "'Bruce Richardson'" <bruce.richardson@intel.com>,
"'Konstantin Ananyev'" <konstantin.ananyev@intel.com>,
"'Thomas Monjalon'" <thomas.monjalon@6wind.com>,
"'Cristian Dumitrescu'" <cristian.dumitrescu@intel.com>,
"'Pradeep'" <pradeep@us.ibm.com>
Subject: Re: [dpdk-dev] [PATCH v6 0/9] enable lpm, acl and other missing libraries in ppc64le
Date: Wed, 17 Aug 2016 16:48:58 +0800 [thread overview]
Message-ID: <001f01d1f864$46b65eb0$d4231c10$@linux.vnet.ibm.com> (raw)
In-Reply-To: <1471343279-24014-1-git-send-email-gowrishankar.m@linux.vnet.ibm.com>
-----Original Message-----
From: Gowrishankar Muthukrishnan [mailto:gowrishankar.m@linux.vnet.ibm.com]
Sent: 2016年8月16日 18:28
To: dev@dpdk.org
Cc: Chao Zhu <chaozhu@linux.vnet.ibm.com>; Bruce Richardson
<bruce.richardson@intel.com>; Konstantin Ananyev
<konstantin.ananyev@intel.com>; Thomas Monjalon <thomas.monjalon@6wind.com>;
Cristian Dumitrescu <cristian.dumitrescu@intel.com>; Pradeep
<pradeep@us.ibm.com>
Subject: [PATCH v6 0/9] enable lpm, acl and other missing libraries in
ppc64le
This patchset enables LPM, ACL and other few missing libs in ppc64le and
also address few patches in related examples (ip_pipeline and l3fwd).
Test report:
1. LPM and ACL unit tests passed.
Steps:
compile test app and run (with any needed params)
lpm_autotest
acl_autotest
table_autotest
test_table_lpm_combined fails same as in intel in current master.
2. Example ip_pipeline application verified for port forwarding.
compile examples/ip_pipeline (adjust app_init_core_map
param for ht to 1 in case of ppc64le in SMT=off mode).
modify config/l3fwd.cfg as per enabled PMD ports.
run ip_pipeline with config file option and check packets fwd.
v6 changes:
- added cache alignment fix for rte hash table structs.
v5 changes:
- no change in lpm lib enablement
- no change in acl lib enablement
- config file changes individually for sched,table,port,pipeline
lib enablement
- ip_pipeline patch description and changes flagged only for ppc64le.
app_init_core_map changes removed (due to bug found and under
investigation only on ppc64le/smt=off case).
v4 changes:
- fix transition4 in acl_run_altivec.h for gcc strict-aliasing error.
Thanks to Chao Zhu for bringing up.
v3 changes:
- rebase over master to fix conflict in examples/l3fwd/l3fwd_em.c
v2 changes:
- enabling libs in config included as part of lib changes itself.
Gowrishankar Muthukrishnan (9):
lpm: add altivec intrinsics for dpdk lpm on ppc_64
acl: add altivec intrinsics for dpdk acl on ppc_64
l3fwd: add altivec support for em_hash_key
table: enable table library for ppc64le
sched: enable sched library for ppc64le
port: enable port library for ppc64le
pipeline: enable pipeline library for ppc64le
ip_pipeline: fix lcore mapping for varying SMT threads as in ppc64
table: align rte table hash structs for cache line size
app/test-acl/main.c | 4 +
app/test/test_xmmt_ops.h | 16 +
config/defconfig_ppc_64-power8-linuxapp-gcc | 7 -
examples/ip_pipeline/cpu_core_map.c | 8 +
examples/l3fwd/l3fwd_em.c | 10 +-
lib/librte_acl/Makefile | 2 +
lib/librte_acl/acl.h | 4 +
lib/librte_acl/acl_run.h | 2 +
lib/librte_acl/acl_run_altivec.c | 47 +++
lib/librte_acl/acl_run_altivec.h | 329
+++++++++++++++++++++
lib/librte_acl/rte_acl.c | 13 +
lib/librte_acl/rte_acl.h | 1 +
.../common/include/arch/ppc_64/rte_vect.h | 60 ++++
lib/librte_lpm/Makefile | 2 +
lib/librte_lpm/rte_lpm.h | 2 +
lib/librte_lpm/rte_lpm_altivec.h | 154 ++++++++++
lib/librte_table/rte_table_hash_key16.c | 4 +-
lib/librte_table/rte_table_hash_key32.c | 4 +-
lib/librte_table/rte_table_hash_key8.c | 2 +-
19 files changed, 658 insertions(+), 13 deletions(-) create mode 100644
lib/librte_acl/acl_run_altivec.c create mode 100644
lib/librte_acl/acl_run_altivec.h create mode 100644
lib/librte_eal/common/include/arch/ppc_64/rte_vect.h
create mode 100644 lib/librte_lpm/rte_lpm_altivec.h
--
1.9.1
I think this patch set is good enough.
Acked-by: Chao Zhu <chaozhu@linux.vnet.ibm.com>
next prev parent reply other threads:[~2016-08-17 8:54 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-16 10:27 Gowrishankar Muthukrishnan
2016-08-16 10:27 ` [dpdk-dev] [PATCH v6 1/9] lpm: add altivec intrinsics for dpdk lpm on ppc_64 Gowrishankar Muthukrishnan
2016-09-07 9:21 ` Bruce Richardson
2016-08-16 10:27 ` [dpdk-dev] [PATCH v6 2/9] acl: add altivec intrinsics for dpdk acl " Gowrishankar Muthukrishnan
2016-08-31 13:08 ` Ananyev, Konstantin
2016-08-16 10:27 ` [dpdk-dev] [PATCH v6 3/9] l3fwd: add altivec support for em_hash_key Gowrishankar Muthukrishnan
2016-08-16 10:27 ` [dpdk-dev] [PATCH v6 4/9] table: enable table library for ppc64le Gowrishankar Muthukrishnan
2016-08-16 10:27 ` [dpdk-dev] [PATCH v6 5/9] sched: enable sched " Gowrishankar Muthukrishnan
2016-08-16 10:27 ` [dpdk-dev] [PATCH v6 6/9] port: enable port " Gowrishankar Muthukrishnan
2016-08-16 10:27 ` [dpdk-dev] [PATCH v6 7/9] pipeline: enable pipeline " Gowrishankar Muthukrishnan
2016-08-16 10:27 ` [dpdk-dev] [PATCH v6 8/9] ip_pipeline: fix lcore mapping for varying SMT threads as in ppc64 Gowrishankar Muthukrishnan
2016-08-31 17:33 ` Dumitrescu, Cristian
2016-08-16 10:27 ` [dpdk-dev] [PATCH v6 9/9] table: align rte table hash structs for cache line size Gowrishankar Muthukrishnan
2016-08-31 17:29 ` Dumitrescu, Cristian
2016-09-08 9:36 ` Thomas Monjalon
2016-09-08 9:40 ` Dumitrescu, Cristian
2016-09-08 16:52 ` gowrishankar muthukrishnan
2016-08-17 8:48 ` Chao Zhu [this message]
2016-08-26 10:55 ` [dpdk-dev] FW: [PATCH v6 0/9] enable lpm, acl and other missing libraries in ppc64le Chao Zhu
2016-08-26 15:29 ` Thomas Monjalon
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='001f01d1f864$46b65eb0$d4231c10$@linux.vnet.ibm.com' \
--to=chaozhu@linux.vnet.ibm.com \
--cc=bruce.richardson@intel.com \
--cc=cristian.dumitrescu@intel.com \
--cc=dev@dpdk.org \
--cc=gowrishankar.m@linux.vnet.ibm.com \
--cc=konstantin.ananyev@intel.com \
--cc=pradeep@us.ibm.com \
--cc=thomas.monjalon@6wind.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).