Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/139010/?format=api
{ "id": 139010, "url": "http://patchwork.dpdk.org/api/patches/139010/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/patch/20240330164433.50144-15-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": "<20240330164433.50144-15-stephen@networkplumber.org>", "list_archive_url": "https://inbox.dpdk.org/dev/20240330164433.50144-15-stephen@networkplumber.org", "date": "2024-03-30T16:42:25", "name": "[v20,14/14] log: colorize log output", "commit_ref": null, "pull_url": null, "state": "new", "archived": false, "hash": "34cf2d5910aae4fc05c492662c9448ff06e17adb", "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/20240330164433.50144-15-stephen@networkplumber.org/mbox/", "series": [ { "id": 31649, "url": "http://patchwork.dpdk.org/api/series/31649/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/list/?series=31649", "date": "2024-03-30T16:42:11", "name": "Logging unification and improvements", "version": 20, "mbox": "http://patchwork.dpdk.org/series/31649/mbox/" } ], "comments": "http://patchwork.dpdk.org/api/patches/139010/comments/", "check": "warning", "checks": "http://patchwork.dpdk.org/api/patches/139010/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 29EDF43D85;\n\tSat, 30 Mar 2024 17:46:26 +0100 (CET)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 2AC6B41148;\n\tSat, 30 Mar 2024 17:45:08 +0100 (CET)", "from mail-pf1-f170.google.com (mail-pf1-f170.google.com\n [209.85.210.170])\n by mails.dpdk.org (Postfix) with ESMTP id 4E9E640689\n for <dev@dpdk.org>; Sat, 30 Mar 2024 17:44:56 +0100 (CET)", "by mail-pf1-f170.google.com with SMTP id\n d2e1a72fcca58-6ea9a605ca7so2030402b3a.0\n for <dev@dpdk.org>; Sat, 30 Mar 2024 09:44:56 -0700 (PDT)", "from hermes.local (204-195-123-203.wavecable.com. [204.195.123.203])\n by smtp.gmail.com with ESMTPSA id\n by1-20020a056a02058100b005dc26144d96sm4192417pgb.75.2024.03.30.09.44.54\n (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n Sat, 30 Mar 2024 09:44:55 -0700 (PDT)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1711817095;\n x=1712421895; 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=7RX6O0wRocTiTmWMorYuwBBl7oR+i829l0UHwL0eAjY=;\n b=J0T8LXat5lEJ7G3gw+vsuzd++PxeGsqubbQxZLQ/UsS5Y6SeAsQ3aQHsrtAJwRGSJX\n HsTXq4J8EG795uhVBubombeMv+yL/0Hs6ikGniloCm+98/B79daDqL7Bl7yAGu328bnP\n /ELsltloA8oTUwm4WRMDK7/WLNHpE61C3WiYo8hBCE3cqgsj3hhHNeTmg9v2ruiqnf6A\n VhzJZ0PTmB8dTkl7HcgVtTLTxL3sact80kuYBk3fyXil6/x5ESafwDIvHAQAP/TCg1Pa\n BH5Cq+7YRyMTxMJxNQw7pgGIRKaCmXR34tnXKRKi0b9Mh21EXhxQ/x1g+9pJ2FKdu4jB\n EHQw==", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20230601; t=1711817095; x=1712421895;\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=7RX6O0wRocTiTmWMorYuwBBl7oR+i829l0UHwL0eAjY=;\n b=OGVGviDTtLnCZw2kfjtGgN1HmtcpePYfhyNss++eum//WNkoJhCukeAbAgfgvDYJ0i\n +tUXWIL4mmQOXm76r/UQY3CJbhjnHx3HFzMM4jQUN3r62Ev9AJJyXmY+vTwo0sSIOI41\n WRRMZoS4ctt+hTQpNgfMZ8j+zc7DRjw/xlfgmxSm7QcsPpLQzp2jQVnjSyTG+s37DyZl\n 0iEl1v9GuYHXLHvFOrv7h20R5dpRFeOi5GcpR9Fn6IIQ9+BJql0XYbGfcRWCNjQszhWj\n ro/xNjmMeDvmADM2HdGzRPzaptmSxlVcFNgcyDvBG24utHYEJXEq3NfSK1FwbenQaZok\n Lp4w==", "X-Gm-Message-State": "AOJu0YwyrcrmDI/3CxPudyQyAnpdUYEmcm0fp4rCCOI0STOY+8Ft4VP5\n /+MN8iEoC119A6UdoTAfSBeMdwoLs+WPwDiS1PmS6LEtym5ztqbMjYCnNCnMhy+smlOKoBhv6XE\n +", "X-Google-Smtp-Source": "\n AGHT+IHmppXbzLNDAwAkp50MhoI+UVYGtBypZplbY0KGjNXXTncLWdp+OLiYfk70M7pZgk2gwbi35Q==", "X-Received": "by 2002:a05:6a00:2395:b0:6ea:b606:6550 with SMTP id\n f21-20020a056a00239500b006eab6066550mr12274372pfc.4.1711817095414;\n Sat, 30 Mar 2024 09:44:55 -0700 (PDT)", "From": "Stephen Hemminger <stephen@networkplumber.org>", "To": "dev@dpdk.org", "Cc": "Stephen Hemminger <stephen@networkplumber.org>", "Subject": "[PATCH v20 14/14] log: colorize log output", "Date": "Sat, 30 Mar 2024 09:42:25 -0700", "Message-ID": "<20240330164433.50144-15-stephen@networkplumber.org>", "X-Mailer": "git-send-email 2.43.0", "In-Reply-To": "<20240330164433.50144-1-stephen@networkplumber.org>", "References": "<20200814173441.23086-1-stephen@networkplumber.org>\n <20240330164433.50144-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 | 6 +-\n lib/log/log.c | 24 +++--\n lib/log/log_color.c | 148 ++++++++++++++++++++++++++++\n lib/log/log_internal.h | 5 +\n lib/log/log_private.h | 8 ++\n lib/log/meson.build | 1 +\n lib/log/version.map | 1 +\n 10 files changed, 235 insertions(+), 9 deletions(-)\n create mode 100644 lib/log/log_color.c", "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 476dedb097..f46720fe34 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@@ -115,7 +130,6 @@ There are three possible settings for this option:\n *always*\n Always try to direct messages to journal socket.\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 9a82118184..70fdf3f5a1 100644\n--- a/lib/eal/common/eal_common_options.c\n+++ b/lib/eal/common/eal_common_options.c\n@@ -73,6 +73,7 @@ eal_long_options[] = {\n \t{OPT_HUGE_UNLINK, 2, NULL, OPT_HUGE_UNLINK_NUM },\n \t{OPT_IOVA_MODE,\t 1, NULL, OPT_IOVA_MODE_NUM },\n \t{OPT_LCORES, 1, NULL, OPT_LCORES_NUM },\n+\t{OPT_LOG_COLOR,\t\t2, NULL, OPT_LOG_COLOR_NUM\t },\n \t{OPT_LOG_LEVEL, 1, NULL, OPT_LOG_LEVEL_NUM },\n \t{OPT_LOG_JOURNAL,\t2, NULL, OPT_LOG_JOURNAL_NUM\t },\n \t{OPT_LOG_TIMESTAMP, 2, NULL, OPT_LOG_TIMESTAMP_NUM },\n@@ -1620,6 +1621,7 @@ eal_log_level_parse(int argc, char * const argv[])\n \t\tcase OPT_SYSLOG_NUM:\n \t\tcase OPT_LOG_JOURNAL_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@@ -1872,6 +1874,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@@ -2240,6 +2250,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 c5a1c70288..c0686c2544 100644\n--- a/lib/eal/common/eal_options.h\n+++ b/lib/eal/common/eal_options.h\n@@ -33,10 +33,12 @@ enum {\n \tOPT_HUGE_UNLINK_NUM,\n #define OPT_LCORES \"lcores\"\n \tOPT_LCORES_NUM,\n-#define OPT_LOG_LEVEL \"log-level\"\n-\tOPT_LOG_LEVEL_NUM,\n+#define OPT_LOG_COLOR\t \"log-color\"\n+\tOPT_LOG_COLOR_NUM,\n #define OPT_LOG_JOURNAL \"log-journal\"\n \tOPT_LOG_JOURNAL_NUM,\n+#define OPT_LOG_LEVEL \"log-level\"\n+\tOPT_LOG_LEVEL_NUM,\n #define OPT_LOG_TIMESTAMP \"log-timestamp\"\n \tOPT_LOG_TIMESTAMP_NUM,\n #define OPT_TRACE \"trace\"\ndiff --git a/lib/log/log.c b/lib/log/log.c\nindex 13d68402d9..856c0a0a29 100644\n--- a/lib/log/log.c\n+++ b/lib/log/log.c\n@@ -503,10 +503,14 @@ rte_log(uint32_t level, uint32_t logtype, const char *format, ...)\n void\n eal_log_init(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 #ifdef RTE_EXEC_ENV_LINUX\n \tif (log_journal_enabled())\n@@ -516,11 +520,19 @@ eal_log_init(const char *id)\n \tif (log_syslog_enabled(is_terminal))\n \t\tlog_syslog_open(id, is_terminal);\n \telse\n-#endif\n-\tif (log_timestamp_enabled())\n-\t\trte_logs.print_func = log_print_with_timestamp;\n-\telse\n-\t\trte_logs.print_func = vfprintf;\n+#endif /* RTE_EXEC_ENV_WINDOWS */\n+\n+\tif (log_color_enabled(is_terminal)) {\n+\t\tif (log_timestamp_enabled())\n+\t\t\trte_logs.print_func = color_print_with_timestamp;\n+\t\telse\n+\t\t\trte_logs.print_func = color_print;\n+\t} else {\n+\t\tif (log_timestamp_enabled())\n+\t\t\trte_logs.print_func = log_print_with_timestamp;\n+\t\telse\n+\t\t\trte_logs.print_func = vfprintf;\n+\t}\n \n #if RTE_LOG_DP_LEVEL >= RTE_LOG_DEBUG\n \tRTE_LOG(NOTICE, EAL,\ndiff --git a/lib/log/log_color.c b/lib/log/log_color.c\nnew file mode 100644\nindex 0000000000..897a10a342\n--- /dev/null\n+++ b/lib/log/log_color.c\n@@ -0,0 +1,148 @@\n+/* SPDX-License-Identifier: BSD-3-Clause */\n+\n+#include <limits.h>\n+#include <stdbool.h>\n+#include <stdio.h>\n+#include <stdint.h>\n+#include <stdarg.h>\n+#include <stdlib.h>\n+#include <string.h>\n+\n+#include <rte_common.h>\n+#include <rte_log.h>\n+\n+#ifdef RTE_EXEC_ENV_WINDOWS\n+#include <rte_os_shim.h>\n+#endif\n+\n+#include \"log_internal.h\"\n+#include \"log_private.h\"\n+\n+enum {\n+\tLOG_COLOR_AUTO = 0,\t/* default */\n+\tLOG_COLOR_NEVER,\n+\tLOG_COLOR_ALWAYS,\n+} log_color_mode;\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+/* Standard terminal escape codes for colors and bold */\n+static const char * const color_code[] = {\n+\t[COLOR_NONE]\t= \"\",\n+\t[COLOR_RED]\t= \"\\033[31m\",\n+\t[COLOR_GREEN]\t= \"\\033[32m\",\n+\t[COLOR_YELLOW]\t= \"\\033[33m\",\n+\t[COLOR_BLUE]\t= \"\\033[34m\",\n+\t[COLOR_MAGENTA] = \"\\033[35m\",\n+\t[COLOR_CYAN] = \"\\033[36m\",\n+\t[COLOR_WHITE]\t= \"\\033[37m\",\n+\t[COLOR_BOLD]\t= \"\\033[1m\",\n+\t[COLOR_CLEAR]\t= \"\\033[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+/*\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\tlog_color_mode = LOG_COLOR_ALWAYS;\n+\telse if (strcmp(mode, \"never\") == 0)\n+\t\tlog_color_mode = LOG_COLOR_NEVER;\n+\telse if (strcmp(mode, \"auto\") == 0)\n+\t\tlog_color_mode = LOG_COLOR_AUTO;\n+\telse\n+\t\treturn -1;\n+\n+\treturn 0;\n+}\n+\n+bool\n+log_color_enabled(bool is_terminal)\n+{\n+\tswitch (log_color_mode) {\n+\tdefault:\n+\tcase LOG_COLOR_NEVER:\n+\t\treturn false;\n+\tcase LOG_COLOR_ALWAYS:\n+\t\treturn true;\n+\tcase LOG_COLOR_AUTO:\n+\t\treturn is_terminal;\n+\t}\n+}\n+\n+int\n+color_print(FILE *f, const char *format, va_list ap)\n+{\n+\tconst int level = rte_log_cur_msg_loglevel();\n+\tchar *cp, *msg;\n+\tchar buf[LINE_MAX];\n+\tint ret = 0;\n+\n+\tvsnprintf(buf, LINE_MAX, format, ap);\n+\tmsg = buf;\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(f, 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+int\n+color_print_with_timestamp(FILE *f,\n+\t\t\t const char *format, va_list ap)\n+{\n+\tchar tsbuf[128];\n+\n+\tif (log_timestamp(tsbuf, sizeof(tsbuf)) > 0)\n+\t\tcolor_fprintf(f, COLOR_GREEN, \"[%s] \", tsbuf);\n+\n+\treturn color_print(f, format, ap);\n+}\ndiff --git a/lib/log/log_internal.h b/lib/log/log_internal.h\nindex 82fdc21ac2..bba7041ea3 100644\n--- a/lib/log/log_internal.h\n+++ b/lib/log/log_internal.h\n@@ -50,5 +50,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/log_private.h b/lib/log/log_private.h\nindex 91976ab313..5952bfb453 100644\n--- a/lib/log/log_private.h\n+++ b/lib/log/log_private.h\n@@ -23,4 +23,12 @@ void log_syslog_open(const char *id, bool is_terminal);\n bool log_journal_enabled(void);\n void log_journal_open(const char *id);\n \n+bool log_color_enabled(bool is_tty);\n+\n+__rte_format_printf(2, 0)\n+int color_print(FILE *f, const char *format, va_list ap);\n+\n+__rte_format_printf(2, 0)\n+int color_print_with_timestamp(FILE *f, const char *format, va_list ap);\n+\n #endif /* LOG_PRIVATE_H */\ndiff --git a/lib/log/meson.build b/lib/log/meson.build\nindex 0a80027326..4041feed13 100644\n--- a/lib/log/meson.build\n+++ b/lib/log/meson.build\n@@ -4,6 +4,7 @@\n includes += global_inc\n sources = files(\n 'log.c',\n+ 'log_color.c',\n 'log_timestamp.c',\n )\n \ndiff --git a/lib/log/version.map b/lib/log/version.map\nindex 7af97ece43..eb3200dcde 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_journal;\n \teal_log_level2str;\n", "prefixes": [ "v20", "14/14" ] }