From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f66.google.com (mail-wm0-f66.google.com [74.125.82.66]) by dpdk.org (Postfix) with ESMTP id D8E852BF7 for ; Wed, 23 May 2018 16:34:22 +0200 (CEST) Received: by mail-wm0-f66.google.com with SMTP id m129-v6so9803126wmb.3 for ; Wed, 23 May 2018 07:34:22 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=gv1WJBwkfznolHBml3K4HMiGCIO8MFrfxJ592PzqUFk=; b=EZcVbxdqYK1WCHkk7f0hmw5Z1WjqCJ7ThdreVw/weIdAU9sywjwweJpOsS8i6Vj1WU teCjyQmJbIdT/JHQRetcFZpBUR9JP2GkmVp0z+4KwePKzFDfQ0y3Ksf+gBELljY9n5DO TP/Rll/8LIfyXYaB6hjxuQxtD1dyRT6scmSQfu5i6Z9KP8uDRZesphDPxXUarCrHmH5z 2NMfS78+rOK5aO3XF6PqShFSV6eVHvE/QSMXg2VtryHGljRctmYELnQFWRydBBPRigvY pPuPJtUZH36Or/JKJ/i5wyR7+abO25SKN1ARu35/XXh/PB5xOR5kQ0JoUdfihcFcSCQ/ SJRQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=gv1WJBwkfznolHBml3K4HMiGCIO8MFrfxJ592PzqUFk=; b=K0ziiPHAS+u78JhwBXuPkQTExol42NJSUnf9XCbbDkYCdPZKOTqBCS6CbFZ4T6LguE Ob29VCFCW9xPIhCaaKNdCvYhwSukm06J4mBhm0Exk65PwM3lmcF+vuKCmjTvKtvh5Y9E G3cwuO3OAXuqR17MUBcSPSu+ZfBAmt+SEtylrpzsL1PwoZgSNyLvYUxUF+afFHltQtjG eIPBo1hQnIXN2AQv5T8poQHrV98+hxERbNZvsylnDcU7kZ05Y8loXXQFpKTWcDR1gkxX kSQmXaOdT9nfB68EHSYXzQQU4KS6g7sj7nM79m/inKqKBxNltaErvRfSivEGqo+ViOAm 1y4g== X-Gm-Message-State: ALKqPwfqFzxPH13tTyH9LRm18MBymQRSttJHXQ2/InwNFqMVz4orwUz0 Bf2fUUga9C5fPHWlM5el3HE= X-Google-Smtp-Source: AB8JxZpi8176OHOX6xN61UXlocrMnKfPPQcTLidpcweYB6bJDSbyi1ANdh0wMIp+YUvK0XWVrVZ7xQ== X-Received: by 2002:a1c:101:: with SMTP id 1-v6mr4501592wmb.28.1527086062551; Wed, 23 May 2018 07:34:22 -0700 (PDT) Received: from localhost ([2a00:23c5:be9a:5200:ce4c:82c0:d567:ecbb]) by smtp.gmail.com with ESMTPSA id r10-v6sm8622397wrn.6.2018.05.23.07.34.21 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Wed, 23 May 2018 07:34:21 -0700 (PDT) From: luca.boccassi@gmail.com To: Shahed Shaikh Cc: dpdk stable Date: Wed, 23 May 2018 15:33:41 +0100 Message-Id: <20180523143344.16885-20-luca.boccassi@gmail.com> X-Mailer: git-send-email 2.14.2 In-Reply-To: <20180523143344.16885-1-luca.boccassi@gmail.com> References: <20180516100945.4449-2-luca.boccassi@gmail.com> <20180523143344.16885-1-luca.boccassi@gmail.com> Subject: [dpdk-stable] patch 'net/qede: fix unicast filter routine return code' has been queued to LTS release 16.11.7 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: , X-List-Received-Date: Wed, 23 May 2018 14:34:23 -0000 Hi, FYI, your patch has been queued to LTS release 16.11.7 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/25/18. So please shout if anyone has objections. Thanks. Luca Boccassi --- >>From 20a4c778f2191a79886accf32f5ec373277bae87 Mon Sep 17 00:00:00 2001 From: Shahed Shaikh Date: Sat, 19 May 2018 17:15:46 -0700 Subject: [PATCH] net/qede: fix unicast filter routine return code [ upstream commit f8d2581ecbba056db08888d562eafe5181a009ba ] There is no need to return failure (-EEXIST) when the requested filter is already configured. Fixes: d6cb17535f88 ("net/qede: fix VLAN filters") Fixes: 77fac1b54fc9 ("net/qede: fix filtering code") Signed-off-by: Shahed Shaikh --- drivers/net/qede/qede_ethdev.c | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/net/qede/qede_ethdev.c b/drivers/net/qede/qede_ethdev.c index 5275ef9d2..ce3a0936d 100644 --- a/drivers/net/qede/qede_ethdev.c +++ b/drivers/net/qede/qede_ethdev.c @@ -248,10 +248,10 @@ qede_ucast_filter(struct rte_eth_dev *eth_dev, struct ecore_filter_ucast *ucast, if ((memcmp(mac_addr, &tmp->mac, ETHER_ADDR_LEN) == 0) && ucast->vlan == tmp->vlan) { - DP_ERR(edev, "Unicast MAC is already added" - " with vlan = %u, vni = %u\n", - ucast->vlan, ucast->vni); - return -EEXIST; + DP_INFO(edev, "Unicast MAC is already added" + " with vlan = %u, vni = %u\n", + ucast->vlan, ucast->vni); + return 0; } } u = rte_malloc(NULL, sizeof(struct qede_ucast_entry), @@ -613,9 +613,9 @@ static int qede_vlan_filter_set(struct rte_eth_dev *eth_dev, SLIST_FOREACH(tmp, &qdev->vlan_list_head, list) { if (tmp->vid == vlan_id) { - DP_ERR(edev, "VLAN %u already configured\n", - vlan_id); - return -EEXIST; + DP_INFO(edev, "VLAN %u already configured\n", + vlan_id); + return 0; } } -- 2.14.2