From patchwork Thu Aug 2 21:06:31 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Yongseok Koh X-Patchwork-Id: 43541 X-Patchwork-Delegate: shahafs@mellanox.com 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 A427C1B50B; Thu, 2 Aug 2018 23:06:53 +0200 (CEST) Received: from EUR01-HE1-obe.outbound.protection.outlook.com (mail-he1eur01on0045.outbound.protection.outlook.com [104.47.0.45]) by dpdk.org (Postfix) with ESMTP id C685A1B4FA; Thu, 2 Aug 2018 23:06:51 +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=IAlzNZlBRnv/byqa/qEFFEkWEV+ofoKr2R9GbGfjLYYmkPgL973n4iwAmgEVD3yxEtfXiRI9h+Z++5iTCqOzAw6Bavpah4LmzVUQbNNffg1x40lt39nT26G62v3FxeGdt4QaZZXJ/J/5UdmPPA4l6EIpso5pGkLxSxHmJ3Jj76U= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=yskoh@mellanox.com; Received: from mellanox.com (209.116.155.178) by HE1PR0501MB2041.eurprd05.prod.outlook.com (2603:10a6:3:35::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.995.21; Thu, 2 Aug 2018 21:06:47 +0000 From: Yongseok Koh To: shahafs@mellanox.com Cc: dev@dpdk.org, Yongseok Koh , stable@dpdk.org Date: Thu, 2 Aug 2018 14:06:31 -0700 Message-Id: <20180802210632.11006-1-yskoh@mellanox.com> X-Mailer: git-send-email 2.11.0 In-Reply-To: <20180802010557.16648-1-yskoh@mellanox.com> References: <20180802010557.16648-1-yskoh@mellanox.com> MIME-Version: 1.0 X-Originating-IP: [209.116.155.178] X-ClientProxiedBy: BYAPR02CA0036.namprd02.prod.outlook.com (2603:10b6:a02:ee::49) To HE1PR0501MB2041.eurprd05.prod.outlook.com (2603:10a6:3:35::19) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: dd938599-6f16-4028-6432-08d5f8bbdcb1 X-MS-Office365-Filtering-HT: Tenant X-Microsoft-Antispam: BCL:0; PCL:0; RULEID:(7020095)(4652040)(8989117)(5600074)(711020)(4618075)(2017052603328)(7153060)(7193020); SRVR:HE1PR0501MB2041; X-Microsoft-Exchange-Diagnostics: 1; HE1PR0501MB2041; 3:WL1Sc6Ls8KmdFGcH7q3yBXqsX6WhgZqSxGd21xxIxUXImht84jh+LgD+y8kx5d77CRIQHbv5YZeM9LrYZUn1DCr8CBCEUQ/1B9FvwV3I467J9t3F2vFWuiwZlG6Nib6eHfum3RqTAk7oMiM38SE2yMbszMul07XqF+6pJ8y0/znMlIOZataIwPN0eEcGK5ha4nbwEEpZ+8u7t5R0gTYr4zRHuUJcKyfjBRiUCdBmwqfQxQxDOnnSxrtSoUZ7VIYZ; 25:iqDsO+VhVNEhIkDcluM111ib0bzxxY5v/1gIT6DSvzNirvKe7/V5qv204zXbC5pqhfP27r3Sp0eVpD9PwNQ9WBzZN8ebHRr2fxenQFsZq9NsS6OLm1XRjrsYmLVnMC0UNkR66viaQl5rO8FENBt2G7MvU0aENEj6KyW13cC5EvfpVoCu9PrralsnBefsE1GYkA81AMmeCVk4G+om13nba2Ya/xpX2yehVhozMDGsZjrLP9tzj8vZzAnDLxC8VX7OgwV5iOSGNktmA2bba4pWfJJnzacTREJtUOVBVJcKLogr5DAUXnQeS7SxzEyGoAGTkpibZqD00ztYuV5F7eH+ag==; 31:DtSlu458R78gwDhJa9IeYF3gMeBCyaKYvumFWfGucCCbYFojZUZzCY1P/SAzcExYA1+fWAg6J6XdFYwDduxgrgDBVHO/g1sX+mp/aK/53BLmHRyh7TdIbH+VagLF/1YknzgX+LeEIYPhwuF8y4hvcURtrB7a/P2HgdZl6fEtKpnwa1voT9DJp8cJycN6FY5FQofDysx1HomFPVOJOdowWffgReBV4ZTVZprVNMslT4o= X-MS-TrafficTypeDiagnostic: HE1PR0501MB2041: X-Microsoft-Exchange-Diagnostics: 1; HE1PR0501MB2041; 20:6zTpPMUYf4Udv91jZcWoPM291lLWnA6GMQn0wPfRp83zy08Xl3Z75wAIfNyXJyAycEFzlVc+C3wfK9dIbKOcbET2YpX6hDHV4MmcDnZXS/yX64rkG0yqo7WCyfbjTiM2/OyEe42lQEmqEhBMVuB9gASESZ1RbLip2qFFpHigrwKZVGKphEgMfAQGw5hkc+dfBmoTFUKv2ZpDOgXiHJ2rywJcU7pgwQNfrItFi6vyl5zsvYg+hTTbb04YPFPY7KEC05MTvpjrCmJ6DdBFyQ96rU9JB3NtBdvnKmJ0WqOCe4BKmqR8NRPD4CQ+3CXDDTpvVoI8bSicid4nVDocONrahpi0tdUWNbm5VmnQmdWZ/u1odizuXW9VtYLKdsSTnOMupfV/lg4eUUCX1JPdyPQhD3EDW2dAYvwuUBIsdt+rL7SdADolCs+xygZrM9DnbSXsGemQ8BjV55Bm1xDmWriCKtiOa0wClG6o6S/9R7w2VXLjOlLL40VlPXmnHErj9/6P; 4:Kb3GavwLfCKQZw+Vsgwnpc0aTkQ0tCqjfnhWE5sjix4QYRVXFqi0+Wa85XKHTX6qtT2AlMPXPYnECHYZMkWtvFu+OZTY/MTtAP/M7ZlqgrC67MdSW9jb2vUnMbEch6aLOCWvWVSpHZpiT2T/psEjlIaJnF4XNfIq5COvqfZHMMuxt39qW/0DgP9c0mqLoqvX3GclMhSHIC20ERU60bJYpkdvgYwICpR2YoMVv0U22ez/Czi3Uk+xzH6fIHjnXG++2r9Lwdh9WHCarDFB5VIl6A== 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)(8121501046)(5005006)(3002001)(3231311)(944501410)(52105095)(10201501046)(93006095)(93001095)(6055026)(149027)(150027)(6041310)(20161123560045)(20161123564045)(20161123558120)(20161123562045)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(6072148)(201708071742011)(7699016); SRVR:HE1PR0501MB2041; BCL:0; PCL:0; RULEID:; SRVR:HE1PR0501MB2041; X-Forefront-PRVS: 07521929C1 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(979002)(136003)(39860400002)(366004)(396003)(346002)(376002)(199004)(189003)(11346002)(86362001)(105586002)(450100002)(106356001)(21086003)(34206002)(25786009)(386003)(47776003)(66066001)(305945005)(53936002)(36756003)(6116002)(97736004)(3846002)(2906002)(7696005)(2351001)(4326008)(2616005)(478600001)(55016002)(48376002)(69596002)(50466002)(37006003)(68736007)(8936002)(2361001)(6666003)(16526019)(14444005)(26005)(486006)(51416003)(7736002)(446003)(956004)(52116002)(76176011)(81156014)(5660300001)(476003)(50226002)(81166006)(16586007)(316002)(1076002)(8676002)(6636002)(969003)(989001)(999001)(1009001)(1019001); DIR:OUT; SFP:1101; SCL:1; SRVR:HE1PR0501MB2041; 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; HE1PR0501MB2041; 23:Dr54/gtvJbaEwl0DPyowokpki0HZcl+jpZyoAzf?= fgu7UR0GKl5+2koGkOGmTRyREw/jKDav5AmC5ZiBqmR0osLagPbRaE3Ee+qsVUiG4zJoq0k79kcd3V0+I7oa/NWLkIemD1i2iF+WMcJMp/tjseMLeEfWpwjXGJMn4LuMFS7BQbhnNzs1n+EpC5e38/dIwLwtobnz+L0aFgNXX3g99Z/nMMo80UA+NU2DOxshvVVUD70Ha6Q59FVb+FYIlsue5H8DQip9to2wMmdWUdWJWBni0wOQi8T48h7x/2gbZ9abZlIH8dJ4eRyhUofPiKjA4PVY5OagLb5ZrCIiwPszSYyEK6MsDn9jioxB4itY09JwwsiGsIbCkBbK376bNxx+xg1iGWC6Axzi/rSf9mX40aDZ9BuRt6IY1SjDT6PK9YkYdrBzIQtvRgIEDAVxKIQ4HW6/tL/Mx7FzPvURJPSCUKj+Q0oNVWas0mh5nklZ3EilJ3mxMIwCiW/9LR8Hikrvhmjih5TSThuVfPFNJL4Cu9EHNCJjfhSqoCNPSacsSKEr+yu8IDvKF4Ii8Qwq7KJxXAKXxQuLYndk3f2LZrfGCbs339KiAoCnrWi4Djs+zGcdxrbXxxFDx9LhXbjSbmCpA88NGQRdakYlf10ppNwkX8d+lWqz7Uwvtba0mfiBnqeb5HwyXv51cclfCzTNxnJTW25Fn9laJ8VGJgDYn/DRZXqiharjblVqjpO5NDBqV3GNRw/HdJQkhLFAlV1vDl4DekYMKwY8LlgRsnqSHAwfTgIk7fZGbg7WGllftYv2y2X21Hf3+Tvpqfofo1Xmwpe1lalel7OvJNkyMos9ccC2fE5CFyHLvJY425pOwLuMYY8q7tFuQv5M/8eTtDRvl265pJAqbRxPYxBqxd4dH7/fvubVkZcUEMfy9UqLkWiaICsCopYT6EannbyS4To7c9c/XrkTBqQYLEy6daO2MfwbpxZWN7ic9YwUDVE6rB5Q5A9y20A07q3tuu4qImNmX4GVVq1zj4FbIzTwp1u7qyDAmBAyPAwaa9NsZpLUVJ8GkLHA3R6AR02ppBfEcbZKyRL9Vvl10IXOSWhAbcrLGq8EEKMmmaoEoAur1DOXav7sDjXdkGpSyR5BYZuI1W6aoYrt6S7J+CWQn+q1vGQ9QhYjG7CuAB6+aO9oVZ9BVz/GrRebTCUBEF8TIlXrPXwbix+N87nFjfHbLgXJ+NSxnMaU2MUpBzLyCcl/Jtefcza/21HSOsX3mwfr1KjKfvWfKIfOU6qV/ejLydOi2q7E5Gh/AaATwoy+JLaq39jlv2c8V2AKWZFPemaZONJO1PfXtIqNrM2R71HuhRjxOX2NCy3SQ8EYudd0LG1DAe9mdHst3fpPaek96XJEd0ZCLWnbdqFz3J8kISpqK1K4y5XKS5GUV0w== X-Microsoft-Antispam-Message-Info: rxvRTxLMFl3N5Pjlw2q7uwVxV+SHH7+FsOB1r1NIRWTJ9BRwkb9u0Q2TkMMhWYucuDytqkJCihBKahXtteTKCi6t4dlp9Yrm/1feNOXmlLFea0TqX749VuYtuYW8um6pCnP9MKiIk84JW8IcXfifbWaO3QVhWVlJvUprC10RD7Tw+FHVjKsNRrVR32dpQ55F1h/3pSEZhFnmqZ6s0RZBx+DRsgvxOD99vAPSVNVFP5PpsPTuiHrcEefRF9lTSwDkomHS/pavptgynPnQ24Ih7vhfLkHdtQpxjAVPpDPi+ol6LPKsEdVgO72ggUFQbQ/17nUowKkfYYVpWzc5VsmOJknN7TFfkR46ZkHm5XitUuQ= X-Microsoft-Exchange-Diagnostics: 1; HE1PR0501MB2041; 6:/BZVJTAaUiR+rVs45423pwaRD/tek3tuzBM3kou/DzcCiwfIJcu/BSuEatUdpucZfusU/yyNFmSkMadXbZCnCuy6e8gV5W6CvwQoN/tqnhevclo4ry01uy4zWMgE3ugWKpylKkMgCQdc4hS96gnESTLfVI2C7PqRO3n9U+4CmEGmd4Qquuv1Mzpxre0CHUfdlnXY9ehkscJIT4HodSCiR3LbL/XxYE46Gu94bBP9Xs981etsg9WJ2cNg9d5bPuwQiDLanPWaghGC0aXMSPiElUG83RlE60uwmyXYz7/807EU6hmSuXqOIAqgU0abGkBuUMjeGa0RAheSeFwl1LDl7nVDOqbdJZuggNCbSW/dVs9tw1sjzSSJgjDEa2ebjApWpKDnDPBmIhF2GNrKoy5vr7IrvSoU4e6a/zR9mlOJmb+Pk6RiiP3kUYuou2XX6PA7M9jl+Bm2I052bNHVAaz7iw==; 5:LqSg7JvuDoH1hmr/15wDZttn9FRRAFVyWnVGdTM60RSF87dyU6OMEy2Aeo9o5xDpsF82nAjYMBF1Ko/sVGdza7DCyYALUbGrWzvWJ5jA1Q/mn+j/Y4KSbC8rxvfYZ1nxccMc1SRrHHsNsMg1lE4EeOgjwySKlju9ftaScEKoK5g=; 7:nrlhg3J6WviTY8qlXI8VQNGJkhdDSe35UacvR10sEuonE70krRxaznqruCvTLWz7ia4Fw5RKVugrHnJnlAKqErlBQGhUCbFkvwcFoxnjNsY1gYLsiOk144+LgCinFBEY/1572oHkvp7DbVZB3ZrYXzZYjV26J9ZduyE/prBd+aInzCaBCApUkiPvAWEA+W+sJT+upWCSdV1amgmCRv13EuPFvw2T2Fw8YNV9En0oAhz6EdmHiFrv3ZWw6jLP2TaC SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: Mellanox.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Aug 2018 21:06:47.7079 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: dd938599-6f16-4028-6432-08d5f8bbdcb1 X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: a652971c-7d2e-4d9b-a6a4-d149256f461b X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR0501MB2041 Subject: [dpdk-dev] [PATCH v2 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 21:06:32 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Yongseok Koh X-Patchwork-Id: 43542 X-Patchwork-Delegate: shahafs@mellanox.com 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 E9A621B51D; Thu, 2 Aug 2018 23:06:54 +0200 (CEST) Received: from EUR01-HE1-obe.outbound.protection.outlook.com (mail-he1eur01on0045.outbound.protection.outlook.com [104.47.0.45]) by dpdk.org (Postfix) with ESMTP id F03871B503; Thu, 2 Aug 2018 23:06:51 +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=fQ3kFOn9H3hozcCaIie7q40u+4EDvWvQWc4SlrjACx6hFm8dx4OAKY6MMkd/O2uKQgADe5Yc9j+yzlR0wK8j2wavodHstUh9HD6AbNXVids6gpDHYCJL6fMSHp2ut7qRIAF/XQpo0EbFPoM7p067l5HD1+6q545M3mFOBVoOdB0= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=yskoh@mellanox.com; Received: from mellanox.com (209.116.155.178) by HE1PR0501MB2041.eurprd05.prod.outlook.com (2603:10a6:3:35::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.995.21; Thu, 2 Aug 2018 21:06:49 +0000 From: Yongseok Koh To: shahafs@mellanox.com Cc: dev@dpdk.org, Yongseok Koh , stable@dpdk.org Date: Thu, 2 Aug 2018 14:06:32 -0700 Message-Id: <20180802210632.11006-2-yskoh@mellanox.com> X-Mailer: git-send-email 2.11.0 In-Reply-To: <20180802210632.11006-1-yskoh@mellanox.com> References: <20180802010557.16648-1-yskoh@mellanox.com> <20180802210632.11006-1-yskoh@mellanox.com> MIME-Version: 1.0 X-Originating-IP: [209.116.155.178] X-ClientProxiedBy: BYAPR02CA0036.namprd02.prod.outlook.com (2603:10b6:a02:ee::49) To HE1PR0501MB2041.eurprd05.prod.outlook.com (2603:10a6:3:35::19) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 0fe476e1-7862-4826-7e09-08d5f8bbdd9f X-MS-Office365-Filtering-HT: Tenant X-Microsoft-Antispam: BCL:0; PCL:0; RULEID:(7020095)(4652040)(8989117)(5600074)(711020)(4618075)(2017052603328)(7153060)(7193020); SRVR:HE1PR0501MB2041; X-Microsoft-Exchange-Diagnostics: 1; HE1PR0501MB2041; 3:wM8jA1Tn+B08m1lAsPo4M62Swu6V3rlvOKjqXk0LFU9enUJPXQMnq/5f7IeT5KwfmQFcQLR1wp3IS08RK5Tz8nYqw0AXbflhKuVyJSff3hpeOBjdQVAavliPkDiw7MBnGbeR0ws79haDwYH71ozOcIXL1gRUsLHVO6wfZkWT/9mOI7xupZqGwmAHvTt0R8ZKas/Da9Ak/g+uFFEnkPCyuqAlbCSXE1gea1jWChqvYuTXUt1SONNHJv4Xa8oXKgWw; 25:xNxvfwDkE7ORLiZrWax6XSc7U0h33msWJ7RWKPdJgv5W59zavFBiGu7xqk0fnqxJio+y0JhqfnwL/fmKIuwSaTLT/BEBXRYDXv/fJUwIacqvRIfz3PDrONW8LcoV1YrxpgLdJz10svGKbbNAmde3rO/uWNzML2LwgbrGiucz1Y2HIctWueAaj0E6lSpi33CSq24oqPNusw91v8c9E1NeKTTwDRLTwO2t08OnYHkNnEw7YIR6g+NQrFG8saWh/FVnKkOSrShkIgKhLE4EOaRT5MdoOqJVcfMn6UdaC7ACEBcNce7/nKBtHw3vdHjy7K0V3Vxm0EGfsRuaqPBYNH33zA==; 31:dpp6ngmaffbcV6z0rXq9FBHzVjNtwZloaezZaRhEAvweazbQI4PCdU1aIg/I5XtMP/IctzsoZ57Ypt9qyS/yhJeFjT6SPnvfmkqxMf0jxCtOhL9IKLFEE+B9nj1C/5QNF4LOcNuWznsF9OO8qwcTma9QgI0AKck14TcTTA/RCJJRTeDNFf2bhwixHZD5Hu1bBnCKmDYwRkNK/vn6G+6QgvlHowKH8cW6+eexP4AgCe8= X-MS-TrafficTypeDiagnostic: HE1PR0501MB2041: X-Microsoft-Exchange-Diagnostics: 1; HE1PR0501MB2041; 20:NLd0grd4Jr1x8FP2EJjmD/MtySfNfygjv21RrXXzuhSjlQJ1m24DytcBuMywAjrvmEYSY9W3qEd9Dlrg4KS4tdvSJAoRwoxFI5O86aNSHptLLVJrojn+4toQFheXJSHKfHFCk3FAmuSFRKaRfkDJSW6GbLq8X4wv2P8erLsMo0Biks09Gkfx9GTuzNF0w1VpCei/ChvvRWb/DV8n+UgJwHNBj83MHY3p3gVm6OH69ukgJXqTPiWHFJWnkqAXZ0HQrhlueeI0Y/LUH3jrZkzqRQZVMy/A71rWa6QlIbi5osqWMLutHIaohkxBYHLOhu6q/EOmx/eYSGE9hlt/X+qyJoGZgZvZ4MEJkMJFLcjT6w4ljakUtF53NMhEcHxzI7JVjAl574Ks+fRjI53ouLLQDJDEZxfbpfEHODkff6faHGcEubXlr0XPFJGilzOW+lIcbNH7HkCTNUjzsWdh5LDkVpiFvrI0SCx+m7DnTgymb0ZSOFDMfUdp95c+vh7FDjJd; 4:e5bOLTjrP6ZUwIOglL+wUlzH5oE4RN2Fm+8vgI8/C3gLLf/LnHnNEBaKFZZuPAhCOhl0D5yWC9sp3mMCqNep7L56NO7pF3XFMKt4O23rDV9iVPjxSyXilaROT/p/H58xQVhJQkKuVFYHakFHvxLJxEXM/G6xuoJ7AV+H2eh62EpAYBT5xQnNhX8Eg2qqVmtgZ1DHMLmGS7IPmBLxkv89uiI8pJkkGvOzxZw/dGeoMnBSlGfMhFIWu3YsnVqo26JqFs5StH0szUMn4EIDTj5//w== 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)(8121501046)(5005006)(3002001)(3231311)(944501410)(52105095)(10201501046)(93006095)(93001095)(6055026)(149027)(150027)(6041310)(20161123560045)(20161123564045)(20161123558120)(20161123562045)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(6072148)(201708071742011)(7699016); SRVR:HE1PR0501MB2041; BCL:0; PCL:0; RULEID:; SRVR:HE1PR0501MB2041; X-Forefront-PRVS: 07521929C1 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(136003)(39860400002)(366004)(396003)(346002)(376002)(199004)(189003)(11346002)(86362001)(105586002)(450100002)(106356001)(21086003)(34206002)(25786009)(386003)(47776003)(66066001)(305945005)(53936002)(36756003)(6116002)(97736004)(3846002)(2906002)(7696005)(2351001)(4326008)(2616005)(478600001)(55016002)(48376002)(69596002)(50466002)(37006003)(68736007)(8936002)(2361001)(6666003)(16526019)(14444005)(26005)(486006)(51416003)(7736002)(446003)(956004)(52116002)(76176011)(81156014)(5660300001)(476003)(50226002)(81166006)(16586007)(316002)(1076002)(8676002)(6636002); DIR:OUT; SFP:1101; SCL:1; SRVR:HE1PR0501MB2041; 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; HE1PR0501MB2041; 23:sJdHHDwE+lK0SYPIVMWnwP1GSWHxzmMcbLo3DV5?= Nvb6H3TRxEXV68+5PFyyxriUJgmuwYsZtMslCZP9KYVdcK6LrPOcbXFd7NFkxVG/HewPoGsrnzDBlhO3GPk3F2DoPc89hXKeoJYXyMw1lpLUqMn3UCsD3GpuMDtYZPqcnmVjWwXY6rIcIrRCIOQ7gBwUsS6sbi3MS6U9ox0zoffwvAhCggUkVGPM4IBSk6HGMwR6+Aj5yeBXOrb7LmKihzoJmpGrWou+f6dXoXFv95fkMOPCu2EdC8GKdijDpFT+FmPoD3Z35poAYfTfqU5GtrquQvCM8qYUwSxmtM5amwOKLVk5mEwMbuXPluPhsG6DM/HExW1gAxQ6V2yEowRYeXnVxgpbL5Uv5DsFHSZtKU3bQgvKNWY5jHWsJg0VRnOQbzeAAi/z7csA7ZWTP65oNCWnLdN5z3HfwE08gB9P/Ja3+ZQcTNz+8CcQw79FB7bVVe5dhzs5KIROyWTJ9yTivEBz6cbwAHlhkWUSm/u7QMZucz27A0Vs3qTicY0tWat/oi5AlM7468DoLNcOpT7J4k3XzJ/P2P8lx6kGe891u3LoWEGyxV9UJXFq9QsAFPv4xaDJF+tBfp5e3L+dxF0S51HeJXXYZQSmLkbw6YLrlOE4h3+DBlJ+52FwHEzY/M9JYjbjbf11fV6ARvRpXdCbeUASrK3+X1VEGXEz/kWYIWPeozpecFLKRQn37rEbyxd1RVAuT8ijdY5k1QVH6N5EYDiiaC+0mOcvK4imL8WA6kDskhyOt8QC1X1UDxPgy3o23IjhvNEkto1HoDWds0HO7MyQflzpkT6f7TeGfa1SzZwIETz02tJCKUZCxSwiuroeuyVch7+T0mp/sa55xLHJdEbfBHC9EUee8rW1aqE59yC4h6SW53yG91q1kjBdMqMsad7dObheTj9/ZhUtiIuCZDZ5RzXvEo+f0f6yq9b2vVxEP3gi21O77WVQoO0D2WSTOWuJO9epMGy1+957Q0OaOaZz6ue3RXtxQiZrP+aKFL1vgIZpZUsrNQDek5mMOxGWMOYrIC2OTU5gqaVR9VWZcZMNcsHJM/ISRBj9wwcqxuV1fcKUSfhj0Ldr6gb5RbALhc9ru90h0ls+JE7CemDLscX0VbvIip4XzAteg0mUiTb1og0J2dlOuTI3irF4ck1G8psuEnHvFSt7++YRTikz7B1r9wfzclnO925cpMnyg6XR4stFleZij2IAbaEeOQEMAaQ9jilGHK5Srx1Xyz2DO6YLQ0RQGnoLL+Bi4t5x3Xx+Awbm/5j+4VRKOIwMZ6M9PSTo= X-Microsoft-Antispam-Message-Info: UYWm/9+4mGS6XeRWTMozaTWLy+xWft39V0pnyaLG/YoCv+8/3iutQQk+CdTSkhB4KML5yuWp5Zfm/Yve4hCHNOsKC/2Q9nuTe/S0awS2gumr5Eqb2tqyYsP67m56m1p074pMe1xPuemSVjFaL9RMGFmVLv2twY6ybUckN3VYtEEU9Q1inE6HdgJ0Cp3ZnnYHc0b9FAKu6ht8zB4IEGqfgBqjEpbuwfWK/u7ga2qrpzooTNgZSXpCwoZ8RthmMkk7yVji6ee5FsXmpf+1ra1EXBXg+Ru3Bkyt+WA19euqM+mZ/hasGCBFqthQNHRIKJJB/yvL2QL5i2yNNLyRN8GlFa9l/qaWslRIzCwDpLulLr0= X-Microsoft-Exchange-Diagnostics: 1; HE1PR0501MB2041; 6:IMH9i1Z6e045Hka1sU9pweEMwoLeu1qP9lurD1dmRftBC/WeN2ngebrXeEFF2hbleUfq32ChdSw+PWiv+4725DwoNZQPpVdF3M/TgSGE9OGu5kTkn9FJ5GzJRpKviXiqVcvm6mwswpImJihaf1lMaM9FsnHkBOmnCg+eU4rg+xMiSj1kLhGZZ9hTbRr2WxWJd+APDg6roF3gx2WdawzwbmGh757XAjTFeN4B8pl5JD3uerdY6bD7VfD7AfgW/4e419mXWJq/6tGZ2lDSDgBwHCDP/vpo3COXeNb+O22h2eWxluvVQG27jWZtQQkOGMJ0NxKiRW9sPI11fDb8MbtVv8A4K0x8u4usOmsSxuAeGZbJ2PbdQrGsi2+6aTg5PfX2B+dOTvpe/EyiKnZ4w4SuOWd1zVyiblXtTlcpnAin0IkflFq2g5vN7dKLX/KsMQvbrpgKEO5UQXXGB8N/TovT8g==; 5:ErgnNSe0qnun1jD7eR9KPNqdUF8GS/A0QU0WSi+vKcEcHtdKdbgJZQb8c7JfaHKK0CeMqCrtO9My/FIFWQwTi9mAYFHdUN+HPpX2BTMLyZcsyrF2yb/Bc5/7jBHT72WPBeIYBoBxvCswBNI6YYL8r5GuqlU4VNFg8D0wu39C3cs=; 7:h9OZr69E9kYqTEmGyRIXg5ml9hhZGZRBxi01cN5Fp39oBoe3EoGoPY5JZP+YrsAUO1FbK8LO7Cunzz4GIjNuyiMSXcbyuftANNs2W6tXgtKM/YV6TG9oVziJ02QzU0A+VRNfzzzy1kiQcvcWCSyf5S8CuDKRrn2iwrVBpcL45/4+9YdC5UXeKPs8NdjqI5R3bMlmz+K3QO6D68VSmGdCEW4iP6XocSPv7MEjEwV3/8HZm8z4iQKK0cAPjXht1PGo SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: Mellanox.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Aug 2018 21:06:49.2693 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 0fe476e1-7862-4826-7e09-08d5f8bbdd9f X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: a652971c-7d2e-4d9b-a6a4-d149256f461b X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR0501MB2041 Subject: [dpdk-dev] [PATCH v2 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)