Hello Dharmik,
On Wed, May 22, 2019 at 9:38 PM Dharmik Thakkar <dharmik.thak...@arm.com>
wrote:
> > On May 22, 2019, at 12:17 PM, David Marchand <david.march...@redhat.com>
> wrote:
> > On Wed, May 22, 2019 at 6:27 PM Dharmik Thakkar <dharmik.thak...@arm.com>
> wrote:
> > > @@ -738,10 +736,9 @@ struct {
> > > rte_eal_remote_launch(test_rwc_reader,
> > > (void
> *)(uintptr_t)read_type,
> > >
> enabled_core_ids[i]);
> > > - rte_eal_mp_wait_lcore();
> > >
> > > for (i = 1; i <= rwc_core_cnt[n]; i++)
> > > - if (lcore_config[i].ret < 0)
> > > + if (rte_eal_wait_lcore(i) < 0)
> > if (rte_eal_wait_lcore(enabled_core_ids[i]) < 0)
> >
> > (There are similar changes in other functions too.
> > I realize that this is a separate issue than what the patch is aimed for.
> > If you see fit, please integrate it, else I will put out a patch once
> your patch has been merged.)
> >
> > Indeed, reproduced.
> >
>
Did you have a chance to look at this ?
These loops could be reworked by using RTE_LCORE_FOREACH_SLAVE() (so that
it automatically skips the master lcore) removing the need for
enabled_core_ids[].
--
David Marchand
