> > 28/06/2019 11:09, David Marchand: > > On Fri, Jun 28, 2019 at 5:44 AM Honnappa Nagarahalli > > > --- a/app/test/test_rcu_qsbr.c > > > +++ b/app/test/test_rcu_qsbr.c > > > @@ -27,7 +27,7 @@ > > > #define TEST_RCU_QSBR_CNT_INIT 1 > > > > > > uint16_t enabled_core_ids[RTE_MAX_LCORE]; -uint8_t num_cores; > > > +uint16_t num_cores; > > > > If we want to be closer to the eal API, those should be unsigned int. > > I agree, please use unsigned int where relevant. Sent v2. There are other similar cases, may be in a follow up patch.
> >