Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/129634/?format=api
{ "id": 129634, "url": "http://patchwork.dpdk.org/api/patches/129634/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/patch/20230718164802.110560-3-bruce.richardson@intel.com/", "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": "<20230718164802.110560-3-bruce.richardson@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20230718164802.110560-3-bruce.richardson@intel.com", "date": "2023-07-18T16:48:02", "name": "[v3,2/2] doc/contributing: guidelines for logging, tracing and telemetry", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "56ab101644a25aa12cd1fc2aff30dae8f760c308", "submitter": { "id": 20, "url": "http://patchwork.dpdk.org/api/people/20/?format=api", "name": "Bruce Richardson", "email": "bruce.richardson@intel.com" }, "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/20230718164802.110560-3-bruce.richardson@intel.com/mbox/", "series": [ { "id": 28968, "url": "http://patchwork.dpdk.org/api/series/28968/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/list/?series=28968", "date": "2023-07-18T16:48:00", "name": "Improve docs on getting info on running process", "version": 3, "mbox": "http://patchwork.dpdk.org/series/28968/mbox/" } ], "comments": "http://patchwork.dpdk.org/api/patches/129634/comments/", "check": "fail", "checks": "http://patchwork.dpdk.org/api/patches/129634/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 458B942EAA;\n\tTue, 18 Jul 2023 18:48:28 +0200 (CEST)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 0DA5442D85;\n\tTue, 18 Jul 2023 18:48:19 +0200 (CEST)", "from mga12.intel.com (mga12.intel.com [192.55.52.136])\n by mails.dpdk.org (Postfix) with ESMTP id EB27C42D7E\n for <dev@dpdk.org>; Tue, 18 Jul 2023 18:48:16 +0200 (CEST)", "from fmsmga005.fm.intel.com ([10.253.24.32])\n by fmsmga106.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 18 Jul 2023 09:48:16 -0700", "from unknown (HELO silpixa00401385.ir.intel.com) ([10.237.214.14])\n by fmsmga005.fm.intel.com with ESMTP; 18 Jul 2023 09:48:15 -0700" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple;\n d=intel.com; i=@intel.com; q=dns/txt; s=Intel;\n t=1689698897; x=1721234897;\n h=from:to:cc:subject:date:message-id:in-reply-to:\n references:mime-version:content-transfer-encoding;\n bh=bWtXtYWbOjLVmLVdKE0cU7F6l/kgO6Ypqs7/0vFTCiU=;\n b=X/FusVuApPSLv3x6Vtc4GCO6bqgEpRq/0UCo1MArl4nClBetgLukTP/U\n KG8FgOSHUEtLaRQjSpW6PfdJmN9byiPGwsuy8nf3FQArNRRLv8C0YiyQH\n omY7tQS0dxORMondq9xUkagY05bLUkKWBFhHjQJLgXDaP8EQYnVcqgCrV\n 7O82I46OmdIUR2had2XTenlK6LMtN1iB/0izJqIcx2fElO3hW4OYiJ7NO\n BM0ax3hl/FW1QNcPB/z70A4uw+DPgeWe4CpbSsEpsvRR7Kl+cXCNf0HWH\n p4p040OsQznw7BDtnZLp/4IYwWgE4CJBHzOQn0DakynxHbDn9P+Krsgeu Q==;", "X-IronPort-AV": [ "E=McAfee;i=\"6600,9927,10775\"; a=\"345848549\"", "E=Sophos;i=\"6.01,214,1684825200\"; d=\"scan'208\";a=\"345848549\"", "E=McAfee;i=\"6600,9927,10775\"; a=\"1054367390\"", "E=Sophos;i=\"6.01,214,1684825200\"; d=\"scan'208\";a=\"1054367390\"" ], "X-ExtLoop1": "1", "From": "Bruce Richardson <bruce.richardson@intel.com>", "To": "dev@dpdk.org", "Cc": "david.marchand@redhat.com, Bruce Richardson <bruce.richardson@intel.com>,\n\t=?utf-8?q?Morten_Br=C3=B8rup?= <mb@smartsharesystems.com>", "Subject": "[PATCH v3 2/2] doc/contributing: guidelines for logging,\n tracing and telemetry", "Date": "Tue, 18 Jul 2023 17:48:02 +0100", "Message-Id": "<20230718164802.110560-3-bruce.richardson@intel.com>", "X-Mailer": "git-send-email 2.39.2", "In-Reply-To": "<20230718164802.110560-1-bruce.richardson@intel.com>", "References": "<20230613143355.77914-1-bruce.richardson@intel.com>\n <20230718164802.110560-1-bruce.richardson@intel.com>", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=UTF-8", "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": "As discussed by DPDK technical board [1], our contributor guide should\ninclude some details as to when to use logging vs tracing vs telemetry\nto provide the end user with information about the running process and\nthe DPDK libraries it uses.\n\n[1] https://mails.dpdk.org/archives/dev/2023-March/265204.html\n\nSigned-off-by: Bruce Richardson <bruce.richardson@intel.com>\nAcked-by: Morten Brørup <mb@smartsharesystems.com>\nReviewed-by: David Marchand <david.marchand@redhat.com>\n---\n doc/guides/contributing/coding_style.rst | 2 +\n doc/guides/contributing/design.rst | 49 ++++++++++++++++++++++++\n doc/guides/prog_guide/telemetry_lib.rst | 2 +\n doc/guides/prog_guide/trace_lib.rst | 2 +\n 4 files changed, 55 insertions(+)", "diff": "diff --git a/doc/guides/contributing/coding_style.rst b/doc/guides/contributing/coding_style.rst\nindex f41f88c632..2d8d9a1469 100644\n--- a/doc/guides/contributing/coding_style.rst\n+++ b/doc/guides/contributing/coding_style.rst\n@@ -794,6 +794,8 @@ Control Statements\n /* NOTREACHED */\n }\n \n+.. _dynamic_logging:\n+\n Dynamic Logging\n ---------------\n \ndiff --git a/doc/guides/contributing/design.rst b/doc/guides/contributing/design.rst\nindex d24a7ff6a0..5a4c4b7ed1 100644\n--- a/doc/guides/contributing/design.rst\n+++ b/doc/guides/contributing/design.rst\n@@ -4,6 +4,7 @@\n Design\n ======\n \n+\n Environment or Architecture-specific Sources\n --------------------------------------------\n \n@@ -68,6 +69,54 @@ Adding a new static field or flag must be an exception matching many criteria\n like (non exhaustive): wide usage, performance, size.\n \n \n+Runtime Information - Logging, Tracing and Telemetry\n+-------------------------------------------------------\n+\n+It is often desirable to provide information to the end-user as to what is happening to the application at runtime.\n+DPDK provides a number of built-in mechanisms to provide this introspection:\n+\n+* :ref:`Logging<dynamic_logging>`\n+* :ref:`Tracing<trace_library>`\n+* :ref:`Telemetry<telemetry_library>`\n+\n+Each of these has its own strengths and suitabilities for use within DPDK components.\n+\n+Below are some guidelines for when each should be used:\n+\n+* For reporting error conditions, or other abnormal runtime issues, *logging* should be used.\n+ Depending on the severity of the issue, the appropriate log level, for example,\n+ ``ERROR``, ``WARNING`` or ``NOTICE``, should be used.\n+\n+.. note::\n+\n+ Drivers of all classes, including both bus and device drivers,\n+ should not output any log information if the hardware they support is not present.\n+ This is to avoid any changes in output for existing users when a new driver is added to DPDK.\n+\n+* For component initialization, or other cases where a path through the code is only likely to be taken once,\n+ either *logging* at ``DEBUG`` level or *tracing* may be used, or potentially both.\n+ In the latter case, tracing can provide basic information as to the code path taken,\n+ with debug-level logging providing additional details on internal state,\n+ not possible to emit via tracing.\n+\n+* For a component's data-path, where a path is to be taken multiple times within a short timeframe,\n+ *tracing* should be used.\n+ Since DPDK tracing uses `Common Trace Format <https://diamon.org/ctf/>`_ for its tracing logs,\n+ post-analysis can be done using a range of external tools.\n+\n+* For numerical or statistical data generated by a component, for example, per-packet statistics,\n+ *telemetry* should be used.\n+\n+* For any data where the data may need to be gathered at any point in the execution to help assess the state of the application component,\n+ for example, core configuration, device information, *telemetry* should be used.\n+ Telemetry callbacks should not modify any program state, but be \"read-only\".\n+\n+Many libraries also include a ``rte_<libname>_dump()`` function as part of their API,\n+writing verbose internal details to a given file-handle.\n+New libraries are encouraged to provide such functions where it makes sense to do so,\n+as they provide an additional application-controlled mechanism to get details of the internals of a DPDK component.\n+\n+\n Library Statistics\n ------------------\n \ndiff --git a/doc/guides/prog_guide/telemetry_lib.rst b/doc/guides/prog_guide/telemetry_lib.rst\nindex 32f525a67f..71f8bd735e 100644\n--- a/doc/guides/prog_guide/telemetry_lib.rst\n+++ b/doc/guides/prog_guide/telemetry_lib.rst\n@@ -1,6 +1,8 @@\n .. SPDX-License-Identifier: BSD-3-Clause\n Copyright(c) 2020 Intel Corporation.\n \n+.. _telemetry_library:\n+\n Telemetry Library\n =================\n \ndiff --git a/doc/guides/prog_guide/trace_lib.rst b/doc/guides/prog_guide/trace_lib.rst\nindex e5718feddc..a3b8a7c2eb 100644\n--- a/doc/guides/prog_guide/trace_lib.rst\n+++ b/doc/guides/prog_guide/trace_lib.rst\n@@ -1,6 +1,8 @@\n .. SPDX-License-Identifier: BSD-3-Clause\n Copyright(C) 2020 Marvell International Ltd.\n \n+.. _trace_library:\n+\n Trace Library\n =============\n \n", "prefixes": [ "v3", "2/2" ] }