On Wed, Oct 14, 2020 at 12:32 PM Ananyev, Konstantin <konstantin.anan...@intel.com> wrote: > > 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". > > Yes, you right. > Do you want me to re-spin?
No need for this, thanks. -- David Marchand
