Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/135220/?format=api
{ "id": 135220, "url": "http://patches.dpdk.org/api/patches/135220/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20231215064355.1429709-2-adwivedi@marvell.com/", "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": "<20231215064355.1429709-2-adwivedi@marvell.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20231215064355.1429709-2-adwivedi@marvell.com", "date": "2023-12-15T06:43:54", "name": "[v6,1/2] devtools: move build map changes function", "commit_ref": null, "pull_url": null, "state": "new", "archived": false, "hash": "ac036da8c1705b05ab3888737f15f2b7e5f3a038", "submitter": { "id": 1561, "url": "http://patches.dpdk.org/api/people/1561/?format=api", "name": "Ankur Dwivedi", "email": "adwivedi@marvell.com" }, "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/20231215064355.1429709-2-adwivedi@marvell.com/mbox/", "series": [ { "id": 30572, "url": "http://patches.dpdk.org/api/series/30572/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=30572", "date": "2023-12-15T06:43:53", "name": "devtools: add tracepoint check in checkpatch", "version": 6, "mbox": "http://patches.dpdk.org/series/30572/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/135220/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/135220/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 7C9C5436F8;\n\tFri, 15 Dec 2023 07:44:20 +0100 (CET)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id EE4C9432C6;\n\tFri, 15 Dec 2023 07:44:18 +0100 (CET)", "from mx0b-0016f401.pphosted.com (mx0b-0016f401.pphosted.com\n [67.231.156.173])\n by mails.dpdk.org (Postfix) with ESMTP id 04833432A6\n for <dev@dpdk.org>; Fri, 15 Dec 2023 07:44:17 +0100 (CET)", "from pps.filterd (m0045851.ppops.net [127.0.0.1])\n by mx0b-0016f401.pphosted.com (8.17.1.24/8.17.1.24) with ESMTP id\n 3BF5EV8A020537; Thu, 14 Dec 2023 22:44:17 -0800", "from dc5-exch02.marvell.com ([199.233.59.182])\n by mx0b-0016f401.pphosted.com (PPS) with ESMTPS id 3v06m0t92x-1\n (version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NOT);\n Thu, 14 Dec 2023 22:44:17 -0800 (PST)", "from DC5-EXCH02.marvell.com (10.69.176.39) by DC5-EXCH02.marvell.com\n (10.69.176.39) with Microsoft SMTP Server (TLS) id 15.0.1497.48;\n Thu, 14 Dec 2023 22:44:15 -0800", "from maili.marvell.com (10.69.176.80) by DC5-EXCH02.marvell.com\n (10.69.176.39) with Microsoft SMTP Server id 15.0.1497.48 via Frontend\n Transport; Thu, 14 Dec 2023 22:44:15 -0800", "from localhost.localdomain (unknown [10.28.36.185])\n by maili.marvell.com (Postfix) with ESMTP id B0F2F3F706F;\n Thu, 14 Dec 2023 22:44:13 -0800 (PST)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.com; h=\n from:to:cc:subject:date:message-id:in-reply-to:references\n :mime-version:content-transfer-encoding:content-type; s=\n pfpt0220; bh=9OHPL+/HLP84ouTIr8hlV8TWuzRfyEQkcA8aWMZuvnI=; b=Dtc\n bo3jGD8ULB1caip7I/GQhBv91IoyVqAS0Kn0jsGdGusHopLkcUFY5Em1nWdrvp8h\n 0xX9EKdVAswzdeElZ+6uUxMCnHeCloczC8CeASQKRuMdDG1TdY272VQWMpCvTTHn\n T3MdNQJpJai1APMOktnFq1cV3LMo6mJfG9JWzDFawu8W4FMT+sFokIiaBvGA3Tdy\n O5czxyNZqtzjx4LQROXbqYtevNOf5QLWyjle428FAhCCnQG6xnIIngN4V2RtpREN\n Wsc0OVSSEoe0/DmNW0JvynhT9xRbTwUi+c9NeQBgHj4oQdvJ4vx2te1+B7NrtCJ8\n ECIIj24gsvAEAqalDlA==", "From": "Ankur Dwivedi <adwivedi@marvell.com>", "To": "<dev@dpdk.org>", "CC": "<thomas@monjalon.net>, <jerinj@marvell.com>, Ankur Dwivedi\n <adwivedi@marvell.com>", "Subject": "[PATCH v6 1/2] devtools: move build map changes function", "Date": "Fri, 15 Dec 2023 12:13:54 +0530", "Message-ID": "<20231215064355.1429709-2-adwivedi@marvell.com>", "X-Mailer": "git-send-email 2.25.1", "In-Reply-To": "<20231215064355.1429709-1-adwivedi@marvell.com>", "References": "<20230307120514.2774917-1-adwivedi@marvell.com>\n <20231215064355.1429709-1-adwivedi@marvell.com>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Content-Type": "text/plain", "X-Proofpoint-GUID": "4kzn9WokSquWuKasTLMxWL8kSaMfUD4W", "X-Proofpoint-ORIG-GUID": "4kzn9WokSquWuKasTLMxWL8kSaMfUD4W", "X-Proofpoint-Virus-Version": "vendor=baseguard\n engine=ICAP:2.0.272,Aquarius:18.0.997,Hydra:6.0.619,FMLib:17.11.176.26\n definitions=2023-12-09_02,2023-12-07_01,2023-05-22_02", "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": "This patch moves the build_map_changes function from\ncheck-symbol-change.sh to a new symbol-map-util.sh file. This function\ncan be used in other scripts by including symbol-map-util.sh file. The\nlicense in check-symbol-change.sh is copied to symbol-map-util.sh.\n\nSigned-off-by: Ankur Dwivedi <adwivedi@marvell.com>\n---\n devtools/check-symbol-change.sh | 76 +-------------------------------\n devtools/symbol-map-util.sh | 78 +++++++++++++++++++++++++++++++++\n 2 files changed, 80 insertions(+), 74 deletions(-)\n create mode 100644 devtools/symbol-map-util.sh", "diff": "diff --git a/devtools/check-symbol-change.sh b/devtools/check-symbol-change.sh\nindex 8992214ac8..68613f63cc 100755\n--- a/devtools/check-symbol-change.sh\n+++ b/devtools/check-symbol-change.sh\n@@ -2,80 +2,8 @@\n # SPDX-License-Identifier: BSD-3-Clause\n # Copyright(c) 2018 Neil Horman <nhorman@tuxdriver.com>\n \n-build_map_changes()\n-{\n-\tlocal fname=\"$1\"\n-\tlocal mapdb=\"$2\"\n-\n-\tcat \"$fname\" | awk '\n-\t\t# Initialize our variables\n-\t\tBEGIN {map=\"\";sym=\"\";ar=\"\";sec=\"\"; in_sec=0; in_map=0}\n-\n-\t\t# Anything that starts with + or -, followed by an a\n-\t\t# and ends in the string .map is the name of our map file\n-\t\t# This may appear multiple times in a patch if multiple\n-\t\t# map files are altered, and all section/symbol names\n-\t\t# appearing between a triggering of this rule and the\n-\t\t# next trigger of this rule are associated with this file\n-\t\t/[-+] [ab]\\/.*\\.map/ {map=$2; in_map=1; next}\n-\n-\t\t# The previous rule catches all .map files, anything else\n-\t\t# indicates we left the map chunk.\n-\t\t/[-+] [ab]\\// {in_map=0}\n-\n-\t\t# Triggering this rule, which starts a line and ends it\n-\t\t# with a { identifies a versioned section. The section name is\n-\t\t# the rest of the line with the + and { symbols removed.\n-\t\t# Triggering this rule sets in_sec to 1, which actives the\n-\t\t# symbol rule below\n-\t\t/^.*{/ {\n-\t\t\tgsub(\"+\", \"\");\n-\t\t\tif (in_map == 1) {\n-\t\t\t\tsec=$(NF-1); in_sec=1;\n-\t\t\t}\n-\t\t}\n-\n-\t\t# This rule identifies the end of a section, and disables the\n-\t\t# symbol rule\n-\t\t/.*}/ {in_sec=0}\n-\n-\t\t# This rule matches on a + followed by any characters except a :\n-\t\t# (which denotes a global vs local segment), and ends with a ;.\n-\t\t# The semicolon is removed and the symbol is printed with its\n-\t\t# association file name and version section, along with an\n-\t\t# indicator that the symbol is a new addition. Note this rule\n-\t\t# only works if we have found a version section in the rule\n-\t\t# above (hence the in_sec check) And found a map file (the\n-\t\t# in_map check). If we are not in a map chunk, do nothing. If\n-\t\t# we are in a map chunk but not a section chunk, record it as\n-\t\t# unknown.\n-\t\t/^+[^}].*[^:*];/ {gsub(\";\",\"\");sym=$2;\n-\t\t\tif (in_map == 1) {\n-\t\t\t\tif (in_sec == 1) {\n-\t\t\t\t\tprint map \" \" sym \" \" sec \" add\"\n-\t\t\t\t} else {\n-\t\t\t\t\tprint map \" \" sym \" unknown add\"\n-\t\t\t\t}\n-\t\t\t}\n-\t\t}\n-\n-\t\t# This is the same rule as above, but the rule matches on a\n-\t\t# leading - rather than a +, denoting that the symbol is being\n-\t\t# removed.\n-\t\t/^-[^}].*[^:*];/ {gsub(\";\",\"\");sym=$2;\n-\t\t\tif (in_map == 1) {\n-\t\t\t\tif (in_sec == 1) {\n-\t\t\t\t\tprint map \" \" sym \" \" sec \" del\"\n-\t\t\t\t} else {\n-\t\t\t\t\tprint map \" \" sym \" unknown del\"\n-\t\t\t\t}\n-\t\t\t}\n-\t\t}' > \"$mapdb\"\n-\n-\t\tsort -u \"$mapdb\" > \"$mapdb.2\"\n-\t\tmv -f \"$mapdb.2\" \"$mapdb\"\n-\n-}\n+selfdir=$(dirname $(readlink -f $0))\n+. $selfdir/symbol-map-util.sh\n \n is_stable_section() {\n \t[ \"$1\" != 'EXPERIMENTAL' ] && [ \"$1\" != 'INTERNAL' ]\ndiff --git a/devtools/symbol-map-util.sh b/devtools/symbol-map-util.sh\nnew file mode 100644\nindex 0000000000..86e4279cdc\n--- /dev/null\n+++ b/devtools/symbol-map-util.sh\n@@ -0,0 +1,78 @@\n+#!/bin/sh\n+# SPDX-License-Identifier: BSD-3-Clause\n+# Copyright(c) 2018 Neil Horman <nhorman@tuxdriver.com>\n+\n+build_map_changes()\n+{\n+\tlocal fname=\"$1\"\n+\tlocal mapdb=\"$2\"\n+\n+\tcat \"$fname\" | awk '\n+\t\t# Initialize our variables\n+\t\tBEGIN {map=\"\";sym=\"\";ar=\"\";sec=\"\"; in_sec=0; in_map=0}\n+\n+\t\t# Anything that starts with + or -, followed by an a\n+\t\t# and ends in the string .map is the name of our map file\n+\t\t# This may appear multiple times in a patch if multiple\n+\t\t# map files are altered, and all section/symbol names\n+\t\t# appearing between a triggering of this rule and the\n+\t\t# next trigger of this rule are associated with this file\n+\t\t/[-+] [ab]\\/.*\\.map/ {map=$2; in_map=1; next}\n+\n+\t\t# The previous rule catches all .map files, anything else\n+\t\t# indicates we left the map chunk.\n+\t\t/[-+] [ab]\\// {in_map=0}\n+\n+\t\t# Triggering this rule, which starts a line and ends it\n+\t\t# with a { identifies a versioned section. The section name is\n+\t\t# the rest of the line with the + and { symbols removed.\n+\t\t# Triggering this rule sets in_sec to 1, which actives the\n+\t\t# symbol rule below\n+\t\t/^.*{/ {\n+\t\t\tgsub(\"+\", \"\");\n+\t\t\tif (in_map == 1) {\n+\t\t\t\tsec=$(NF-1); in_sec=1;\n+\t\t\t}\n+\t\t}\n+\n+\t\t# This rule identifies the end of a section, and disables the\n+\t\t# symbol rule\n+\t\t/.*}/ {in_sec=0}\n+\n+\t\t# This rule matches on a + followed by any characters except a :\n+\t\t# (which denotes a global vs local segment), and ends with a ;.\n+\t\t# The semicolon is removed and the symbol is printed with its\n+\t\t# association file name and version section, along with an\n+\t\t# indicator that the symbol is a new addition. Note this rule\n+\t\t# only works if we have found a version section in the rule\n+\t\t# above (hence the in_sec check) And found a map file (the\n+\t\t# in_map check). If we are not in a map chunk, do nothing. If\n+\t\t# we are in a map chunk but not a section chunk, record it as\n+\t\t# unknown.\n+\t\t/^+[^}].*[^:*];/ {gsub(\";\",\"\");sym=$2;\n+\t\t\tif (in_map == 1) {\n+\t\t\t\tif (in_sec == 1) {\n+\t\t\t\t\tprint map \" \" sym \" \" sec \" add\"\n+\t\t\t\t} else {\n+\t\t\t\t\tprint map \" \" sym \" unknown add\"\n+\t\t\t\t}\n+\t\t\t}\n+\t\t}\n+\n+\t\t# This is the same rule as above, but the rule matches on a\n+\t\t# leading - rather than a +, denoting that the symbol is being\n+\t\t# removed.\n+\t\t/^-[^}].*[^:*];/ {gsub(\";\",\"\");sym=$2;\n+\t\t\tif (in_map == 1) {\n+\t\t\t\tif (in_sec == 1) {\n+\t\t\t\t\tprint map \" \" sym \" \" sec \" del\"\n+\t\t\t\t} else {\n+\t\t\t\t\tprint map \" \" sym \" unknown del\"\n+\t\t\t\t}\n+\t\t\t}\n+\t\t}' > \"$mapdb\"\n+\n+\t\tsort -u \"$mapdb\" > \"$mapdb.2\"\n+\t\tmv -f \"$mapdb.2\" \"$mapdb\"\n+\n+}\n", "prefixes": [ "v6", "1/2" ] }