On Wed, Sep 28, 2022 at 8:14 PM Gujjar, Abhinandan S <abhinandan.guj...@intel.com> wrote: > > > > > -----Original Message----- > > From: Jerin Jacob <jerinjac...@gmail.com> > > Sent: Tuesday, September 27, 2022 6:56 PM > > To: Gujjar, Abhinandan S <abhinandan.guj...@intel.com> > > Cc: Volodymyr Fialko <vfia...@marvell.com>; dev@dpdk.org; Jerin Jacob > > Kollanukkaran <jer...@marvell.com>; Pavan Nikhilesh Bhagavatula > > <pbhagavat...@marvell.com>; Shijith Thotton <sthot...@marvell.com>; > > Hemant Agrawal <hemant.agra...@nxp.com>; Sachin Saxena > > <sachin.sax...@oss.nxp.com>; Jayatheerthan, Jay > > <jay.jayatheert...@intel.com>; Akhil Goyal <gak...@marvell.com>; Anoob > > Joseph <ano...@marvell.com> > > Subject: Re: [PATCH 1/3] eventdev: introduce event cryptodev vector type > > > > > > > > > > > > > > > Right now we are targeting crypto_cn10k PMD and ipsec-secgw > > > > > > event mode to support vectorization. > > > > > Is there a way to test this? When can be dataplane changes expected? > > > > > > > > > If the spec looks okay, support in s/w crypto adapter and other h/w > > > > PMDs can be added by respective maintainers. Currently, we are > > > > adding library change, support for one PMD and an application to > > > > test the feature. Feature is exposed with capability flag to not break > > existing functionality. > > > Got it. How do I test this feature without data plane changes? > > > > > > Hi @Gujjar, Abhinandan S > > > > > If there is a way to test this, please let me know. > > > > Dataplane changes can be tested on the cn10k platform. > > This feature is a hardware assisted feature. > > > > > This design is right now tested for cn10k, I am not sure this works for sw > > adapter. > > > > SW driver support is not added in this series as in order to accept a API > > change, > > one would need, 1)API spec 2)One of the driver 3)Test application to > > exercise > > the API. > > > > It is a similar case for all ethdev, rte_flow features etc. > > Community can add SW driver support just like any other subsystem APIs. > > > > Also, The proposed library changes don't differentiate between SW & HW > > PMDs. > > The proposed changes are exposed with a capability flag and so SW crypto > > adapter will not have any perf impact. > > > > > I need to have perf data with and without vectorization support to > > > approve. > > > > On the cn10k platform, we see nearly 2.5x performance with vectorization. > > Eth > > rx adapter already supports vectorization and this spec change is in line > > with > > that. > > > > Also IPsec gateway update to exercise these APIs. See > > http://patches.dpdk.org/project/dpdk/patch/20220804103626.102688-6- > > vfia...@marvell.com/ > > > > Command to test on drivers which have this functionality. > > > > ./dpdk-ipsec-secgw -c 0xff0000 -a 0002:01:00.1 -a 0002:20:00.1 -a > > 0002:1e:00.0 -- -P -p 0x1 -P --transfer-mode event -l --event-schedule-type > > parallel --desc-nb 8192 --event-vector -f simple.conf > > > > sample.conf > > > > sp ipv4 out esp protect 19 pri 1 dst 192.18.0.0/32 sport 0:65535 dport > > 0:65535 > > sa out 19 aead_algo aes-128-gcm aead_key > > 73:69:78:74:65:65:6e:62:79:74:65:73:20:6b:65:79:64:70:64:6b mode ipv4- > > tunnel src 2.1.1.1 dst 1.1.1.1 type lookaside-protocol-offload port_id 0 > > > > neigh port 0 d0:37:45:02:b0:d3 > > rt ipv4 dst 1.1.0.0/16 port 0 > > > > In order to make forward progress and merge patch in RC1, I would request > > 1)Review the API specific patch(eventdev: introduce event cryptodev vector > > type), If spec needs to be changed to adapt any other driver(SW or HW) then > > the author should address that. > > 2)If you think, API usage is not enough with dpdk-ipsec-secgw application, I > > think, author should update the test-eventdev application to support the new > > mode.Which can be merged after RC1 as it is a test application change. > > > > Let us know what you think to make forward progress. > > Hi @Jerin Jacob & @Volodymyr Fialko, > Thanks for clarifying. We would like to have a call to discuss on this. > Could you please setup a call & include @Jayatheerthan, Jay also into it?
Sure. Setting up one at 4PM IST on 29 Oct. Please find the details. Hi there, Jerin Jacob Kollanukkaran is inviting you to a scheduled Zoom meeting. Topic: Jerin Jacob Kollanukkaran's Personal Meeting Room Join Zoom Meeting: https://marvell.zoom.us/j/9901077677?pwd=T2lTTGMwYlc1YTQzMnR4eGRWQXR6QT09 Password: 339888 Or Telephone: Dial(for higher quality, dial a number based on your current location): US: +1 646 558 8656 or +1 646 931 3860 or +1 669 444 9171 or +1 669 900 6833 or +1 719 359 4580 or +1 253 215 8782 or +1 301 715 8592 or +1 309 205 3325 or +1 312 626 6799 or +1 346 248 7799 or +1 386 347 5053 or +1 564 217 2000 or 888 788 0099 (Toll Free) or 833 548 0276 (Toll Free) or 833 548 0282 (Toll Free) or 833 928 4608 (Toll Free) or 833 928 4609 (Toll Free) or 833 928 4610 (Toll Free) or 877 853 5247 (Toll Free) Meeting ID: 990 107 7677 Password: 358309 International numbers available: https://marvell.zoom.us/u/adpcCpMHYt Or a Video Conference Room: >From Touchpad: Tap Join Zoom button. When prompted, enter 990 107 7677 Password: 358309 For China locations, from Touchpad: Dial* then 990 107 7677 Password: 358309 > > Thanks > Abhinandan