On Tue, Oct 6, 2020 at 5:13 PM Konstantin Ananyev <konstantin.anan...@intel.com> wrote: > > Add AVX512 classify to the test coverage. > > Signed-off-by: Konstantin Ananyev <konstantin.anan...@intel.com> > --- > app/test/test_acl.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/app/test/test_acl.c b/app/test/test_acl.c > index 333b347579..5b32347954 100644 > --- a/app/test/test_acl.c > +++ b/app/test/test_acl.c > @@ -278,8 +278,8 @@ test_classify_alg(struct rte_acl_ctx *acx, struct > ipv4_7tuple test_data[], > > /* set given classify alg, skip test if alg is not supported */ > ret = rte_acl_set_ctx_classify(acx, alg); > - if (ret == -ENOTSUP) > - return 0; > + if (ret != 0) > + return (ret == -ENOTSUP) ? 0 : ret;
Does this really belong to this patch? I would expect it in "test/acl: expand classify test coverage". -- David Marchand
