Cover Detail
Show a cover letter.
GET /api/covers/30/?format=api
{ "id": 30, "url": "http://patchwork.dpdk.org/api/covers/30/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/cover/20230809133553.1396119-1-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": "<20230809133553.1396119-1-bruce.richardson@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20230809133553.1396119-1-bruce.richardson@intel.com", "date": "2023-08-09T13:35:50", "name": "[v8,0/3] Split logging functionality out of EAL", "submitter": { "id": 20, "url": "http://patchwork.dpdk.org/api/people/20/?format=api", "name": "Bruce Richardson", "email": "bruce.richardson@intel.com" }, "mbox": "http://patchwork.dpdk.org/project/dpdk/cover/20230809133553.1396119-1-bruce.richardson@intel.com/mbox/", "series": [ { "id": 29143, "url": "http://patchwork.dpdk.org/api/series/29143/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/list/?series=29143", "date": "2023-08-09T13:35:50", "name": "Split logging functionality out of EAL", "version": 8, "mbox": "http://patchwork.dpdk.org/series/29143/mbox/" } ], "comments": "http://patchwork.dpdk.org/api/covers/30/comments/", "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 C717843018;\n\tWed, 9 Aug 2023 15:36:28 +0200 (CEST)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 5B38940691;\n\tWed, 9 Aug 2023 15:36:28 +0200 (CEST)", "from mgamail.intel.com (mgamail.intel.com [134.134.136.31])\n by mails.dpdk.org (Postfix) with ESMTP id 461F8400D6\n for <dev@dpdk.org>; Wed, 9 Aug 2023 15:36:26 +0200 (CEST)", "from fmsmga001.fm.intel.com ([10.253.24.23])\n by orsmga104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 09 Aug 2023 06:36:01 -0700", "from silpixa00401385.ir.intel.com ([10.237.214.156])\n by fmsmga001.fm.intel.com with ESMTP; 09 Aug 2023 06:36:03 -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=1691588186; x=1723124186;\n h=from:to:cc:subject:date:message-id:in-reply-to:\n references:mime-version:content-transfer-encoding;\n bh=++8M6qzAHk5+Mopi8W/Yw4IfxXErrxDdmMOMsJzE12U=;\n b=Iqujxl94U81Wstlv7F3kSLv+PQ++lFmzFeuUi70ohkmJas3X48D40VCq\n 2T0CGgqMohNag6in0orKXFbCrgY4/jFteGe9ccErHc3vS7yoba/8NEbhd\n LDfZByXwdJ6+lfiJmhDzvTVqDGydIoMLVl4IZytDER9+ktc0/coNYktRC\n SuN9ALD+OW8iblMKLAhonQ/iu6WbRKpBdv8Q7WJNBUqhaDVUl+zsSJf2M\n C9ob39TJMTJr6aF6ti7t1v+E9m31nL1+zQVs7MilMgqiYpXx/gBLfANPh\n qfw9KGSZ7MJFE0OSMGYbuJ0pmLdAGLyUa4m8oqjzTgK+53j4JKY8zCN89 w==;", "X-IronPort-AV": [ "E=McAfee;i=\"6600,9927,10795\"; a=\"435015570\"", "E=Sophos;i=\"6.01,159,1684825200\"; d=\"scan'208\";a=\"435015570\"", "E=Sophos;i=\"6.01,202,1684825200\"; d=\"scan'208\";a=\"875261426\"" ], "X-ExtLoop1": "1", "From": "Bruce Richardson <bruce.richardson@intel.com>", "To": "dev@dpdk.org", "Cc": "david.marchand@redhat.com,\n\tBruce Richardson <bruce.richardson@intel.com>", "Subject": "[PATCH v8 0/3] Split logging functionality out of EAL", "Date": "Wed, 9 Aug 2023 14:35:50 +0100", "Message-Id": "<20230809133553.1396119-1-bruce.richardson@intel.com>", "X-Mailer": "git-send-email 2.39.2", "In-Reply-To": "<20220829151901.376754-1-bruce.richardson@intel.com>", "References": "<20220829151901.376754-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": "There is a general desire to reduce the size and scope of EAL. To this\nend, this patchset makes a (very) small step in that direction by taking\nthe logging functionality out of EAL and putting it into its own library\nthat can be built and maintained separately.\n\nAs with the first RFC for this, the main obstacle is the \"fnmatch\"\nfunction which is needed by both EAL and the new log function when\nbuilding on windows. While the function cannot stay in EAL - or we would\nhave a circular dependency, moving it to a new library or just putting\nit in the log library have the disadvantages that it then \"leaks\" into\nthe public namespace without an rte_prefix, which could cause issues.\nSince only a single function is involved, subsequent versions take a\ndifferent approach to v1, and just moves the offending function to be a\nstatic function in a header file. This allows use by multiple libs\nwithout conflicting names or making it public.\n\nThe other complication, as explained in v1 RFC was that of multiple\nimplementations for different OS's. This is solved here in the same\nway as v1, by including the OS in the name and having meson pick the\ncorrect file for each build. Since only one file is involved, there\nseemed little need for replicating EAL's separate subdirectories\nper-OS.\n\nV8:\n* Added \"inline\" to static functions in fnmatch header\n* Removed SCCS tag as unneeded carryover from .c file\n* Corrected doc cross-references and headers\n* Added maintainers entry\n\nV7:\n* re-submit to re-run CI with ABI checks disabled\n\nv6:\n* Updated ABI version to DPDK_24 for new log library for 23.11 release.\n\nv5:\n* rebased to latest main branch\n* fixed trailing whitespace issues in new doc section\n\nv4:\n* Fixed windows build error, due to missing strdup (_strdup on windows)\n* Added doc updates to programmers guide.\n\nv3:\n* Fixed missing log file for BSD\n* Removed \"eal\" from the filenames of files in the log directory\n* added prefixes to elements in the fnmatch header to avoid conflicts\n* fixed space indentation in new lines in telemetry.c (checkpatch)\n* removed \"extern int logtype\" definition in telemetry.c (checkpatch)\n* added log directory to list for doxygen scanning\n\nBruce Richardson (3):\n eal/windows: move fnmatch function to header file\n log: separate logging functions out of EAL\n telemetry: use standard logging\n\n MAINTAINERS | 6 +-\n doc/api/doxy-api.conf.in | 1 +\n .../prog_guide/env_abstraction_layer.rst | 4 +-\n doc/guides/prog_guide/index.rst | 1 +\n doc/guides/prog_guide/log_lib.rst | 113 ++++++++++++\n lib/eal/common/eal_common_options.c | 2 +-\n lib/eal/common/eal_private.h | 7 -\n lib/eal/common/meson.build | 1 -\n lib/eal/freebsd/eal.c | 6 +-\n lib/eal/include/meson.build | 1 -\n lib/eal/linux/eal.c | 8 +-\n lib/eal/linux/meson.build | 1 -\n lib/eal/meson.build | 2 +-\n lib/eal/version.map | 17 --\n lib/eal/windows/eal.c | 2 +-\n lib/eal/windows/fnmatch.c | 172 ------------------\n lib/eal/windows/include/fnmatch.h | 169 +++++++++++++++--\n lib/eal/windows/meson.build | 2 -\n lib/kvargs/meson.build | 3 +-\n .../common/eal_common_log.c => log/log.c} | 7 +-\n lib/log/log_freebsd.c | 12 ++\n .../common/eal_log.h => log/log_internal.h} | 18 +-\n lib/{eal/linux/eal_log.c => log/log_linux.c} | 2 +-\n .../windows/eal_log.c => log/log_windows.c} | 2 +-\n lib/log/meson.build | 9 +\n lib/{eal/include => log}/rte_log.h | 0\n lib/log/version.map | 34 ++++\n lib/meson.build | 1 +\n lib/telemetry/meson.build | 3 +-\n lib/telemetry/telemetry.c | 11 +-\n lib/telemetry/telemetry_internal.h | 3 +-\n 31 files changed, 367 insertions(+), 253 deletions(-)\n create mode 100644 doc/guides/prog_guide/log_lib.rst\n delete mode 100644 lib/eal/windows/fnmatch.c\n rename lib/{eal/common/eal_common_log.c => log/log.c} (99%)\n create mode 100644 lib/log/log_freebsd.c\n rename lib/{eal/common/eal_log.h => log/log_internal.h} (69%)\n rename lib/{eal/linux/eal_log.c => log/log_linux.c} (97%)\n rename lib/{eal/windows/eal_log.c => log/log_windows.c} (93%)\n create mode 100644 lib/log/meson.build\n rename lib/{eal/include => log}/rte_log.h (100%)\n create mode 100644 lib/log/version.map\n\n--\n2.39.2" }