Hi Akhil, We have decided that we will not be upstreaming this patchset in this release (I have changed its status to 'deferred').
Regards, Arek > -----Original Message----- > From: Akhil Goyal <akhil.go...@nxp.com> > Sent: środa, 15 lipca 2020 22:16 > To: Thomas Monjalon <tho...@monjalon.net>; Kusztal, ArkadiuszX > <arkadiuszx.kusz...@intel.com> > Cc: dev@dpdk.org; Trahe, Fiona <fiona.tr...@intel.com>; > techbo...@dpdk.org; Anoob Joseph <ano...@marvell.com>; Somalapuram, > Amaranath <amaranath.somalapu...@amd.com>; Ankur Dwivedi > <adwiv...@marvell.com>; ruifeng.w...@arm.com; De Lara Guarch, Pablo > <pablo.de.lara.gua...@intel.com>; Nagadheeraj Rottela > <rnagadhee...@marvell.com> > Subject: RE: [PATCH v3 0/5] app: add multi process crypto application > > > > I see this application as a useful tool to test the readiness of a > > > driver to be > > used > > > in a multi process environment. If app is not a correct place to > > > host it, should it > > be > > > added in examples/multi_process/. I also suggested that in v2 but it > > > makes > > more > > > sense in app as it is a unit test application which does not have > > > any relevance > > as > > > standalone application as crypto may not be used standalone without > > > ethernet for multi process scenario. > > > My first preference was to modify l2fwd-crypto to be used as a multi > > > process > > proof > > > Application but it also make sense to have a unit test application > > > to verify > > standalone > > > crypto PMDs. > > > Open for comments from other crypto PMD owners. > > > > I agree it looks like unit tests. > > Can it be added to app/test/test_cryptodev* ? > > > > Running two instances of test application will be a challenge I guess. > If it can be done, I think all the cases covered in test app other than crypto > would be affected/tested. > > Test-crypto-perf can be a better option but it may defeat the purpose of test- > crypto-perf. > Best would be to make l2fwd-crypto compliant with multi process. > But still I am ok to have a unit test application. > >
