Hi Jerin, Will do the suggested change (RX/rx-> Rx & TX/tx->Tx). Do you want me to trim the headline as well?
@Akhil, did you get a chance to review the series? Do you have any comments on the patches? Thanks, Anoob > -----Original Message----- > From: dev <dev-boun...@dpdk.org> On Behalf Of Jerin Jacob > Sent: Saturday, January 18, 2020 8:09 PM > To: Anoob Joseph <ano...@marvell.com> > Cc: Akhil Goyal <akhil.go...@nxp.com>; Declan Doherty > <declan.dohe...@intel.com>; Thomas Monjalon <tho...@monjalon.net>; Jerin > Jacob Kollanukkaran <jer...@marvell.com>; Narayana Prasad Raju Athreya > <pathr...@marvell.com>; Kiran Kumar Kokkilagadda > <kirankum...@marvell.com>; Nithin Kumar Dabilpuram > <ndabilpu...@marvell.com>; Pavan Nikhilesh Bhagavatula > <pbhagavat...@marvell.com>; Ankur Dwivedi <adwiv...@marvell.com>; > Archana Muniganti <march...@marvell.com>; Tejasree Kondoj > <ktejas...@marvell.com>; Vamsi Krishna Attunuru <vattun...@marvell.com>; > Lukas Bartosik <lbarto...@marvell.com>; dpdk-dev <dev@dpdk.org> > Subject: Re: [dpdk-dev] [PATCH v2 00/15] add OCTEONTX2 inline IPsec support > > On Sat, Jan 18, 2020 at 4:19 PM Anoob Joseph <ano...@marvell.com> wrote: > > > > This series adds inline IPsec support in OCTEONTX2 PMD. > > > > In the inbound path, rte_flow framework need to be used to configure > > the NPC block, which does the h/w lookup. The packets would get > > processed by the crypto block and would submit to the scheduling > > block, SSO. So inline IPsec mode can be enabled only when traffic is > > received via event device using Rx adapter. > > > > In the outbound path, the core would submit to the crypto block and > > the crypto block would submit the packet for Tx internally. > > > Please fix following check-git-log.sh issues. > > Wrong headline lowercase: > net/octeontx2: add inline ipsec rx path changes > drivers/octeontx2: add sec in compiler optimized RX fastpath framework > drivers/octeontx2: add sec in compiler optimized TX fastpath framework > crypto/octeontx2: add inline tx path changes Headline too long: > drivers/octeontx2: add sec in compiler optimized RX fastpath framework > drivers/octeontx2: add sec in compiler optimized TX fastpath framework > crypto/octeontx2: sync inline tag type cfg with Rx adapter > configuration > > Changing to Rx and Tx will fix most of the issues. > > > > > v2: > > * Minimized additions to common/octeontx2 > > * Updated release notes > > * Renamed otx2_is_ethdev to otx2_ethdev_is_sec_capable > > > > Ankur Dwivedi (3): > > crypto/octeontx2: add eth security capabilities > > crypto/octeontx2: add datapath ops in eth security ctx > > crypto/octeontx2: add inline tx path changes > > > > Anoob Joseph (4): > > common/octeontx2: add CPT LF mbox for inline inbound > > crypto/octeontx2: create eth security ctx > > crypto/octeontx2: enable CPT to share QP with ethdev > > crypto/octeontx2: add eth security session operations > > > > Archana Muniganti (3): > > crypto/octeontx2: add lookup mem changes to hold sa indices > > drivers/octeontx2: add sec in compiler optimized RX fastpath framework > > drivers/octeontx2: add sec in compiler optimized TX fastpath > > framework > > > > Tejasree Kondoj (3): > > crypto/octeontx2: configure for inline IPsec > > crypto/octeontx2: add security in eth dev configure > > net/octeontx2: add inline ipsec rx path changes > > > > Vamsi Attunuru (2): > > common/octeontx2: add routine to check if sec capable otx2 > > crypto/octeontx2: sync inline tag type cfg with Rx adapter > > configuration > > > > doc/guides/nics/octeontx2.rst | 20 + > > doc/guides/rel_notes/release_20_02.rst | 9 + > > drivers/common/octeontx2/otx2_common.c | 22 + > > drivers/common/octeontx2/otx2_common.h | 22 + > > drivers/common/octeontx2/otx2_mbox.h | 7 + > > .../octeontx2/rte_common_octeontx2_version.map | 3 + > > drivers/crypto/octeontx2/Makefile | 7 +- > > drivers/crypto/octeontx2/meson.build | 7 +- > > drivers/crypto/octeontx2/otx2_cryptodev.c | 8 + > > .../crypto/octeontx2/otx2_cryptodev_hw_access.h | 22 +- > > drivers/crypto/octeontx2/otx2_cryptodev_mbox.c | 53 ++ > > drivers/crypto/octeontx2/otx2_cryptodev_mbox.h | 7 + > > drivers/crypto/octeontx2/otx2_cryptodev_ops.c | 56 ++ > > drivers/crypto/octeontx2/otx2_cryptodev_qp.h | 35 + > > drivers/crypto/octeontx2/otx2_ipsec_fp.h | 348 +++++++++ > > drivers/crypto/octeontx2/otx2_security.c | 870 > +++++++++++++++++++++ > > drivers/crypto/octeontx2/otx2_security.h | 158 ++++ > > drivers/crypto/octeontx2/otx2_security_tx.h | 175 +++++ > > drivers/event/octeontx2/Makefile | 1 + > > drivers/event/octeontx2/meson.build | 5 +- > > drivers/event/octeontx2/otx2_evdev.c | 170 ++-- > > drivers/event/octeontx2/otx2_evdev.h | 4 +- > > drivers/event/octeontx2/otx2_worker.c | 6 +- > > drivers/event/octeontx2/otx2_worker.h | 6 + > > drivers/event/octeontx2/otx2_worker_dual.c | 6 +- > > drivers/net/octeontx2/Makefile | 1 + > > drivers/net/octeontx2/meson.build | 3 + > > drivers/net/octeontx2/otx2_ethdev.c | 46 +- > > drivers/net/octeontx2/otx2_ethdev.h | 2 + > > drivers/net/octeontx2/otx2_ethdev_devargs.c | 19 + > > drivers/net/octeontx2/otx2_flow.c | 26 + > > drivers/net/octeontx2/otx2_lookup.c | 11 +- > > drivers/net/octeontx2/otx2_rx.c | 27 +- > > drivers/net/octeontx2/otx2_rx.h | 377 ++++++--- > > drivers/net/octeontx2/otx2_tx.c | 29 +- > > drivers/net/octeontx2/otx2_tx.h | 271 +++++-- > > 36 files changed, 2556 insertions(+), 283 deletions(-) create mode > > 100644 drivers/crypto/octeontx2/otx2_cryptodev_qp.h > > create mode 100644 drivers/crypto/octeontx2/otx2_ipsec_fp.h > > create mode 100644 drivers/crypto/octeontx2/otx2_security.c > > create mode 100644 drivers/crypto/octeontx2/otx2_security.h > > create mode 100644 drivers/crypto/octeontx2/otx2_security_tx.h > > > > -- > > 2.7.4 > >
