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 39B8346BE8; Tue, 22 Jul 2025 16:05:01 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 378C740685; Tue, 22 Jul 2025 16:04:34 +0200 (CEST) Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.14]) by mails.dpdk.org (Postfix) with ESMTP id AA973402DE for ; Tue, 22 Jul 2025 16:04:29 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1753193070; x=1784729070; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=soyt7NdE8agqpvYcfCeG5DgT8k2Qs3F2uAattaMgb28=; b=Zdg79h4NZOBBC4wkwmHsCKLueQ83B6WmTe2Cw74cdRZ0gemhWFKwGv5k r2cW4b/FPU6cAehPbAdG9WnBhEgf8Uu1uec4A0EaUgN0bKrHTrv5VkMzD JdyJmTEqZB3hySlDINxRVoQW30YOqMPjUbqOdw7cFJgaIpC3tGNwcQ/i2 qCAPhmdRDRq0GHNIP76u9wL3csSMpOZz1Em7n82t6y0jlXi2GT2IuyLpj K7gUByrU72Ec/kpP2yycykoUfbNNbr8OrOjOmqUbfEAbIBRsPaXwNXOva S9aYAl4BfyVir14vKlj7ghXN3iWdDpBa9EuhRxlVtEfO9nx+YsbKzSVx0 w==; X-CSE-ConnectionGUID: oSybcfYwTdSt9vQsgCsjeg== X-CSE-MsgGUID: 1dD0H13GTeCKzvrT6+Q0PQ== X-IronPort-AV: E=McAfee;i="6800,10657,11500"; a="59241953" X-IronPort-AV: E=Sophos;i="6.16,331,1744095600"; d="scan'208";a="59241953" Received: from fmviesa001.fm.intel.com ([10.60.135.141]) by orvoesa106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 22 Jul 2025 07:04:02 -0700 X-CSE-ConnectionGUID: cKdQaXAqQF+H62RlEa721Q== X-CSE-MsgGUID: CIz7mQrSSXWNfj35AvY9Gg== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.16,331,1744095600"; d="scan'208";a="190104404" Received: from silpixa00401385.ir.intel.com ([10.237.214.33]) by fmviesa001.fm.intel.com with ESMTP; 22 Jul 2025 07:04:00 -0700 From: Bruce Richardson To: dev@dpdk.org Cc: david.marchand@redhat.com, Bruce Richardson , Tyler Retzlaff Subject: [PATCH v6 4/9] eal: add long options for each short option Date: Tue, 22 Jul 2025 15:03:09 +0100 Message-ID: <20250722140316.3568603-5-bruce.richardson@intel.com> X-Mailer: git-send-email 2.48.1 In-Reply-To: <20250722140316.3568603-1-bruce.richardson@intel.com> References: <20250520164025.2055721-1-bruce.richardson@intel.com> <20250722140316.3568603-1-bruce.richardson@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 To simplify future rework of the EAL arg handling, add a long-option equivalent for each short option that doesn't already have one. When updating the docs with the new long options, standardize the format of options which have both short and long variants, and drop the deprecated service-coremask option from the docs, rather than adding its new long option. Signed-off-by: Bruce Richardson --- doc/guides/linux_gsg/eal_args.include.rst | 20 ++++++++------------ lib/eal/common/eal_common_options.c | 9 +++++++++ lib/eal/common/eal_options.h | 16 ++++++++++++++++ 3 files changed, 33 insertions(+), 12 deletions(-) diff --git a/doc/guides/linux_gsg/eal_args.include.rst b/doc/guides/linux_gsg/eal_args.include.rst index 9ced54af40..0b17879d42 100644 --- a/doc/guides/linux_gsg/eal_args.include.rst +++ b/doc/guides/linux_gsg/eal_args.include.rst @@ -4,7 +4,7 @@ Lcore-related options ~~~~~~~~~~~~~~~~~~~~~ -* ``-l/--lcores `` +* ``-l, --lcores `` List of cores to run on @@ -71,11 +71,7 @@ Lcore-related options Core ID that is used as main. -* ``-s `` - - Hexadecimal bitmask of cores to be used as service cores. - -* ``-S `` +* ``-S, --service-corelist `` List of cores to be used as service cores. @@ -108,7 +104,7 @@ Device-related options --vdev 'net_pcap0,rx_pcap=input.pcap,tx_pcap=output.pcap' -* ``-d `` +* ``-d, --driver-path `` Load external drivers. An argument can be a single shared object file, or a directory containing multiple driver shared objects. Multiple -d options are @@ -134,15 +130,15 @@ Multiprocessing-related options Memory-related options ~~~~~~~~~~~~~~~~~~~~~~ -* ``-n `` +* ``-n, --memory-channels `` Set the number of memory channels to use. -* ``-r `` +* ``-r, --memory-ranks `` Set the number of memory ranks (auto-detected by default). -* ``-m `` +* ``-m, --memory-size `` Amount of memory to preallocate at startup. @@ -236,11 +232,11 @@ Debugging options Other options ~~~~~~~~~~~~~ -* ``-h``, ``--help`` +* ``-h, --help`` Display help message listing all EAL parameters. -* ``-v`` +* ``-v, --version`` Display the version information on startup. diff --git a/lib/eal/common/eal_common_options.c b/lib/eal/common/eal_common_options.c index 3169dd069f..226acea467 100644 --- a/lib/eal/common/eal_common_options.c +++ b/lib/eal/common/eal_common_options.c @@ -66,7 +66,9 @@ eal_short_options[] = const struct option eal_long_options[] = { {OPT_BASE_VIRTADDR, 1, NULL, OPT_BASE_VIRTADDR_NUM }, + {OPT_COREMASK, 1, NULL, OPT_COREMASK_NUM }, {OPT_CREATE_UIO_DEV, 0, NULL, OPT_CREATE_UIO_DEV_NUM }, + {OPT_DRIVER_PATH, 1, NULL, OPT_DRIVER_PATH_NUM }, {OPT_FILE_PREFIX, 1, NULL, OPT_FILE_PREFIX_NUM }, {OPT_HELP, 0, NULL, OPT_HELP_NUM }, {OPT_HUGE_DIR, 1, NULL, OPT_HUGE_DIR_NUM }, @@ -76,6 +78,11 @@ eal_long_options[] = { {OPT_LOG_COLOR, 2, NULL, OPT_LOG_COLOR_NUM }, {OPT_LOG_LEVEL, 1, NULL, OPT_LOG_LEVEL_NUM }, {OPT_LOG_TIMESTAMP, 2, NULL, OPT_LOG_TIMESTAMP_NUM }, + {OPT_MEMORY_CHANNELS, 1, NULL, OPT_MEMORY_CHANNELS_NUM }, + {OPT_MEMORY_RANKS, 1, NULL, OPT_MEMORY_RANKS_NUM }, + {OPT_MEMORY_SIZE, 1, NULL, OPT_MEMORY_SIZE_NUM }, + {OPT_SERVICE_CORELIST, 1, NULL, OPT_SERVICE_CORELIST_NUM }, + {OPT_SERVICE_COREMASK, 1, NULL, OPT_SERVICE_COREMASK_NUM }, {OPT_TRACE, 1, NULL, OPT_TRACE_NUM }, {OPT_TRACE_DIR, 1, NULL, OPT_TRACE_DIR_NUM }, {OPT_TRACE_BUF_SIZE, 1, NULL, OPT_TRACE_BUF_SIZE_NUM }, @@ -109,6 +116,8 @@ eal_long_options[] = { {OPT_NO_TELEMETRY, 0, NULL, OPT_NO_TELEMETRY_NUM }, {OPT_FORCE_MAX_SIMD_BITWIDTH, 1, NULL, OPT_FORCE_MAX_SIMD_BITWIDTH_NUM}, {OPT_HUGE_WORKER_STACK, 2, NULL, OPT_HUGE_WORKER_STACK_NUM }, + {OPT_VERSION, 0, NULL, OPT_VERSION_NUM }, + {0, 0, NULL, 0 } }; diff --git a/lib/eal/common/eal_options.h b/lib/eal/common/eal_options.h index 7a56aa3810..6ef45559f0 100644 --- a/lib/eal/common/eal_options.h +++ b/lib/eal/common/eal_options.h @@ -17,8 +17,24 @@ enum { OPT_DEV_ALLOW_NUM = 'a', #define OPT_DEV_BLOCK "block" OPT_DEV_BLOCK_NUM = 'b', +#define OPT_COREMASK "coremask" + OPT_COREMASK_NUM = 'c', +#define OPT_DRIVER_PATH "driver-path" + OPT_DRIVER_PATH_NUM = 'd', #define OPT_LCORES "lcores" OPT_LCORES_NUM = 'l', +#define OPT_MEMORY_SIZE "memory-size" + OPT_MEMORY_SIZE_NUM = 'm', +#define OPT_MEMORY_CHANNELS "memory-channels" + OPT_MEMORY_CHANNELS_NUM = 'n', +#define OPT_MEMORY_RANKS "memory-ranks" + OPT_MEMORY_RANKS_NUM = 'r', +#define OPT_SERVICE_COREMASK "service-coremask" + OPT_SERVICE_COREMASK_NUM = 's', +#define OPT_SERVICE_CORELIST "service-corelist" + OPT_SERVICE_CORELIST_NUM = 'S', +#define OPT_VERSION "version" + OPT_VERSION_NUM = 'v', /* first long only option value must be >= 256, so that we won't * conflict with short options */ -- 2.48.1