Re: [PATCH 3/3] net/netvsc: not finding VF should not cause failure

2 messages, 1 author, 2018-12-17 · open the first message on its own page

Re: [PATCH 3/3] net/netvsc: not finding VF should not cause failure

From: Ferruh Yigit <hidden>
Date: 2018-12-17 22:11:10

On 12/17/2018 7:59 PM, Stephen Hemminger wrote:
This can go to 18.11 stable
I will add fixes and stable tags while merging, please provide them with commit
to ensure the backport of the patches to the stable trees.
-----Original Message-----
From: Ferruh Yigit <redacted> 
Sent: Monday, December 17, 2018 2:49 AM
To: Stephen Hemminger <stephen@networkplumber.org>; dev@dpdk.org
Cc: Stephen Hemminger <redacted>; Kevin Traynor <redacted>
Subject: Re: [dpdk-dev] [PATCH 3/3] net/netvsc: not finding VF should not cause failure

On 12/14/2018 1:26 AM, Stephen Hemminger wrote:
quoted
It is possible that the VF device exists but DPDK doesn't know
about it. This could happen if device was blacklisted or more
likely the necessary device (Mellanox) was not part of the DPDK
configuration.

In either case, the right thing to do is just keep working
but only with the slower para-virtual device.
Same question for this one, is this something that should be backported?
Is it intentionally left out from backporting?

Just a reminder, for backport, a patch needs a few markers,
- fix patch with fixes line
- Cc: stable@dpdk.org line
quoted
Signed-off-by: Stephen Hemminger <redacted>
---
 drivers/net/netvsc/hn_ethdev.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/netvsc/hn_ethdev.c b/drivers/net/netvsc/hn_ethdev.c
index de872212d3f3..1f7a7e66a51b 100644
--- a/drivers/net/netvsc/hn_ethdev.c
+++ b/drivers/net/netvsc/hn_ethdev.c
@@ -788,7 +788,7 @@ eth_hn_dev_init(struct rte_eth_dev *eth_dev)
 
 		err = hn_vf_add(eth_dev, hv);
 		if (err)
-			goto failed;
+			hv->vf_present = 0;
 	}
 
 	return 0;

Re: [PATCH 3/3] net/netvsc: not finding VF should not cause failure

From: Ferruh Yigit <hidden>
Date: 2018-12-17 22:12:24

On 12/17/2018 10:11 PM, Ferruh Yigit wrote:
On 12/17/2018 7:59 PM, Stephen Hemminger wrote:
quoted
This can go to 18.11 stable
I will add fixes and stable tags while merging, please provide them with commit
to ensure the backport of the patches to the stable trees.
Added:

    Fixes: dc7680e8597c ("net/netvsc: support integrated VF")
    Cc: stable@dpdk.org
quoted
-----Original Message-----
From: Ferruh Yigit <redacted> 
Sent: Monday, December 17, 2018 2:49 AM
To: Stephen Hemminger <stephen@networkplumber.org>; dev@dpdk.org
Cc: Stephen Hemminger <redacted>; Kevin Traynor <redacted>
Subject: Re: [dpdk-dev] [PATCH 3/3] net/netvsc: not finding VF should not cause failure

On 12/14/2018 1:26 AM, Stephen Hemminger wrote:
quoted
It is possible that the VF device exists but DPDK doesn't know
about it. This could happen if device was blacklisted or more
likely the necessary device (Mellanox) was not part of the DPDK
configuration.

In either case, the right thing to do is just keep working
but only with the slower para-virtual device.
Same question for this one, is this something that should be backported?
Is it intentionally left out from backporting?

Just a reminder, for backport, a patch needs a few markers,
- fix patch with fixes line
- Cc: stable@dpdk.org line
quoted
Signed-off-by: Stephen Hemminger <redacted>
---
 drivers/net/netvsc/hn_ethdev.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/netvsc/hn_ethdev.c b/drivers/net/netvsc/hn_ethdev.c
index de872212d3f3..1f7a7e66a51b 100644
--- a/drivers/net/netvsc/hn_ethdev.c
+++ b/drivers/net/netvsc/hn_ethdev.c
@@ -788,7 +788,7 @@ eth_hn_dev_init(struct rte_eth_dev *eth_dev)
 
 		err = hn_vf_add(eth_dev, hv);
 		if (err)
-			goto failed;
+			hv->vf_present = 0;
 	}
 
 	return 0;
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help