* [dpdk-dev] [PATCH] app/test: Removed vdev test in EAL flags unit test @ 2014-06-30 14:42 Pablo de Lara 2014-07-01 8:31 ` Olivier MATZ 0 siblings, 1 reply; 9+ messages in thread From: Pablo de Lara @ 2014-06-30 14:42 UTC (permalink / raw) To: dev From: Pablo de Lara <pablo.de.lara.guarch@intel.com> 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@intel.com> --- app/test/test_eal_flags.c | 4 ---- 1 files changed, 0 insertions(+), 4 deletions(-) diff --git a/app/test/test_eal_flags.c b/app/test/test_eal_flags.c index 1b80b80..7c6656b 100644 --- a/app/test/test_eal_flags.c +++ b/app/test/test_eal_flags.c @@ -58,7 +58,6 @@ #define no_huge "--no-huge" #define no_shconf "--no-shconf" #define pci_whitelist "--pci-whitelist" -#define vdev "--vdev" #define memtest "memtest" #define memtest1 "memtest1" #define memtest2 "memtest2" @@ -317,9 +316,6 @@ test_whitelist_flag(void) const char *wlval3[] = {prgname, prefix, mp_flag, "-n", "1", "-c", "1", pci_whitelist, "09:0B.3,type=test", pci_whitelist, "08:00.1,type=normal", -#ifdef RTE_LIBRTE_PMD_RING - vdev, "eth_ring,arg=test", -#endif }; for (i = 0; i < sizeof(wlinval) / sizeof(wlinval[0]); i++) { -- 1.7.0.7 ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [dpdk-dev] [PATCH] app/test: Removed vdev test in EAL flags unit test 2014-06-30 14:42 [dpdk-dev] [PATCH] app/test: Removed vdev test in EAL flags unit test Pablo de Lara @ 2014-07-01 8:31 ` Olivier MATZ 2014-07-01 8:32 ` [dpdk-dev] [PATCH] app/test: fix vdev test in test_eal_flags Olivier Matz 2014-07-01 8:59 ` [dpdk-dev] [PATCH] app/test: Removed vdev test in EAL flags unit test De Lara Guarch, Pablo 0 siblings, 2 replies; 9+ messages in thread From: Olivier MATZ @ 2014-07-01 8:31 UTC (permalink / raw) To: Pablo de Lara, dev Hi Pablo, On 06/30/2014 04:42 PM, Pablo de Lara wrote: > From: Pablo de Lara <pablo.de.lara.guarch@intel.com> > > 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@intel.com> 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 ^ permalink raw reply [flat|nested] 9+ messages in thread
* [dpdk-dev] [PATCH] app/test: fix vdev test in test_eal_flags 2014-07-01 8:31 ` Olivier MATZ @ 2014-07-01 8:32 ` Olivier Matz 2014-07-01 8:59 ` [dpdk-dev] [PATCH] app/test: Removed vdev test in EAL flags unit test De Lara Guarch, Pablo 1 sibling, 0 replies; 9+ messages in thread From: Olivier Matz @ 2014-07-01 8:32 UTC (permalink / raw) To: dev, pablox.de.lara.guarch In whitelist test, within EAL flags unit test, the vdev flag test fails because it is started in a secondary process, which is not able to allocate memzones and this is required to instanciate a pmd_ring. This patch changes the vdev flag test to run it in a primary process. Patch "app/test: fix build switches to enable cmdline tests" from Thomas Monjalon is needed for this patch. Signed-off-by: Olivier Matz <olivier.matz@6wind.com> --- app/test/test_eal_flags.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/app/test/test_eal_flags.c b/app/test/test_eal_flags.c index 1b80b80..ca47bb2 100644 --- a/app/test/test_eal_flags.c +++ b/app/test/test_eal_flags.c @@ -59,6 +59,7 @@ #define no_shconf "--no-shconf" #define pci_whitelist "--pci-whitelist" #define vdev "--vdev" +#define whitelist_prefix "whitelist" #define memtest "memtest" #define memtest1 "memtest1" #define memtest2 "memtest2" @@ -314,7 +315,8 @@ test_whitelist_flag(void) pci_whitelist, "00FF:09:0B.3"}; const char *wlval2[] = {prgname, prefix, mp_flag, "-n", "1", "-c", "1", pci_whitelist, "09:0B.3", pci_whitelist, "0a:0b.1"}; - const char *wlval3[] = {prgname, prefix, mp_flag, "-n", "1", "-c", "1", + const char *wlval3[] = {prgname, "--prefix=" whitelist_prefix, + "-n", "1", "-c", "1", pci_whitelist, "09:0B.3,type=test", pci_whitelist, "08:00.1,type=normal", #ifdef RTE_LIBRTE_PMD_RING -- 1.9.2 ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [dpdk-dev] [PATCH] app/test: Removed vdev test in EAL flags unit test 2014-07-01 8:31 ` Olivier MATZ 2014-07-01 8:32 ` [dpdk-dev] [PATCH] app/test: fix vdev test in test_eal_flags Olivier Matz @ 2014-07-01 8:59 ` De Lara Guarch, Pablo 2014-07-01 9:20 ` Olivier MATZ 1 sibling, 1 reply; 9+ messages in thread From: De Lara Guarch, Pablo @ 2014-07-01 8:59 UTC (permalink / raw) To: Olivier MATZ, dev Hi Olivier, > -----Original Message----- > From: Olivier MATZ [mailto:olivier.matz@6wind.com] > Sent: Tuesday, July 01, 2014 9:31 AM > To: De Lara Guarch, Pablo; dev@dpdk.org > Cc: Olivier MATZ > Subject: Re: [dpdk-dev] [PATCH] app/test: Removed vdev test in EAL flags > unit test > > Hi Pablo, > > On 06/30/2014 04:42 PM, Pablo de Lara wrote: > > From: Pablo de Lara <pablo.de.lara.guarch@intel.com> > > > > 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@intel.com> > > 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 I checked your patch, and I got this: ... EAL: Detected lcore 38 as core 11 on socket 1 EAL: Detected lcore 39 as core 12 on socket 1 EAL: Support maximum 64 logical core(s) by configuration. EAL: Detected 40 lcore(s) ./x86_64-native-linuxapp-gcc/app/test: unrecognized option '--prefix=whitelist' Also, I thought we could not run two primary process at the same time (considering that the test app is a primary process). Thanks, Pablo ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [dpdk-dev] [PATCH] app/test: Removed vdev test in EAL flags unit test 2014-07-01 8:59 ` [dpdk-dev] [PATCH] app/test: Removed vdev test in EAL flags unit test De Lara Guarch, Pablo @ 2014-07-01 9:20 ` Olivier MATZ 2014-07-01 9:21 ` [dpdk-dev] [PATCH v2] app/test: fix vdev test in test_eal_flags Olivier Matz 0 siblings, 1 reply; 9+ messages in thread From: Olivier MATZ @ 2014-07-01 9:20 UTC (permalink / raw) To: De Lara Guarch, Pablo, dev Hi Pablo, On 07/01/2014 10:59 AM, De Lara Guarch, Pablo wrote: > I checked your patch, and I got this: > > ... > > EAL: Detected lcore 38 as core 11 on socket 1 > EAL: Detected lcore 39 as core 12 on socket 1 > EAL: Support maximum 64 logical core(s) by configuration. > EAL: Detected 40 lcore(s) > ./x86_64-native-linuxapp-gcc/app/test: unrecognized option '--prefix=whitelist' I'm sorry, I cleaned the patch without retesting it, and I introduced a bug in it (--prefix instead of --file-prefix). Please find the new one as a reply to this mail. When I test it, I get: [...] EAL: Ask a virtual area of 0x200000 bytes EAL: Virtual area found at 0x2aaba4c00000 (size = 0x200000) EAL: Ask a virtual area of 0x200000 bytes EAL: Virtual area found at 0x2aaba4e00000 (size = 0x200000) EAL: Ask a virtual area of 0x200000 bytes EAL: Virtual area found at 0x2aaba5000000 (size = 0x200000) EAL: Requesting 984 pages of size 2MB from socket 0 EAL: Requesting 1000 pages of size 2MB from socket 1 EAL: TSC frequency is ~3333293 KHz EAL: Master core 0 is ready (tid=ab9044e0) PMD: Initializing pmd_ring for eth_ring PMD: Error parsing device, invalid key <arg> PMD: Ignoring unsupported parameters when creating rings-backed ethernet device PMD: Creating rings-backed ethdev on numa socket 0 > Also, I thought we could not run two primary process at the same time > (considering that the test app is a primary process). Some other tests in the same file launch a primary process, I suppose it should work if the prefix for huge-pages is different. Regards, Olivier ^ permalink raw reply [flat|nested] 9+ messages in thread
* [dpdk-dev] [PATCH v2] app/test: fix vdev test in test_eal_flags 2014-07-01 9:20 ` Olivier MATZ @ 2014-07-01 9:21 ` Olivier Matz 2014-07-01 11:44 ` De Lara Guarch, Pablo 0 siblings, 1 reply; 9+ messages in thread From: Olivier Matz @ 2014-07-01 9:21 UTC (permalink / raw) To: dev, pablox.de.lara.guarch In whitelist test, within EAL flags unit test, the vdev flag test fails because it is started in a secondary process, which is not able to allocate memzones and this is required to instanciate a pmd_ring. This patch changes the vdev flag test to run it in a primary process. Patch "app/test: fix build switches to enable cmdline tests" from Thomas Monjalon is needed for this patch. Signed-off-by: Olivier Matz <olivier.matz@6wind.com> --- app/test/test_eal_flags.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/app/test/test_eal_flags.c b/app/test/test_eal_flags.c index 1b80b80..4a50d37 100644 --- a/app/test/test_eal_flags.c +++ b/app/test/test_eal_flags.c @@ -59,6 +59,7 @@ #define no_shconf "--no-shconf" #define pci_whitelist "--pci-whitelist" #define vdev "--vdev" +#define whitelist_prefix "whitelist" #define memtest "memtest" #define memtest1 "memtest1" #define memtest2 "memtest2" @@ -314,7 +315,8 @@ test_whitelist_flag(void) pci_whitelist, "00FF:09:0B.3"}; const char *wlval2[] = {prgname, prefix, mp_flag, "-n", "1", "-c", "1", pci_whitelist, "09:0B.3", pci_whitelist, "0a:0b.1"}; - const char *wlval3[] = {prgname, prefix, mp_flag, "-n", "1", "-c", "1", + const char *wlval3[] = {prgname, "--file-prefix=" whitelist_prefix, + "-n", "1", "-c", "1", pci_whitelist, "09:0B.3,type=test", pci_whitelist, "08:00.1,type=normal", #ifdef RTE_LIBRTE_PMD_RING -- 1.9.2 ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [dpdk-dev] [PATCH v2] app/test: fix vdev test in test_eal_flags 2014-07-01 9:21 ` [dpdk-dev] [PATCH v2] app/test: fix vdev test in test_eal_flags Olivier Matz @ 2014-07-01 11:44 ` De Lara Guarch, Pablo 2014-07-01 12:00 ` Olivier MATZ 0 siblings, 1 reply; 9+ messages in thread From: De Lara Guarch, Pablo @ 2014-07-01 11:44 UTC (permalink / raw) To: Olivier Matz, dev Hi Olivier, > -----Original Message----- > From: Olivier Matz [mailto:olivier.matz@6wind.com] > Sent: Tuesday, July 01, 2014 10:21 AM > To: dev@dpdk.org; De Lara Guarch, Pablo > Cc: olivier.matz@6wind.com > Subject: [PATCH v2] app/test: fix vdev test in test_eal_flags > > In whitelist test, within EAL flags unit test, the vdev flag test fails > because it is started in a secondary process, which is not able to > allocate memzones and this is required to instanciate a pmd_ring. > > This patch changes the vdev flag test to run it in a primary process. > > Patch "app/test: fix build switches to enable cmdline tests" > from Thomas Monjalon is needed for this patch. > > Signed-off-by: Olivier Matz <olivier.matz@6wind.com> > --- > app/test/test_eal_flags.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/app/test/test_eal_flags.c b/app/test/test_eal_flags.c > index 1b80b80..4a50d37 100644 > --- a/app/test/test_eal_flags.c > +++ b/app/test/test_eal_flags.c > @@ -59,6 +59,7 @@ > #define no_shconf "--no-shconf" > #define pci_whitelist "--pci-whitelist" > #define vdev "--vdev" > +#define whitelist_prefix "whitelist" > #define memtest "memtest" > #define memtest1 "memtest1" > #define memtest2 "memtest2" > @@ -314,7 +315,8 @@ test_whitelist_flag(void) > pci_whitelist, "00FF:09:0B.3"}; > const char *wlval2[] = {prgname, prefix, mp_flag, "-n", "1", "-c", "1", > pci_whitelist, "09:0B.3", pci_whitelist, "0a:0b.1"}; > - const char *wlval3[] = {prgname, prefix, mp_flag, "-n", "1", "-c", "1", > + const char *wlval3[] = {prgname, "--file-prefix=" whitelist_prefix, > + "-n", "1", "-c", "1", > pci_whitelist, "09:0B.3,type=test", > pci_whitelist, "08:00.1,type=normal", > #ifdef RTE_LIBRTE_PMD_RING > -- > 1.9.2 Thanks for this. It works fine :) Actually, I am thinking of creating a separate test for vdev, out of whitelist test, as it has no relation with it. Do you agree on this? Thanks, Pablo ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [dpdk-dev] [PATCH v2] app/test: fix vdev test in test_eal_flags 2014-07-01 11:44 ` De Lara Guarch, Pablo @ 2014-07-01 12:00 ` Olivier MATZ 2014-07-01 12:11 ` De Lara Guarch, Pablo 0 siblings, 1 reply; 9+ messages in thread From: Olivier MATZ @ 2014-07-01 12:00 UTC (permalink / raw) To: De Lara Guarch, Pablo, dev Hi Pablo, On 07/01/2014 01:44 PM, De Lara Guarch, Pablo wrote: > Thanks for this. It works fine :) Thanks for testing! > Actually, I am thinking of creating a separate test > for vdev, out of whitelist test, as it has no relation with it. Do you agree on this? I have no problem with that, it makes sense. Just to confirm: it means that Thomas should not apply our previous patches from this thread and wait for your next one, right? Regards, Olivier ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [dpdk-dev] [PATCH v2] app/test: fix vdev test in test_eal_flags 2014-07-01 12:00 ` Olivier MATZ @ 2014-07-01 12:11 ` De Lara Guarch, Pablo 0 siblings, 0 replies; 9+ messages in thread From: De Lara Guarch, Pablo @ 2014-07-01 12:11 UTC (permalink / raw) To: Olivier MATZ, dev > -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Olivier MATZ > Sent: Tuesday, July 01, 2014 1:01 PM > To: De Lara Guarch, Pablo; dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2] app/test: fix vdev test in test_eal_flags > > Hi Pablo, > > On 07/01/2014 01:44 PM, De Lara Guarch, Pablo wrote: > > Thanks for this. It works fine :) > > Thanks for testing! > > > Actually, I am thinking of creating a separate test > > for vdev, out of whitelist test, as it has no relation with it. Do you agree on > this? > > I have no problem with that, it makes sense. Just to confirm: it means > that Thomas should not apply our previous patches from this thread and > wait for your next one, right? That's it :) > > Regards, > Olivier Thanks, Pablo ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2014-07-01 12:12 UTC | newest] Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2014-06-30 14:42 [dpdk-dev] [PATCH] app/test: Removed vdev test in EAL flags unit test Pablo de Lara 2014-07-01 8:31 ` Olivier MATZ 2014-07-01 8:32 ` [dpdk-dev] [PATCH] app/test: fix vdev test in test_eal_flags Olivier Matz 2014-07-01 8:59 ` [dpdk-dev] [PATCH] app/test: Removed vdev test in EAL flags unit test De Lara Guarch, Pablo 2014-07-01 9:20 ` Olivier MATZ 2014-07-01 9:21 ` [dpdk-dev] [PATCH v2] app/test: fix vdev test in test_eal_flags Olivier Matz 2014-07-01 11:44 ` De Lara Guarch, Pablo 2014-07-01 12:00 ` Olivier MATZ 2014-07-01 12:11 ` De Lara Guarch, Pablo
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).