All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Monjalon <thomas@monjalon.net>
To: David Marchand <david.marchand@redhat.com>
Cc: dev@dpdk.org, Bruce Richardson <bruce.richardson@intel.com>,
	solal.pirelli@gmail.com, dpdk stable <stable@dpdk.org>
Subject: Re: [PATCH] eal: fix out of bound access when no cpu is available
Date: Thu, 17 Jan 2019 18:38:32 +0100	[thread overview]
Message-ID: <20373872.tSZagaFhzs@xps> (raw)
In-Reply-To: <CAJFAV8zs69dv5CfJ+XXL225rKvq+FgRfaVPbLaYtVEOWwia1WQ@mail.gmail.com>

17/01/2019 18:17, David Marchand:
> On Thu, Jan 17, 2019 at 5:34 PM Thomas Monjalon <thomas@monjalon.net> wrote:
> 
> > 17/01/2019 14:12, David Marchand:
> > > In the unlikely case when the dpdk application is started with no cpu
> > > available in the [0, RTE_MAX_LCORE - 1] range, the master_lcore is
> > > automatically chosen as RTE_MAX_LCORE which triggers an out of bound
> > > access.
> > >
> > > Either you have a crash then, or the initialisation fails later when
> > > trying to pin the master thread on it.
> > > In my test, with RTE_MAX_LCORE == 2:
> > >
> > > $ taskset -c 2 ./master/app/testpmd --no-huge -m 512 --log-level *:debug
> > > [...]
> > > EAL: pthread_setaffinity_np failed
> > > PANIC in eal_thread_init_master():
> > > cannot set affinity
> > > 7: [./master/app/testpmd() [0x47f629]]
> > >
> > > Bugzilla ID: 19
> > > Signed-off-by: David Marchand <david.marchand@redhat.com>
> >
> > We should backport this fix.
> > When this bug has been introduced?
> >
> 
> Indeed, at first, I thought the problem had always been there, but it
> should be starting 17.02:
> Fixes: 2eba8d21f3c9 ("eal: restrict cores auto detection")
> 
> + CC stable
> 
> Do you want a v2 ?

Applied, thanks

      reply	other threads:[~2019-01-17 17:38 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-01-17 13:12 [PATCH] eal: fix out of bound access when no cpu is available David Marchand
2019-01-17 16:34 ` Thomas Monjalon
2019-01-17 17:17   ` David Marchand
2019-01-17 17:38     ` Thomas Monjalon [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20373872.tSZagaFhzs@xps \
    --to=thomas@monjalon.net \
    --cc=bruce.richardson@intel.com \
    --cc=david.marchand@redhat.com \
    --cc=dev@dpdk.org \
    --cc=solal.pirelli@gmail.com \
    --cc=stable@dpdk.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.