From: Olivier MATZ <olivier.matz-pdR9zngts4EAvxtiuMwx3w@public.gmane.org>
To: Pablo de Lara
<pablox.de.lara.guarch-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>,
dev-VfR2kkLFssw@public.gmane.org
Subject: Re: [PATCH] app/test: Removed vdev test in EAL flags unit test
Date: Tue, 01 Jul 2014 10:31:29 +0200 [thread overview]
Message-ID: <53B271E1.1010304@6wind.com> (raw)
In-Reply-To: <1404139377-24580-1-git-send-email-pablox.de.lara.guarch-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Hi Pablo,
On 06/30/2014 04:42 PM, Pablo de Lara wrote:
> From: Pablo de Lara <pablo.de.lara.guarch-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
>
> In whitelist test, within EAL flags unit test, vdev flag
> was being tested, but test fails, as a secondary process
> cannot allocate memzones, so it cannot create the device
> and application exits. Also, this can be tested in ring
> pmd test, which needs vdev and not in whitelist, which
> should only test --pci-whitelist.
>
> Patch "app/test: fix build switches to enable cmdline tests"
> from Thomas Monjalon is needed for this patch.
>
> Signed-off-by: Pablo de Lara <pablo.de.lara.guarch-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
If the test cannot success because it is started from a secondary
process, do you think it is possible to do it with a primary process?
I did a quick test and it works (see my patch in the same thread), but
as I'm not really familiar with this part of the code, I may have
forgotten something.
Regards,
Olivier
next prev parent reply other threads:[~2014-07-01 8:31 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-30 14:42 [PATCH] app/test: Removed vdev test in EAL flags unit test Pablo de Lara
[not found] ` <1404139377-24580-1-git-send-email-pablox.de.lara.guarch-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-07-01 8:31 ` Olivier MATZ [this message]
[not found] ` <53B271E1.1010304-pdR9zngts4EAvxtiuMwx3w@public.gmane.org>
2014-07-01 8:32 ` [PATCH] app/test: fix vdev test in test_eal_flags Olivier Matz
2014-07-01 8:59 ` [PATCH] app/test: Removed vdev test in EAL flags unit test De Lara Guarch, Pablo
[not found] ` <E115CCD9D858EF4F90C690B0DCB4D8971A797D35-kPTMFJFq+rELt2AQoY/u9bfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2014-07-01 9:20 ` Olivier MATZ
[not found] ` <53B27D70.4090102-pdR9zngts4EAvxtiuMwx3w@public.gmane.org>
2014-07-01 9:21 ` [PATCH v2] app/test: fix vdev test in test_eal_flags Olivier Matz
[not found] ` <1404206489-30703-1-git-send-email-olivier.matz-pdR9zngts4EAvxtiuMwx3w@public.gmane.org>
2014-07-01 11:44 ` De Lara Guarch, Pablo
[not found] ` <E115CCD9D858EF4F90C690B0DCB4D8971A797E1F-kPTMFJFq+rELt2AQoY/u9bfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2014-07-01 12:00 ` Olivier MATZ
[not found] ` <53B2A2F8.4060901-pdR9zngts4EAvxtiuMwx3w@public.gmane.org>
2014-07-01 12:11 ` De Lara Guarch, Pablo
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=53B271E1.1010304@6wind.com \
--to=olivier.matz-pdr9zngts4eavxtiumwx3w@public.gmane.org \
--cc=dev-VfR2kkLFssw@public.gmane.org \
--cc=pablox.de.lara.guarch-ral2JQCrhuEAvxtiuMwx3w@public.gmane.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.