Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/74775/?format=api
{ "id": 74775, "url": "http://patches.dpdk.org/api/patches/74775/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20200724142403.6132-11-parav@mellanox.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": "<20200724142403.6132-11-parav@mellanox.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20200724142403.6132-11-parav@mellanox.com", "date": "2020-07-24T14:24:03", "name": "[v9,10/10] common/mlx5: remove class check from class drivers", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "6883cf88957a9b323d191111f2ef4589d0072606", "submitter": { "id": 1780, "url": "http://patches.dpdk.org/api/people/1780/?format=api", "name": "Parav Pandit", "email": "parav@mellanox.com" }, "delegate": { "id": 3268, "url": "http://patches.dpdk.org/api/users/3268/?format=api", "username": "rasland", "first_name": "Raslan", "last_name": "Darawsheh", "email": "rasland@nvidia.com" }, "mbox": "http://patches.dpdk.org/project/dpdk/patch/20200724142403.6132-11-parav@mellanox.com/mbox/", "series": [ { "id": 11290, "url": "http://patches.dpdk.org/api/series/11290/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=11290", "date": "2020-07-24T14:23:54", "name": "Improve mlx5 PMD driver framework for multiple classes", "version": 9, "mbox": "http://patches.dpdk.org/series/11290/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/74775/comments/", "check": "fail", "checks": "http://patches.dpdk.org/api/patches/74775/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 dpdk.org (dpdk.org [92.243.14.124])\n\tby inbox.dpdk.org (Postfix) with ESMTP id 4866AA0526;\n\tFri, 24 Jul 2020 16:26:23 +0200 (CEST)", "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id 52D2F1C11B;\n\tFri, 24 Jul 2020 16:24:52 +0200 (CEST)", "from EUR02-VE1-obe.outbound.protection.outlook.com\n (mail-eopbgr20047.outbound.protection.outlook.com [40.107.2.47])\n by dpdk.org (Postfix) with ESMTP id 6AF181C0C9\n for <dev@dpdk.org>; Fri, 24 Jul 2020 16:24:45 +0200 (CEST)", "from AM0PR05MB4866.eurprd05.prod.outlook.com (2603:10a6:208:c0::32)\n by AM4PR05MB3203.eurprd05.prod.outlook.com (2603:10a6:205:6::28) with\n Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3216.20; Fri, 24 Jul\n 2020 14:24:44 +0000", "from AM0PR05MB4866.eurprd05.prod.outlook.com\n ([fe80::eccf:72b3:bacb:f09d]) by AM0PR05MB4866.eurprd05.prod.outlook.com\n ([fe80::eccf:72b3:bacb:f09d%5]) with mapi id 15.20.3216.024; Fri, 24 Jul 2020\n 14:24:44 +0000", "from sw-mtx-036.mtx.labs.mlnx (208.176.44.194) by\n DM6PR02CA0136.namprd02.prod.outlook.com (2603:10b6:5:1b4::38) with Microsoft\n SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id\n 15.20.3216.21 via Frontend Transport; Fri, 24 Jul 2020 14:24:42 +0000" ], "ARC-Seal": "i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none;\n b=VKXRKdnstchdR05XHPiV3G6h7W2Ka/vNWRbT1d1tNJdttnrRJudfy+qED5UMFR0jvDeF8xv8ulwb/Gjn8szOxiDgw+kM6p3Ev/NRvCiYyJQiqFKRne/UWQp/KOzM6f8tXViRJFT1DDIgiAM2KWicmTALKe9Nygf5vF2u7ZNFoUA7rPpzcZq1NqIxak1KdGy0jYdJuB1OTsWis0h82qH6GL2+9gwglFPSeQxSzvxpYFV0zaJlXr2u1CEMGixVXiTUHYVgA8qAu69pV75wWZwb/75BHmRlknXAwRLPcTKoBQSbj+zt6rT7CCWRHhZ9W6blgTs9+NS6YV/jLSaR6T+PFA==", "ARC-Message-Signature": "i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com;\n s=arcselector9901;\n h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;\n bh=7RRTCf2cFt1yYGpc94tcUcshT/4Ysfkpd/B2GqYUzMc=;\n b=XImurOh0ANzC6NtyZi9t4ePu+rTRS6KKYgCYxX1gFiZRfPILmrdfBd0ujsFe6wzr9D/dcfJaIbKdjAoq9sm6pwFucEJtKxd+AZY+6G/4Zivf4CJ7G+nxyG4HWIt9mAQyjY+pw5Uavbonwa44siOuM1deM+uJTDcIum2Ygt+qBKhFCxGwbO3z8a9x2Vx8UaxA+BfwTJrNUJLJ6f7K30FC09eGeSU2gwjVumJOFp27FvkJxtRtl3gy9atS5cXsOM2Jaf0w0wuL5kqjj07sN+Kyge53DgfLAycPJvwtEUwK8eccmucIz8Mz9keMtgjTt5nLJX3Tu9b8r4DINZ8pDeSBvw==", "ARC-Authentication-Results": "i=1; mx.microsoft.com 1; spf=pass\n smtp.mailfrom=mellanox.com; dmarc=pass action=none header.from=mellanox.com;\n dkim=pass header.d=mellanox.com; arc=none", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=Mellanox.com;\n s=selector1;\n h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;\n bh=7RRTCf2cFt1yYGpc94tcUcshT/4Ysfkpd/B2GqYUzMc=;\n b=rOKlgaHmXejZQELI8Qhm2FKeIvE57EOUGmH+xJfkCYEhchOXnmoicbuwC2ZixeSdHD6s7dA83ur0X9Eguyv5RSdRvzpuP+cs+ZnKBA4n5bL2ooIfx+1Q4+GQhS3gziSG2hd93ldMnUJnYeZ6vzMFn+bHqSFHURqkACjxddksets=", "Authentication-Results": "dpdk.org; dkim=none (message not signed)\n header.d=none;dpdk.org; dmarc=none action=none header.from=mellanox.com;", "From": "Parav Pandit <parav@mellanox.com>", "To": "dev@dpdk.org, grive@u256.net, ferruh.yigit@intel.com, thomas@monjalon.net", "Cc": "rasland@mellanox.com, orika@mellanox.com, matan@mellanox.com,\n joyce.kong@arm.com, Parav Pandit <parav@mellanox.com>", "Date": "Fri, 24 Jul 2020 17:24:03 +0300", "Message-Id": "<20200724142403.6132-11-parav@mellanox.com>", "X-Mailer": [ "git-send-email 2.26.2", "git-send-email 2.26.2" ], "In-Reply-To": "<20200724142403.6132-1-parav@mellanox.com>", "References": "<20200610171728.89-2-parav@mellanox.com>\n <20200724142403.6132-1-parav@mellanox.com>", "Content-Transfer-Encoding": "8bit", "Content-Type": "text/plain", "X-ClientProxiedBy": "DM6PR02CA0136.namprd02.prod.outlook.com\n (2603:10b6:5:1b4::38) To AM0PR05MB4866.eurprd05.prod.outlook.com\n (2603:10a6:208:c0::32)", "MIME-Version": "1.0", "X-MS-Exchange-MessageSentRepresentingType": "1", "X-Originating-IP": "[208.176.44.194]", "X-MS-PublicTrafficType": "Email", "X-MS-Office365-Filtering-HT": "Tenant", "X-MS-Office365-Filtering-Correlation-Id": "1a3345dc-f542-4d03-b4b9-08d82fdd4f5d", "X-MS-TrafficTypeDiagnostic": "AM4PR05MB3203:", "X-LD-Processed": "a652971c-7d2e-4d9b-a6a4-d149256f461b,ExtFwd,ExtAddr", "X-MS-Exchange-Transport-Forked": "True", "X-Microsoft-Antispam-PRVS": "\n <AM4PR05MB3203A1B5B1070C05C6CE5DBBD1770@AM4PR05MB3203.eurprd05.prod.outlook.com>", "X-MS-Oob-TLC-OOBClassifiers": "OLM:7219;", "X-MS-Exchange-SenderADCheck": "1", "X-Microsoft-Antispam": "BCL:0;", "X-Microsoft-Antispam-Message-Info": "\n 7nZkTHWNnTtOUMya9zmdrb9eNLlisocrjfM7nKmByNivkWk6hTCnIR5i2BjYnmxquMZUQzJTBAHYLGL5g41Rxf8uOCDBvq3oC/HTyLExg7zGz7i3v3ZbEWglcZcMfrKQuTNiDZ2uyclAlWgWd1D09yd80Uk8Acun5mzwCPMawVWy2RhTNhtdQeFD2gzLvxKGCWPGpbvXGoj2LFrvTbRWnOeyGdB+il66Wp735BwN1honPu+ALOhKFLdwhjc/4m+IMV5FDc3g1IwqnV/Rb4DTN/zVCdvkTQ+jnnW/BMsr91MRZPk7innFGwf7JI50pE7w", "X-Forefront-Antispam-Report": "CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:;\n IPV:NLI; SFV:NSPM; H:AM0PR05MB4866.eurprd05.prod.outlook.com; PTR:; CAT:NONE;\n SFTY:;\n SFS:(4636009)(376002)(346002)(136003)(366004)(396003)(39860400002)(5660300002)(86362001)(2616005)(956004)(6512007)(2906002)(66946007)(66476007)(66556008)(316002)(6506007)(4326008)(16526019)(186003)(26005)(107886003)(8936002)(478600001)(8676002)(6666004)(52116002)(6486002)(83380400001)(1076003)(36756003);\n DIR:OUT; SFP:1101;", "X-MS-Exchange-AntiSpam-MessageData": "\n jggOVlWyhu2eQ072iz/CC/mfLn+qB2Qmsqp0RVqMu4XXeu30PW956SJmm0/ehQbh/XAksah3RKsQwmZUpy4krQn/kzhB/v+jqhlv1c4zf0iFynN1cu8vBQ1d0ghEOvYO3lhDg/xGPCUNpUcyDk0OtD9QQ2/582GZn12lJbP/STFSUfJHykb4/380BETCQ7KeTrnTfxig9PX1We/DKI7gEkS6JEgFH6xSmjrbupIm696FAy5IKddXiqROiNy8EqOC3M0gJGq9BiU8+S1XEWzLh4tuOkI/kB1ppoP9SS0uBljNnj7Y1ZZmp4E0q9nPoLQbnnZtSOb0d5eex8hd7v54VDDpo1sT92GSD6NVonYoSmO/omLAjUKq8QP/Aj0d6SZRfRctM9EUioX1MbMrn1mv3URIOAUcBOierJc5kqoc5Cj+Xi/PdOQQXes2E957BMU7Vj+tJKwq2XB3o7bwi5ZdgZZIWIO8uOx4XQQwhQLXY1+D6ijBBRSksb/kx2bVGdy7", "X-OriginatorOrg": "Mellanox.com", "X-MS-Exchange-CrossTenant-Network-Message-Id": "\n 1a3345dc-f542-4d03-b4b9-08d82fdd4f5d", "X-MS-Exchange-CrossTenant-AuthSource": "AM0PR05MB4866.eurprd05.prod.outlook.com", "X-MS-Exchange-CrossTenant-AuthAs": "Internal", "X-MS-Exchange-CrossTenant-OriginalArrivalTime": "24 Jul 2020 14:24:44.0203 (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": "\n Xnojy5b+O9Iu6mRlt+tEMKv1/rNcMcSTt2KH3QPvAyDiSlkYjsphnB44p0xMe7BU22mvKkKKlgKQx1iwkD+pdw==", "X-MS-Exchange-Transport-CrossTenantHeadersStamped": "AM4PR05MB3203", "Subject": "[dpdk-dev] [PATCH v9 10/10] common/mlx5: remove class check from\n\tclass drivers", "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 <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", "Sender": "\"dev\" <dev-bounces@dpdk.org>" }, "content": "Now that mlx5_pci PMD checks for enabled classes and performs\nprobe(), remove() of associated classes, individual class driver\ndoes not need to check if other driver is enabled.\n\nSigned-off-by: Parav Pandit <parav@mellanox.com>\nAcked-by: Matan Azrad <matan@mellanox.com>\n---\n drivers/common/mlx5/mlx5_common.c | 37 -------------------\n drivers/common/mlx5/mlx5_common.h | 2 -\n .../common/mlx5/rte_common_mlx5_version.map | 3 +-\n drivers/net/mlx5/linux/mlx5_os.c | 5 ---\n drivers/vdpa/mlx5/mlx5_vdpa.c | 5 ---\n 5 files changed, 1 insertion(+), 51 deletions(-)", "diff": "diff --git a/drivers/common/mlx5/mlx5_common.c b/drivers/common/mlx5/mlx5_common.c\nindex fd818ef24..06f0a6400 100644\n--- a/drivers/common/mlx5/mlx5_common.c\n+++ b/drivers/common/mlx5/mlx5_common.c\n@@ -20,43 +20,6 @@ int mlx5_common_logtype;\n \n uint8_t haswell_broadwell_cpu;\n \n-static int\n-mlx5_class_check_handler(__rte_unused const char *key, const char *value,\n-\t\t\t void *opaque)\n-{\n-\tenum mlx5_class *ret = opaque;\n-\n-\tif (strcmp(value, \"vdpa\") == 0) {\n-\t\t*ret = MLX5_CLASS_VDPA;\n-\t} else if (strcmp(value, \"net\") == 0) {\n-\t\t*ret = MLX5_CLASS_NET;\n-\t} else {\n-\t\tDRV_LOG(ERR, \"Invalid mlx5 class %s. Maybe typo in device\"\n-\t\t\t\" class argument setting?\", value);\n-\t\t*ret = MLX5_CLASS_INVALID;\n-\t}\n-\treturn 0;\n-}\n-\n-enum mlx5_class\n-mlx5_class_get(struct rte_devargs *devargs)\n-{\n-\tstruct rte_kvargs *kvlist;\n-\tconst char *key = MLX5_CLASS_ARG_NAME;\n-\tenum mlx5_class ret = MLX5_CLASS_NET;\n-\n-\tif (devargs == NULL)\n-\t\treturn ret;\n-\tkvlist = rte_kvargs_parse(devargs->args, NULL);\n-\tif (kvlist == NULL)\n-\t\treturn ret;\n-\tif (rte_kvargs_count(kvlist, key))\n-\t\trte_kvargs_process(kvlist, key, mlx5_class_check_handler, &ret);\n-\trte_kvargs_free(kvlist);\n-\treturn ret;\n-}\n-\n-\n /* In case this is an x86_64 intel processor to check if\n * we should use relaxed ordering.\n */\ndiff --git a/drivers/common/mlx5/mlx5_common.h b/drivers/common/mlx5/mlx5_common.h\nindex 864e82ff0..2cdb226f3 100644\n--- a/drivers/common/mlx5/mlx5_common.h\n+++ b/drivers/common/mlx5/mlx5_common.h\n@@ -247,8 +247,6 @@ struct mlx5_klm {\n \n LIST_HEAD(mlx5_dbr_page_list, mlx5_devx_dbr_page);\n \n-__rte_internal\n-enum mlx5_class mlx5_class_get(struct rte_devargs *devargs);\n __rte_internal\n void mlx5_translate_port_name(const char *port_name_in,\n \t\t\t struct mlx5_switch_info *port_info_out);\ndiff --git a/drivers/common/mlx5/rte_common_mlx5_version.map b/drivers/common/mlx5/rte_common_mlx5_version.map\nindex 73cf72548..7729d0dd0 100644\n--- a/drivers/common/mlx5/rte_common_mlx5_version.map\n+++ b/drivers/common/mlx5/rte_common_mlx5_version.map\n@@ -1,9 +1,8 @@\n INTERNAL {\n \tglobal:\n \n-\tmlx5_class_get;\n-\n \tmlx5_common_init;\n+\n \tmlx5_common_verbs_reg_mr;\n \tmlx5_common_verbs_dereg_mr;\n \ndiff --git a/drivers/net/mlx5/linux/mlx5_os.c b/drivers/net/mlx5/linux/mlx5_os.c\nindex 97d7b0e7b..fc339affa 100644\n--- a/drivers/net/mlx5/linux/mlx5_os.c\n+++ b/drivers/net/mlx5/linux/mlx5_os.c\n@@ -1524,11 +1524,6 @@ mlx5_os_pci_probe(struct rte_pci_driver *pci_drv __rte_unused,\n \tunsigned int dev_config_vf;\n \tint ret;\n \n-\tif (mlx5_class_get(pci_dev->device.devargs) != MLX5_CLASS_NET) {\n-\t\tDRV_LOG(DEBUG, \"Skip probing - should be probed by other mlx5\"\n-\t\t\t\" driver.\");\n-\t\treturn 1;\n-\t}\n \tif (rte_eal_process_type() == RTE_PROC_PRIMARY)\n \t\tmlx5_pmd_socket_init();\n \tret = mlx5_init_once();\ndiff --git a/drivers/vdpa/mlx5/mlx5_vdpa.c b/drivers/vdpa/mlx5/mlx5_vdpa.c\nindex ffe2f00ac..c0b87bcc0 100644\n--- a/drivers/vdpa/mlx5/mlx5_vdpa.c\n+++ b/drivers/vdpa/mlx5/mlx5_vdpa.c\n@@ -680,11 +680,6 @@ mlx5_vdpa_pci_probe(struct rte_pci_driver *pci_drv __rte_unused,\n \tstruct mlx5_hca_attr attr;\n \tint ret;\n \n-\tif (mlx5_class_get(pci_dev->device.devargs) != MLX5_CLASS_VDPA) {\n-\t\tDRV_LOG(DEBUG, \"Skip probing - should be probed by other mlx5\"\n-\t\t\t\" driver.\");\n-\t\treturn 1;\n-\t}\n \tibv = mlx5_vdpa_get_ib_device_match(&pci_dev->addr);\n \tif (!ibv) {\n \t\tDRV_LOG(ERR, \"No matching IB device for PCI slot \"\n", "prefixes": [ "v9", "10/10" ] }