From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 87EA232A5 for ; Tue, 20 Nov 2018 20:14:31 +0100 (CET) Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id E7C94309174F; Tue, 20 Nov 2018 19:14:30 +0000 (UTC) Received: from ktraynor.remote.csb (unknown [10.36.118.7]) by smtp.corp.redhat.com (Postfix) with ESMTP id 1566060141; Tue, 20 Nov 2018 19:14:29 +0000 (UTC) From: Kevin Traynor To: Stephen Hemminger Cc: dpdk stable Date: Tue, 20 Nov 2018 19:11:58 +0000 Message-Id: <20181120191252.30277-8-ktraynor@redhat.com> In-Reply-To: <20181120191252.30277-1-ktraynor@redhat.com> References: <20181120191252.30277-1-ktraynor@redhat.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Scanned-By: MIMEDefang 2.79 on 10.5.11.11 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.41]); Tue, 20 Nov 2018 19:14:30 +0000 (UTC) Subject: [dpdk-stable] patch 'net/netvsc: fix chimney buffer size error handling' has been queued to stable release 18.08.1 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: Tue, 20 Nov 2018 19:14:31 -0000 Hi, FYI, your patch has been queued to stable release 18.08.1 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 11/23/18. 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. If the code is different (ie: not only metadata diffs), due for example to a change in context or macro names, please double check it. Thanks. Kevin Traynor --- >>From 66b526bffe5b09d7991e6ac5449b1e1336197489 Mon Sep 17 00:00:00 2001 From: Stephen Hemminger Date: Thu, 9 Aug 2018 10:50:05 -0700 Subject: [PATCH] net/netvsc: fix chimney buffer size error handling [ upstream commit 99e3d0e72f0bf34a39bf57d4b363730994eb5a59 ] Fix the error handling in setting up transmit buffer. If setting up chimney buffer fails, then it is not connected so no need to send disconnect. Allow for some unused area if full area is not used. Fixes: 4e9c73e96e83 ("net/netvsc: add Hyper-V network device") Signed-off-by: Stephen Hemminger --- drivers/net/netvsc/hn_nvs.c | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/drivers/net/netvsc/hn_nvs.c b/drivers/net/netvsc/hn_nvs.c index 77d3b839f..a458bb720 100644 --- a/drivers/net/netvsc/hn_nvs.c +++ b/drivers/net/netvsc/hn_nvs.c @@ -280,5 +280,5 @@ hn_nvs_conn_chim(struct hn_data *hv) if (error) { PMD_DRV_LOG(ERR, "exec nvs chim conn failed"); - goto cleanup; + return error; } @@ -286,6 +286,5 @@ hn_nvs_conn_chim(struct hn_data *hv) PMD_DRV_LOG(ERR, "nvs chim conn failed: %x", resp.status); - error = -EIO; - goto cleanup; + return -EIO; } @@ -296,5 +295,6 @@ hn_nvs_conn_chim(struct hn_data *hv) "invalid chimney sending buffer section size: %u", sectsz); - return 0; + error = -EINVAL; + goto cleanup; } @@ -305,9 +305,4 @@ hn_nvs_conn_chim(struct hn_data *hv) len, hv->chim_szmax, hv->chim_cnt); - if (len % hv->chim_szmax != 0) { - PMD_DRV_LOG(NOTICE, - "chimney sending sections are not properly aligned"); - } - /* Done! */ return 0; -- 2.19.0 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2018-11-20 17:53:07.665777909 +0000 +++ 0008-net-netvsc-fix-chimney-buffer-size-error-handling.patch 2018-11-20 17:53:07.000000000 +0000 @@ -1,8 +1,10 @@ -From 99e3d0e72f0bf34a39bf57d4b363730994eb5a59 Mon Sep 17 00:00:00 2001 +From 66b526bffe5b09d7991e6ac5449b1e1336197489 Mon Sep 17 00:00:00 2001 From: Stephen Hemminger Date: Thu, 9 Aug 2018 10:50:05 -0700 Subject: [PATCH] net/netvsc: fix chimney buffer size error handling +[ upstream commit 99e3d0e72f0bf34a39bf57d4b363730994eb5a59 ] + Fix the error handling in setting up transmit buffer. If setting up chimney buffer fails, then it is not connected so no need to send disconnect. @@ -10,7 +12,6 @@ Allow for some unused area if full area is not used. Fixes: 4e9c73e96e83 ("net/netvsc: add Hyper-V network device") -Cc: stable@dpdk.org Signed-off-by: Stephen Hemminger ---