Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/107272/?format=api
{ "id": 107272, "url": "http://patches.dpdk.org/api/patches/107272/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20220210154239.587185-2-bruce.richardson@intel.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": "<20220210154239.587185-2-bruce.richardson@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20220210154239.587185-2-bruce.richardson@intel.com", "date": "2022-02-10T15:42:33", "name": "[v4,1/7] eal: fix header build with C++", "commit_ref": null, "pull_url": null, "state": "accepted", "archived": true, "hash": "75edc81f5f82923183067638161e15c054e5ccb0", "submitter": { "id": 20, "url": "http://patches.dpdk.org/api/people/20/?format=api", "name": "Bruce Richardson", "email": "bruce.richardson@intel.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/20220210154239.587185-2-bruce.richardson@intel.com/mbox/", "series": [ { "id": 21607, "url": "http://patches.dpdk.org/api/series/21607/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=21607", "date": "2022-02-10T15:42:32", "name": "Verify C++ compatibility of public headers", "version": 4, "mbox": "http://patches.dpdk.org/series/21607/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/107272/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/107272/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 EF54AA00BE;\n\tThu, 10 Feb 2022 16:43:05 +0100 (CET)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id DFA764274B;\n\tThu, 10 Feb 2022 16:43:05 +0100 (CET)", "from mga11.intel.com (mga11.intel.com [192.55.52.93])\n by mails.dpdk.org (Postfix) with ESMTP id 71C5B426ED;\n Thu, 10 Feb 2022 16:43:04 +0100 (CET)", "from orsmga006.jf.intel.com ([10.7.209.51])\n by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 10 Feb 2022 07:42:53 -0800", "from silpixa00399126.ir.intel.com ([10.237.223.162])\n by orsmga006.jf.intel.com with ESMTP; 10 Feb 2022 07:42:47 -0800" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple;\n d=intel.com; i=@intel.com; q=dns/txt; s=Intel;\n t=1644507784; x=1676043784;\n h=from:to:cc:subject:date:message-id:in-reply-to:\n references:mime-version:content-transfer-encoding;\n bh=T0owC88/PabvnVJE2FcQA+oqFbMvWokbTc1k9v9LjfU=;\n b=GJOouaap9oH3t9kYVcpZ7djVT5CmE1C09/blZRF77+Z+F6PtmQB90Bob\n Va6CshiaqFPDEDZ+EVvhZfPGzqKuw2+o62Eaz2c5xWmRYDGy7Nzd0bn4J\n wMUy5c8vZgYFPKDJTwkioBs9EXHwBhYqAOZL9tpKJzhfiKPPnei0cbTP5\n 9jcTc5V0gEDwn6SdHhLfOy7ujtaIaGJ+K7mzgKHzpb5PLg+vDq2+fKU9o\n Qoj2BUiBlaZQ6Fbv0FWY+eEPuf5c/Ftj91uS9PBxrkbH9qKHPJXhPBxTr\n AlqWnsvaVbcVNuhezqsrSCTpzKOjPNbARtJ2nxg2d5ZIDhYX3bU8ELh02 g==;", "X-IronPort-AV": [ "E=McAfee;i=\"6200,9189,10254\"; a=\"247101111\"", "E=Sophos;i=\"5.88,359,1635231600\"; d=\"scan'208\";a=\"247101111\"", "E=Sophos;i=\"5.88,359,1635231600\"; d=\"scan'208\";a=\"485723380\"" ], "X-ExtLoop1": "1", "From": "Bruce Richardson <bruce.richardson@intel.com>", "To": "dev@dpdk.org", "Cc": "Bruce Richardson <bruce.richardson@intel.com>, joyce.kong@arm.com,\n jerinj@marvell.com, stable@dpdk.org, Gavin Hu <gavin.hu@arm.com>,\n Konstantin Ananyev <konstantin.ananyev@intel.com>,\n Honnappa Nagarahalli <honnappa.nagarahalli@arm.com>,\n Ola Liljedahl <ola.liljedahl@arm.com>,\n Sunil Kumar Kori <skori@marvell.com>,\n David Marchand <david.marchand@redhat.com>", "Subject": "[PATCH v4 1/7] eal: fix header build with C++", "Date": "Thu, 10 Feb 2022 15:42:33 +0000", "Message-Id": "<20220210154239.587185-2-bruce.richardson@intel.com>", "X-Mailer": "git-send-email 2.32.0", "In-Reply-To": "<20220210154239.587185-1-bruce.richardson@intel.com>", "References": "<20220204174209.440207-1-bruce.richardson@intel.com>\n <20220210154239.587185-1-bruce.richardson@intel.com>", "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": "C++ files could not include some headers because:\n\n* \"new\" is a keyword in C++, so can't be a variable name\n* there is no automatic casting to/from void *\n\nFixes: 184104fc6121 (\"ticketlock: introduce fair ticket based locking\")\nFixes: 032a7e5499a0 (\"trace: implement provider payload\")\nCc: joyce.kong@arm.com\nCc: jerinj@marvell.com\nCc: stable@dpdk.org\n\nSigned-off-by: Bruce Richardson <bruce.richardson@intel.com>\n---\n lib/eal/include/generic/rte_ticketlock.h | 14 +++++++-------\n lib/eal/include/rte_trace_point.h | 2 +-\n 2 files changed, 8 insertions(+), 8 deletions(-)", "diff": "diff --git a/lib/eal/include/generic/rte_ticketlock.h b/lib/eal/include/generic/rte_ticketlock.h\nindex c1b8808f51..693c67b517 100644\n--- a/lib/eal/include/generic/rte_ticketlock.h\n+++ b/lib/eal/include/generic/rte_ticketlock.h\n@@ -91,13 +91,13 @@ rte_ticketlock_unlock(rte_ticketlock_t *tl)\n static inline int\n rte_ticketlock_trylock(rte_ticketlock_t *tl)\n {\n-\trte_ticketlock_t old, new;\n-\told.tickets = __atomic_load_n(&tl->tickets, __ATOMIC_RELAXED);\n-\tnew.tickets = old.tickets;\n-\tnew.s.next++;\n-\tif (old.s.next == old.s.current) {\n-\t\tif (__atomic_compare_exchange_n(&tl->tickets, &old.tickets,\n-\t\t new.tickets, 0, __ATOMIC_ACQUIRE, __ATOMIC_RELAXED))\n+\trte_ticketlock_t oldl, newl;\n+\toldl.tickets = __atomic_load_n(&tl->tickets, __ATOMIC_RELAXED);\n+\tnewl.tickets = oldl.tickets;\n+\tnewl.s.next++;\n+\tif (oldl.s.next == oldl.s.current) {\n+\t\tif (__atomic_compare_exchange_n(&tl->tickets, &oldl.tickets,\n+\t\t newl.tickets, 0, __ATOMIC_ACQUIRE, __ATOMIC_RELAXED))\n \t\t\treturn 1;\n \t}\n \ndiff --git a/lib/eal/include/rte_trace_point.h b/lib/eal/include/rte_trace_point.h\nindex e226f073f7..0f8700974f 100644\n--- a/lib/eal/include/rte_trace_point.h\n+++ b/lib/eal/include/rte_trace_point.h\n@@ -370,7 +370,7 @@ do { \\\n do { \\\n \tif (unlikely(in == NULL)) \\\n \t\treturn; \\\n-\trte_strscpy(mem, in, __RTE_TRACE_EMIT_STRING_LEN_MAX); \\\n+\trte_strscpy((char *)mem, in, __RTE_TRACE_EMIT_STRING_LEN_MAX); \\\n \tmem = RTE_PTR_ADD(mem, __RTE_TRACE_EMIT_STRING_LEN_MAX); \\\n } while (0)\n \n", "prefixes": [ "v4", "1/7" ] }