Hi Akhil, > -----Original Message----- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Iremonger, Bernard > Sent: Thursday, April 18, 2019 3:59 PM > To: Akhil Goyal <akhil.go...@nxp.com>; dev@dpdk.org; Ananyev, > Konstantin <konstantin.anan...@intel.com> > Cc: sta...@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v4 1/2] examples/ipsec-secgw: fix 1st packet > dropped for inline crypto > > Hi Akhil, > > <snip> > > > Subject: RE: [PATCH v4 1/2] examples/ipsec-secgw: fix 1st packet > > dropped for inline crypto > > <snip> > > > + if (sa->type == > > RTE_SECURITY_ACTION_TYPE_LOOKASIDE_PROTOCOL) { > > > + ctx = (struct rte_security_ctx *) > > > + rte_eth_dev_get_sec_ctx(sa->portid); > > > > This is breaking the lookaside mode. Ctx was retrieved using the > > ipsec_ctx- > > >tbl struct rte_security_ctx *ctx = (struct rte_security_ctx *) > > rte_cryptodev_get_sec_ctx( > > ipsec_ctx->tbl[cdev_id_qp].id); > > > > I am looking into it, but I don't have time left to get it integrated > > in RC2. So this has to be pushed to RC3 > > <snip> > > Unfortunately we do not have the HW to test this feature. > What HW are you using to test this? > > Having looked at the code previously > ipsec_ctx->tbl[cdev_id_qp].id turned out to be the port_id. > > So we had expected it to work. > > We will need your help with this. > > Regards, > > Bernard.
Just had another look at the 19.05.rc1 code Line 1546 in ipsec-secgw.c: ipsec_ctx->tbl[i].id = cdev_id; The id is the cryptodev id, not the port_id Regards, Bernard.
