Re: [dpdk-dev] [PATCH v2 0/5] cryptodev: hide internal structures

2021-10-19 Thread Ananyev, Konstantin
> > > > Just to let you know this patch set causes to seg-fault ipsec-secgw: > > examples/ipsec-secgw/test/run_test.sh -46m > > ... > > [23695833.390785] dpdk-ipsec-secg[2491066]: segfault at 0 ip > > 564325730963 sp 7fffb9111d00 error 4 in dpdk-ipsec- > > secgw[564324df+134d000] > >

Re: [dpdk-dev] [PATCH v2 0/5] cryptodev: hide internal structures

2021-10-15 Thread Akhil Goyal
> > Just to let you know this patch set causes to seg-fault ipsec-secgw: > examples/ipsec-secgw/test/run_test.sh -46m > ... > [23695833.390785] dpdk-ipsec-secg[2491066]: segfault at 0 ip > 564325730963 sp 7fffb9111d00 error 4 in dpdk-ipsec- > secgw[564324df+134d000] > [23695833.39079

Re: [dpdk-dev] [PATCH v2 0/5] cryptodev: hide internal structures

2021-10-15 Thread Ananyev, Konstantin
Hi Akhil, > Structures rte_cryptodev and rte_cryptodev_data are not > supposed to be directly used by the application. These > are made public as they are used by inline datapath > public APIs. > This patchset, creates a new rte_cryptodev_core.h file > which helps in defining a data structure to

Re: [dpdk-dev] [PATCH v2 0/5] cryptodev: hide internal structures

2021-10-11 Thread Zhang, Roy Fan
De Lara Guarch, Pablo > > ; Trahe, Fiona ; > > Doherty, Declan ; ma...@nvidia.com; > > g.si...@nxp.com; jianjay.z...@huawei.com; asoma...@amd.com; > > ruifeng.w...@arm.com; Ananyev, Konstantin > > ; Nicolau, Radu > ; > > ajit.khapa...@broadcom.com; rnagadhee

Re: [dpdk-dev] [PATCH v2 0/5] cryptodev: hide internal structures

2021-10-11 Thread Ji, Kai
oadcom.com; rnagadhee...@marvell.com; > adwiv...@marvell.com; Power, Ciara > Subject: Re: [dpdk-dev] [PATCH v2 0/5] cryptodev: hide internal structures > > Hi Akhil, > > The approach looks great but we may have to check if it works in multi- > process environment - since all

Re: [dpdk-dev] [PATCH v2 0/5] cryptodev: hide internal structures

2021-10-11 Thread Zhang, Roy Fan
Hi Akhil, The approach looks great but we may have to check if it works in multi-process environment - since all enqueue/dequeue handlers are set by primary process the secondary process may not recognize the fp_ops data. We will run a quick test to see if it is true. Regards, Fan > -Origin

[dpdk-dev] [PATCH v2 0/5] cryptodev: hide internal structures

2021-10-11 Thread Akhil Goyal
Structures rte_cryptodev and rte_cryptodev_data are not supposed to be directly used by the application. These are made public as they are used by inline datapath public APIs. This patchset, creates a new rte_cryptodev_core.h file which helps in defining a data structure to hold datapath APIs in a