get:
Show a patch.

patch:
Update a patch.

put:
Update a patch.

GET /api/patches/138951/?format=api
HTTP 200 OK
Allow: GET, PUT, PATCH, HEAD, OPTIONS
Content-Type: application/json
Vary: Accept

{
    "id": 138951,
    "url": "http://patches.dpdk.org/api/patches/138951/?format=api",
    "web_url": "http://patches.dpdk.org/project/dpdk/patch/20240327233001.83505-16-stephen@networkplumber.org/",
    "project": {
        "id": 1,
        "url": "http://patches.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": "<20240327233001.83505-16-stephen@networkplumber.org>",
    "list_archive_url": "https://inbox.dpdk.org/dev/20240327233001.83505-16-stephen@networkplumber.org",
    "date": "2024-03-27T23:28:38",
    "name": "[v17,15/15] log: colorize log output",
    "commit_ref": null,
    "pull_url": null,
    "state": "superseded",
    "archived": true,
    "hash": "8c94fa89979df4beb7be7eff1c140c4f26471637",
    "submitter": {
        "id": 27,
        "url": "http://patches.dpdk.org/api/people/27/?format=api",
        "name": "Stephen Hemminger",
        "email": "stephen@networkplumber.org"
    },
    "delegate": {
        "id": 1,
        "url": "http://patches.dpdk.org/api/users/1/?format=api",
        "username": "tmonjalo",
        "first_name": "Thomas",
        "last_name": "Monjalon",
        "email": "thomas@monjalon.net"
    },
    "mbox": "http://patches.dpdk.org/project/dpdk/patch/20240327233001.83505-16-stephen@networkplumber.org/mbox/",
    "series": [
        {
            "id": 31635,
            "url": "http://patches.dpdk.org/api/series/31635/?format=api",
            "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=31635",
            "date": "2024-03-27T23:28:23",
            "name": "Logging unification and improvements",
            "version": 17,
            "mbox": "http://patches.dpdk.org/series/31635/mbox/"
        }
    ],
    "comments": "http://patches.dpdk.org/api/patches/138951/comments/",
    "check": "fail",
    "checks": "http://patches.dpdk.org/api/patches/138951/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 09C9643D5B;\n\tThu, 28 Mar 2024 00:31:49 +0100 (CET)",
            "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 86C1A42DE4;\n\tThu, 28 Mar 2024 00:30:37 +0100 (CET)",
            "from mail-pl1-f182.google.com (mail-pl1-f182.google.com\n [209.85.214.182])\n by mails.dpdk.org (Postfix) with ESMTP id B34E142D55\n for <dev@dpdk.org>; Thu, 28 Mar 2024 00:30:24 +0100 (CET)",
            "by mail-pl1-f182.google.com with SMTP id\n d9443c01a7336-1e0411c0a52so3872545ad.0\n for <dev@dpdk.org>; Wed, 27 Mar 2024 16:30:24 -0700 (PDT)",
            "from hermes.local (204-195-123-203.wavecable.com. [204.195.123.203])\n by smtp.gmail.com with ESMTPSA id\n n15-20020a170902e54f00b001e0fdc6e4ebsm80772plf.175.2024.03.27.16.30.23\n (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n Wed, 27 Mar 2024 16:30:23 -0700 (PDT)"
        ],
        "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1711582224;\n x=1712187024; darn=dpdk.org;\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=f6TdB4wD1wzUlrX9hQ8IEek2JyDnmijhwd4pTTDQhLg=;\n b=tDfrmsoRaelIDWigsAiJQJtvgaAebLUe9cWV/xiU5ECnNK4Hp9iCbVdROZqqdIXRkB\n xvHqm5/WCIeX5PtM3YUVDbO8+H5j2nHIYeDd9+7idAatGVwi64sAjqk9dbGU35bpQFWs\n HibDnaUrKK1IznqDB1dv3XhWJLyeJ2MMUSWPGszo6SJtrlPw1ED8BTFr2/vhp1bEGaKR\n aaLNB8/QlD2Lovva4/X6mLMF0PyEzx4/nc3CocZkQyAY5q2ifw/F76etQLcvLAZGlVhU\n sl+AZbaTOzwTn27QJcKYIlw0ZgoXa4CWd69sGwZHGPinK3GivjpB+q4Yl0yIGfe/pY5p\n Fp+w==",
        "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20230601; t=1711582224; x=1712187024;\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=f6TdB4wD1wzUlrX9hQ8IEek2JyDnmijhwd4pTTDQhLg=;\n b=cBh8faK3el9CrVshhkbB1zGhuehTT0n59QIwN8lyDhQ8J3suCazydtVLOXpNlsE0HR\n LQqqUVF9qHuhIuq6myhz5NokHmWdINznJrQS1ND8cQMbduNBzpHz9P5xGyuTsbVLARWc\n D1Xy5uD3AGADzleAJCVB5utx5rlzoHEZ0DojvziLSsXp6mUuviYqSmOk7jrm/v6R/KkW\n XHmCaomQySSAWLyUo0uXxAh3JMdRaQdOaYwG8bO2Njaj3kQeJNZ07fe5yPxAuS02mMZK\n PywPU68qP4KQlA08iixBEJ68cPwrSISRm/enFIz0Js3HIOnI9bSV7S677lohE/oAYuF1\n VtMQ==",
        "X-Gm-Message-State": "AOJu0YxEO3o26sMrdWaO1A3XkVXtJ4kOoBZ04qcF6q5KpwVWL5uQmFsa\n lSVMjHiuZB4jv76WZtUwr/k5gkCDSfPZ1MkhdJtLVby5AtD/c3Sbzlec2pDyJD0ptp/vFJQ8Cbs\n Z",
        "X-Google-Smtp-Source": "\n AGHT+IH+1y+prGDIY+7m+Zub5mfLe+NfR1A5+1NlEBRBe+ELRkwe0GU5M1MWTqIBKF6PTb8/OTNJjA==",
        "X-Received": "by 2002:a17:902:b901:b0:1e0:ca47:4d9b with SMTP id\n bf1-20020a170902b90100b001e0ca474d9bmr1182544plb.29.1711582223715;\n Wed, 27 Mar 2024 16:30:23 -0700 (PDT)",
        "From": "Stephen Hemminger <stephen@networkplumber.org>",
        "To": "dev@dpdk.org",
        "Cc": "Stephen Hemminger <stephen@networkplumber.org>",
        "Subject": "[PATCH v17 15/15] log: colorize log output",
        "Date": "Wed, 27 Mar 2024 16:28:38 -0700",
        "Message-ID": "<20240327233001.83505-16-stephen@networkplumber.org>",
        "X-Mailer": "git-send-email 2.43.0",
        "In-Reply-To": "<20240327233001.83505-1-stephen@networkplumber.org>",
        "References": "<20200814173441.23086-1-stephen@networkplumber.org>\n <20240327233001.83505-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": "Like dmesg, colorize the log output (unless redirected to file).\nTimestamp is green, the subsystem is in yellow and the message\nis red if urgent, boldface if an error, and normal for info and\ndebug messages.\n\nSigned-off-by: Stephen Hemminger <stephen@networkplumber.org>\n---\n app/test/test_eal_flags.c           |  24 ++++\n doc/guides/prog_guide/log_lib.rst   |  16 ++-\n lib/eal/common/eal_common_options.c |  11 ++\n lib/eal/common/eal_options.h        |   2 +\n lib/log/log.c                       | 167 +++++++++++++++++++++++++++-\n lib/log/log_internal.h              |   5 +\n lib/log/version.map                 |   1 +\n 7 files changed, 222 insertions(+), 4 deletions(-)",
    "diff": "diff --git a/app/test/test_eal_flags.c b/app/test/test_eal_flags.c\nindex 08f4866461..c6c05e2e1d 100644\n--- a/app/test/test_eal_flags.c\n+++ b/app/test/test_eal_flags.c\n@@ -1067,6 +1067,18 @@ test_misc_flags(void)\n \tconst char * const argv25[] = {prgname, prefix, mp_flag,\n \t\t\t\t       \"--log-timestamp=invalid\" };\n \n+\t/* Try running with --log-color */\n+\tconst char * const argv26[] = {prgname, prefix, mp_flag,\n+\t\t\t\t       \"--log-color\" };\n+\n+\t/* Try running with --log-color=never */\n+\tconst char * const argv27[] = {prgname, prefix, mp_flag,\n+\t\t\t\t       \"--log-color=never\" };\n+\n+\t/* Try running with --log-color=invalid */\n+\tconst char * const argv28[] = {prgname, prefix, mp_flag,\n+\t\t\t\t       \"--log-color=invalid\" };\n+\n \n \t/* run all tests also applicable to FreeBSD first */\n \n@@ -1187,6 +1199,18 @@ test_misc_flags(void)\n \t\tprintf(\"Error - process did run ok with --log-timestamp=invalid parameter\\n\");\n \t\tgoto fail;\n \t}\n+\tif (launch_proc(argv26) != 0) {\n+\t\tprintf(\"Error - process did not run ok with --log-color parameter\\n\");\n+\t\tgoto fail;\n+\t}\n+\tif (launch_proc(argv27) != 0) {\n+\t\tprintf(\"Error - process did not run ok with --log-color=none parameter\\n\");\n+\t\tgoto fail;\n+\t}\n+\tif (launch_proc(argv28) == 0) {\n+\t\tprintf(\"Error - process did run ok with --log-timestamp=invalid parameter\\n\");\n+\t\tgoto fail;\n+\t}\n \n \n \trmdir(hugepath_dir3);\ndiff --git a/doc/guides/prog_guide/log_lib.rst b/doc/guides/prog_guide/log_lib.rst\nindex abaedc7212..40727ebaae 100644\n--- a/doc/guides/prog_guide/log_lib.rst\n+++ b/doc/guides/prog_guide/log_lib.rst\n@@ -59,6 +59,21 @@ For example::\n \n Within an application, the same result can be got using the ``rte_log_set_level_pattern()`` or ``rte_log_set_level_regex()`` APIs.\n \n+Color output\n+~~~~~~~~~~~~\n+\n+The log library will highlight important messages.\n+This is controlled by the ``--log-color`` option.\n+he optional argument ``when`` can be ``auto``, ``never``, or ``always``.\n+The default setting is ``auto`` which enables color when the output to\n+``stderr`` is a terminal.\n+If the ``when`` argument is omitted, it defaults to ``always``.\n+\n+For example to turn off all coloring::\n+\n+\t/path/to/app --log-color=none\n+\n+\n Log timestamp\n ~~~~~~~~~~~~~\n \n@@ -101,7 +116,6 @@ option. There are three possible settings for this option:\n If ``--syslog`` option is not specified, then only console (stderr) will be used.\n \n \n-\n Using Logging APIs to Generate Log Messages\n -------------------------------------------\n \ndiff --git a/lib/eal/common/eal_common_options.c b/lib/eal/common/eal_common_options.c\nindex 9ca7db04aa..5e7ab29ae3 100644\n--- a/lib/eal/common/eal_common_options.c\n+++ b/lib/eal/common/eal_common_options.c\n@@ -75,6 +75,7 @@ eal_long_options[] = {\n \t{OPT_LCORES,            1, NULL, OPT_LCORES_NUM           },\n \t{OPT_LOG_LEVEL,         1, NULL, OPT_LOG_LEVEL_NUM        },\n \t{OPT_LOG_TIMESTAMP,     2, NULL, OPT_LOG_TIMESTAMP_NUM    },\n+\t{OPT_LOG_COLOR,\t\t2, NULL, OPT_LOG_COLOR_NUM\t  },\n \t{OPT_TRACE,             1, NULL, OPT_TRACE_NUM            },\n \t{OPT_TRACE_DIR,         1, NULL, OPT_TRACE_DIR_NUM        },\n \t{OPT_TRACE_BUF_SIZE,    1, NULL, OPT_TRACE_BUF_SIZE_NUM   },\n@@ -1618,6 +1619,7 @@ eal_log_level_parse(int argc, char * const argv[])\n \t\tcase OPT_LOG_LEVEL_NUM:\n \t\tcase OPT_SYSLOG_NUM:\n \t\tcase OPT_LOG_TIMESTAMP_NUM:\n+\t\tcase OPT_LOG_COLOR_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@@ -1862,6 +1864,14 @@ eal_parse_common_option(int opt, const char *optarg,\n \t\t}\n \t\tbreak;\n \n+\tcase OPT_LOG_COLOR_NUM:\n+\t\tif (eal_log_color(optarg) < 0) {\n+\t\t\tEAL_LOG(ERR, \"invalid parameters for --\"\n+\t\t\t\tOPT_LOG_COLOR);\n+\t\t\treturn -1;\n+\t\t}\n+\t\tbreak;\n+\n #ifndef RTE_EXEC_ENV_WINDOWS\n \tcase OPT_TRACE_NUM: {\n \t\tif (eal_trace_args_save(optarg) < 0) {\n@@ -2229,6 +2239,7 @@ eal_common_usage(void)\n \t       \"                      Set specific log level\\n\"\n \t       \"  --\"OPT_LOG_LEVEL\"=help    Show log types and levels\\n\"\n \t       \"  --\"OPT_LOG_TIMESTAMP\"[=<format>]  Timestamp log output\\n\"\n+\t       \"  --\"OPT_LOG_COLOR\"[=<when>] Colorize log messages\\n\"\n #ifndef RTE_EXEC_ENV_WINDOWS\n \t       \"  --\"OPT_TRACE\"=<regex-match>\\n\"\n \t       \"                      Enable trace based on regular expression trace name.\\n\"\ndiff --git a/lib/eal/common/eal_options.h b/lib/eal/common/eal_options.h\nindex e24c9eca53..5a63c1dd3a 100644\n--- a/lib/eal/common/eal_options.h\n+++ b/lib/eal/common/eal_options.h\n@@ -37,6 +37,8 @@ enum {\n \tOPT_LOG_LEVEL_NUM,\n #define OPT_LOG_TIMESTAMP     \"log-timestamp\"\n \tOPT_LOG_TIMESTAMP_NUM,\n+#define OPT_LOG_COLOR\t      \"log-color\"\n+\tOPT_LOG_COLOR_NUM,\n #define OPT_TRACE             \"trace\"\n \tOPT_TRACE_NUM,\n #define OPT_TRACE_DIR         \"trace-dir\"\ndiff --git a/lib/log/log.c b/lib/log/log.c\nindex 650d294120..16c0369083 100644\n--- a/lib/log/log.c\n+++ b/lib/log/log.c\n@@ -23,6 +23,7 @@\n #include <sys/un.h>\n #endif\n \n+#include <rte_common.h>\n #include <rte_log.h>\n #include <rte_per_lcore.h>\n \n@@ -49,6 +50,12 @@ enum eal_log_syslog {\n \tEAL_LOG_SYSLOG_BOTH,\t\t/* log to both syslog and stderr */\n };\n \n+enum eal_log_color {\n+\tEAL_LOG_COLOR_AUTO = 0,\t/* default */\n+\tEAL_LOG_COLOR_NEVER,\n+\tEAL_LOG_COLOR_ALWAYS,\n+};\n+\n typedef int (*log_print_t)(FILE *f, uint32_t level, const char *fmt, va_list ap);\n static int log_print(FILE *f, uint32_t level, const char *format, va_list ap);\n \n@@ -64,6 +71,7 @@ static struct rte_logs {\n #endif\n \tlog_print_t print_func;\n \n+\tenum eal_log_color color_mode;\n \tenum eal_log_time_format time_format;\n \tstruct timespec started;   /* when log was initialized */\n \tstruct timespec previous;  /* when last msg was printed */\n@@ -715,6 +723,75 @@ format_timestamp(char *tsbuf, size_t tsbuflen)\n \treturn 0;\n }\n \n+enum color {\n+\tCOLOR_NONE,\n+\tCOLOR_RED,\n+\tCOLOR_GREEN,\n+\tCOLOR_YELLOW,\n+\tCOLOR_BLUE,\n+\tCOLOR_MAGENTA,\n+\tCOLOR_CYAN,\n+\tCOLOR_WHITE,\n+\tCOLOR_BOLD,\n+\tCOLOR_CLEAR\n+};\n+\n+static const char * const color_code[] = {\n+\t[COLOR_NONE]\t= \"\",\n+\t[COLOR_RED]\t= \"\\e[31m\",\n+\t[COLOR_GREEN]\t= \"\\e[32m\",\n+\t[COLOR_YELLOW]\t= \"\\e[33m\",\n+\t[COLOR_BLUE]\t= \"\\e[34m\",\n+\t[COLOR_MAGENTA] = \"\\e[35m\",\n+\t[COLOR_CYAN]    = \"\\e[36m\",\n+\t[COLOR_WHITE]\t= \"\\e[37m\",\n+\t[COLOR_BOLD]\t= \"\\e[1m\",\n+\t[COLOR_CLEAR]\t= \"\\e[0m\",\n+};\n+\n+__rte_format_printf(3, 4)\n+static int color_fprintf(FILE *out, enum color color, const char *fmt, ...)\n+{\n+\tva_list args;\n+\tint ret = 0;\n+\n+\tva_start(args, fmt);\n+\tret = fprintf(out, \"%s\", color_code[color]);\n+\tret += vfprintf(out, fmt, args);\n+\tret += fprintf(out, \"%s\", color_code[COLOR_CLEAR]);\n+\tva_end(args);\n+\n+\treturn ret;\n+}\n+\n+static ssize_t\n+color_log_write(FILE *f, int level, char *msg)\n+{\n+\tchar *cp;\n+\tssize_t ret = 0;\n+\n+\t/*\n+\t * use convention that first part of message (up to the ':' character)\n+\t * is the subsystem id and should be highlighted.\n+\t */\n+\tcp = strchr(msg, ':');\n+\tif (cp) {\n+\t\t/* print first part in yellow */\n+\t\tret = color_fprintf(stderr, COLOR_YELLOW, \"%.*s\",\n+\t\t\t\t    (int)(cp - msg + 1), msg);\n+\t\tmsg = cp + 1;\n+\t}\n+\n+\tif (level <= 0 || level >= (int)RTE_LOG_INFO)\n+\t\tret += fprintf(f, \"%s\", msg);\n+\telse if (level >= (int)RTE_LOG_ERR)\n+\t\tret += color_fprintf(f, COLOR_BOLD, \"%s\", msg);\n+\telse\n+\t\tret += color_fprintf(f, COLOR_RED, \"%s\", msg);\n+\n+\treturn ret;\n+}\n+\n /* default log print function */\n __rte_format_printf(3, 0)\n static int\n@@ -930,14 +1007,87 @@ log_open_syslog(const char *id, bool is_terminal)\n }\n #endif\n \n+__rte_format_printf(3, 0)\n+static int\n+color_print(FILE *f, uint32_t level, const char *format, va_list ap)\n+{\n+\tchar *buf = NULL;\n+\tint ret;\n+\n+\t/* need to make temporary buffer for color scan */\n+\tret = vasprintf(&buf, format, ap);\n+\tif (ret > 0) {\n+\t\tret = color_log_write(f, level, buf);\n+\t\tfree(buf);\n+\t\treturn ret;\n+\t}\n+\n+\t/* if vasprintf fails, print without color */\n+\treturn log_print(f, level, format, ap);\n+}\n+\n+__rte_format_printf(3, 0)\n+static int\n+color_print_with_timestamp(FILE *f, uint32_t level,\n+\t\t\t   const char *format, va_list ap)\n+{\n+\tchar tsbuf[128];\n+\n+\tif (format_timestamp(tsbuf, sizeof(tsbuf)) > 0)\n+\t\tcolor_fprintf(f, COLOR_GREEN, \"[%s] \", tsbuf);\n+\n+\treturn color_print(f, level, format, ap);\n+}\n+\n+/*\n+ * Controls whether color is enabled:\n+ * modes are:\n+ *   always - enable color output regardless\n+ *   auto - enable if stderr is a terminal\n+ *   never - color output is disabled.\n+ */\n+int\n+eal_log_color(const char *mode)\n+{\n+\tif (mode == NULL || strcmp(mode, \"always\") == 0)\n+\t\trte_logs.color_mode = EAL_LOG_COLOR_ALWAYS;\n+\telse if (strcmp(mode, \"never\") == 0)\n+\t\trte_logs.color_mode = EAL_LOG_COLOR_NEVER;\n+\telse if (strcmp(mode, \"auto\") == 0)\n+\t\trte_logs.color_mode = EAL_LOG_COLOR_AUTO;\n+\telse\n+\t\treturn -1;\n+\n+\treturn 0;\n+}\n+\n+static inline bool\n+use_color(bool is_terminal)\n+{\n+\tswitch (rte_logs.color_mode) {\n+\tdefault:\n+\tcase EAL_LOG_COLOR_NEVER:\n+\t\treturn false;\n+\tcase EAL_LOG_COLOR_ALWAYS:\n+\t\treturn true;\n+\tcase EAL_LOG_COLOR_AUTO:\n+\t\treturn is_terminal;\n+\t}\n+\n+}\n+\n /* Choose how log output is directed */\n static void\n log_output_selection(const char *id)\n {\n+\tbool is_terminal;\n+\n #ifdef RTE_EXEC_ENV_WINDOWS\n \tRTE_SET_USED(id);\n+\n+\tis_terminal = _isatty(fileno(stderr));\n #else\n-\tbool is_terminal = isatty(STDERR_FILENO);\n+\tis_terminal = isatty(STDERR_FILENO);\n \n \t/* If stderr is redirected to systemd journal then upgrade */\n \tif (!is_terminal && is_journal(STDERR_FILENO)) {\n@@ -957,8 +1107,19 @@ log_output_selection(const char *id)\n \t\treturn;\n \t}\n #endif\n-\tif (rte_logs.time_format != EAL_LOG_TIMESTAMP_NONE)\n-\t\trte_logs.print_func = log_print_with_timestamp;\n+\n+\tif (use_color(is_terminal)) {\n+\t\tif (rte_logs.time_format == EAL_LOG_TIMESTAMP_NONE)\n+\t\t\trte_logs.print_func = color_print;\n+\t\telse\n+\t\t\trte_logs.print_func = color_print_with_timestamp;\n+\t} else {\n+\t\tif (rte_logs.time_format == EAL_LOG_TIMESTAMP_NONE)\n+\t\t\trte_logs.print_func = log_print;\n+\t\telse\n+\t\t\trte_logs.print_func = log_print_with_timestamp;\n+\t}\n+\n }\n \n /*\ndiff --git a/lib/log/log_internal.h b/lib/log/log_internal.h\nindex 7c7d44eed2..525e1397fd 100644\n--- a/lib/log/log_internal.h\n+++ b/lib/log/log_internal.h\n@@ -47,5 +47,10 @@ void rte_eal_log_cleanup(void);\n __rte_internal\n int eal_log_timestamp(const char *fmt);\n \n+/*\n+ * Enable or disable color in log messages\n+ */\n+__rte_internal\n+int eal_log_color(const char *mode);\n \n #endif /* LOG_INTERNAL_H */\ndiff --git a/lib/log/version.map b/lib/log/version.map\nindex 14d6681a5f..7fa79b20ff 100644\n--- a/lib/log/version.map\n+++ b/lib/log/version.map\n@@ -25,6 +25,7 @@ DPDK_24 {\n INTERNAL {\n \tglobal:\n \n+\teal_log_color;\n \teal_log_init;\n \teal_log_level2str;\n \teal_log_save_pattern;\n",
    "prefixes": [
        "v17",
        "15/15"
    ]
}