[PATCH v2 07/10] test/eal: fix core check in c flag test

[Stephen Hemminger]

On Fri, 15 Nov 2024 09:07:42 +0000
Bruce Richardson <bruce.richardson at intel.com> wrote:

> On Thu, Nov 14, 2024 at 11:25:05AM -0800, Stephen Hemminger wrote:
> > The expression for checking which lcore is enabled for 0-7
> > was wrong (missing case for 6).
> > 
> > Link: https://pvs-studio.com/en/blog/posts/cpp/1179/
> > 
> > Fixes: b0209034f2bb ("test/eal: check number of cores before running subtests")
> > Cc: msantana at redhat.com
> > Cc: stable at dpdk.org
> > 
> > Signed-off-by: Stephen Hemminger <stephen at networkplumber.org>  
> 
> Just wondering would it not be better/safer to put in an actual loop check
> here? 
> However, I'm also ok with keeping the fix as-is, so:
> 
> Acked-by: Bruce Richardson <bruce.richardson at intel.com>

My goal was to do minimum changes for now, to avoid introducing new bugs.