Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/139116/?format=api
https://patches.dpdk.org/api/patches/139116/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/patch/20240405111310.3280084-1-sebastian.brzezinka@intel.com/", "project": { "id": 1, "url": "https://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": "<20240405111310.3280084-1-sebastian.brzezinka@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20240405111310.3280084-1-sebastian.brzezinka@intel.com", "date": "2024-04-05T11:13:10", "name": "meson/mlx5: Suppress -Wunused-value diagnostic", "commit_ref": null, "pull_url": null, "state": "not-applicable", "archived": false, "hash": "c9cd0c09f15bebdf72cb4b0cab503a26c845f06a", "submitter": { "id": 3289, "url": "https://patches.dpdk.org/api/people/3289/?format=api", "name": "Sebastian Brzezinka", "email": "sebastian.brzezinka@intel.com" }, "delegate": { "id": 3268, "url": "https://patches.dpdk.org/api/users/3268/?format=api", "username": "rasland", "first_name": "Raslan", "last_name": "Darawsheh", "email": "rasland@nvidia.com" }, "mbox": "https://patches.dpdk.org/project/dpdk/patch/20240405111310.3280084-1-sebastian.brzezinka@intel.com/mbox/", "series": [ { "id": 31685, "url": "https://patches.dpdk.org/api/series/31685/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/list/?series=31685", "date": "2024-04-05T11:13:10", "name": "meson/mlx5: Suppress -Wunused-value diagnostic", "version": 1, "mbox": "https://patches.dpdk.org/series/31685/mbox/" } ], "comments": "https://patches.dpdk.org/api/patches/139116/comments/", "check": "warning", "checks": "https://patches.dpdk.org/api/patches/139116/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 CF6A743E00;\n\tFri, 5 Apr 2024 13:13:19 +0200 (CEST)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id BD7E3402D4;\n\tFri, 5 Apr 2024 13:13:19 +0200 (CEST)", "from mgamail.intel.com (mgamail.intel.com [192.198.163.12])\n by mails.dpdk.org (Postfix) with ESMTP id BDEA6402CE\n for <dev@dpdk.org>; Fri, 5 Apr 2024 13:13:17 +0200 (CEST)", "from fmviesa010.fm.intel.com ([10.60.135.150])\n by fmvoesa106.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 05 Apr 2024 04:13:17 -0700", "from sbrzez.igk.intel.com ([10.211.11.150])\n by fmviesa010.fm.intel.com with ESMTP; 05 Apr 2024 04:13: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=1712315598; x=1743851598;\n h=from:to:cc:subject:date:message-id:mime-version:\n content-transfer-encoding;\n bh=WGUz4QKtj8EBIOWu7nVxdT14EtZ0T7rGo618OMwqPW4=;\n b=UnfBSF69DITLQlhdD6uBvTPRxXNPn9DmPKCxjlPzKZ1mekUP+afiE6kt\n oaRIHrcSEd7OmJHS47NxFqZBv9lYT8nmoYbbQI9expTA+XbfZwmLQnRzd\n dWaX6uJmsClnE2ZZNDzEZCLo2UwFTXNSIaEbdMliEExf5gGwSTszVzulU\n 37YbB/DWnTzNKsZ671eF3FUSu2IXtdSTcywagDgsjqNKUnXxaSgpMR+9Z\n ZfcUc/lF5qivzCjJs3/NH8rDf0HS3ZSTDaBJ0C2Mluj+3TotOKK4hBbRR\n EueD+bC2Hq5mMQ7W83Bj0Ntjb0KlGzfsZzn8wxclZ9cegokmtjN6ReMhH Q==;", "X-CSE-ConnectionGUID": [ "o3pT7LBFR6G2xczKBqJ1xw==", "wTmybQWRQdWFnyeZq9jBZg==" ], "X-CSE-MsgGUID": [ "r2vvf+cPTDKcce+3Rye82w==", "pyS7NcK6TpW8y0aHEUk5ZQ==" ], "X-IronPort-AV": [ "E=McAfee;i=\"6600,9927,11034\"; a=\"11415260\"", "E=Sophos;i=\"6.07,181,1708416000\"; d=\"scan'208\";a=\"11415260\"", "E=Sophos;i=\"6.07,181,1708416000\"; d=\"scan'208\";a=\"19051974\"" ], "X-ExtLoop1": "1", "From": "Sebastian Brzezinka <sebastian.brzezinka@intel.com>", "To": "dev@dpdk.org", "Cc": "Alexey Marchuk <alexeymar@mellanox.com>,\n Sebastian Brzezinka <sebastian.brzezinka@intel.com>", "Subject": "[PATCH] meson/mlx5: Suppress -Wunused-value diagnostic", "Date": "Fri, 5 Apr 2024 13:13:10 +0200", "Message-Id": "<20240405111310.3280084-1-sebastian.brzezinka@intel.com>", "X-Mailer": "git-send-email 2.34.1", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=\"us-ascii\"", "Content-Transfer-Encoding": "7bit", "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": "From: Alexey Marchuk <alexeymar@mellanox.com>\n\nmlx5 common library checks if several symbols/definitions\nare presented in system header files. If some are not\npresented, they will be enabled by mlx5_glue library.\nThe problem appears with clang and '-Werror' - code\ngenerated by meson is not compiled due to unused variable:\n\nCode:\n\n #include <infiniband/mlx5dv.h>\n int main(void) {\n /* If it's not defined as a macro, try to use as a symbol */\n #ifndef mlx5dv_create_flow_action_packet_reformat\n mlx5dv_create_flow_action_packet_reformat;\n #endif\n return 0;\n }\nCompiler stdout:\n\nCompiler stderr:\n /hpc/local/work/alexeymar/repo/spdk/dpdk/build-tmp/meson-private/tmp5obnak86/testfile.c:6:17: error: expression result unused [-Werror,-Wunused-value]\n mlx5dv_create_flow_action_packet_reformat;\n ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~\n\nAs result, almost all symbols are enabled in mlx5_glue while\nthey exist is system headers. As result, we get multiple\nsymbols redefenitions when we compile mlx5_common.\nAs a solution for this problem we can suppress\n-Wunused-vaurable using pragma\n\nDPDK 23.11 note:\nStarting with commit bellow, all cflags are passed to the has_header_symbol().\n(33d6694) build: use C11 standard\nTo make sure that the symbol is properly detected, the pedantic flags needs to\nbe removed.\n\nSigned-off-by: Alexey Marchuk <alexeymar@mellanox.com>\nSigned-off-by: Sebastian Brzezinka <sebastian.brzezinka@intel.com>\n---\n drivers/common/mlx5/linux/meson.build | 6 +++++-\n 1 file changed, 5 insertions(+), 1 deletion(-)", "diff": "diff --git a/drivers/common/mlx5/linux/meson.build b/drivers/common/mlx5/linux/meson.build\nindex b3a64547c5..84b2c64041 100644\n--- a/drivers/common/mlx5/linux/meson.build\n+++ b/drivers/common/mlx5/linux/meson.build\n@@ -231,7 +231,11 @@ if libmtcr_ul_found\n endif\n \n foreach arg:has_sym_args\n- mlx5_config.set(arg[0], cc.has_header_symbol(arg[1], arg[2], dependencies: libs, args: cflags))\n+ file_prefix = '#pragma clang diagnostic ignored \"-Wunused-value\"'\n+ cflags += [\n+ '-Wno-pedantic',\n+ ]\n+ mlx5_config.set(arg[0], cc.has_header_symbol(arg[1], arg[2], prefix : file_prefix, dependencies: libs, args: cflags))\n endforeach\n foreach arg:has_member_args\n file_prefix = '#include <' + arg[1] + '>'\n", "prefixes": [] }{ "id": 139116, "url": "