Re: [PATCH v15 10/11] app/test: replace .sh script with .py script

O Thu, Feb 03, 2022 at 12:15:19PM +0100, Thomas Monjalon wrote: > 26/01/2022 06:10, Jie Zhou: > > - Add python script to check if system supports hugepages > > - Remove corresponding .sh script > > - Replace calling of .sh with corresponding .py in meson.build > >

[PATCH v15 09/11] app/test: skip tests that are not supported yet

: Execution enters an infinite loops, requires investigation. * test_rcu_qsbr_perf: Execution hangs on Windows, requires investigation. Signed-off-by: Jie Zhou Signed-off-by: Dmitry Kozlyuk Acked-by: Tyler Retzlaff --- app/test/test_acl.c | 12 app/test

[PATCH v15 11/11] app/test: enable unit test on Windows

s patchset to be merged). - For dump tests, currently the tests hang on Windows which require further investigation. - For telemetry tests, it has POSIX socket specific codes which require replacement for Windows. Will investigate and work on a separate patch. Signed-off-by: Jie Zhou Acked

[PATCH v15 08/11] app/test: resolve name collision

Add OP_ prefix to resolve name collision to enable hash_perf test Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- app/test/test_hash_perf.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/app/test/test_hash_perf.c b/app/test/test_hash_perf.c index

[PATCH v15 10/11] app/test: replace .sh script with .py script

- Add python script to check if system supports hugepages - Remove corresponding .sh script - Replace calling of .sh with corresponding .py in meson.build Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- app/test/has-hugepage.sh | 11 --- app/test/has_hugepage.py | 26

[PATCH v15 07/11] app/test: remove two alarm_autotest cases

Remove two alarm_autotest test cases which do bogus range check on Windows. Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- app/test/test_alarm.c | 4 1 file changed, 4 insertions(+) diff --git a/app/test/test_alarm.c b/app/test/test_alarm.c index b4034339b8..70e97a3109 100644 --- a

[PATCH v15 06/11] app/test: differentiate a strerror on different OS

On Windows, strerror returns just "Unknown error" for errnum greater than MAX_ERRNO, while linux and freebsd returns "Unknown error ", which is the current expectation for errno_autotest. Differentiate the error string on Windows to remove a "duplicate error code" fai

[PATCH v15 05/11] app/test: skip two logs_autotest cases on Windows

ndef these two test cases, and for the rte_log_set_level_pattern validation case following these two cases, differentiate the expected log level passed into macro CHECK_LEVELS Now logs_autotest completes for all dynamic log types and static log types. Signed-off-by: Jie Zhou Acked-by: Dmit

[PATCH v15 00/11] app/test: enable subset of tests on Windows

s not supported on Windows yet, thus no corresponding header files, and undefined APIs used in test helper functions. Just adding the regular if statement in the test main function is not sufficient. Tested-by: Pallavi Kadam Jie Zhou (11): eal/windows: return ENOTSUP for not supporte

[PATCH v15 02/11] app/test: remove POSIX-specific code

- Replace POSIX-specific code with DPDK equivalents or conditionally disable it on Windows - Use NUL on Windows as /dev/null for Unix - Exclude tests not supported on Windows yet * multi-process * PMD performance statistics display on signal Signed-off-by: Jie Zhou Signed-off-by: Dmitry

[PATCH v15 04/11] app/test: skip interrupt tests on Windows

Even though test_interrupts.c can compile on Windows, skip interrupt tests for now since majority of eal_interrupt on Windows are stubs. Will remove the skip after interrupt being fully enabled on Windows. Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- app/test/test_interrupts.c | 3

[PATCH v15 01/11] eal/windows: return ENOTSUP for not supported API

. Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- lib/eal/windows/eal_memalloc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/eal/windows/eal_memalloc.c b/lib/eal/windows/eal_memalloc.c index 55d6dcc71c..aa7589b81d 100644 --- a/lib/eal/windows/eal_memalloc.c

[PATCH v15 03/11] app/test: fix incorrect errno variable

Fix incorrect errno variable used in memory autotest. Use rte_errno instead. Fixes: 086d426406bd ("app/test: fix memory autotests on FreeBSD") Cc: bruce.richard...@intel.com Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- app/test/test_memory.c | 2 +- 1 file changed, 1 inser

[PATCH v14 09/11] app/test: add test stubs for not supported ones

. * test_lcores: Execution enters an infinite loops, requires investigation. * test_rcu_qsbr_perf: Execution hangs on Windows, requires investigation. Signed-off-by: Jie Zhou Signed-off-by: Dmitry Kozlyuk --- app/test/test_acl.c | 12 app/test/test_bpf.c

[PATCH v14 11/11] app/test: enable unit test on Windows

s patchset to be merged). - For dump tests, currently the tests hang on Windows which require further investigation. - For telemetry tests, it has POSIX socket specific codes which require replacement for Windows. Will investigate and work on a separate patch. Signed-off-by: Jie Zhou Acked

[PATCH v14 07/11] app/test: remove two alarm_autotest cases

Remove two alarm_autotest test cases which do bogus range check on Windows. Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- app/test/test_alarm.c | 4 1 file changed, 4 insertions(+) diff --git a/app/test/test_alarm.c b/app/test/test_alarm.c index b4034339b8..70e97a3109 100644

[PATCH v14 06/11] app/test: differentiate a strerror on different OS

On Windows, strerror returns just "Unknown error" for errnum greater than MAX_ERRNO, while linux and freebsd returns "Unknown error ", which is the current expectation for errno_autotest. Differentiate the error string on Windows to remove a "duplicate error code" fai

[PATCH v14 08/11] app/test: resolve name collision

Add OP_ prefix to resolve name collision to enable hash_perf test Signed-off-by: Jie Zhou --- app/test/test_hash_perf.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/app/test/test_hash_perf.c b/app/test/test_hash_perf.c index 76cdac5d53..7e98ec3964 100644

[PATCH v14 10/11] app/test: replace .sh script with .py script

- Add python script to check if system supports hugepages - Remove corresponding .sh script - Replace calling of .sh with corresponding .py in meson.build Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- app/test/has-hugepage.sh | 11 --- app/test/has_hugepage.py | 26

[PATCH v14 05/11] app/test: skip two logs_autotest cases on Windows

ndef these two test cases, and for the rte_log_set_level_pattern validation case following these two cases, differentiate the expected log level passed into macro CHECK_LEVELS Now logs_autotest completes for all dynamic log types and static log types. Signed-off-by: Jie Zhou Acked-by: Dmit

[PATCH v14 02/11] app/test: remove POSIX-specific code

- Replace POSIX-specific code with DPDK equivalents or conditionally disable it on Windows - Use NUL on Windows as /dev/null for Unix - Exclude tests not supported on Windows yet * multi-process * PMD performance statistics display on signal Signed-off-by: Jie Zhou Signed-off-by: Dmitry

[PATCH v14 04/11] app/test: skip interrupt tests on Windows

Even though test_interrupts.c can compile on Windows, skip interrupt tests for now since majority of eal_interrupt on Windows are stubs. Will remove the skip after interrupt being fully enabled on Windows. Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- app/test/test_interrupts.c | 10

[PATCH v14 01/11] eal/windows: return ENOTSUP for not supported API

. Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- lib/eal/windows/eal_memalloc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/eal/windows/eal_memalloc.c b/lib/eal/windows/eal_memalloc.c index 55d6dcc71c..aa7589b81d 100644 --- a/lib/eal/windows

[PATCH v14 00/11] app/test: enable subset of tests on Windows

e reason of skipping telemetry tests in commit log Tested-by: Pallavi Kadam Jie Zhou (11): eal/windows: return ENOTSUP for not supported API app/test: remove POSIX-specific code app/test: fix incorrect errno variable app/test: skip interrupt tests on Windows app/test: skip two logs_autotest ca

[PATCH v14 03/11] app/test: fix incorrect errno variable

Fix incorrect errno variable used in memory autotest. Use rte_errno instead. Fixes: 086d426406bd ("app/test: fix memory autotests on FreeBSD") Cc: bruce.richard...@intel.com Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- app/test/test_memory.c | 2 +- 1 file changed, 1 inser

Re: [PATCH v13 11/11] app/test: enable unit test on Windows

On Wed, Dec 08, 2021 at 06:23:30PM +, Bruce Richardson wrote: > On Wed, Dec 08, 2021 at 10:19:27AM -0800, Jie Zhou wrote: > > On Wed, Dec 08, 2021 at 04:57:52PM +, Bruce Richardson wrote: > > > On Tue, Dec 07, 2021 at 05:51:02PM -0800, Jie Zhou wrote: > > > >

Re: [PATCH v13 11/11] app/test: enable unit test on Windows

On Wed, Dec 08, 2021 at 04:57:52PM +, Bruce Richardson wrote: > On Tue, Dec 07, 2021 at 05:51:02PM -0800, Jie Zhou wrote: > > Enable a subset of unit tests for Windows CI > > > > - For driver tests, driver owners should enable corresponding tests when > > enabl

[PATCH v13 06/11] app/test: differentiate a strerror on different OS

On Windows, strerror returns just "Unknown error" for errnum greater than MAX_ERRNO, while linux and freebsd returns "Unknown error ", which is the current expectation for errno_autotest. Differentiate the error string on Windows to remove a "duplicate error code" fai

[PATCH v13 11/11] app/test: enable unit test on Windows

s patchset to be merged). - For dump tests, currently the tests hang on Windows which require further investigation. Signed-off-by: Jie Zhou --- app/test/meson.build | 111 ++- 1 file changed, 58 insertions(+), 53 deletions(-) diff --git a/app/test/mes

[PATCH v13 09/11] app/test: add test stubs for not supported ones

. * test_lcores: Execution enters an infinite loops, requires investigation. * test_rcu_qsbr_perf: Execution hangs on Windows, requires investigation. Signed-off-by: Jie Zhou Signed-off-by: Dmitry Kozlyuk --- app/test/test_acl.c | 12 app/test/test_bpf.c

[PATCH v13 08/11] app/test: resolve name collision

Add OP_ prefix to resolve name collision to enable hash_perf test Signed-off-by: Jie Zhou Signed-off-by: Dmitry Kozlyuk --- app/test/test_hash_perf.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/app/test/test_hash_perf.c b/app/test/test_hash_perf.c

[PATCH v13 10/11] app/test: replace .sh script with .py script

- Add python script to check if system supports hugepages - Remove corresponding .sh script - Replace calling of .sh with corresponding .py in meson.build Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- app/test/has-hugepage.sh | 11 --- app/test/has_hugepage.py | 26

[PATCH v13 07/11] app/test: remove two alarm_autotest cases

Remove two alarm_autotest test cases which do bogus range check on Windows. Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- app/test/test_alarm.c | 4 1 file changed, 4 insertions(+) diff --git a/app/test/test_alarm.c b/app/test/test_alarm.c index b4034339b8..70e97a3109 100644

[PATCH v13 05/11] app/test: skip two logs_autotest cases on Windows

ndef these two test cases, and for the rte_log_set_level_pattern validation case following these two cases, differentiate the expected log level passed into macro CHECK_LEVELS Now logs_autotest completes for all dynamic log types and static log types. Signed-off-by: Jie Zhou Acked-by: Dmit

[PATCH v13 02/11] app/test: remove POSIX-specific code

- Replace POSIX-specific code with DPDK equivalents or conditionally disable it on Windows - Use NUL on Windows as /dev/null for Unix - Exclude tests not supported on Windows yet * multi-process * PMD performance statistics display on signal Signed-off-by: Jie Zhou Signed-off-by: Dmitry

[PATCH v13 00/11] app/test: enable subset of tests on Windows

Patch#9 in V11 into two patches for better structure - Reorder some of the patches for better structure - Document more details in commit message for issue tracking --- V13 changes: - Fix misc coding style issue - Fix build issue on Ubuntu 18.04 Tested-by: Pallavi Kadam Jie Zhou

[PATCH v13 04/11] app/test: skip interrupt tests on Windows

Even though test_interrupts.c can compile on Windows, skip interrupt tests for now since majority of eal_interrupt on Windows are stubs. Will remove the skip after interrupt being fully enabled on Windows. Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- app/test/test_interrupts.c | 10

[PATCH v13 03/11] app/test: fix incorrect errno variable

Fix incorrect errno variable used in memory autotest. Use rte_errno instead. Fixes: 086d426406bd ("app/test: fix memory autotests on FreeBSD") Cc: bruce.richard...@intel.com Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- app/test/test_memory.c | 2 +- 1 file changed, 1 inser

[PATCH v13 01/11] eal/windows: return ENOTSUP for not supported API

. Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- lib/eal/windows/eal_memalloc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/eal/windows/eal_memalloc.c b/lib/eal/windows/eal_memalloc.c index 55d6dcc71c..aa7589b81d 100644 --- a/lib/eal/windows

[PATCH v12 02/11] app/test: remove POSIX-specific code

-by: Jie Zhou Signed-off-by: Dmitry Kozlyuk --- app/test/commands.c | 2 -- app/test/packet_burst_generator.c | 1 + app/test/process.h| 4 +++- app/test/test.c | 5 - app/test/test_byteorder.c | 2 +- app/test

[PATCH v12 09/11] app/test: add test stubs for not supported ones

. * test_lcores: Execution enters an infinite loops, requires investigation. * test_rcu_qsbr_perf: Execution hangs on Windows, requires investigation. Signed-off-by: Jie Zhou Signed-off-by: Dmitry Kozlyuk --- app/test/test_acl.c | 12 app/test/test_bpf.c

[PATCH v12 11/11] app/test: enable unit test for Windows

For dump tests, currently the tests hang on Windows which require further investigation. Signed-off-by: Jie Zhou --- app/test/meson.build | 111 ++- 1 file changed, 58 insertions(+), 53 deletions(-) diff --git a/app/test/meson.build b/app/test/meson.build i

[PATCH v12 10/11] app/test: replace .sh script with .py script

- Add python script to check if system supports hugepages - Remove corresponding .sh script - Replace calling of .sh with corresponding .py in meson.build Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- app/test/has-hugepage.sh | 11 --- app/test/has_hugepage.py | 26

[PATCH v12 05/11] app/test: skip two logs_autotest cases on Windows

ndef these two test cases, and for the rte_log_set_level_pattern validation case following these two cases, differentiate the expected log level passed into macro CHECK_LEVELS Now logs_autotest completes for all dynamic log types and static log types. Signed-off-by: Jie Zhou Acked-by: Dmit

[PATCH v12 08/11] app/test: resolve name collision

Add OP_ prefix to resolve name collision to enable hash_perf test Signed-off-by: Jie Zhou Signed-off-by: Dmitry Kozlyuk --- app/test/test_hash_perf.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/app/test/test_hash_perf.c b/app/test/test_hash_perf.c

[PATCH v12 07/11] app/test: remove two alarm_autotest cases

Remove two alarm_autotest test cases which do bogus range check on Windows. Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- app/test/test_alarm.c | 4 1 file changed, 4 insertions(+) diff --git a/app/test/test_alarm.c b/app/test/test_alarm.c index b4034339b8..70e97a3109 100644

[PATCH v12 06/11] app/test: differentiate a strerror on different OS

On Windows, strerror returns just "Unknown error" for errnum greater than MAX_ERRNO, while linux and freebsd returns "Unknown error ", which is the current expectation for errno_autotest. Differentiate the error string on Windows to remove a "duplicate error code" fai

[PATCH v12 03/11] app/test: fix incorrect errno variable

Fix incorrect errno variable used in memory autotest. Use rte_errno instead. Fixes: 086d426406bd ("app/test: fix memory autotests on FreeBSD") Cc: bruce.richard...@intel.com Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- app/test/test_memory.c | 2 +- 1 file changed, 1 inser

[PATCH v12 04/11] app/test: skip interrupt tests on Windows

Even though test_interrupts.c can compile on Windows, skip interrupt tests for now since majority of eal_interrupt on Windows are stubs. Will remove the skip after interrupt being fully enabled on Windows. Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- app/test/test_interrupts.c | 10

[PATCH v12 00/11] app/test: enable subset of tests on Windows

ch#9 in V11 into two patches for better structure - Reorder some of the patches for better structure - Document more details in commit message for issue tracking Tested-by: Pallavi Kadam Jie Zhou (11): eal/windows: return ENOTSUP for not supported API app/test: remove POSIX-specific

[PATCH v12 01/11] eal/windows: return ENOTSUP for not supported API

. Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- lib/eal/windows/eal_memalloc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/eal/windows/eal_memalloc.c b/lib/eal/windows/eal_memalloc.c index 55d6dcc71c..aa7589b81d 100644 --- a/lib/eal/windows

Re: [PATCH v11 6/9] app/test: differentiate a strerror on different OS

On Sat, Dec 04, 2021 at 04:33:37AM +0300, Dmitry Kozlyuk wrote: > 2021-12-02 16:06 (UTC-0800), Jie Zhou: > > On Windows, strerror returns just "Unknown error" for errnum greater > > than MAX_ERRNO, while linux and freebsd returns "Unknown error ", >

[PATCH v11 7/9] app/test: remove two alarm_autotest cases

Remove two alarm_autotest test cases which do bogus range check on Windows. Signed-off-by: Jie Zhou --- app/test/test_alarm.c | 4 1 file changed, 4 insertions(+) diff --git a/app/test/test_alarm.c b/app/test/test_alarm.c index b4034339b8..70e97a3109 100644 --- a/app/test/test_alarm.c

[PATCH v11 9/9] app/test: enable subset of unit tests on Windows

) which will be enabled by "patch-18949: app/test: enable crypto unit tests on Windows". - For dump tests, currently the tests hang on Windows which require further investigation. Keep the dump test list just for non-Windows for easier tracking. Signed-off-by: Jie Zhou --- app/test/m

[PATCH v11 6/9] app/test: differentiate a strerror on different OS

On Windows, strerror returns just "Unknown error" for errnum greater than MAX_ERRNO, while linux and freebsd returns "Unknown error ", which is the current expectation for errno_autotest. Differentiate the error string on Windows to remove a "duplicate error code" fai

[PATCH v11 8/9] app/test: replace .sh script with .py script

- Add python script to check if system supports hugepages - Remove corresponding .sh script - Replace calling of .sh with corresponding .py in meson.build Signed-off-by: Jie Zhou --- app/test/has-hugepage.sh | 11 --- app/test/has_hugepage.py | 26 ++ app/test

[PATCH v11 5/9] app/test: skip two logs_autotest cases on Windows

ndef these two test cases, and for the rte_log_set_level_pattern validation case following these two cases, differentiate the expected log level passed into macro CHECK_LEVELS Now logs_autotest completes for all dynamic log types and static log types. Signed-off-by: Jie Zhou --- app/test/test_

[PATCH v11 2/9] app/test: remove POSIX-specific code

netinet/in.h u6_addr and better have a separate patch to add such tests on Windows) Signed-off-by: Jie Zhou --- app/test/commands.c | 2 -- app/test/packet_burst_generator.c | 1 + app/test/process.h| 4 +++- app/test/test.c | 5 - app/test

[PATCH v11 4/9] app/test: skip interrupt tests on Windows

Even though test_interrupts.c can compile on Windows, skip interrupt tests for now since majority of eal_interrupt on Windows are stubs. Will remove the skip after interrupt being fully enabled on Windows. Signed-off-by: Jie Zhou --- app/test/test_interrupts.c | 10 ++ 1 file changed

[PATCH v11 3/9] app/test: fix incorrect errno variable

Fix incorrect errno variable used in memory autotest. Use rte_errno instead. Fixes: 086d426406bd ("app/test: fix memory autotests on FreeBSD") Cc: bruce.richard...@intel.com Signed-off-by: Jie Zhou --- app/test/test_memory.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) di

[PATCH v11 0/9] app/test: enable subset of tests on Windows

datory dependency on bitratestats, latencystats, and metrics libs in test meson.build, which was reintroduced at rebase in V9. Jie Zhou (9): eal/windows: return ENOTSUP for not supported API app/test: remove POSIX-specific code app/test: fix incorrect errno variable app/test: ski

[PATCH v11 1/9] eal/windows: return ENOTSUP for not supported API

. Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- lib/eal/windows/eal_memalloc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/eal/windows/eal_memalloc.c b/lib/eal/windows/eal_memalloc.c index 55d6dcc71c..aa7589b81d 100644 --- a/lib/eal/windows

Re: [PATCH v10 9/9] app/test: enable subset of unit tests on Windows

On Thu, Dec 02, 2021 at 10:17:48AM +0100, David Marchand wrote: > Hello, > > On Wed, Dec 1, 2021 at 7:45 PM Jie Zhou wrote: > > @@ -158,34 +152,17 @@ test_sources = files( > > ) > > > > test_deps = [ > > -'acl', > >

Re: [PATCH v9 9/9] app/test: enable subset of unit tests on Windows

On Wed, Dec 01, 2021 at 10:45:35AM -0800, Stephen Hemminger wrote: > On Wed, 1 Dec 2021 10:05:38 -0800 > Jie Zhou wrote: > > > diff --git a/app/test/test_dmadev.c b/app/test/test_dmadev.c > > index b206db27ae..c2ea9090cf 100644 > > --- a/app/test/test_dmadev.c >

[PATCH v10 9/9] app/test: enable subset of unit tests on Windows

) which will be enabled by "patch-18949: app/test: enable crypto unit tests on Windows". - For dump tests, currently the tests hang on Windows which require further investigation. Keep the dump test list just for non-Windows for easier tracking. Signed-off-by: Jie Zhou --- app/test/m

[PATCH v10 6/9] app/test: differentiate a strerror on different OS

On Windows, strerror returns just "Unknown error" for errnum greater than MAX_ERRNO, while linux and freebsd returns "Unknown error ", which is the current expectation for errno_autotest. Differentiate the error string on Windows to remove a "duplicate error code" fai

[PATCH v10 5/9] app/test: skip two logs_autotest cases on Windows

ndef these two test cases, and for the rte_log_set_level_pattern validation case following these two cases, differentiate the expected log level passed into macro CHECK_LEVELS Now logs_autotest completes for all dynamic log types and static log types. Signed-off-by: Jie Zhou --- app/test/test_

[PATCH v10 8/9] app/test: replace .sh script with .py script

- Add python script to check if system supports hugepages - Remove corresponding .sh script - Replace calling of .sh with corresponding .py in meson.build Signed-off-by: Jie Zhou --- app/test/has-hugepage.sh | 11 --- app/test/has_hugepage.py | 26 ++ app/test

[PATCH v10 7/9] app/test: remove two alarm_autotest cases

Remove two alarm_autotest test cases which do bogus range check on Windows. Signed-off-by: Jie Zhou --- app/test/test_alarm.c | 4 1 file changed, 4 insertions(+) diff --git a/app/test/test_alarm.c b/app/test/test_alarm.c index b4034339b8..70e97a3109 100644 --- a/app/test/test_alarm.c

[PATCH v10 0/9] app/test: enable subset of tests on Windows

e previously skipped test_cmdline_socket_fns test cases - Revise title and message, and add Fixes info for current Patch#3 - Combine 2 patches (previous #2 and #3 in V8) into one and with more detailed message V10 changes: - Fix indentation Jie Zhou (9): eal/windows: return ENOTSUP f

[PATCH v10 2/9] app/test: remove POSIX-specific code

netinet/in.h u6_addr and better have a sperate patch to add such tests on Windows) Signed-off-by: Jie Zhou --- app/test/commands.c | 2 -- app/test/packet_burst_generator.c | 1 + app/test/process.h| 4 +++- app/test/test.c | 5 - app/test

[PATCH v10 4/9] app/test: skip interrupt tests on Windows

Even though test_interrupts.c can compile on Windows, skip interrupt tests for now since majority of eal_interrupt on Windows are stubs. Will remove the skip after interrupt being fully enabled on Windows. Signed-off-by: Jie Zhou --- app/test/test_interrupts.c | 10 ++ 1 file changed

[PATCH v10 3/9] app/test: fix incorrect errno variable

Fix incorrect errno variable used in memory autotest. Use rte_errno instead. Fixes: 086d426406bd ("app/test: fix memory autotests on FreeBSD") Signed-off-by: Jie Zhou --- app/test/test_memory.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/test/test_memo

[PATCH v10 1/9] eal/windows: return ENOTSUP for not supported API

. Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- lib/eal/windows/eal_memalloc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/eal/windows/eal_memalloc.c b/lib/eal/windows/eal_memalloc.c index 55d6dcc71c..aa7589b81d 100644 --- a/lib/eal/windows

[PATCH v9 9/9] app/test: enable subset of unit tests on Windows

) which will be enabled by "patch-18949: app/test: enable crypto unit tests on Windows". - For dump tests, currently the tests hang on Windows which require further investigation. Keep the dump test list just for non-Windows for easier tracking. Signed-off-by: Jie Zhou --- app/test/m

[PATCH v9 6/9] app/test: differentiate a strerror on different OS

On Windows, strerror returns just "Unknown error" for errnum greater than MAX_ERRNO, while linux and freebsd returns "Unknown error ", which is the current expectation for errno_autotest. Differentiate the error string on Windows to remove a "duplicate error code" fai

[PATCH v9 7/9] app/test: remove two alarm_autotest cases

Remove two alarm_autotest test cases which do bogus range check on Windows. Signed-off-by: Jie Zhou --- app/test/test_alarm.c | 4 1 file changed, 4 insertions(+) diff --git a/app/test/test_alarm.c b/app/test/test_alarm.c index b4034339b8..70e97a3109 100644 --- a/app/test/test_alarm.c

[PATCH v9 8/9] app/test: replace .sh script with .py script

- Add python script to check if system supports hugepages - Remove corresponding .sh script - Replace calling of .sh with corresponding .py in meson.build Signed-off-by: Jie Zhou --- app/test/has-hugepage.sh | 11 --- app/test/has_hugepage.py | 26 ++ app/test

[PATCH v9 5/9] app/test: skip two logs_autotest cases on Windows

ndef these two test cases, and for the rte_log_set_level_pattern validation case following these two cases, differentiate the expected log level passed into macro CHECK_LEVELS Now logs_autotest completes for all dynamic log types and static log types. Signed-off-by: Jie Zhou --- app/test/test_

[PATCH v9 4/9] app/test: skip interrupt tests on Windows

Even though test_interrupts.c can compile on Windows, skip interrupt tests for now since majority of eal_interrupt on Windows are stubs. Will remove the skip after interrupt being fully enabled on Windows. Signed-off-by: Jie Zhou --- app/test/test_interrupts.c | 10 ++ 1 file changed

[PATCH v9 2/9] app/test: remove POSIX-specific code

netinet/in.h u6_addr and better have a separate patch to add such tests on Windows) Signed-off-by: Jie Zhou --- app/test/commands.c | 2 -- app/test/packet_burst_generator.c | 1 + app/test/process.h| 4 +++- app/test/test.c | 5 - app/test

[PATCH v9 0/9] app/test: enable subset of tests on Windows

From: Jie Zhou This patchset is to enable a subset of unit tests on windows. It mainly includes: - Replace POSIX specific codes - Add test stubs for not supported ones on Windows - Fix some lib and tests per failures investigation - Replace .sh script with .py script for meson.build - Enable

[PATCH v9 3/9] app/test: fix incorrect errno variable

Fix incorrect errno variable used in memory autotest. Use rte_errno instead. Fixes: 086d426406bd ("app/test: fix memory autotests on FreeBSD") Cc: bruce.richard...@intel.com Signed-off-by: Jie Zhou --- app/test/test_memory.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) di

[PATCH v9 1/9] eal/windows: return ENOTSUP for not supported API

. Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- lib/eal/windows/eal_memalloc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/eal/windows/eal_memalloc.c b/lib/eal/windows/eal_memalloc.c index 55d6dcc71c..aa7589b81d 100644 --- a/lib/eal/windows

Re: [PATCH v8 03/11] app/test: replace POSIX specific code

On Wed, Nov 24, 2021 at 01:02:06AM +0300, Dmitry Kozlyuk wrote: > 2021-10-25 19:45 (UTC-0700), Jie Zhou: > > - Include rte_os_shim.h > > - Replace sleep and usleep with rte_delay_us_sleep > > - #ifndef RTE_EXEC_ENV_WINDOWS for POSIX code only > > > > Signed-of

Re: [PATCH v8 04/11] app/test: exclude ENOTSUP as failure

On Wed, Nov 24, 2021 at 01:02:10AM +0300, Dmitry Kozlyuk wrote: > 2021-10-25 19:45 (UTC-0700), Jie Zhou: > > Check rte_errno to exclude ENOTSUP as failures in test_memory.c > > > > Signed-off-by: Jie Zhou > > --- > > app/test/test_memory.c | 3 ++- > &

Re: [PATCH v8 06/11] app/test: temporarily "skip" one cmdline test case

On Wed, Nov 24, 2021 at 01:02:13AM +0300, Dmitry Kozlyuk wrote: > 2021-10-25 19:45 (UTC-0700), Jie Zhou: > > cmdline tests pass except one failure at the test_cmdline_socket_fns > > test case with error: failed to open /dev/null for reading! > > Can't it be somethi

Re: [PATCH v8 10/11] app/test: replace .sh scripts with .py scripts

On Wed, Nov 24, 2021 at 01:15:33AM +0300, Dmitry Kozlyuk wrote: > 2021-10-25 19:46 (UTC-0700), Jie Zhou: > > - Add python script to check if system supports hugepages > > - Remove corresponding .sh scripts > > - Replace calling of .sh with corresponding .py in meson.build &

[dpdk-dev] [PATCH v8 11/11] app/test: enable subset of unit tests on Windows

) which will be enabled by "patch-18949: app/test: enable crypto unit tests on Windows". - For dump tests, currently the tests hang on Windows which require further investigation. Keep the dump test list just for non-Windows for easier tracking. Signed-off-by: Jie Zhou --- app/test/m

[dpdk-dev] [PATCH v8 08/11] app/test: differentiate a strerror on different OS

On Windows, strerror returns just "Unknown error" for errnum greater than MAX_ERRNO, while linux and freebsd returns "Unknown error ", which is the current expectation for errno_autotest. Differentiate the error string on Windows to remove a "duplicate error code" fai

[dpdk-dev] [PATCH v8 07/11] app/test: skip two logs_autotest cases on Windows

ndef these two test cases, and for the rte_log_set_level_pattern validation case following these two cases, differentiate the expected log level passed into macro CHECK_LEVELS Now logs_autotest completes for all dynamic log types and static log types. Signed-off-by: Jie Zhou --- app/test/test_

[dpdk-dev] [PATCH v8 10/11] app/test: replace .sh scripts with .py scripts

- Add python script to check if system supports hugepages - Remove corresponding .sh scripts - Replace calling of .sh with corresponding .py in meson.build Signed-off-by: Jie Zhou --- app/test/has-hugepage.sh | 11 --- app/test/has_hugepage.py | 25 + app/test

[dpdk-dev] [PATCH v8 05/11] app/test: skip interrupt tests on Windows

Even though test_interrupts.c can compile on Windows, skip interrupt tests for now since majority of eal_interrupt on Windows are stubs. Will remove the skip after interrupt being fully enabled on Windows. Signed-off-by: Jie Zhou --- app/test/test_interrupts.c | 10 ++ 1 file changed

[dpdk-dev] [PATCH v8 06/11] app/test: temporarily "skip" one cmdline test case

ned-off-by: Jie Zhou --- app/test/test_cmdline_lib.c | 4 1 file changed, 4 insertions(+) diff --git a/app/test/test_cmdline_lib.c b/app/test/test_cmdline_lib.c index 4715edc341..eb54ef06d7 100644 --- a/app/test/test_cmdline_lib.c +++ b/app/test/test_cmdline_lib.c @@ -14

[dpdk-dev] [PATCH v8 09/11] app/test: remove two alarm_autotest cases

Remove two alarm_autotest test cases which do bogus range check on Windows. Signed-off-by: Jie Zhou --- app/test/test_alarm.c | 4 1 file changed, 4 insertions(+) diff --git a/app/test/test_alarm.c b/app/test/test_alarm.c index b4034339b8..70e97a3109 100644 --- a/app/test/test_alarm.c

[dpdk-dev] [PATCH v8 03/11] app/test: replace POSIX specific code

- Include rte_os_shim.h - Replace sleep and usleep with rte_delay_us_sleep - #ifndef RTE_EXEC_ENV_WINDOWS for POSIX code only Signed-off-by: Jie Zhou --- app/test/packet_burst_generator.c | 1 + app/test/process.h| 4 +++- app/test/test.c | 4 app/test

[dpdk-dev] [PATCH v8 04/11] app/test: exclude ENOTSUP as failure

Check rte_errno to exclude ENOTSUP as failures in test_memory.c Signed-off-by: Jie Zhou --- app/test/test_memory.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/app/test/test_memory.c b/app/test/test_memory.c index dbf6871e71..379b0f99ca 100644 --- a/app/test

[dpdk-dev] [PATCH v8 02/11] app/test: remove unnecessary headers

Remove unnecessary header inclusion Signed-off-by: Jie Zhou --- app/test/commands.c | 2 -- app/test/test.c | 1 - app/test/test_cmdline_lib.c | 1 - 3 files changed, 4 deletions(-) diff --git a/app/test/commands.c b/app/test/commands.c index 76f6ee5d23..82bd8bf474 100644

[dpdk-dev] [PATCH v8 00/11] app/test: enable subset of tests on Windows

e patches after this patch set. V8 changes: - Fix coding style issue of using C99 // comments Jie Zhou (11): eal/windows: return ENOTSUP for not supported API app/test: remove unnecessary headers app/test: replace POSIX specific code app/test: exclude ENOTSUP as failure app/test: ski

[dpdk-dev] [PATCH v8 01/11] eal/windows: return ENOTSUP for not supported API

. Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- lib/eal/windows/eal_memalloc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/eal/windows/eal_memalloc.c b/lib/eal/windows/eal_memalloc.c index 55d6dcc71c..aa7589b81d 100644 --- a/lib/eal/windows

  1   2   3   4   >