From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 7A9824264C; Wed, 27 Sep 2023 05:52:24 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 10D0E4028C; Wed, 27 Sep 2023 05:52:24 +0200 (CEST) Received: from mgamail.intel.com (mgamail.intel.com [192.55.52.88]) by mails.dpdk.org (Postfix) with ESMTP id B7F8740277; Wed, 27 Sep 2023 05:52:21 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1695786742; x=1727322742; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=BiDibJIElfZ0J8cnXEc/QVm2pTmgOxI07EhuRkh8FHc=; b=O9YJjiaN3Z2r/MydyiZKn++bF/IkaEjaJJ6xJcsoyE6n7MgDQMIHRuAI I4soJ9uNOVkp9hBrM8qiRNRnoiuL19vz7Jte7ng7SnES0pYtyCST0Cwcw fRAz8iwvPhxeStpUXF2N6T7OIjOAfwBl6CURAHotxvtn8S8V0w7gUAziy VE2dZ9dIqHlcQ12l9f/G3ZbcffzV+XrjWpYMMxxRSNmlZlJ//428Qndxk fYEUj57FEdeA/uI/zn650+SUdufgiYlqiQZj6oicqvJcNslkKwdYxyzZb jI5IcbQAyHCIuyN0CIPAIPGYZnbVFWJYxBfOFenDSleEIUAqzdj2oeD+K Q==; X-IronPort-AV: E=McAfee;i="6600,9927,10845"; a="412637936" X-IronPort-AV: E=Sophos;i="6.03,179,1694761200"; d="scan'208";a="412637936" Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by fmsmga101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Sep 2023 20:52:20 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10845"; a="996021903" X-IronPort-AV: E=Sophos;i="6.03,179,1694761200"; d="scan'208";a="996021903" Received: from unknown (HELO localhost.localdomain) ([10.239.252.253]) by fmsmga006-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Sep 2023 20:52:18 -0700 From: Mingjin Ye To: dev@dpdk.org Cc: qiming.yang@intel.com, yidingx.zhou@intel.com, Mingjin Ye , stable@dpdk.org Subject: [PATCH v4] app/test: append 'allow' parameters to secondary processes Date: Wed, 27 Sep 2023 03:42:04 +0000 Message-Id: <20230927034204.2279012-1-mingjinx.ye@intel.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20230925094240.2122544-1-mingjinx.ye@intel.com> References: <20230925094240.2122544-1-mingjinx.ye@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit 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 The 'allow' parameters are not passed to the secondary process, and all devices will be loaded by default. When the primary process specifies the 'allow' parameters and does not specify all devices that use the vfio-pci driver, the secondary process will not load the devices as expected, which will return incorrect test results. This patch fixes this issue by appending the 'allow' parameters to the secondary process. Fixes: 086eb64db39e ("test/pdump: add unit test for pdump library") Fixes: 148f963fb532 ("xen: core library changes") Fixes: af75078fece3 ("first public release") Fixes: b8d5e544e73e ("test: add procfs error message for multi-process launch") Cc: stable@dpdk.org Signed-off-by: Mingjin Ye --- v4: Resolve patch conflicts and optimize code. --- app/test/process.h | 60 +++++++++++++++++++++++++++++++++++++++++++--- 1 file changed, 57 insertions(+), 3 deletions(-) diff --git a/app/test/process.h b/app/test/process.h index af7bc3e0de..1cdf28752e 100644 --- a/app/test/process.h +++ b/app/test/process.h @@ -18,6 +18,8 @@ #include /* strlcpy */ +#include + #ifdef RTE_EXEC_ENV_FREEBSD #define self "curproc" #define exe "file" @@ -34,6 +36,47 @@ extern uint16_t flag_for_send_pkts; #endif #endif +#define PREFIX_ALLOW "--allow" + +static int +add_parameter_allow(char **argv, int max_capacity) +{ + struct rte_devargs *devargs; + int count = 0; + int name_length, data_length; + char *dev; + int malloc_size; + + RTE_EAL_DEVARGS_FOREACH(NULL, devargs) { + if (count >= max_capacity) + return count; + + name_length = strlen(devargs->name); + if (name_length == 0) + continue; + + if (devargs->data != NULL) + data_length = strlen(devargs->data); + else + data_length = 0; + + malloc_size = name_length + data_length + 1; + dev = malloc(malloc_size); + + memcpy(dev, devargs->name, name_length); + if (data_length > 0) + memcpy(dev + name_length, devargs->data, data_length); + memset(dev + malloc_size - 1, 0, 1); + + *(argv + count) = strdup(PREFIX_ALLOW); + count++; + + *(argv + count) = dev; + count++; + } + return count; +} + /* * launches a second copy of the test process using the given argv parameters, * which should include argv[0] as the process name. To identify in the @@ -44,7 +87,9 @@ static inline int process_dup(const char *const argv[], int numargs, const char *env_value) { int num; - char *argv_cpy[numargs + 1]; + char **argv_cpy; + unsigned int allow_num; + unsigned int argv_num; int i, status; char path[32]; #ifdef RTE_LIB_PDUMP @@ -58,12 +103,15 @@ process_dup(const char *const argv[], int numargs, const char *env_value) if (pid < 0) return -1; else if (pid == 0) { + allow_num = rte_devargs_type_count(RTE_DEVTYPE_ALLOWED); + argv_num = numargs + allow_num * 2 + 1; + argv_cpy = malloc(argv_num * sizeof(char *)); /* make a copy of the arguments to be passed to exec */ for (i = 0; i < numargs; i++) argv_cpy[i] = strdup(argv[i]); - argv_cpy[i] = NULL; num = numargs; - + num += add_parameter_allow(&argv_cpy[i], allow_num * 2); + argv_cpy[argv_num - 1] = NULL; #ifdef RTE_EXEC_ENV_LINUX { const char *procdir = "/proc/" self "/fd/"; @@ -131,6 +179,12 @@ process_dup(const char *const argv[], int numargs, const char *env_value) } rte_panic("Cannot exec: %s\n", strerror(errno)); } + + for (i = 0; i < num; i++) { + if (argv_cpy[i] != NULL) + free(argv_cpy[i]); + } + free(argv_cpy); } /* parent process does a wait */ #ifdef RTE_LIB_PDUMP -- 2.25.1