From patchwork Thu Aug 2 20:58:33 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Yongseok Koh X-Patchwork-Id: 43538 Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 115DC1B502; Thu, 2 Aug 2018 22:58:49 +0200 (CEST) Received: from EUR04-DB3-obe.outbound.protection.outlook.com (mail-eopbgr60069.outbound.protection.outlook.com [40.107.6.69]) by dpdk.org (Postfix) with ESMTP id D1C911B4FF; Thu, 2 Aug 2018 22:58:47 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Mellanox.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=vZx7EAS5fXt/NJrsoFrLbSdrATA8XdXrck/OPVIMBiA=; b=ID6EdNmvfmE2eAQIG30KEpiDm9GuvTA5VciPHyHOSkYWbyrd49h4TXjwtVlgvy7vYjPGe/rBp110ST+0tbgj7WOmH4T/vgrRNoCwOWPIow9JytyWRIg3tVINfw0Ch6vEMqvGFb4f/lF/GrUdWtKNUqQIaRgd5fvTKD3sdV4G+Rw= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=yskoh@mellanox.com; Received: from mellanox.com (209.116.155.178) by VI1PR0501MB2045.eurprd05.prod.outlook.com (2603:10a6:800:36::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1017.15; Thu, 2 Aug 2018 20:58:44 +0000 From: Yongseok Koh To: shahafs@mellanox.com Cc: dev@dpdk.org, Yongseok Koh , stable@dpdk.org Date: Thu, 2 Aug 2018 13:58:33 -0700 Message-Id: <20180802205834.10491-1-yskoh@mellanox.com> X-Mailer: git-send-email 2.11.0 MIME-Version: 1.0 X-Originating-IP: [209.116.155.178] X-ClientProxiedBy: BN6PR03CA0011.namprd03.prod.outlook.com (2603:10b6:404:23::21) To VI1PR0501MB2045.eurprd05.prod.outlook.com (2603:10a6:800:36::19) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 93f49eb7-6653-48dc-189e-08d5f8babcc7 X-MS-Office365-Filtering-HT: Tenant X-Microsoft-Antispam: BCL:0; PCL:0; RULEID:(7020095)(4652040)(8989117)(5600074)(711020)(4618075)(2017052603328)(7153060)(7193020); SRVR:VI1PR0501MB2045; X-Microsoft-Exchange-Diagnostics: 1; VI1PR0501MB2045; 3:ibLVb8ivYjXAka7QEAWidKiV8T2znSzzljd670jKCWpeQECKlMA/OF6shBxlLvDmjBgI03JGNm+88Ex0lNHxqEGDgcFmn14h3gzR1tdsNCMMRA4i7JVZI22OaJ9Jt3/+RmZvqF8s7KA/vcU4k7Ia9E0PvBLqngNdk1ma8R0WT24ovguBrogtFyLudNm4bPelUaijOT6zammciys/+WEqqwOgwq/89JNW4XrufqZ0DraDsWy63aXHoMATIljt4q8c; 25:5GAjArZEeifEB1Di0elXJqYriBAEp3ecVc5pWNLCUJUc+hySKB8mt/KekGYl9+MxD+9aZQB1vXOsgTFpHZVyRGFfrHniP8jS8BRttiEKvRi+v9GLu6GoVVB3yOgrpVK0r0Wlvem78kBM03EhftUJUTnw6MwptqPeOKT00outpw6TDkBa2wGLP00Ld1Ji5O1nsZT/JrAsQsge0fRPJVO9OTqtsFByQaJ0aSMrJWYm7s8/H3wi/2CaQCfXm+NRkk8DjfuF4P3DcMbKh0SlPMMdBavi/ItumQKHLpJeWP2f+YuWWATxKfys++wpTTopZajs0h13YR/jt21hj22633YoEg==; 31:ipsGtzBxy93QvMf3viuwqXQgLYowXz2BYZ1GDuuIpB+Ylzm4sBdqZ8WlRL4FWMAdDKsHMti3PkiN8wit6Xcloxyec826SER54Bz3rHaKO77EOkZmiX7KwHoOdvYITiFjjXzIBfFLObIQ+FfSDWEQrDJjKE48TEgWKM4p22uWTSak9yiJH5LOSXgtvM50A9P3iXLl9/L5N9+MH6L9f0fV6Nn9nnC5KA2v1R8hp6zuknw= X-MS-TrafficTypeDiagnostic: VI1PR0501MB2045: X-Microsoft-Exchange-Diagnostics: 1; VI1PR0501MB2045; 20:055btKI1IERn53r48IGf6h2BpdkFcGSh/UsZgR42RHEjcm5rzypsXBt97nAqlInOHBpWxQfFv0QnbTfa+wG6PKJzh1kkzU9h6r6xdqLJgrZHf4AuK7yuY/fs0EKVCet2GTGYvr6YbbUYdSAoLvUc06hgyoyru3BDFO7RVxtbbwE5nVyYBUCvmMJSJSXVTYZrv2/K28iRggszwHe7Zl6bpkQYIJtFbtTLLd0FiKd2xyhe7YhbhKvmEm6HpWf5Gt1+9HYyCgH1H//jnMWWacfqdOO4qQNUdAUjrJDrHOD0oice54rvzIZgwql+zHkhLy0hIvN0OCPfPSqapXRaBc+AFcGhDqUqiVfv6K9bWabwCMKLbpDj01gxNnGZt+O9YRT+btdAU+UX1IcROK83XZsZ1kMYky+xo0lpUvucKYVbZHKrm1/KQTAtESOhXQSrEVfrCGgVXNx9H7+SzpOaDjIuUeQykB2tKWodu9JNrI/x1RtfTOGqy8rIWvtSydYmkrii; 4:GBslW1FY4DcvKsaYEqQtumXNVPwOrYB22JuVARJCN89F9LUb26B9kM3dUqJwT+0HqAurz+6bRyWWQ8WNfBU6cnRzAIPRkzQnJg48k1xwzTAfE0AXG+2h57/8yruIzBTUNkCjTYiQ6fhr87luhNgCjwrsKxIgpojiPQO4C6adDivfIcX8QOr26VWe8yoj+TINiqkblkPjMPEu3746HdgYAVWVakHZg8Jc9LTxXZI6JFDxniqbiiuwkG0dZlxhCyWo4SFIXs4sQ+N0AN2PUAASwA== X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-MS-Exchange-SenderADCheck: 1 X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(8211001083)(6040522)(2401047)(5005006)(8121501046)(93006095)(93001095)(10201501046)(3231311)(944501410)(52105095)(3002001)(6055026)(149027)(150027)(6041310)(20161123560045)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123562045)(20161123558120)(20161123564045)(6072148)(201708071742011)(7699016); SRVR:VI1PR0501MB2045; BCL:0; PCL:0; RULEID:; SRVR:VI1PR0501MB2045; X-Forefront-PRVS: 07521929C1 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(396003)(366004)(346002)(136003)(376002)(39860400002)(199004)(189003)(53936002)(51416003)(478600001)(55016002)(26005)(14444005)(486006)(316002)(386003)(476003)(2616005)(2906002)(956004)(50466002)(16586007)(37006003)(48376002)(97736004)(450100002)(105586002)(4326008)(25786009)(6636002)(106356001)(2361001)(7696005)(52116002)(86362001)(6666003)(50226002)(5660300001)(8936002)(305945005)(47776003)(81166006)(7736002)(66066001)(3846002)(16526019)(81156014)(6116002)(34206002)(69596002)(8676002)(1076002)(21086003)(2351001)(36756003)(68736007); DIR:OUT; SFP:1101; SCL:1; SRVR:VI1PR0501MB2045; H:mellanox.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1; Received-SPF: None (protection.outlook.com: mellanox.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; VI1PR0501MB2045; 23:Vrv2Z62vdxs3e/h6SpCeuG3ddE7ewb5jPP+If+3?= SZLnwyJdj2SZL20kUi+QgNUDzRJsyUdT996ot2pBaBlNWxogjYxPdSD3Dq3Gr2Ozr3bWWlKmnAOqqqUVjy8IDYYB9xYvFRZbqPWHvYW+O7vRJyEvLvT5YGTAJA1/HPpUXMBZ9MGX3xzsUB1Y2ZY2ERrJ3M9RGlUt8Ppjdgi6wtg5FK6rQedusV234s2euhKrUtevBtlymtW5nep2hJprrriZdtt1H1wrIN6oy4TItbfnD/P4VRfqv2yMpZMx1CJos8serApSgFTzIleHyrSW8yg6HHDagxbROAUddnI8Hh6X8tL/lLDfhqTfHEbeh9Jkt/HCvLpChToZGH7fsLiWXFpExC1e5KAzDMq63FLTlhKEBikWtaOH46XtZfiHXDJAaE6tCefzeOXJp31p9e/vPsJtj8yq/2T+Y36gFBzPMBxWXukWKMxYi+3NlplmCTJTb5pdpNFY1bas+mo3Qzd8p3dM1GYkSXsYzS+UYGOlZ+xjEwglHJ6tfZPydA+FHKudwOD2jcbkFtR0WM41qiUERBGI6OoAhB37cyrWjG4JhWW5O2BxNk30eSok/7iQbIbJ2iO+x9CuOcC1OR67DHmGwUWMc31FSeOHm+GYHiXn17xIYNTK169RD3sHulz30dNqASKnxy++xRQ51ODAldAMrjl55GvL1asuuoz3VNHhIyg3uO3wGDOxkG9PST82TRfXfJK/7u0hp2X2+aCR8wnijXnjrQ+39jCN5PJ2DjxeeoT9sbR38hh600irzGCuuknXxrbRCg1KqJFmohHGzIBCmcprZEpJh3ZCY7vpfu7oXIcuO7nWnfNZebQZj0T6H8Sa9UytWz+T8Rw+tcFabJPJA54otJW/heDdLahKLkbv9TgBrPKlgT84wQKmDFor1n91pddBoYMjD4nzzeNbztTgJaWDDfaI/BCO/8/AcTYTyfzhvx1755d2GxsY3KlNMdJbPLKVvKyv2Oywscg1x0J966os7a+4bA8riY2qhStMYs7CBy+Kl0UZMbhr2uhQGTNBOu/Syt28skH2VX5HE8vg9Es+fzpTb9KlPEg8/OfYuULcI/ik0HR+v6/8b7mTaY98eF6LNZSXR7oFd8QzB77c5G4lTejRZQxnXz/Cm4JewYex+ck3dbc58nGlWi43tk1bH+cN9BgZ+uJ/jkLfDfU3l3wX1ghRbFSqs/+gbCFbZwE/n8nUabGpsGJhcsa0fY5pTyoMZrJy6Csz1CZiqjbl4TnKk X-Microsoft-Antispam-Message-Info: T/fmnuF8yS86tmWBkXaI5MgBCC2Q8bxO9LxzcVo230BsLNBcnczXQaMbo8BSnBJNAgNyzefNwwz85X16cf8M4cmDQ4GPwuxYD9YrioNQ2bc+kkyEOTAYG0tLrm+8HDYCRiiVkcFMUOPpWHc+AJyHqIi0Ff6LCegKSfc8GD30DJA3tWG8A08X/YxmMBfnEZkxOm58DK4zEgLgSkdOB4ldULYbmfyT0mLIXdjMEgRI8kXVwUfVYgdhElID9iJNtzPNfABCD2RmDWUo7t9i9iyehHokBH8LQ7H+V5pwysuXOieu7dLp3N7hdS2P+YEhzrIUyClohpsHunmFGsFrr5bYSvRsrnbfMuFxBZ+oRCr2U6I= X-Microsoft-Exchange-Diagnostics: 1; VI1PR0501MB2045; 6:o0Gd81/nr3p8d6TP+cBK7MA+DfUFug1T3k/R9yJ/gyt1mFNgbbeKEe9FjTSXnb9zylV3zfDtV3TZbOivM9zABchG204xQ8sDjZj+2o6e3UAEUwa8Ur1G0bhQayX7en+MDxfLqdKfe4iyj0rDlmgCyb9UEGrG2ex0MW3nHcOoWH3433ArMgqElkveIzeZYdHgt7lY/a88eqq5Z8uWmdlz98+IpAAlOx9GeXNHejHj7S/cUjiccYsSwQTdthwatam1gkB/bAWAPiYPGm3GkTYu14sUX5YCp7D5yp4CdfSlLwEX5Eh+p1JKKedd0FdRryYYkwna8idAKYaWOWZuok5rhRcyPjXnrAQybHQAsrB2oM4rv8wkPAVaIffumfOL/MONN5Nl6TGOhJYgfLfitDGCdUuIvhIgud9tmFzlyeDUxeJAWmXLV7JogORqQ2lFCX8eI0bxLhzOcTx/GinPLrKL2A==; 5:7v+N4yrGPBd6V2d4hgSZUUFPc9Rr61/CpgXW6qo/nd2JQ6/y2ni77VMGvnNcbsXrn7Sgt/umMF8PBLDcnDvL4qK0IjT5DTtBbnG0cqd1xqy0Ww6Aq0wUG6Ghf1mnEBgnlzaG9QNlJaSvGWg2hCU3Cnz6pnp/j17GMrZ3LFKLSs4=; 7:W5G5UDHGJU8+eMdbDYa9ahFIwwA0SGrXdktrkuvMIiKvgFm5I3DoSGrbuqZL6jh1M84zaJEZpbDgSODg6sjHdLDlcnegPRD5XQK5MtA8zDqDeB0TgSBzlj28iQuIcGrIPf79/UFqee0m87MUVaqbKlqizde+Pz9dj7splsL6qu7HjpDC2VbVFdevRWyNeiz9kPr+JWF1NV2SQ6mWNzX/9EhAaTWOVxfOLaIX3diZwBWKRbNkdCwApxSerMYhOV1+ SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: Mellanox.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Aug 2018 20:58:44.7304 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 93f49eb7-6653-48dc-189e-08d5f8babcc7 X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: a652971c-7d2e-4d9b-a6a4-d149256f461b X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR0501MB2045 Subject: [dpdk-dev] [PATCH 1/2] net/mlx5: preserve promisc flag for flow isolation mode X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" mlx5_dev_ops_isolate doesn't have APIs for enabling/disabling promiscuous mode as it can't be enabled in flow isolation mode. If the function pointers are null, librte APIs such as rte_eth_promiscuous_enable/disable() fail to set the flag (dev->data->promiscuous). The flag is used when starting traffic by mlx5_traffic_enable(). When switching out of flow isolation mode, promiscuous mode will not be set even though it has been enabled. Fixes: 0887aa7f27f3 ("net/mlx5: add new operations for isolated mode") Cc: stable@dpdk.org Signed-off-by: Yongseok Koh --- v2: * do not toggle promisc mode when switching flow isolation mode * add warning message when attempting to enable promisc in flow isolation mode drivers/net/mlx5/mlx5.c | 2 ++ drivers/net/mlx5/mlx5_rxmode.c | 13 +++++++++++-- 2 files changed, 13 insertions(+), 2 deletions(-) diff --git a/drivers/net/mlx5/mlx5.c b/drivers/net/mlx5/mlx5.c index e3e2a181ac..83b82f11ab 100644 --- a/drivers/net/mlx5/mlx5.c +++ b/drivers/net/mlx5/mlx5.c @@ -399,6 +399,8 @@ const struct eth_dev_ops mlx5_dev_ops_isolate = { .dev_set_link_down = mlx5_set_link_down, .dev_set_link_up = mlx5_set_link_up, .dev_close = mlx5_dev_close, + .promiscuous_enable = mlx5_promiscuous_enable, + .promiscuous_disable = mlx5_promiscuous_disable, .link_update = mlx5_link_update, .stats_get = mlx5_stats_get, .stats_reset = mlx5_stats_reset, diff --git a/drivers/net/mlx5/mlx5_rxmode.c b/drivers/net/mlx5/mlx5_rxmode.c index 80824bc43b..3c0373bb4d 100644 --- a/drivers/net/mlx5/mlx5_rxmode.c +++ b/drivers/net/mlx5/mlx5_rxmode.c @@ -32,10 +32,18 @@ void mlx5_promiscuous_enable(struct rte_eth_dev *dev) { + struct priv *priv = dev->data->dev_private; int ret; dev->data->promiscuous = 1; - if (((struct priv *)dev->data->dev_private)->config.vf) + if (priv->isolated) { + DRV_LOG(WARNING, + "port %u cannot enable promiscuous mode" + " in flow isolation mode", + dev->data->port_id); + return; + } + if (priv->config.vf) mlx5_nl_promisc(dev, 1); ret = mlx5_traffic_restart(dev); if (ret) @@ -52,10 +60,11 @@ mlx5_promiscuous_enable(struct rte_eth_dev *dev) void mlx5_promiscuous_disable(struct rte_eth_dev *dev) { + struct priv *priv = dev->data->dev_private; int ret; dev->data->promiscuous = 0; - if (((struct priv *)dev->data->dev_private)->config.vf) + if (priv->config.vf) mlx5_nl_promisc(dev, 0); ret = mlx5_traffic_restart(dev); if (ret) From patchwork Thu Aug 2 20:58:34 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Yongseok Koh X-Patchwork-Id: 43539 Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 2FB401B50C; Thu, 2 Aug 2018 22:59:01 +0200 (CEST) Received: from EUR04-VI1-obe.outbound.protection.outlook.com (mail-eopbgr80052.outbound.protection.outlook.com [40.107.8.52]) by dpdk.org (Postfix) with ESMTP id 9ED591B503; Thu, 2 Aug 2018 22:58:58 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Mellanox.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=6n7Dq5lmSgwudkLjklFXgE87hGbsaCXQpBasjj49H1o=; b=k7p4tsUQL7Jxq9wjTgsFpa+6Z4AKCPItNFAp42S1VBQx7ApGaQZT13UC4io03TP3tVYZigAsS3QeVNdy8KeAmwaWWCGdTp6H7kV9+ogZCpZKwqNb3zgnCwLT1w0i7fLU0UwH5GalB+FoRVMsVLO7zdFCLj3lHbmZO7ZqgeHQnAc= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=yskoh@mellanox.com; Received: from mellanox.com (209.116.155.178) by VI1PR0501MB2045.eurprd05.prod.outlook.com (2603:10a6:800:36::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1017.15; Thu, 2 Aug 2018 20:58:56 +0000 From: Yongseok Koh To: shahafs@mellanox.com Cc: dev@dpdk.org, Yongseok Koh , stable@dpdk.org Date: Thu, 2 Aug 2018 13:58:34 -0700 Message-Id: <20180802205834.10491-2-yskoh@mellanox.com> X-Mailer: git-send-email 2.11.0 In-Reply-To: <20180802205834.10491-1-yskoh@mellanox.com> References: <20180802205834.10491-1-yskoh@mellanox.com> MIME-Version: 1.0 X-Originating-IP: [209.116.155.178] X-ClientProxiedBy: BN6PR03CA0011.namprd03.prod.outlook.com (2603:10b6:404:23::21) To VI1PR0501MB2045.eurprd05.prod.outlook.com (2603:10a6:800:36::19) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 270047a8-a4b4-4ce3-6766-08d5f8bac3bd X-MS-Office365-Filtering-HT: Tenant X-Microsoft-Antispam: BCL:0; PCL:0; RULEID:(7020095)(4652040)(8989117)(5600074)(711020)(4618075)(2017052603328)(7153060)(7193020); SRVR:VI1PR0501MB2045; X-Microsoft-Exchange-Diagnostics: 1; VI1PR0501MB2045; 3:GRoblLS0bb1Zh98x0EIzW8qww0LQs/EM3Lb5ptiPXTt4GhvfBx5URm/HK4OrkPblRKh0ff16ggNMWFC7xobQdC8mIcWIpr4G/ITQMkPwwhlZrhVlysTJMRM6coyqF1hpONmWx2LQLFAl81bttlxFs5hf3wonBWmeI7EeRsPKIOlYHjlK+sCSUZBnQ84AaA8byoVNlRdECYK9yY5cSouBD3jsIQWBYlYxLg5w7SOrt8Jl4cBUR7U1kY5YeTkR+EYz; 25:2NvD4UukoxV6tP3Tm9+Wjuys+1YQCGb3PWPpiWtFtDC4LeMWq9V2vB6g3Prme9Bistsg70qsbHfquVEpvOQGeC6JffktCm7k4dHIlHvbDoswna0qK71bLxh9WS2cQZ2LGjhDiUCQFUsGxcIAMBIFafOZUGkgxrbECasl4boiBIp/ossJ5+hibd73KYXMy8oC/dwqg6lH9Eqz9eZJ1ceqWCjfyPYv1F/mQ8CfLPFc00ppFuDvs0yy+nfWb6XUX8GtUWjff3J7T9BOWD16Xl3Fl0hpd4Ns/odXFMBR48MSVl9Px/ioGBk9x6sRnE5XXLbX9RnL5N8G4nJto89zvfjeCw==; 31:L74JRqb2SBfy8UUmLV1XK6DLlCKbtZlyCMAxaUfrg2l3o6YBK11OaE/DwJSbDjlvyHlcNS5eGFRRXMWxInrpMiBBajEwSxy9fHgAiEm7ffqN0Vvpu/FHSng93ZnGZMxjLLsCY9JFpkFXkiCbSUiog2nskN6K6QqeFsiRNW9qA4ETwB3Wv4bTBccQdLM8IUomcY1yoQcbRpCr8WXP8wp2KX8PvDIon23/+qoPfV5ZmXQ= X-MS-TrafficTypeDiagnostic: VI1PR0501MB2045: X-Microsoft-Exchange-Diagnostics: 1; VI1PR0501MB2045; 20:iy53EtCqEuhJQNdKLx9/qW7/dpxyNIaKO+MV6azfFpwuLQUjhG8LSmGviUn5Rw7mDGtW2NSeypMPcd6QOYzKzEd7wsQiyg5QruVZYn8Pqsrs8cH8jK41IHDqR94hJD/SNAqwdbMVMnailY+D1+QFkt6djnDzRnRJV/RYFGonfZMvTyIsmynDc/g/I5GSd8AvVK5xnkjxGTezdBHLYTW5rtwxI5CtZf0DoURig90K7L75bgoyY9NS2/RednX+X9bpJozr3aNOYqktBuW/Fi4vwwEDacKPiYmd05d5emBji0tjm+biHSi8YCufQNWCPzbKrNFDlvlIZJ8uAGBOGp+tW2wFy50i93uFZLSP4dTvONu8ISS9FgGziLFRrB/vYYuWwZh41iU22J6x3tZG3b3zwxCtJGAJQXKJIvSgynYUZTv5JOJ6KSaf84xSx/2Iob9zXRNUDTj1b4Um5NfkF1snMiIuhBr3dJVG+VCDmZy6zvnjFfP1U+g/yDP9mjDU6mQi; 4:sa4yyaHI/Yda5AApskEfWDSADZa5a3FWq5ROgr6VW/orkhKjuPA2Z60kMlgeo8WhQ7EJ8e6bFBRi4AVcIpqDPQAeKZg5yMPwe7XPEJ7MHfYGckWy3T4yLan9W0iucRWwe/88LdL43FFoC8yvTsEIoh90U8uqtM86V69KReP9hAk8YL75BUkmPvYMIkqtjkuKhPNvmn9iJ1FtrobG68AUgsAlkejRu+7xyIv9tPJw+coD1l3L5T1NKi1JbxBJDWPsue+gj5pJAlmqdgtkXa5YBg== X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-MS-Exchange-SenderADCheck: 1 X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(8211001083)(6040522)(2401047)(5005006)(8121501046)(93006095)(93001095)(10201501046)(3231311)(944501410)(52105095)(3002001)(6055026)(149027)(150027)(6041310)(20161123560045)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123562045)(20161123558120)(20161123564045)(6072148)(201708071742011)(7699016); SRVR:VI1PR0501MB2045; BCL:0; PCL:0; RULEID:; SRVR:VI1PR0501MB2045; X-Forefront-PRVS: 07521929C1 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(396003)(366004)(346002)(136003)(376002)(39860400002)(199004)(189003)(53936002)(51416003)(478600001)(55016002)(26005)(14444005)(486006)(316002)(386003)(476003)(2616005)(2906002)(956004)(50466002)(16586007)(37006003)(48376002)(97736004)(450100002)(105586002)(4326008)(76176011)(25786009)(6636002)(106356001)(2361001)(7696005)(52116002)(86362001)(6666003)(50226002)(5660300001)(8936002)(305945005)(47776003)(81166006)(446003)(11346002)(7736002)(66066001)(3846002)(16526019)(81156014)(6116002)(34206002)(69596002)(8676002)(1076002)(21086003)(2351001)(36756003)(68736007); DIR:OUT; SFP:1101; SCL:1; SRVR:VI1PR0501MB2045; H:mellanox.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1; Received-SPF: None (protection.outlook.com: mellanox.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; VI1PR0501MB2045; 23:AXZoA64lbNqNiY98nb+2Uyjc2nkBa9+bozCj3Nf?= PbTGUYCHCVWYrAy1LzY1UgNlbbC2wqXAgdH1AiOwa2rOHoLQKPfVfhI4Ovx4ikHYg7ucuDLtk3rksMmNIEAy0G5MpX/GN7y7xN75gJzK4mA8F+SQ49Am9KJU7+nNu3+kDLWXGU28eV587671QlNiLjMGQyN+6SZA9ZP+YLUPUvQu4yt5CV8G4FH4A2Kl+ODZKZwHkkAPLQf72wlT917haaNNas0YGhMIkB/3PgePvTpnA5360XdREiEJiYgmcHFaNEDaNRnUbQVD0epysO8IMZ31WuRYwP8P+JOL53pKpNLTHvyQNPUtFN/hqQlZWKfN9Blg5rgYTvNgm2npPQNIppJAfZbWMKGADljkFUaQ3avs1bxnBHzA6VJZYiPMeyHYwRAl0gdFg/moRwbv2cs52J8dNfvLT0Dionv/eTY/5s8RdF3tGcCKVCJu+vD/+1lrTJKDwak75I+tbXX7GJq/8QwYRRLpxBFnOBnfLWm3v00utxrKMbvOlz47Yc6tT1Q/fWP67MkU78phB+l7zsM72rjI7w+mFP5GuxGkF83uxM7WWRQLQCgq5Y1B8OH97S+58l5vDluWtzZ8oRbALypD8t+++npWAkVo092aaoN2PBjm/tVem2/69ZXLSB8vq4q04bXAN7ZWT6Pfev9zqy4pb3yp+jr7iunhwJPXWkJwzjRvaUdsSnDJpyuSYY7leqGfde2kpT5RN9Ydki+M7nTN5fe4TtLNrdfJxyH2akoKEqWUMjyTgNafdLghkbstaB8UX3ldUA8i0/5kgaq7fHcsGNS6Wqo3txVRWA6awx8OaIlio+LGuGu2SFiqNIk+9PEjAefNzkUvvDJ9I4x4YnZ62IHPCBfqRLco3OytPTPtYDTZq8EORY+tbfPag0JI9I8pWV+MVEL9cLEnCMtk5/tFqMmRDN6xuaUgKZrx4gqNE514E6ZTyLjdjBP9ZKaaN12Qh48wAsobM/ZE151qqPUUcLT8n7fXdoMKc7rI1oIaIAxLC3STgCWDR+qQLeYdQKZGvHv4YK/WJKDB2DxgXyeVZom2DzJLWGY66zdS2u+g/boynuWhEXXcqoIygi2kwEcRHHdA4nEHzQN8Ah+rLjzR8aNuMu1zRY9XJsZHVf1ET9RQ3irS2CBOMCeWqKILOf4uGTPsEd1av1qjz7t3bzwYAQu2KRx8g8i2USqxdZ3DyvmP5m0UuKcqH8hgavQLH4l7yPOGV9cWPl6qSVbMLjYkPRn4ziN87aWpUwsq18pd2gVprLJla+3oc4dkzn5FJyeoBqT8= X-Microsoft-Antispam-Message-Info: Opt4iqSlSbQ/E4582dsMCNMBOqguwgds7jaQQX2iitOH2lf3pzC2VIHSpg/N6NXHwgs39HE0HBTswl9B6vRDITrdZK8SI4DhbTonzspCdx9EuWvPQACjpQ9DYkiFpTWrcxU/+miCfQaPrpMpGVc/qhHxRdWb+FJycpHoHfb3Om8Sbjgq6I2ONvL7lw7BRg5YWiNw2EWb+rqSBQz4Bds+BAfHr2M/U1G0DVMp59SFPaeyGIREQYVvXe9byBF8/yRfnTFcyBQNTgRkIEPX5X8G0dj8UTMA+aFQBoEyWHKqvQedRQiciNr/srNq2HS0kbSMbYrQWIoAycAQifYRg3fe+pIwMIjNqGvDPxNt2nZjbVo= X-Microsoft-Exchange-Diagnostics: 1; VI1PR0501MB2045; 6:g5m7VSCTqgCDY096fGEJ6uyRsC4jOU7Zk4tuoT6JMx1LHVIXwTZFgpYqg5Z+YVmVI2fRXlmsHYYv58FtZpddxVmSBZ4qGfI5REJEkOKUeu7VMVrYEU3iZuIH89QQ5gd0eqDCyygYzqdc3bdro+V9aXLSs0kwIzTDjVjIQhbJZ+DMNy6k9t7Vfq6SauW5VIpyWpLTDR0hU0FOGTiBc6XF7lrSqa9EuCI2WoqSy3j3ewZ5E8YUlsDV/aCm3bGJuwWnbM7jpi5cKYSxLQSYXE+cEgniEVo03hV0IlsF0gasBSDbOmnHq7VJiZk5BcWv0Sdbiol1xl2CkKQi1ycm+fx6lJC8+LC7PqRbLq9tLe87RHNYb/dkazz84oTP3Z7epxE9Fqs2AerlDVipljzK93Y4bio4abTDaSRYbvleQvEbAOt4/HV6COIvPBWKxTwZKzEp2y57X9BvMXNp8yJqDhZ5zw==; 5:oD/k/YGg6bJebT6BXwnWFgNiHz4KFS6G4matAAQTnhxrokgdufj7FgML3yYVtZn8a9PA5HaSXY1zTDYi9AVGnWtHZUmDEGrG1DhCWFZL+rkg9iCNp4ytSh8YKoBee3lWTlckCqTBQCUkYXNo1xp59hMG3Pxc/vAQS/hG3HWW0ow=; 7:f3fPvJodwm04ASU/jTNnyZ+VXof2X2f2V0APctHxx2zkIToFZFtpxoDPZxqHCezNHBc3sLcTuPI+gCJVAhD+Orevcty07dySiDKA/Q13PQe2sY1MDHjHN+l/vsM1ef/O5hmjMLBg1FvLSA/tE8ZJkJGslP8f75jC9y+rm+OYE2Eikit53n0BUOUmLPtJS9za9s39G5tk37YmHHsM2norBpw1LEm/SmP4794p0l5jW3wbu+QhPWMrpxQ6OBBUslcM SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: Mellanox.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Aug 2018 20:58:56.4110 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 270047a8-a4b4-4ce3-6766-08d5f8bac3bd X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: a652971c-7d2e-4d9b-a6a4-d149256f461b X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR0501MB2045 Subject: [dpdk-dev] [PATCH 2/2] net/mlx5: preserve allmulti flag for flow isolation mode X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" mlx5_dev_ops_isolate doesn't have APIs for enabling/disabling allmulti mode as it can't be enabled in flow isolation mode. If the function pointers are null, librte APIs such as rte_eth_allmulticast_enable/disable() fail to set the flag (dev->data->all_multicast). The flag is used when starting traffic by mlx5_traffic_enable(). When switching out of flow isolation mode, allmulti mode will not be set even though it has been enabled. Fixes: 0887aa7f27f3 ("net/mlx5: add new operations for isolated mode") Cc: stable@dpdk.org Signed-off-by: Yongseok Koh --- v2: * do not toggle allmulti mode when switching flow isolation mode * add warning message when attempting to enable allmulti in flow isolation mode drivers/net/mlx5/mlx5.c | 2 ++ drivers/net/mlx5/mlx5_rxmode.c | 13 +++++++++++-- 2 files changed, 13 insertions(+), 2 deletions(-) diff --git a/drivers/net/mlx5/mlx5.c b/drivers/net/mlx5/mlx5.c index 83b82f11ab..ec63bc6e22 100644 --- a/drivers/net/mlx5/mlx5.c +++ b/drivers/net/mlx5/mlx5.c @@ -401,6 +401,8 @@ const struct eth_dev_ops mlx5_dev_ops_isolate = { .dev_close = mlx5_dev_close, .promiscuous_enable = mlx5_promiscuous_enable, .promiscuous_disable = mlx5_promiscuous_disable, + .allmulticast_enable = mlx5_allmulticast_enable, + .allmulticast_disable = mlx5_allmulticast_disable, .link_update = mlx5_link_update, .stats_get = mlx5_stats_get, .stats_reset = mlx5_stats_reset, diff --git a/drivers/net/mlx5/mlx5_rxmode.c b/drivers/net/mlx5/mlx5_rxmode.c index 3c0373bb4d..e74fdef8b9 100644 --- a/drivers/net/mlx5/mlx5_rxmode.c +++ b/drivers/net/mlx5/mlx5_rxmode.c @@ -81,10 +81,18 @@ mlx5_promiscuous_disable(struct rte_eth_dev *dev) void mlx5_allmulticast_enable(struct rte_eth_dev *dev) { + struct priv *priv = dev->data->dev_private; int ret; dev->data->all_multicast = 1; - if (((struct priv *)dev->data->dev_private)->config.vf) + if (priv->isolated) { + DRV_LOG(WARNING, + "port %u cannot enable allmulticast mode" + " in flow isolation mode", + dev->data->port_id); + return; + } + if (priv->config.vf) mlx5_nl_allmulti(dev, 1); ret = mlx5_traffic_restart(dev); if (ret) @@ -101,10 +109,11 @@ mlx5_allmulticast_enable(struct rte_eth_dev *dev) void mlx5_allmulticast_disable(struct rte_eth_dev *dev) { + struct priv *priv = dev->data->dev_private; int ret; dev->data->all_multicast = 0; - if (((struct priv *)dev->data->dev_private)->config.vf) + if (priv->config.vf) mlx5_nl_allmulti(dev, 0); ret = mlx5_traffic_restart(dev); if (ret)