15/07/2020 22:15, Akhil Goyal: > > > 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.
Right > 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. Yes, probably a good idea. > But still I am ok to have a unit test application.