From patchwork Fri Dec 15 13:06:56 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Bruce Richardson X-Patchwork-Id: 135229 X-Patchwork-Delegate: thomas@monjalon.net Return-Path: 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]) by inbox.dpdk.org (Postfix) with ESMTP id 13308436FE; Fri, 15 Dec 2023 14:07:11 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 920B94326E; Fri, 15 Dec 2023 14:07:10 +0100 (CET) Received: from mgamail.intel.com (mgamail.intel.com [134.134.136.126]) by mails.dpdk.org (Postfix) with ESMTP id 4A1CE40299 for ; Fri, 15 Dec 2023 14:07:09 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1702645629; x=1734181629; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=1OgiObRELLhyYm61k7H/bJ6LJUSdQSBNX+FdHMYQnIk=; b=m9Um2hpUmWcHWSZKCNIoyc1oxukd55UdwEPvYp2NrwFbQ+gpcAQzs3fa N8ur14Aq1LKyZWlHrHjVxDa9SuM04p8207g7ZRHGCA06AKrhdWp+lEeoB zAv9P09mTqrDI880iWnAt09ZAwyWxbruOMkjGGIs4dT4dwWPIVboBDjD/ aWKLM9rydA+DbR/TxagLzJXQ10aupWxGE58hFeMmQiTOuz2+lOosfXBOz e3Y3C4RBXo+Vfu2niYvVUC6uuohSNrhQjEshtpjL2NsCHzxxsVLFr5dM5 hFQhkCZsm2F674cv9EsYynVbQBioagU70I3bCmvimOI8d0CyrIRUjXhaL A==; X-IronPort-AV: E=McAfee;i="6600,9927,10924"; a="380268495" X-IronPort-AV: E=Sophos;i="6.04,278,1695711600"; d="scan'208";a="380268495" Received: from orviesa001.jf.intel.com ([10.64.159.141]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 15 Dec 2023 05:07:07 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.04,278,1695711600"; d="scan'208";a="22833213" Received: from unknown (HELO silpixa00401385.ir.intel.com) ([10.237.214.152]) by orviesa001.jf.intel.com with ESMTP; 15 Dec 2023 05:07:06 -0800 From: Bruce Richardson To: dev@dpdk.org Cc: Bruce Richardson Subject: [RFC PATCH] app/test: allow passing a parameter string to autotests Date: Fri, 15 Dec 2023 13:06:56 +0000 Message-Id: <20231215130656.247582-1-bruce.richardson@intel.com> X-Mailer: git-send-email 2.40.1 MIME-Version: 1.0 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sometimes it can be nice to have autotests which can take a parameter, or can be tweaked in some ways, e.g. adjust the number of iterations, or the burst size used in the test. Currently there is no way to do so - all test parameters are hardcoded, which makes sense for a generic regression test to be run quickly, but is a bit lacking for those looking to use unit tests for investigations. To that end, we can all the commandline to accept an additional string parameter after each autotest name, and make that available via global variable. Any test which wishes to make use of this can do so, to allow overriding test defaults. Signed-off-by: Bruce Richardson Acked-by: Morten Brørup --- app/test/commands.c | 33 ++++++++++++++++++++++++++- app/test/test.h | 2 ++ doc/guides/contributing/unit_test.rst | 15 ++++++++++++ 3 files changed, 49 insertions(+), 1 deletion(-) diff --git a/app/test/commands.c b/app/test/commands.c index 497d8e9952..7672e9db6b 100644 --- a/app/test/commands.c +++ b/app/test/commands.c @@ -40,6 +40,8 @@ #include "test.h" +char *test_parameter = NULL; + /****************/ static struct test_commands_list commands_list = @@ -55,14 +57,25 @@ struct cmd_autotest_result { cmdline_fixed_string_t autotest; }; +struct cmd_autotest_param_result { + cmdline_fixed_string_t autotest; + cmdline_fixed_string_t param; +}; + static void cmd_autotest_parsed(void *parsed_result, __rte_unused struct cmdline *cl, - __rte_unused void *data) + void *data) { struct test_command *t; struct cmd_autotest_result *res = parsed_result; int ret = 0; + test_parameter = NULL; + if (data == (void *)1) { + struct cmd_autotest_param_result *pres = parsed_result; + test_parameter = pres->param; + } + TAILQ_FOREACH(t, &commands_list, next) { if (!strcmp(res->autotest, t->command)) ret = t->callback(); @@ -92,6 +105,22 @@ cmdline_parse_inst_t cmd_autotest = { }, }; +cmdline_parse_token_string_t cmd_autotest_param_autotest = + TOKEN_STRING_INITIALIZER(struct cmd_autotest_param_result, autotest, ""); +cmdline_parse_token_string_t cmd_autotest_param_param = + TOKEN_STRING_INITIALIZER(struct cmd_autotest_param_result, param, NULL); + +cmdline_parse_inst_t cmd_autotest_with_param = { + .f = cmd_autotest_parsed, + .data = (void *)1, + .help_str = "launch autotest, providing a string parameter", + .tokens = { + (void *)&cmd_autotest_param_autotest, + (void *)&cmd_autotest_param_param, + NULL, + }, +}; + /****************/ struct cmd_dump_result { @@ -348,6 +377,7 @@ cmdline_parse_inst_t cmd_set_rxtx_sc = { cmdline_parse_ctx_t main_ctx[] = { (cmdline_parse_inst_t *)&cmd_autotest, + (cmdline_parse_inst_t *)&cmd_autotest_with_param, (cmdline_parse_inst_t *)&cmd_dump, (cmdline_parse_inst_t *)&cmd_dump_one, (cmdline_parse_inst_t *)&cmd_quit, @@ -378,5 +408,6 @@ int commands_init(void) } cmd_autotest_autotest.string_data.str = commands; + cmd_autotest_param_autotest.string_data.str = commands; return 0; } diff --git a/app/test/test.h b/app/test/test.h index 15e23d297f..3910317027 100644 --- a/app/test/test.h +++ b/app/test/test.h @@ -27,6 +27,8 @@ #include +extern char *test_parameter; + #define TEST_ASSERT RTE_TEST_ASSERT #define TEST_ASSERT_EQUAL RTE_TEST_ASSERT_EQUAL diff --git a/doc/guides/contributing/unit_test.rst b/doc/guides/contributing/unit_test.rst index 063cefa192..2ce64e2db7 100644 --- a/doc/guides/contributing/unit_test.rst +++ b/doc/guides/contributing/unit_test.rst @@ -374,6 +374,21 @@ Example:: ... +Optional test parameters +------------------------- + +Each unit test suite must be runable without any additional test parameters, +since automated testing via ``meson test``, and many DPDK Continuous Integration(CI) frameworks, +will just call each test via cmdline-parameter to the ``dpdk-test`` binary +or by setting the test name in the ``DPDK_TEST`` environment variable. + +However, when run interactively via ``dpdk-test`` CLI, +an additional parameter may be passed to the autotest command. +This parameter will be available to the test in the ``test_parameter`` global variable. +This support is intended for developer use only as a convenience. +It is up to the each test how to use this parameter and what form it may have. + + Checking code coverage ----------------------