Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/53773/?format=api
https://patches.dpdk.org/api/patches/53773/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/patch/51f2f9ff4410bcbd4939f48d9cb7ce2b4fbf17f5.1559049191.git.motih@mellanox.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": "<51f2f9ff4410bcbd4939f48d9cb7ce2b4fbf17f5.1559049191.git.motih@mellanox.com>", "list_archive_url": "https://inbox.dpdk.org/dev/51f2f9ff4410bcbd4939f48d9cb7ce2b4fbf17f5.1559049191.git.motih@mellanox.com", "date": "2019-05-28T13:23:46", "name": "[v3,1/2] net/mlx5: fix crashing testpmd on null drv opts", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "ba047fd1330e781adffcd0a8302cfcb38fb136b4", "submitter": { "id": 748, "url": "https://patches.dpdk.org/api/people/748/?format=api", "name": "Moti Haimovsky", "email": "motih@mellanox.com" }, "delegate": { "id": 6624, "url": "https://patches.dpdk.org/api/users/6624/?format=api", "username": "shahafs", "first_name": "Shahaf", "last_name": "Shuler", "email": "shahafs@mellanox.com" }, "mbox": "https://patches.dpdk.org/project/dpdk/patch/51f2f9ff4410bcbd4939f48d9cb7ce2b4fbf17f5.1559049191.git.motih@mellanox.com/mbox/", "series": [ { "id": 4799, "url": "https://patches.dpdk.org/api/series/4799/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/list/?series=4799", "date": "2019-05-28T13:23:27", "name": "net/mlx5: remove TCF support from PMD", "version": 3, "mbox": "https://patches.dpdk.org/series/4799/mbox/" } ], "comments": "https://patches.dpdk.org/api/patches/53773/comments/", "check": "fail", "checks": "https://patches.dpdk.org/api/patches/53773/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@dpdk.org", "Delivered-To": "patchwork@dpdk.org", "Received": [ "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id D70911B994;\n\tTue, 28 May 2019 15:23:49 +0200 (CEST)", "from EUR02-HE1-obe.outbound.protection.outlook.com\n\t(mail-eopbgr10073.outbound.protection.outlook.com [40.107.1.73])\n\tby dpdk.org (Postfix) with ESMTP id 24E2B1B95C;\n\tTue, 28 May 2019 15:23:48 +0200 (CEST)", "from AM0PR05MB4435.eurprd05.prod.outlook.com (52.134.95.151) by\n\tAM0PR05MB6161.eurprd05.prod.outlook.com (20.178.113.31) with\n\tMicrosoft SMTP Server (version=TLS1_2,\n\tcipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id\n\t15.20.1922.16; Tue, 28 May 2019 13:23:46 +0000", "from AM0PR05MB4435.eurprd05.prod.outlook.com\n\t([fe80::c40a:d6fe:1246:7278]) by\n\tAM0PR05MB4435.eurprd05.prod.outlook.com\n\t([fe80::c40a:d6fe:1246:7278%4]) with mapi id 15.20.1922.021;\n\tTue, 28 May 2019 13:23:46 +0000" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=Mellanox.com;\n\ts=selector2;\n\th=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;\n\tbh=YVzCXx88jQKSoLlAzibLu3bAH+LGZvdlbk2SYk0DmAs=;\n\tb=e0pjayT0pHp5SCuksqQ9FoBsoHdP/khrFekP52mCYmSbmRZB5qJYCFk/tpJ97PYS5Y0G/nXIMbNylnjCqw4Jzg2GvLjVI4/DIap1b8aIOGLvRzuA8A7Nrqx0Wl1ibPwzGNB4LC8hF5PSK5heg8hy2F+r2h7Mwm7IreHKtoXAePM=", "From": "Moti Haimovsky <motih@mellanox.com>", "To": "Shahaf Shuler <shahafs@mellanox.com>", "CC": "\"dev@dpdk.org\" <dev@dpdk.org>, Moti Haimovsky <motih@mellanox.com>,\n\t\"stable@dpdk.org\" <stable@dpdk.org>", "Thread-Topic": "[PATCH v3 1/2] net/mlx5: fix crashing testpmd on null drv opts", "Thread-Index": "AQHVFViUfMGGEWUn9EuqEmPHfAKAog==", "Date": "Tue, 28 May 2019 13:23:46 +0000", "Message-ID": "<51f2f9ff4410bcbd4939f48d9cb7ce2b4fbf17f5.1559049191.git.motih@mellanox.com>", "References": "<cover.1559044929.git.motih@mellanox.com>\n\t<cover.1559049191.git.motih@mellanox.com>", "In-Reply-To": "<cover.1559049191.git.motih@mellanox.com>", "Accept-Language": "he-IL, en-US", "Content-Language": "en-US", "X-MS-Has-Attach": "", "X-MS-TNEF-Correlator": "", "x-clientproxiedby": "VI1PR08CA0095.eurprd08.prod.outlook.com\n\t(2603:10a6:800:d3::21) To AM0PR05MB4435.eurprd05.prod.outlook.com\n\t(2603:10a6:208:61::23)", "authentication-results": "spf=none (sender IP is )\n\tsmtp.mailfrom=motih@mellanox.com; ", "x-ms-exchange-messagesentrepresentingtype": "1", "x-mailer": "git-send-email 1.7.1", "x-originating-ip": "[37.142.13.130]", "x-ms-publictraffictype": "Email", "x-ms-office365-filtering-correlation-id": "b0866738-620e-4404-24a0-08d6e36fb6bc", "x-ms-office365-filtering-ht": "Tenant", "x-microsoft-antispam": "BCL:0; PCL:0;\n\tRULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600148)(711020)(4605104)(1401327)(4618075)(2017052603328)(7193020);\n\tSRVR:AM0PR05MB6161; ", "x-ms-traffictypediagnostic": "AM0PR05MB6161:", "x-microsoft-antispam-prvs": "<AM0PR05MB6161078364FC00AF222F9F69D21E0@AM0PR05MB6161.eurprd05.prod.outlook.com>", "x-ms-oob-tlc-oobclassifiers": "OLM:612;", "x-forefront-prvs": "00514A2FE6", "x-forefront-antispam-report": "SFV:NSPM;\n\tSFS:(10009020)(396003)(376002)(39860400002)(346002)(366004)(136003)(199004)(189003)(486006)(6116002)(450100002)(446003)(11346002)(66066001)(6436002)(66446008)(64756008)(66556008)(66946007)(478600001)(73956011)(76176011)(53936002)(66476007)(386003)(99286004)(102836004)(3846002)(2906002)(4326008)(6506007)(2616005)(6512007)(6636002)(6862004)(71190400001)(71200400001)(52116002)(14454004)(476003)(118296001)(81156014)(81166006)(8676002)(37006003)(54906003)(256004)(26005)(8936002)(186003)(305945005)(50226002)(7736002)(68736007)(5660300002)(316002)(86362001)(36756003)(25786009)(6486002);\n\tDIR:OUT; SFP:1101; SCL:1; SRVR:AM0PR05MB6161;\n\tH:AM0PR05MB4435.eurprd05.prod.outlook.com; FPR:; SPF:None; LANG:en;\n\tPTR:InfoNoRecords; A:1; MX:1; ", "received-spf": "None (protection.outlook.com: mellanox.com does not designate\n\tpermitted sender hosts)", "x-ms-exchange-senderadcheck": "1", "x-microsoft-antispam-message-info": "ULi+dDDJeXPvAsQFVXnqQnx96QM15R/gmXIZiV2dAk25O6Y9KBt/ZNR6jAg6aihJCK2iet6mnE4NIXBhLtbEOe2aVP/luSAOzQoFp6VNlKzkIQytXdqPUBYdVayLfEpicY0mlGrdl2KOvfprtjae3RIhC3RCmsjAzUtQT1BJi8VU69v8awRY/XjNY1L3ubyxcX/WqJ5Yj7XsRiRSMdJlbyCdi/xk36jwWe1zG6uEvqpPH1HsyyC06ptzbwXPiEzIYAoQHoS09hNoy5F1LiNb7K9pJ0UMedoz8JH/arzn3GBVEXmDVgeruZ0PSe5wnj8nFo7oRBNmSjxJK0ItGi5LzUcQxn+i5hLwADwxnAwnl5xJXrNz9zHe04Dl0xQqWrHGuQUrdEwobdFaozIAGppjyxwhPpe9bQdDJ3jdoA+DKAQ=", "Content-Type": "text/plain; charset=\"utf-8\"", "Content-Transfer-Encoding": "base64", "MIME-Version": "1.0", "X-OriginatorOrg": "Mellanox.com", "X-MS-Exchange-CrossTenant-Network-Message-Id": "b0866738-620e-4404-24a0-08d6e36fb6bc", "X-MS-Exchange-CrossTenant-originalarrivaltime": "28 May 2019 13:23:46.8890\n\t(UTC)", "X-MS-Exchange-CrossTenant-fromentityheader": "Hosted", "X-MS-Exchange-CrossTenant-id": "a652971c-7d2e-4d9b-a6a4-d149256f461b", "X-MS-Exchange-CrossTenant-mailboxtype": "HOSTED", "X-MS-Exchange-CrossTenant-userprincipalname": "motih@mellanox.com", "X-MS-Exchange-Transport-CrossTenantHeadersStamped": "AM0PR05MB6161", "Subject": "[dpdk-dev] [PATCH v3 1/2] net/mlx5: fix crashing testpmd on null\n\tdrv opts", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.15", "Precedence": "list", "List-Id": "DPDK patches and discussions <dev.dpdk.org>", "List-Unsubscribe": "<https://mails.dpdk.org/options/dev>,\n\t<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\t<mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org", "Sender": "\"dev\" <dev-bounces@dpdk.org>" }, "content": "mlx5 implements mlx5_flow_null_drv_ops to be used when a specific\nflow typei/driver is not available or invalid.\nThis routines return error without modifying the rte_flow_error\nparameter passed to them which causes testpmd, for example, to crash.\nThis commit addresses the issue by modifying the rte_flow_error\nparameter in theses routines.\n\nFixes: 0c76d1c9a18d (\"net/mlx5: add abstraction for multiple flow drivers\")\nFixes: 684dafe795d0 (\"net/mlx5: add flow query abstraction interface\")\nCc: stable@dpdk.org\n\nSigned-off-by: Moti Haimovsky <motih@mellanox.com>\n---\nv3:\n * Modified patch subject.\n\nv2:\n * Fixed checkpatch warnings.\n---\n drivers/net/mlx5/mlx5_flow.c | 29 +++++++++++++++--------------\n 1 file changed, 15 insertions(+), 14 deletions(-)", "diff": "diff --git a/drivers/net/mlx5/mlx5_flow.c b/drivers/net/mlx5/mlx5_flow.c\r\nindex 9887018..e5a8e33 100644\r\n--- a/drivers/net/mlx5/mlx5_flow.c\r\n+++ b/drivers/net/mlx5/mlx5_flow.c\r\n@@ -1694,19 +1694,20 @@ uint32_t mlx5_flow_adjust_priority(struct rte_eth_dev *dev, int32_t priority,\r\n \t\t const struct rte_flow_attr *attr __rte_unused,\r\n \t\t const struct rte_flow_item items[] __rte_unused,\r\n \t\t const struct rte_flow_action actions[] __rte_unused,\r\n-\t\t struct rte_flow_error *error __rte_unused)\r\n+\t\t struct rte_flow_error *error)\r\n {\r\n-\trte_errno = ENOTSUP;\r\n-\treturn -rte_errno;\r\n+\treturn rte_flow_error_set(error, ENOTSUP,\r\n+\t\t\t\t RTE_FLOW_ERROR_TYPE_UNSPECIFIED, NULL, NULL);\r\n }\r\n \r\n static struct mlx5_flow *\r\n flow_null_prepare(const struct rte_flow_attr *attr __rte_unused,\r\n \t\t const struct rte_flow_item items[] __rte_unused,\r\n \t\t const struct rte_flow_action actions[] __rte_unused,\r\n-\t\t struct rte_flow_error *error __rte_unused)\r\n+\t\t struct rte_flow_error *error)\r\n {\r\n-\trte_errno = ENOTSUP;\r\n+\trte_flow_error_set(error, ENOTSUP,\r\n+\t\t\t RTE_FLOW_ERROR_TYPE_UNSPECIFIED, NULL, NULL);\r\n \treturn NULL;\r\n }\r\n \r\n@@ -1716,19 +1717,19 @@ uint32_t mlx5_flow_adjust_priority(struct rte_eth_dev *dev, int32_t priority,\r\n \t\t const struct rte_flow_attr *attr __rte_unused,\r\n \t\t const struct rte_flow_item items[] __rte_unused,\r\n \t\t const struct rte_flow_action actions[] __rte_unused,\r\n-\t\t struct rte_flow_error *error __rte_unused)\r\n+\t\t struct rte_flow_error *error)\r\n {\r\n-\trte_errno = ENOTSUP;\r\n-\treturn -rte_errno;\r\n+\treturn rte_flow_error_set(error, ENOTSUP,\r\n+\t\t\t\t RTE_FLOW_ERROR_TYPE_UNSPECIFIED, NULL, NULL);\r\n }\r\n \r\n static int\r\n flow_null_apply(struct rte_eth_dev *dev __rte_unused,\r\n \t\tstruct rte_flow *flow __rte_unused,\r\n-\t\tstruct rte_flow_error *error __rte_unused)\r\n+\t\tstruct rte_flow_error *error)\r\n {\r\n-\trte_errno = ENOTSUP;\r\n-\treturn -rte_errno;\r\n+\treturn rte_flow_error_set(error, ENOTSUP,\r\n+\t\t\t\t RTE_FLOW_ERROR_TYPE_UNSPECIFIED, NULL, NULL);\r\n }\r\n \r\n static void\r\n@@ -1748,10 +1749,10 @@ uint32_t mlx5_flow_adjust_priority(struct rte_eth_dev *dev, int32_t priority,\r\n \t\tstruct rte_flow *flow __rte_unused,\r\n \t\tconst struct rte_flow_action *actions __rte_unused,\r\n \t\tvoid *data __rte_unused,\r\n-\t\tstruct rte_flow_error *error __rte_unused)\r\n+\t\tstruct rte_flow_error *error)\r\n {\r\n-\trte_errno = ENOTSUP;\r\n-\treturn -rte_errno;\r\n+\treturn rte_flow_error_set(error, ENOTSUP,\r\n+\t\t\t\t RTE_FLOW_ERROR_TYPE_UNSPECIFIED, NULL, NULL);\r\n }\r\n \r\n /* Void driver to protect from null pointer reference. */\r\n", "prefixes": [ "v3", "1/2" ] }{ "id": 53773, "url": "