On Fri, Oct 11, 2024 at 5:16 PM Morten Brørup <m...@smartsharesystems.com> 
wrote:
>
> > > The Coverity report [1] just gave me an idea:
> > >
> > > worker_lcore_id = rte_get_next_lcore(-1, 1, 0);
> > > + TEST_ASSERT(worker_lcore_id < RTE_MAX_LCORE, "Worker thread
> > allocation failed");
> > >
> > > Or even better:
> > > Improve rte_eal_remote_launch() by checking the validity of the
> > worker_id parameter.
> > >
> > > [1]: https://scan4.scan.coverity.com/#/project-
> > view/60887/10075?selectedIssue=445384
> > >
> >
> > I have the same fix in my workdir, I was about to send, but if you
> > want to do it, go ahead.
>
> No, I only had the idea, didn't actually work on it.
>
> Please send your fix.

Yes, let's ask the maintainer to do the job.
Ok, I'll see later.


-- 
David Marchand

Reply via email to