From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by dpdk.space (Postfix) with ESMTP id 54D8BA05D3 for ; Tue, 23 Apr 2019 15:35:59 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 3C3BC4CC3; Tue, 23 Apr 2019 15:35:59 +0200 (CEST) Received: from mail-wr1-f68.google.com (mail-wr1-f68.google.com [209.85.221.68]) by dpdk.org (Postfix) with ESMTP id 4DFD54CC3 for ; Tue, 23 Apr 2019 15:35:57 +0200 (CEST) Received: by mail-wr1-f68.google.com with SMTP id t17so8529989wrr.4 for ; Tue, 23 Apr 2019 06:35:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=netronome-com.20150623.gappssmtp.com; s=20150623; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-transfer-encoding:content-language; bh=STEnMxYIqxlKEBSQ82g2C+BpDa/LgmNyXW+ih6M2gYM=; b=wedgfMa5VJz4giKB/wht/Jh7GJMXflwd+iPsLefwfAUh8xmVBiyOy3EkGAqaM7AETX HnaTn9KzbBDNK3F9xB4k1tSbj1e6JdAjFgqSn5GqbjwnRaVfjTUrLS08x11U2/XR9mbh 8JrW0jA0fKDOxkskcEzS6jvF/V4Vhhvtc/S9yPsbJELLanTpB7H5t/0EYLR0ZIjjajkQ XenHqPT+lcpPdV0H8P4OJXdBSOKes9sgIGy2IGkJea8k7Jq2y2Y4VF3gAirdTAnFhN9+ RamRL2dHcODgSMuDFsCGlYwBSIdjM/C5DsalQqUzdE7nN/EH289hAZ2k1q1L5+0y/EaQ 6Fgw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-transfer-encoding :content-language; bh=STEnMxYIqxlKEBSQ82g2C+BpDa/LgmNyXW+ih6M2gYM=; b=XPScJwcCDDwrqRxdK8EEWSnuAQF8zJIIb2Ff0rg7QF3wsGwaqwEoBWEt4oqDzxrGFi xZdakw6FavwGzkfFBxnruu6QlnyF/RaZXed82kfHqrbvu1de1dPCGKfE2F5wkxTjMUvJ IrwxW5k802JymkY4RtXUexQiBh0Z8z1HtslYFfh64GeXGQRHFllMdW4Vu7yAwc76gpBw yAJKwGnUfpVGuzxu5V510sSGXMRyV9VELFH28pgTP5lRkOtf/9BTurX9aUx/PCHl6/b7 uSO3oFbhRprJM52RTQCF6utDcTBU8XD/4XBMRujg4Bb9Skimq96VWsssnCok2P+LKcR3 hKCg== X-Gm-Message-State: APjAAAWXmQjQeV4DlbdpFZiuIZOqXsXvTV30VG/EkHPgG/7rLtoqlwpb en2u1WfTBKVh6UcmJp36UXOU+W/1Gzo= X-Google-Smtp-Source: APXvYqxiGEvucSh1hTDMjM3CayL9zZo13uNJy+RuMX2afmj2vXCUnP6ZLHT7mMjg4z924Q7em9GHZQ== X-Received: by 2002:adf:ea86:: with SMTP id s6mr1324275wrm.44.1556026556617; Tue, 23 Apr 2019 06:35:56 -0700 (PDT) Received: from [172.20.1.40] ([217.38.71.146]) by smtp.googlemail.com with ESMTPSA id x18sm16772072wrw.14.2019.04.23.06.35.55 (version=TLS1_3 cipher=AEAD-AES128-GCM-SHA256 bits=128/128); Tue, 23 Apr 2019 06:35:56 -0700 (PDT) To: Kevin Traynor Cc: Alejandro Lucero , dpdk stable References: <20190416143719.21601-1-ktraynor@redhat.com> <20190416143719.21601-7-ktraynor@redhat.com> <6188fc4d-eccb-d461-8df1-1f60a6e68849@netronome.com> <6cb0fd49-52f3-9069-d8e2-6ec2357bafb8@redhat.com> From: =?UTF-8?Q?Pablo_Casc=c3=b3n?= Message-ID: <6b508e91-2113-c2f5-b855-0320f6b6103e@netronome.com> Date: Tue, 23 Apr 2019 14:35:54 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.6.1 MIME-Version: 1.0 In-Reply-To: <6cb0fd49-52f3-9069-d8e2-6ec2357bafb8@redhat.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Content-Language: en-GB Subject: Re: [dpdk-stable] patch 'net/nfp: fix setting MAC address' has been queued to LTS release 18.11.2 X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 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" Thank you very much for walking me through it, it's crystal clear now :)  Please push the changes, at least no objection from me. (Also got the extra tip from a colleague to use Thunderbird's extension Colored Diffs where the tiny extra space that differentiates a context from the actual change is a lot more visible. Wish I had good feedback to contribute, can only think of: a) educating those struggling (thx!) b) no diff context (any downside?) and c) also sharing the stable patch?) Pablo On 23/04/2019 14:03, Kevin Traynor wrote: > On 23/04/2019 11:24, Pablo Cascón wrote: >> Hi, >> > Hi Pablo, > >> thanks for the reply and explaining the process. >> >> Unfortunately the diff at the end of your commit doesn't match what I was expecting. Have tried to generate it here and could only get changes related to the rebase (metada diffs) but no code changes. And your diff seems to imply code changes, not 100% sure. Could you double check the stable patch and diff please. Happy to send a patch of what I would like to land into stable >> > I checked and the change is the same in stable as master, so it looks ok. > > I think I know what's causing the confusion. The diff is a diff of the > *patches*, not the code. Both patches have +/- LOC in them and one is > being brought into context by a metadata change between the patches. > There is a difference in the indent of the +/-'s of the patches > themselves vs the diff between the patches but it's hard to distinguish. > See below: > > -@@ -576,5 +576,8 @@ nfp_set_mac_addr(struct rte_eth_dev *dev, struct > ether_addr *mac_addr) > +@@ -567,5 +567,8 @@ nfp_set_mac_addr(struct rte_eth_dev *dev, struct > > ^ '+' and '-' is a metadata diff between the patches, context below > > ether_addr *mac_addr) > /* Signal the NIC about the change */ > update = NFP_NET_CFG_UPDATE_MACADDR; > - ctrl = hw->ctrl | NFP_NET_CFG_CTRL_LIVE_ADDR; > > ^ '-' is part of both patches that removes this LOC > ^ no diff between the patches > > > Thanks for checking the patch and raising this. We can think about how > to make it clearer - perhaps the diff would be better without any context. > > Kevin. > >> For clarity this is what I tried, perhaps incorrectly, please let me know: >> 1) generate patch from dpdk upstream commit: >>     dpdk⟫ git format-patch -o /tmp/ -n -1 027412f >> 2) import it into stable >>     dpdk-stable⟫ git am /tmp/0001-net-nfp-fix-setting-MAC-address.patch >> 3) generate patch of just imported one: >>     dpdk-stable⟫ git format-patch -o /tmp/stable/ -n -1 HEAD >> 4) diff the checking there are only rebase related changes: >> >> dpdk-stable⟫ diff /tmp/0001-net-nfp-fix-setting-MAC-address.patch /tmp/stable/0001-net-nfp-fix-setting-MAC-address.patch >> 1c1 >> < From 027412fe949c763fd4d536b13dcb4432f2df5534 Mon Sep 17 00:00:00 2001 >> --- >>> From 3e5c4cd12ff01df228124d92905e4e7d563b652e Mon Sep 17 00:00:00 2001 >> 30c30 >> < index a791e95..1b7b6c2 100644 >> --- >>> index 54c6da9..278e154 100644 >> 33c33 >> < @@ -575,7 +575,10 @@ nfp_set_mac_addr(struct rte_eth_dev *dev, struct ether_addr *mac_addr) >> --- >>> @@ -566,7 +566,10 @@ nfp_set_mac_addr(struct rte_eth_dev *dev, struct ether_addr *mac_addr) >> Thanks, >> >> Pablo >> >> On 16/04/2019 15:36, Kevin Traynor wrote: >>> Hi, >>> >>> FYI, your patch has been queued to LTS release 18.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 04/24/19. 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. >>> >>> Thanks. >>> >>> Kevin Traynor >>> >>> --- >>> From 8fb7943ea804d437fc5b7153d7c073c79eeb3837 Mon Sep 17 00:00:00 2001 >>> From: =?UTF-8?q?Pablo=20Casc=C3=B3n?= >>> Date: Fri, 8 Mar 2019 15:40:47 +0000 >>> Subject: [PATCH] net/nfp: fix setting MAC address >>> MIME-Version: 1.0 >>> Content-Type: text/plain; charset=UTF-8 >>> Content-Transfer-Encoding: 8bit >>> >>> [ upstream commit 027412fe949c763fd4d536b13dcb4432f2df5534 ] >>> >>> Some firmwares, mostly for VFs, do not advertise the feature / >>> capability of changing the MAC address while the interface is up. With >>> such firmware a request to change the MAC address that at the same >>> time also tries to enable the not available feature will be denied by >>> the firmware resulting in an error message like: >>> >>> nfp_net_reconfig(): Error nfp_net reconfig for ctrl: 80000000 update: 800 >>> >>> Fix set_mac_addr by not trying to enable a feature if it is not >>> advertised by the firmware. >>> >>> Fixes: 2fe669f4bcd2 ("net/nfp: support MAC address change") >>> >>> Signed-off-by: Pablo Cascón >>> Acked-by: Alejandro Lucero >>> --- >>> drivers/net/nfp/nfp_net.c | 5 ++++- >>> 1 file changed, 4 insertions(+), 1 deletion(-) >>> >>> diff --git a/drivers/net/nfp/nfp_net.c b/drivers/net/nfp/nfp_net.c >>> index 54c6da924..278e154cd 100644 >>> --- a/drivers/net/nfp/nfp_net.c >>> +++ b/drivers/net/nfp/nfp_net.c >>> @@ -567,5 +567,8 @@ nfp_set_mac_addr(struct rte_eth_dev *dev, struct ether_addr *mac_addr) >>> /* Signal the NIC about the change */ >>> update = NFP_NET_CFG_UPDATE_MACADDR; >>> - ctrl = hw->ctrl | NFP_NET_CFG_CTRL_LIVE_ADDR; >>> + ctrl = hw->ctrl; >>> + if ((hw->ctrl & NFP_NET_CFG_CTRL_ENABLE) && >>> + (hw->cap & NFP_NET_CFG_CTRL_LIVE_ADDR)) >>> + ctrl |= NFP_NET_CFG_CTRL_LIVE_ADDR; >>> if (nfp_net_reconfig(hw, ctrl, update) < 0) { >>> PMD_INIT_LOG(INFO, "MAC address update failed");