Cover Detail
Show a cover letter.
GET /api/covers/104833/?format=api
{ "id": 104833, "url": "http://patchwork.dpdk.org/api/covers/104833/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/cover/1638490007-9939-1-git-send-email-jizh@linux.microsoft.com/", "project": { "id": 1, "url": "http://patchwork.dpdk.org/api/projects/1/?format=api", "name": "DPDK", "link_name": "dpdk", "list_id": "dev.dpdk.org", "list_email": "dev@dpdk.org", "web_url": "http://core.dpdk.org", "scm_url": "git://dpdk.org/dpdk", "webscm_url": "http://git.dpdk.org/dpdk", "list_archive_url": "https://inbox.dpdk.org/dev", "list_archive_url_format": "https://inbox.dpdk.org/dev/{}", "commit_url_format": "" }, "msgid": "<1638490007-9939-1-git-send-email-jizh@linux.microsoft.com>", "list_archive_url": "https://inbox.dpdk.org/dev/1638490007-9939-1-git-send-email-jizh@linux.microsoft.com", "date": "2021-12-03T00:06:38", "name": "[v11,0/9] app/test: enable subset of tests on Windows", "submitter": { "id": 1934, "url": "http://patchwork.dpdk.org/api/people/1934/?format=api", "name": "Jie Zhou", "email": "jizh@linux.microsoft.com" }, "mbox": "http://patchwork.dpdk.org/project/dpdk/cover/1638490007-9939-1-git-send-email-jizh@linux.microsoft.com/mbox/", "series": [ { "id": 20842, "url": "http://patchwork.dpdk.org/api/series/20842/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/list/?series=20842", "date": "2021-12-03T00:06:39", "name": "app/test: enable subset of tests on Windows", "version": 11, "mbox": "http://patchwork.dpdk.org/series/20842/mbox/" } ], "comments": "http://patchwork.dpdk.org/api/covers/104833/comments/", "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@inbox.dpdk.org", "Delivered-To": "patchwork@inbox.dpdk.org", "Received": [ "from mails.dpdk.org (mails.dpdk.org [217.70.189.124])\n\tby inbox.dpdk.org (Postfix) with ESMTP id 5B393A0548;\n\tFri, 3 Dec 2021 01:07:02 +0100 (CET)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id EE1F3426F5;\n\tFri, 3 Dec 2021 01:06:56 +0100 (CET)", "from linux.microsoft.com (linux.microsoft.com [13.77.154.182])\n by mails.dpdk.org (Postfix) with ESMTP id 2DC1940041\n for <dev@dpdk.org>; Fri, 3 Dec 2021 01:06:54 +0100 (CET)", "from\n linuxonhyperv3.guj3yctzbm1etfxqx2vob5hsef.xx.internal.cloudapp.net\n (linux.microsoft.com [13.77.154.182])\n by linux.microsoft.com (Postfix) with ESMTPSA id 6DD7A20E67AD;\n Thu, 2 Dec 2021 16:06:53 -0800 (PST)" ], "DKIM-Filter": "OpenDKIM Filter v2.11.0 linux.microsoft.com 6DD7A20E67AD", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.microsoft.com;\n s=default; t=1638490013;\n bh=TQrU6C+//Um4BgugFPIj1JxhBrqwRUCf+wQYZLx/cxA=;\n h=From:To:Cc:Subject:Date:In-Reply-To:References:From;\n b=F0Lj7mxbkv2Xj46KZaCwnvpeFMZvzNVUu2rnmcFoHwgy/HBcNnRSh/RxYNTvS2DZC\n DyfclsS8F5pReHXl0yt6F7RzQsJNYo3eiFoFGPwLkKiirXmvJLivXflQhYY9XXmfMD\n goItFlrKaVKBdinYoe2IwYmAXqdNS8oTwCedpxDM=", "From": "Jie Zhou <jizh@linux.microsoft.com>", "To": "dev@dpdk.org", "Cc": "dmitry.kozliuk@gmail.com, roretzla@microsoft.com,\n navasile@linux.microsoft.com, dmitrym@microsoft.com,\n pallavi.kadam@intel.com, talshn@nvidia.com, thomas@monjalon.net,\n aconole@redhat.com", "Subject": "[PATCH v11 0/9] app/test: enable subset of tests on Windows", "Date": "Thu, 2 Dec 2021 16:06:38 -0800", "Message-Id": "<1638490007-9939-1-git-send-email-jizh@linux.microsoft.com>", "X-Mailer": "git-send-email 1.8.3.1", "In-Reply-To": "<1638384245-12587-1-git-send-email-jizh@linux.microsoft.com>", "References": "<1638384245-12587-1-git-send-email-jizh@linux.microsoft.com>", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.29", "Precedence": "list", "List-Id": "DPDK patches and discussions <dev.dpdk.org>", "List-Unsubscribe": "<https://mails.dpdk.org/options/dev>,\n <mailto:dev-request@dpdk.org?subject=unsubscribe>", "List-Archive": "<http://mails.dpdk.org/archives/dev/>", "List-Post": "<mailto:dev@dpdk.org>", "List-Help": "<mailto:dev-request@dpdk.org?subject=help>", "List-Subscribe": "<https://mails.dpdk.org/listinfo/dev>,\n <mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org" }, "content": "This patchset is to enable a subset of unit tests on windows. It mainly includes:\n- Replace POSIX specific codes\n- Add test stubs for not supported ones on Windows\n- Fix some lib and tests per failures investigation\n- Replace .sh script with .py script for meson.build\n- Enable build and run subset of unit tests on Windows\n\nFuture work:\n- Work with CI/CD lab to onboard unit tests on Windows to catch regression\n- Enable more tests\n\n---\nV2 changes:\n - Fix compilation error on FreeBSD\n - Fix email mismatch issue\n - Add a missing space around \"*\"\n\n---\nV3 changes:\n - Fix a misc c coding style issue\n - Revise some commit title and message body\n - Fix violations of PEP8 in new added Python scripts\n - Add error handling in get_coremask.py\n - Fix has_hugepage.py to check system support of hugepages\n instead of checking privileges\n - Fix test meson.build to run Python scripts using py3\n - Consolidate lists of source files, test dep, etc. across all \n platforms, with conditional extending on some platform(s)\n\n---\nV4 changes:\n - Remove building of ip_frag, rib, and reorder libraries on Windows.\n These three libs usually can be built on Windows without change.\n However, in between the time of V3 and V4, there is regression in\n upstream caused build failures of these three libs. Will separately\n investigate and enable these libraries.\n\n - Remove previous patch#2 (Enable mempool/stack on Windows) from this\n patchset as it was separated out and merged as patch-19314.\n\n - Consolidate the source files, deps, tests lists across platforms as\n much as possible.\n\n---\nV5 changes:\n - Remove a space between function name and open parenthesis '('\n - Add back a header mistakenly deleted\n\n---\nV6 changes:\n - Fix inconsistent static vs. non-static declarations\n\n---\nV7 changes:\n - Remove get_coremask.py as it is not needed any more in meson.build\n - Remove enablement of efd and lpm and their corresponding unit tests.\n The enablement of these two libs and their UTs will be in separate\n patches after this patch set.\n\nV8 changes:\n - Fix coding style issue of using C99 // comments\n\n---\nV9 changes:\n - Fix has_hugepage.py with adding failure handling on Linux, using\n proper variable name to follow Python convention, and removing\n unnecessary comment.\n - Enable previously skipped test_cmdline_socket_fns test cases\n - Revise title and message, and add Fixes info for current Patch#3 \n - Combine 2 patches (previous #2 and #3 in V8) into one and with \n more detailed message\n\n---\nV10 changes:\n - Fix indentation\n\n---\nV11 changes:\n - Remove mandatory dependency on bitratestats, latencystats,\n and metrics libs in test meson.build, which was reintroduced\n at rebase in V9.\n \n\nJie Zhou (9):\n eal/windows: return ENOTSUP for not supported API\n app/test: remove POSIX-specific code\n app/test: fix incorrect errno variable\n app/test: skip interrupt tests on Windows\n app/test: skip two logs_autotest cases on Windows\n app/test: differentiate a strerror on different OS\n app/test: remove two alarm_autotest cases\n app/test: replace .sh script with .py script\n app/test: enable subset of unit tests on Windows\n\n app/test/commands.c | 2 -\n app/test/has-hugepage.sh | 11 ---\n app/test/has_hugepage.py | 26 +++++\n app/test/meson.build | 118 ++++++++++++-----------\n app/test/packet_burst_generator.c | 1 +\n app/test/process.h | 4 +-\n app/test/test.c | 5 +-\n app/test/test_acl.c | 12 +++\n app/test/test_alarm.c | 4 +\n app/test/test_bpf.c | 15 ++-\n app/test/test_byteorder.c | 2 +-\n app/test/test_cksum.c | 12 +++\n app/test/test_cmdline.c | 2 +\n app/test/test_cmdline_ipaddr.c | 5 +\n app/test/test_cmdline_lib.c | 13 ++-\n app/test/test_crc.c | 1 -\n app/test/test_cryptodev.c | 4 +\n app/test/test_cryptodev_asym.c | 4 +\n app/test/test_cryptodev_blockcipher.c | 4 +\n app/test/test_cryptodev_security_ipsec.c | 4 +\n app/test/test_cryptodev_security_pdcp.c | 4 +\n app/test/test_debug.c | 17 +++-\n app/test/test_distributor.c | 13 +++\n app/test/test_distributor_perf.c | 13 +++\n app/test/test_dmadev.c | 14 ++-\n app/test/test_dmadev_api.c | 4 +\n app/test/test_eal_flags.c | 90 +++++++++++++++++\n app/test/test_eal_fs.c | 12 +++\n app/test/test_efd.c | 15 ++-\n app/test/test_efd_perf.c | 16 ++-\n app/test/test_errno.c | 12 ++-\n app/test/test_event_crypto_adapter.c | 15 ++-\n app/test/test_event_eth_rx_adapter.c | 25 ++++-\n app/test/test_event_eth_tx_adapter.c | 12 +++\n app/test/test_event_ring.c | 16 ++-\n app/test/test_event_timer_adapter.c | 16 ++-\n app/test/test_eventdev.c | 20 +++-\n app/test/test_external_mem.c | 18 +++-\n app/test/test_fib.c | 22 ++++-\n app/test/test_fib6.c | 24 ++++-\n app/test/test_fib6_perf.c | 16 ++-\n app/test/test_fib_perf.c | 15 ++-\n app/test/test_flow_classify.c | 13 +++\n app/test/test_func_reentrancy.c | 12 +++\n app/test/test_graph.c | 18 +++-\n app/test/test_graph_perf.c | 16 ++-\n app/test/test_hash_perf.c | 12 +++\n app/test/test_interrupts.c | 10 ++\n app/test/test_ipfrag.c | 16 ++-\n app/test/test_ipsec.c | 15 ++-\n app/test/test_ipsec_perf.c | 15 ++-\n app/test/test_ipsec_sad.c | 14 ++-\n app/test/test_kni.c | 10 +-\n app/test/test_lcores.c | 12 +++\n app/test/test_logs.c | 6 +-\n app/test/test_lpm.c | 14 ++-\n app/test/test_lpm6.c | 14 ++-\n app/test/test_lpm6_perf.c | 14 ++-\n app/test/test_lpm_perf.c | 13 ++-\n app/test/test_malloc.c | 17 +++-\n app/test/test_mbuf.c | 13 ++-\n app/test/test_member.c | 16 ++-\n app/test/test_member_perf.c | 16 ++-\n app/test/test_memcpy_perf.c | 18 +++-\n app/test/test_memory.c | 2 +-\n app/test/test_mempool_perf.c | 12 +++\n app/test/test_mp_secondary.c | 12 +++\n app/test/test_pie.c | 30 +++++-\n app/test/test_pmd_perf.c | 6 +-\n app/test/test_rawdev.c | 17 +++-\n app/test/test_rcu_qsbr_perf.c | 12 +++\n app/test/test_reciprocal_division.c | 12 +++\n app/test/test_reciprocal_division_perf.c | 12 +++\n app/test/test_red.c | 29 +++++-\n app/test/test_reorder.c | 15 ++-\n app/test/test_rib.c | 22 ++++-\n app/test/test_rib6.c | 22 ++++-\n app/test/test_ring_stress_impl.h | 2 +-\n app/test/test_sched.c | 14 ++-\n app/test/test_security.c | 16 ++-\n app/test/test_table.c | 13 +++\n app/test/test_table_acl.c | 3 +\n app/test/test_table_combined.c | 4 +\n app/test/test_table_pipeline.c | 4 +\n app/test/test_table_ports.c | 4 +\n app/test/test_table_tables.c | 4 +\n app/test/test_telemetry_data.c | 2 +\n app/test/test_timer_secondary.c | 13 +++\n app/test/test_trace.c | 32 +++++-\n lib/eal/common/eal_common_errno.c | 4 +\n lib/eal/windows/eal_memalloc.c | 6 +-\n 91 files changed, 1130 insertions(+), 156 deletions(-)\n delete mode 100755 app/test/has-hugepage.sh\n create mode 100644 app/test/has_hugepage.py" }