From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 34D0AA0A0E for ; Mon, 10 May 2021 18:23:03 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 2E1DA410FE; Mon, 10 May 2021 18:23:03 +0200 (CEST) Received: from NAM02-SN1-obe.outbound.protection.outlook.com (mail-eopbgr770080.outbound.protection.outlook.com [40.107.77.80]) by mails.dpdk.org (Postfix) with ESMTP id 56DA74003E for ; Mon, 10 May 2021 18:23:01 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=hrrWa5fNOWvtD455/akgkkusCtE1QHYdx/yFM11S3mWd3GKogrx73j2krJHirAvi8TcckvkGH13XznhgDjmYLOwbMDeQXDCMymCPsNu4EEawKtymNwsmWsz2pmj2PgjHoEL/MRMqnlz/pOQkiQ4JtmpTiUzDZiHGxfhSiD+fyd+O+GC/CGMc2pS5dwg1ru0yw5gvbA1EyFJckdzTjSJ0+1Yew0xKKTV48K0De4vnKVneE2aj4oBKOKpBTYXkPztN4+3P4m+HAYVazi8YmA0rPnNtRc+gnx5Hu0KOkRkwW6eDAu8UDFCiuANtQvE5GjflOJKyIKEb1fyCJmdhg1/IRA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=JHTmxyDUadHOVvBf6GTMvWmUR6BfPb3Tn2Sk5sd6HSE=; b=mIiwaks/0QQOD9oECgTouqxU90amI3IrEBv6SGk2XwRB7snTBoBoYerExIuVl3tgMyatu3FEHE2Kg2XuaMTLID6CMdR8My1MUZhaffRXLm1dXy01VuNEuOd4I1dxY7SO0oA5eFknT5o2+JBKi2HieGOozjP7oh06oyJg6y/2yGb/1F13GmA3whqGlL4I9QABHOzQZ573FI+dX8nQLX2xHTEC8h3i5g/PqKoUMAk/SppI+IKMy9iamMYeIJQ6JpQZBzcNLDbZwZmonei6W48XZZSeGa99VqldcQYrrNxZyCHUgq5Wkj+qXZenrKzFGZTJnN4Avm2/wDrYHjfl4OFLdw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 216.228.112.34) smtp.rcpttodomain=intel.com smtp.mailfrom=nvidia.com; dmarc=pass (p=none sp=none pct=100) action=none header.from=nvidia.com; dkim=none (message not signed); arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=JHTmxyDUadHOVvBf6GTMvWmUR6BfPb3Tn2Sk5sd6HSE=; b=S8xvJdB5mIKpO54zIfEfalyBX5O9qohETxvOTFtZFF5uQ7Z6ay9lW08dm4Nj34YrTVNNmR2d0mdawXyAGGoVaSOEswx/av12MvliTRyl4ms3H+m+914XghFONUglKDX9sq/Zp+vdOjNkIbx0s2kPxGyfFiqkYCLXHEC1pyDlUvFenThBz3eXvnwtJpE/pFI+dNmBt5kGlTqhXQvWFqrEvQ8khhZCf6NLlns1i9Uyk7PFuA5rKOHKi4mYGkZ6IeLzGU5kWp0U0p1YMB86xpwZFVlx6uzrgNOjlhybz71Ck1MrhkhK3LymaU9OgNUDtLpzuyCks85kn038dOs686ECkQ== Received: from BN6PR11CA0023.namprd11.prod.outlook.com (2603:10b6:405:2::33) by MWHPR12MB1725.namprd12.prod.outlook.com (2603:10b6:300:106::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4108.27; Mon, 10 May 2021 16:22:56 +0000 Received: from BN8NAM11FT005.eop-nam11.prod.protection.outlook.com (2603:10b6:405:2:cafe::bd) by BN6PR11CA0023.outlook.office365.com (2603:10b6:405:2::33) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4108.25 via Frontend Transport; Mon, 10 May 2021 16:22:56 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 216.228.112.34) smtp.mailfrom=nvidia.com; intel.com; dkim=none (message not signed) header.d=none;intel.com; dmarc=pass action=none header.from=nvidia.com; Received-SPF: Pass (protection.outlook.com: domain of nvidia.com designates 216.228.112.34 as permitted sender) receiver=protection.outlook.com; client-ip=216.228.112.34; helo=mail.nvidia.com; Received: from mail.nvidia.com (216.228.112.34) by BN8NAM11FT005.mail.protection.outlook.com (10.13.176.69) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.20.4108.25 via Frontend Transport; Mon, 10 May 2021 16:22:56 +0000 Received: from nvidia.com (172.20.145.6) by HQMAIL107.nvidia.com (172.20.187.13) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Mon, 10 May 2021 16:22:53 +0000 From: Xueming Li To: Robin Zhang CC: Luca Boccassi , Yan Xia , dpdk stable Date: Tue, 11 May 2021 00:02:27 +0800 Message-ID: <20210510160258.30982-198-xuemingl@nvidia.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20210510160258.30982-1-xuemingl@nvidia.com> References: <20210510160258.30982-1-xuemingl@nvidia.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain X-Originating-IP: [172.20.145.6] X-ClientProxiedBy: HQMAIL111.nvidia.com (172.20.187.18) To HQMAIL107.nvidia.com (172.20.187.13) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 48d57ba0-feda-478a-346e-08d913cfdea9 X-MS-TrafficTypeDiagnostic: MWHPR12MB1725: X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:1091; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: l8DF/oxXPcv3ptfFMezjGt9I1li+ff+DJZmDH0hvYC2zPOubBpKTJhbTEQp422GK6+ZLUWHFCuru8Ir6e5MLaX+Md7w3FPVYArkUkoKfgZducA15hzIK3lJojZVOWEcDysUooacBlIPjZ22XCa2bHYhhImL5TkbHh/h49ZsJ30pjGJda1dLDy3l0A9L9LX42yw5zTfoYPg8oI6UHvSd0ZdLytdELeEHcqNtZjxnZgLyn35eiZNJBlnCBmOl+f38RL9GUXYpiVuQApQOGd3a5/sBXPKIhkDIjtPl7juql174vNvsVMDpuJHUIMZ0xc2wAhG7hi/q7QUEs3/FZGoDWMFfa2K4YFCApvmMlILj5U7O5dtRpJBk6IeZsVpmCNaeoaC+FQ0Er5BxnuNsl9UeTIFA7JcHERxgrxdpoAC7S1oXE4DGUmwJRpWZNs1qV4gXCXQAFQA0xtrMIxw/kPmN7nlbpCzytRHRoGcuZZTLAkS1T1/D7N0cqdYbYQwLHATgj8llwqUfSt7FztbmPIMtwyz8ZfBokT3D+iXsJ9ISGR8iO5C5yvlLEI1RpI/CWCIjy/dkdw8MIkCwhNQuTC6+dWTZ4dAE31UnPSlCj4mQI5qy/K+V9gRm4Lp0w9mQvEu8wSCr/UfKo0jaTm9tZlY7cBruz0hQZ3/jMulpbNjsGWb7W8S7qW+C1w7fV/V1o62ENuN2KIxSPnefbhDSmbOmC+XbvjEP59b3mWoICKQoiJZqlarYKCsHHR4Xw4M0jvHV3mEeixiulQmDkln7sc+8jTQ== X-Forefront-Antispam-Report: CIP:216.228.112.34; CTRY:US; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:mail.nvidia.com; PTR:schybrid03.nvidia.com; CAT:NONE; SFS:(4636009)(376002)(346002)(396003)(136003)(39860400002)(46966006)(36840700001)(966005)(70206006)(2616005)(70586007)(478600001)(8676002)(186003)(6916009)(7696005)(16526019)(82740400003)(426003)(336012)(8936002)(47076005)(54906003)(7636003)(4326008)(1076003)(6666004)(36860700001)(82310400003)(53546011)(356005)(316002)(36906005)(26005)(83380400001)(36756003)(55016002)(86362001)(6286002)(5660300002)(2906002); DIR:OUT; SFP:1101; X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 May 2021 16:22:56.1836 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 48d57ba0-feda-478a-346e-08d913cfdea9 X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=43083d15-7273-40c1-b7db-39efd9ccc17a; Ip=[216.228.112.34]; Helo=[mail.nvidia.com] X-MS-Exchange-CrossTenant-AuthSource: BN8NAM11FT005.eop-nam11.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR12MB1725 Subject: [dpdk-stable] patch 'net/iavf: fix lack of MAC type when set MAC address' has been queued to stable release 20.11.2 X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" Hi, FYI, your patch has been queued to stable release 20.11.2 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 05/12/21. So please shout if anyone has objections. Also note that after the patch there's a diff of the upstream commit vs the patch applied to the branch. This will indicate if there was any rebasing needed to apply to the stable branch. If there were code changes for rebasing (ie: not only metadata diffs), please double check that the rebase was correctly done. Queued patches are on a temporary branch at: https://github.com/steevenlee/dpdk This queued commit can be viewed at: https://github.com/steevenlee/dpdk/commit/86ec5aeef9e5e3b31124c89cf71d94e5aae9e38a Thanks. Xueming Li --- >From 86ec5aeef9e5e3b31124c89cf71d94e5aae9e38a Mon Sep 17 00:00:00 2001 From: Robin Zhang Date: Fri, 19 Mar 2021 09:20:19 +0000 Subject: [PATCH] net/iavf: fix lack of MAC type when set MAC address Cc: Luca Boccassi [ upstream commit b335e7203475c82f79692171ce9add9f846d71ef ] When set default MAC address, use type VIRTCHNL_ETHER_ADDR_PRIMARY as this case is changing device/primary unicast MAC. For other cases, such as adding or deleting extra unicast addresses and multicast addresses, use type VIRTCHNL_ETHER_ADDR_EXTRA. Fixes: cb25d4323fbf ("net/avf: enable MAC VLAN and promisc ops") Signed-off-by: Robin Zhang Tested-by: Yan Xia --- drivers/net/iavf/iavf.h | 2 +- drivers/net/iavf/iavf_ethdev.c | 16 +++++++--------- drivers/net/iavf/iavf_vchnl.c | 5 ++++- 3 files changed, 12 insertions(+), 11 deletions(-) diff --git a/drivers/net/iavf/iavf.h b/drivers/net/iavf/iavf.h index 3328bd9327..0196f74721 100644 --- a/drivers/net/iavf/iavf.h +++ b/drivers/net/iavf/iavf.h @@ -314,7 +314,7 @@ int iavf_query_stats(struct iavf_adapter *adapter, int iavf_config_promisc(struct iavf_adapter *adapter, bool enable_unicast, bool enable_multicast); int iavf_add_del_eth_addr(struct iavf_adapter *adapter, - struct rte_ether_addr *addr, bool add); + struct rte_ether_addr *addr, bool add, uint8_t type); int iavf_add_del_vlan(struct iavf_adapter *adapter, uint16_t vlanid, bool add); int iavf_fdir_add(struct iavf_adapter *adapter, struct iavf_fdir_conf *filter); int iavf_fdir_del(struct iavf_adapter *adapter, struct iavf_fdir_conf *filter); diff --git a/drivers/net/iavf/iavf_ethdev.c b/drivers/net/iavf/iavf_ethdev.c index ed69ba483e..649061d2ba 100644 --- a/drivers/net/iavf/iavf_ethdev.c +++ b/drivers/net/iavf/iavf_ethdev.c @@ -954,7 +954,7 @@ iavf_dev_add_mac_addr(struct rte_eth_dev *dev, struct rte_ether_addr *addr, return -EINVAL; } - err = iavf_add_del_eth_addr(adapter, addr, true); + err = iavf_add_del_eth_addr(adapter, addr, true, VIRTCHNL_ETHER_ADDR_EXTRA); if (err) { PMD_DRV_LOG(ERR, "fail to add MAC address"); return -EIO; @@ -976,7 +976,7 @@ iavf_dev_del_mac_addr(struct rte_eth_dev *dev, uint32_t index) addr = &dev->data->mac_addrs[index]; - err = iavf_add_del_eth_addr(adapter, addr, false); + err = iavf_add_del_eth_addr(adapter, addr, false, VIRTCHNL_ETHER_ADDR_EXTRA); if (err) PMD_DRV_LOG(ERR, "fail to delete MAC address"); @@ -1188,17 +1188,15 @@ iavf_dev_set_default_mac_addr(struct rte_eth_dev *dev, struct iavf_adapter *adapter = IAVF_DEV_PRIVATE_TO_ADAPTER(dev->data->dev_private); struct iavf_hw *hw = IAVF_DEV_PRIVATE_TO_HW(adapter); - struct rte_ether_addr *perm_addr, *old_addr; + struct rte_ether_addr *old_addr; int ret; old_addr = (struct rte_ether_addr *)hw->mac.addr; - perm_addr = (struct rte_ether_addr *)hw->mac.perm_addr; - /* If the MAC address is configured by host, skip the setting */ - if (rte_is_valid_assigned_ether_addr(perm_addr)) - return -EPERM; + if (rte_is_same_ether_addr(old_addr, mac_addr)) + return 0; - ret = iavf_add_del_eth_addr(adapter, old_addr, false); + ret = iavf_add_del_eth_addr(adapter, old_addr, false, VIRTCHNL_ETHER_ADDR_PRIMARY); if (ret) PMD_DRV_LOG(ERR, "Fail to delete old MAC:" " %02X:%02X:%02X:%02X:%02X:%02X", @@ -1209,7 +1207,7 @@ iavf_dev_set_default_mac_addr(struct rte_eth_dev *dev, old_addr->addr_bytes[4], old_addr->addr_bytes[5]); - ret = iavf_add_del_eth_addr(adapter, mac_addr, true); + ret = iavf_add_del_eth_addr(adapter, mac_addr, true, VIRTCHNL_ETHER_ADDR_PRIMARY); if (ret) PMD_DRV_LOG(ERR, "Fail to add new MAC:" " %02X:%02X:%02X:%02X:%02X:%02X", diff --git a/drivers/net/iavf/iavf_vchnl.c b/drivers/net/iavf/iavf_vchnl.c index c17ae06227..3d52a8c402 100644 --- a/drivers/net/iavf/iavf_vchnl.c +++ b/drivers/net/iavf/iavf_vchnl.c @@ -1016,6 +1016,7 @@ iavf_add_del_all_mac_addr(struct iavf_adapter *adapter, bool add) continue; rte_memcpy(list->list[j].addr, addr->addr_bytes, sizeof(addr->addr_bytes)); + list->list[j].type = VIRTCHNL_ETHER_ADDR_EXTRA; PMD_DRV_LOG(DEBUG, "add/rm mac:%x:%x:%x:%x:%x:%x", addr->addr_bytes[0], addr->addr_bytes[1], addr->addr_bytes[2], addr->addr_bytes[3], @@ -1111,7 +1112,7 @@ iavf_config_promisc(struct iavf_adapter *adapter, int iavf_add_del_eth_addr(struct iavf_adapter *adapter, struct rte_ether_addr *addr, - bool add) + bool add, uint8_t type) { struct virtchnl_ether_addr_list *list; struct iavf_info *vf = IAVF_DEV_PRIVATE_TO_VF(adapter); @@ -1123,6 +1124,7 @@ iavf_add_del_eth_addr(struct iavf_adapter *adapter, struct rte_ether_addr *addr, list = (struct virtchnl_ether_addr_list *)cmd_buffer; list->vsi_id = vf->vsi_res->vsi_id; list->num_elements = 1; + list->list[0].type = type; rte_memcpy(list->list[0].addr, addr->addr_bytes, sizeof(addr->addr_bytes)); @@ -1377,6 +1379,7 @@ iavf_add_del_mc_addr_list(struct iavf_adapter *adapter, memcpy(list->list[i].addr, mc_addrs[i].addr_bytes, sizeof(list->list[i].addr)); + list->list[i].type = VIRTCHNL_ETHER_ADDR_EXTRA; } args.ops = add ? VIRTCHNL_OP_ADD_ETH_ADDR : VIRTCHNL_OP_DEL_ETH_ADDR; -- 2.25.1 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2021-05-10 23:59:31.709695600 +0800 +++ 0199-net-iavf-fix-lack-of-MAC-type-when-set-MAC-address.patch 2021-05-10 23:59:26.660000000 +0800 @@ -1 +1 @@ -From b335e7203475c82f79692171ce9add9f846d71ef Mon Sep 17 00:00:00 2001 +From 86ec5aeef9e5e3b31124c89cf71d94e5aae9e38a Mon Sep 17 00:00:00 2001 @@ -4,0 +5,3 @@ +Cc: Luca Boccassi + +[ upstream commit b335e7203475c82f79692171ce9add9f846d71ef ] @@ -12 +14,0 @@ -Cc: stable@dpdk.org @@ -23 +25 @@ -index c934d2e614..d1ae5a3f0b 100644 +index 3328bd9327..0196f74721 100644 @@ -26 +28 @@ -@@ -328,7 +328,7 @@ int iavf_query_stats(struct iavf_adapter *adapter, +@@ -314,7 +314,7 @@ int iavf_query_stats(struct iavf_adapter *adapter, @@ -36 +38 @@ -index 51cad48069..8f25a21e85 100644 +index ed69ba483e..649061d2ba 100644 @@ -39 +41 @@ -@@ -997,7 +997,7 @@ iavf_dev_add_mac_addr(struct rte_eth_dev *dev, struct rte_ether_addr *addr, +@@ -954,7 +954,7 @@ iavf_dev_add_mac_addr(struct rte_eth_dev *dev, struct rte_ether_addr *addr, @@ -48 +50 @@ -@@ -1019,7 +1019,7 @@ iavf_dev_del_mac_addr(struct rte_eth_dev *dev, uint32_t index) +@@ -976,7 +976,7 @@ iavf_dev_del_mac_addr(struct rte_eth_dev *dev, uint32_t index) @@ -57 +59 @@ -@@ -1339,17 +1339,15 @@ iavf_dev_set_default_mac_addr(struct rte_eth_dev *dev, +@@ -1188,17 +1188,15 @@ iavf_dev_set_default_mac_addr(struct rte_eth_dev *dev, @@ -79 +81 @@ -@@ -1360,7 +1358,7 @@ iavf_dev_set_default_mac_addr(struct rte_eth_dev *dev, +@@ -1209,7 +1207,7 @@ iavf_dev_set_default_mac_addr(struct rte_eth_dev *dev, @@ -89 +91 @@ -index aae5b900bb..45096cbfae 100644 +index c17ae06227..3d52a8c402 100644 @@ -92 +94 @@ -@@ -1166,6 +1166,7 @@ iavf_add_del_all_mac_addr(struct iavf_adapter *adapter, bool add) +@@ -1016,6 +1016,7 @@ iavf_add_del_all_mac_addr(struct iavf_adapter *adapter, bool add) @@ -100 +102 @@ -@@ -1261,7 +1262,7 @@ iavf_config_promisc(struct iavf_adapter *adapter, +@@ -1111,7 +1112,7 @@ iavf_config_promisc(struct iavf_adapter *adapter, @@ -109 +111 @@ -@@ -1273,6 +1274,7 @@ iavf_add_del_eth_addr(struct iavf_adapter *adapter, struct rte_ether_addr *addr, +@@ -1123,6 +1124,7 @@ iavf_add_del_eth_addr(struct iavf_adapter *adapter, struct rte_ether_addr *addr, @@ -117 +119 @@ -@@ -1550,6 +1552,7 @@ iavf_add_del_mc_addr_list(struct iavf_adapter *adapter, +@@ -1377,6 +1379,7 @@ iavf_add_del_mc_addr_list(struct iavf_adapter *adapter,