Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/129316/?format=api
{ "id": 129316, "url": "http://patchwork.dpdk.org/api/patches/129316/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/patch/20230705224805.286156-4-stephen@networkplumber.org/", "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": "<20230705224805.286156-4-stephen@networkplumber.org>", "list_archive_url": "https://inbox.dpdk.org/dev/20230705224805.286156-4-stephen@networkplumber.org", "date": "2023-07-05T22:48:03", "name": "[v7,3/5] eal: unify logging code", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "1d1784b2583cfed657bb9741eaf3d91915b58497", "submitter": { "id": 27, "url": "http://patchwork.dpdk.org/api/people/27/?format=api", "name": "Stephen Hemminger", "email": "stephen@networkplumber.org" }, "delegate": { "id": 1, "url": "http://patchwork.dpdk.org/api/users/1/?format=api", "username": "tmonjalo", "first_name": "Thomas", "last_name": "Monjalon", "email": "thomas@monjalon.net" }, "mbox": "http://patchwork.dpdk.org/project/dpdk/patch/20230705224805.286156-4-stephen@networkplumber.org/mbox/", "series": [ { "id": 28845, "url": "http://patchwork.dpdk.org/api/series/28845/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/list/?series=28845", "date": "2023-07-05T22:48:00", "name": "Logging timetamp and related patches", "version": 7, "mbox": "http://patchwork.dpdk.org/series/28845/mbox/" } ], "comments": "http://patchwork.dpdk.org/api/patches/129316/comments/", "check": "success", "checks": "http://patchwork.dpdk.org/api/patches/129316/checks/", "tags": {}, "related": [], "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 E1BDF42DDE;\n\tThu, 6 Jul 2023 00:48:27 +0200 (CEST)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 5784242D4B;\n\tThu, 6 Jul 2023 00:48:15 +0200 (CEST)", "from mail-pj1-f42.google.com (mail-pj1-f42.google.com\n [209.85.216.42]) by mails.dpdk.org (Postfix) with ESMTP id BFC0142D50\n for <dev@dpdk.org>; Thu, 6 Jul 2023 00:48:12 +0200 (CEST)", "by mail-pj1-f42.google.com with SMTP id\n 98e67ed59e1d1-262e5e71978so33762a91.1\n for <dev@dpdk.org>; Wed, 05 Jul 2023 15:48:12 -0700 (PDT)", "from hermes.local (204-195-120-218.wavecable.com. [204.195.120.218])\n by smtp.gmail.com with ESMTPSA id\n bb16-20020a17090b009000b002630bfd35b0sm1906044pjb.7.2023.07.05.15.48.10\n (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n Wed, 05 Jul 2023 15:48:11 -0700 (PDT)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=networkplumber-org.20221208.gappssmtp.com; s=20221208; t=1688597292;\n x=1691189292;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:from:to:cc:subject:date\n :message-id:reply-to;\n bh=2oKzsvdvjYPQihIdA7H/3xJp7JRwySRFEKzx+/Tx6+Y=;\n b=G1vTiB1SviY15PMAaYfvNNolkSRvuIVgJA3XSS24ZFNaW2e4h2hebvPObNOEMTQk/Y\n S+ah6csEHViCgvdlpHvdZbN6u7i4pAK7r2QAzPA2+K+t6CxfIBmFvioSx6wCCJx47CSz\n MoMuLiA7XGETUgSrkEtpEk/7v2ThVWMnubaVvdBehN4AdIVwyUTEwV8D3I/8FuDFY8Rf\n wDd217oyV9M/pB6UF39P6Ekd1j5JH7vLyZsEwewOWSckyQJe3m4X83Hs/z0S06Kt4NEI\n m3RfB/jbcilCf88mWcpLcTN/+G/K4fX9SPP5wim7Aj2J5Pp2GwZC4BzbK1UwPuomPbvV\n wK6A==", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20221208; t=1688597292; x=1691189292;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc\n :subject:date:message-id:reply-to;\n bh=2oKzsvdvjYPQihIdA7H/3xJp7JRwySRFEKzx+/Tx6+Y=;\n b=dOOxvyW0aiYWM3bVf8vSXxLR6aDRIecPqC/Zhy4Hs43ps67O7w+KYlI0aSk0c8Vbjp\n HWARLurPx3AYVbV+8xKnn+/+MzIDk0ZsNeB3r/2OQZALxH17Ve61m0AF9bTWkdP6mQYY\n dTzpgo96bQBu9tHgxcSUC+g4Qhd2yoYJavGabyQL7sRYA9MOi3pUaBOnJJZJjef39yPk\n mWvyoFwfzSF8A21ST9GRniJBCqsxNFiD3/qd+hLeCHulnxwFosDEK4HtvRnicM9BpSMF\n PZuolukwg3Imr56928+dGpVE/iF2BSokBf2SYq/S0LXH2OG/T84UP6sEk9LeR04GTFAF\n 3XZQ==", "X-Gm-Message-State": "ABy/qLaoEamUU+5LT6Qh9VmYo1gSxax6SB2DuWhhN2OuKjtjxlxwHeNj\n SgRV0X+rWdf3EWQMg3PIX132a51Yuh91kVC/PEo=", "X-Google-Smtp-Source": "\n APBJJlEik641ndqawo+Uqb/cF8Zcq0UcNufUmNqWSKt8SgIyE9tIR5roHm6L28Ce5g4uQsmihCLF6A==", "X-Received": "by 2002:a17:90a:8b07:b0:263:1e6c:16f4 with SMTP id\n y7-20020a17090a8b0700b002631e6c16f4mr5127145pjn.20.1688597291647;\n Wed, 05 Jul 2023 15:48:11 -0700 (PDT)", "From": "Stephen Hemminger <stephen@networkplumber.org>", "To": "dev@dpdk.org", "Cc": "Stephen Hemminger <stephen@networkplumber.org>,\n Bruce Richardson <bruce.richardson@intel.com>,\n Dmitry Kozlyuk <dmitry.kozliuk@gmail.com>,\n Narcisa Ana Maria Vasile <navasile@linux.microsoft.com>,\n Dmitry Malloy <dmitrym@microsoft.com>,\n Pallavi Kadam <pallavi.kadam@intel.com>", "Subject": "[PATCH v7 3/5] eal: unify logging code", "Date": "Wed, 5 Jul 2023 15:48:03 -0700", "Message-Id": "<20230705224805.286156-4-stephen@networkplumber.org>", "X-Mailer": "git-send-email 2.39.2", "In-Reply-To": "<20230705224805.286156-1-stephen@networkplumber.org>", "References": "<20200814173441.23086-1-stephen@networkplumber.org>\n <20230705224805.286156-1-stephen@networkplumber.org>", "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 <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": "FreeBSD logging code was not using syslog and did not have\nthe same options as Linux. Use a common set of functions for that.\n\nPre-parsing for log level is common to all OS's.\n\nNow the pre-scan can return an error.\nA bad argument give to --log-level option was given the\ncode would keep going.\n\nUse opterr to fix duplicate error message on bad option.\n\nSigned-off-by: Stephen Hemminger <stephen@networkplumber.org>\n---\n .../freebsd_gsg/freebsd_eal_parameters.rst | 27 +++++++++\n lib/eal/common/eal_common_log.c | 49 +++++++++++++++++\n lib/eal/common/eal_log.h | 5 ++\n lib/eal/freebsd/eal.c | 55 ++++---------------\n lib/eal/linux/eal.c | 46 ++--------------\n lib/eal/linux/meson.build | 1 -\n lib/eal/{linux => unix}/eal_log.c | 11 ++++\n lib/eal/unix/meson.build | 1 +\n lib/eal/windows/eal.c | 36 +-----------\n 9 files changed, 112 insertions(+), 119 deletions(-)\n rename lib/eal/{linux => unix}/eal_log.c (76%)", "diff": "diff --git a/doc/guides/freebsd_gsg/freebsd_eal_parameters.rst b/doc/guides/freebsd_gsg/freebsd_eal_parameters.rst\nindex fba467a2ce92..9270d9fa3bfc 100644\n--- a/doc/guides/freebsd_gsg/freebsd_eal_parameters.rst\n+++ b/doc/guides/freebsd_gsg/freebsd_eal_parameters.rst\n@@ -18,3 +18,30 @@ FreeBSD-specific EAL parameters\n -------------------------------\n \n There are currently no FreeBSD-specific EAL command-line parameters available.\n+\n+Other options\n+~~~~~~~~~~~~~\n+\n+* ``--syslog <syslog facility>``\n+\n+ Set syslog facility. Valid syslog facilities are::\n+\n+ auth\n+ cron\n+ daemon\n+ ftp\n+ kern\n+ lpr\n+ mail\n+ news\n+ syslog\n+ user\n+ uucp\n+ local0\n+ local1\n+ local2\n+ local3\n+ local4\n+ local5\n+ local6\n+ local7\ndiff --git a/lib/eal/common/eal_common_log.c b/lib/eal/common/eal_common_log.c\nindex bd7b188ceb4a..1b2080e8ef04 100644\n--- a/lib/eal/common/eal_common_log.c\n+++ b/lib/eal/common/eal_common_log.c\n@@ -2,6 +2,7 @@\n * Copyright(c) 2010-2014 Intel Corporation\n */\n \n+#include <getopt.h>\n #include <stdio.h>\n #include <stdint.h>\n #include <stdarg.h>\n@@ -16,6 +17,8 @@\n #include <rte_per_lcore.h>\n \n #include \"eal_log.h\"\n+#include \"eal_internal_cfg.h\"\n+#include \"eal_options.h\"\n #include \"eal_private.h\"\n \n struct rte_log_dynamic_type {\n@@ -223,6 +226,52 @@ log_save_level(uint32_t priority, const char *regex, const char *pattern)\n \treturn -1;\n }\n \n+\n+/* Parse the all arguments looking for --log-level */\n+int\n+eal_log_level_parse(int argc, char * const argv[])\n+{\n+\tstruct internal_config *internal_conf = eal_get_internal_configuration();\n+\tint option_index, opt;\n+\tconst int old_optind = optind;\n+\tconst int old_optopt = optopt;\n+\tconst int old_opterr = opterr;\n+\tchar *old_optarg = optarg;\n+#ifdef RTE_EXEC_ENV_FREEBSD\n+\tconst int old_optreset = optreset;\n+\toptreset = 1;\n+#endif\n+\n+\toptind = 1;\n+\topterr = 0;\n+\n+\twhile ((opt = getopt_long(argc, argv, eal_short_options,\n+\t\t\t\t eal_long_options, &option_index)) != EOF) {\n+\n+\t\tswitch (opt) {\n+\t\tcase OPT_LOG_LEVEL_NUM:\n+\t\t\tif (eal_parse_common_option(opt, optarg, internal_conf) < 0)\n+\t\t\t\treturn -1;\n+\t\t\tbreak;\n+\t\tcase '?':\n+\t\t\t/* getopt is not happy, stop right now */\n+\t\t\tgoto out;\n+\t\tdefault:\n+\t\t\tcontinue;\n+\t\t}\n+\t}\n+out:\n+\t/* restore getopt lib */\n+\toptind = old_optind;\n+\toptopt = old_optopt;\n+\toptarg = old_optarg;\n+\topterr = old_opterr;\n+#ifdef RTE_EXEC_ENV_FREEBSD\n+\toptreset = old_optreset;\n+#endif\n+\treturn 0;\n+}\n+\n int\n eal_log_save_regexp(const char *regex, uint32_t level)\n {\ndiff --git a/lib/eal/common/eal_log.h b/lib/eal/common/eal_log.h\nindex c784fa604389..31dc489350f6 100644\n--- a/lib/eal/common/eal_log.h\n+++ b/lib/eal/common/eal_log.h\n@@ -13,6 +13,11 @@\n */\n int eal_log_init(const char *id, int facility);\n \n+/*\n+ * Scan command line args for log settings.\n+ */\n+int eal_log_level_parse(int argc, char * const argv[]);\n+\n /*\n * Determine where log data is written when no call to rte_openlog_stream.\n */\ndiff --git a/lib/eal/freebsd/eal.c b/lib/eal/freebsd/eal.c\nindex 7008303e112a..bb9a2b1653d9 100644\n--- a/lib/eal/freebsd/eal.c\n+++ b/lib/eal/freebsd/eal.c\n@@ -52,6 +52,7 @@\n #include \"eal_hugepages.h\"\n #include \"eal_options.h\"\n #include \"eal_memcfg.h\"\n+#include \"eal_log.h\"\n #include \"eal_trace.h\"\n \n #define MEMSIZE_IF_NO_HUGE_PAGE (64ULL * 1024ULL * 1024ULL)\n@@ -363,48 +364,6 @@ eal_get_hugepage_mem_size(void)\n \treturn (size < SIZE_MAX) ? (size_t)(size) : SIZE_MAX;\n }\n \n-/* Parse the arguments for --log-level only */\n-static void\n-eal_log_level_parse(int argc, char **argv)\n-{\n-\tint opt;\n-\tchar **argvopt;\n-\tint option_index;\n-\tconst int old_optind = optind;\n-\tconst int old_optopt = optopt;\n-\tconst int old_optreset = optreset;\n-\tchar * const old_optarg = optarg;\n-\tstruct internal_config *internal_conf =\n-\t\teal_get_internal_configuration();\n-\n-\targvopt = argv;\n-\toptind = 1;\n-\toptreset = 1;\n-\n-\twhile ((opt = getopt_long(argc, argvopt, eal_short_options,\n-\t\t\t\t eal_long_options, &option_index)) != EOF) {\n-\n-\t\tint ret;\n-\n-\t\t/* getopt is not happy, stop right now */\n-\t\tif (opt == '?')\n-\t\t\tbreak;\n-\n-\t\tret = (opt == OPT_LOG_LEVEL_NUM) ?\n-\t\t eal_parse_common_option(opt, optarg, internal_conf) : 0;\n-\n-\t\t/* common parser is not happy */\n-\t\tif (ret < 0)\n-\t\t\tbreak;\n-\t}\n-\n-\t/* restore getopt lib */\n-\toptind = old_optind;\n-\toptopt = old_optopt;\n-\toptreset = old_optreset;\n-\toptarg = old_optarg;\n-}\n-\n /* Parse the argument given in the command line of the application */\n static int\n eal_parse_args(int argc, char **argv)\n@@ -610,7 +569,11 @@ rte_eal_init(int argc, char **argv)\n \teal_save_args(argc, argv);\n \n \t/* set log level as early as possible */\n-\teal_log_level_parse(argc, argv);\n+\tif (eal_log_level_parse(argc, argv) < 0) {\n+\t\trte_eal_init_alert(\"invalid log option.\");\n+\t\trte_errno = EINVAL;\n+\t\treturn -1;\n+\t}\n \n \tif (rte_eal_cpu_init() < 0) {\n \t\trte_eal_init_alert(\"Cannot detect lcores.\");\n@@ -759,6 +722,12 @@ rte_eal_init(int argc, char **argv)\n #endif\n \t}\n \n+\tif (eal_log_init(getprogname(), internal_conf->syslog_facility) < 0) {\n+\t\trte_eal_init_alert(\"Cannot init logging.\");\n+\t\trte_errno = ENOMEM;\n+\t\treturn -1;\n+\t}\n+\n \t/* in secondary processes, memory init may allocate additional fbarrays\n \t * not present in primary processes, so to avoid any potential issues,\n \t * initialize memzones first.\ndiff --git a/lib/eal/linux/eal.c b/lib/eal/linux/eal.c\nindex 145afafde234..c9475560d986 100644\n--- a/lib/eal/linux/eal.c\n+++ b/lib/eal/linux/eal.c\n@@ -546,45 +546,6 @@ eal_parse_vfio_vf_token(const char *vf_token)\n \treturn -1;\n }\n \n-/* Parse the arguments for --log-level only */\n-static void\n-eal_log_level_parse(int argc, char **argv)\n-{\n-\tint opt;\n-\tchar **argvopt;\n-\tint option_index;\n-\tconst int old_optind = optind;\n-\tconst int old_optopt = optopt;\n-\tchar * const old_optarg = optarg;\n-\tstruct internal_config *internal_conf =\n-\t\teal_get_internal_configuration();\n-\n-\targvopt = argv;\n-\toptind = 1;\n-\n-\twhile ((opt = getopt_long(argc, argvopt, eal_short_options,\n-\t\t\t\t eal_long_options, &option_index)) != EOF) {\n-\n-\t\tint ret;\n-\n-\t\t/* getopt is not happy, stop right now */\n-\t\tif (opt == '?')\n-\t\t\tbreak;\n-\n-\t\tret = (opt == OPT_LOG_LEVEL_NUM) ?\n-\t\t\teal_parse_common_option(opt, optarg, internal_conf) : 0;\n-\n-\t\t/* common parser is not happy */\n-\t\tif (ret < 0)\n-\t\t\tbreak;\n-\t}\n-\n-\t/* restore getopt lib */\n-\toptind = old_optind;\n-\toptopt = old_optopt;\n-\toptarg = old_optarg;\n-}\n-\n static int\n eal_parse_huge_worker_stack(const char *arg)\n {\n@@ -993,7 +954,12 @@ rte_eal_init(int argc, char **argv)\n \teal_reset_internal_config(internal_conf);\n \n \t/* set log level as early as possible */\n-\teal_log_level_parse(argc, argv);\n+\tif (eal_log_level_parse(argc, argv) < 0) {\n+\t\trte_eal_init_alert(\"invalid log option.\");\n+\t\trte_errno = EINVAL;\n+\t\t__atomic_store_n(&run_once, 0, __ATOMIC_RELAXED);\n+\t\treturn -1;\n+\t}\n \n \t/* clone argv to report out later in telemetry */\n \teal_save_args(argc, argv);\ndiff --git a/lib/eal/linux/meson.build b/lib/eal/linux/meson.build\nindex 5af456db9edb..e99ebed25692 100644\n--- a/lib/eal/linux/meson.build\n+++ b/lib/eal/linux/meson.build\n@@ -11,7 +11,6 @@ sources += files(\n 'eal_hugepage_info.c',\n 'eal_interrupts.c',\n 'eal_lcore.c',\n- 'eal_log.c',\n 'eal_memalloc.c',\n 'eal_memory.c',\n 'eal_thread.c',\ndiff --git a/lib/eal/linux/eal_log.c b/lib/eal/unix/eal_log.c\nsimilarity index 76%\nrename from lib/eal/linux/eal_log.c\nrename to lib/eal/unix/eal_log.c\nindex d44416fd6570..85d817c2d31e 100644\n--- a/lib/eal/linux/eal_log.c\n+++ b/lib/eal/unix/eal_log.c\n@@ -9,6 +9,8 @@\n #include <rte_log.h>\n \n #include \"eal_log.h\"\n+#include \"eal_internal_cfg.h\"\n+#include \"eal_private.h\"\n \n /*\n * default log function\n@@ -25,6 +27,15 @@ console_log_write(__rte_unused void *c, const char *buf, size_t size)\n \t/* Syslog error levels are from 0 to 7, so subtract 1 to convert */\n \tsyslog(rte_log_cur_msg_loglevel() - 1, \"%.*s\", (int)size, buf);\n \n+#ifdef RTE_EXEC_ENV_LINUX\n+\t/* Handle glibc quirk: write function should return the number of bytes\n+\t * copied from buf, or 0 on error. (The function must not return a negative value.)\n+\t * FreeBSD expects that write function behaves like write(2).\n+\t */\n+\tif (ret < 0)\n+\t\tret = 0;\n+#endif\n+\n \treturn ret;\n }\n \ndiff --git a/lib/eal/unix/meson.build b/lib/eal/unix/meson.build\nindex cc7d67dd321d..37d07594df29 100644\n--- a/lib/eal/unix/meson.build\n+++ b/lib/eal/unix/meson.build\n@@ -6,6 +6,7 @@ sources += files(\n 'eal_file.c',\n 'eal_filesystem.c',\n 'eal_firmware.c',\n+ 'eal_log.c',\n 'eal_unix_memory.c',\n 'eal_unix_thread.c',\n 'eal_unix_timer.c',\ndiff --git a/lib/eal/windows/eal.c b/lib/eal/windows/eal.c\nindex 2d7a0e9ab27e..c6006c48ddef 100644\n--- a/lib/eal/windows/eal.c\n+++ b/lib/eal/windows/eal.c\n@@ -96,41 +96,6 @@ eal_usage(const char *prgname)\n \t}\n }\n \n-/* Parse the arguments for --log-level only */\n-static void\n-eal_log_level_parse(int argc, char **argv)\n-{\n-\tint opt;\n-\tchar **argvopt;\n-\tint option_index;\n-\tstruct internal_config *internal_conf =\n-\t\teal_get_internal_configuration();\n-\n-\targvopt = argv;\n-\n-\teal_reset_internal_config(internal_conf);\n-\n-\twhile ((opt = getopt_long(argc, argvopt, eal_short_options,\n-\t\teal_long_options, &option_index)) != EOF) {\n-\n-\t\tint ret;\n-\n-\t\t/* getopt is not happy, stop right now */\n-\t\tif (opt == '?')\n-\t\t\tbreak;\n-\n-\t\tret = (opt == OPT_LOG_LEVEL_NUM) ?\n-\t\t\teal_parse_common_option(opt, optarg,\n-\t\t\t\tinternal_conf) : 0;\n-\n-\t\t/* common parser is not happy */\n-\t\tif (ret < 0)\n-\t\t\tbreak;\n-\t}\n-\n-\toptind = 0; /* reset getopt lib */\n-}\n-\n /* Parse the argument given in the command line of the application */\n static int\n eal_parse_args(int argc, char **argv)\n@@ -143,6 +108,7 @@ eal_parse_args(int argc, char **argv)\n \t\teal_get_internal_configuration();\n \n \targvopt = argv;\n+\topterr = 1;\n \n \twhile ((opt = getopt_long(argc, argvopt, eal_short_options,\n \t\teal_long_options, &option_index)) != EOF) {\n", "prefixes": [ "v7", "3/5" ] }