2023-01-27 20:46:20

by Laurent Vivier

[permalink] [raw]
Subject: [PATCH v3 0/2] virtio_net: vdpa: update MAC address when it is generated by virtio-net

When the MAC address is not provided by the vdpa device virtio_net
driver assigns a random one without notifying the device.
The consequence, in the case of mlx5_vdpa, is the internal routing
tables of the device are not updated and this can block the
communication between two namespaces.

To fix this problem, use virtnet_send_command(VIRTIO_NET_CTRL_MAC)
to set the address from virtnet_probe() when the MAC address is
not provided by the device.

v3:
- update comments
- fail probe if VIRTIO_NET_CTRL_MAC_ADDR_SET fails
- move the virtnet_send_command() upper, inside the RTNL lock,
this simplifies the cleanup in case of error, and a future patch
from Jason adds an ASSERT_RTNL() in virtnet_send_command()
("virtio-net: convert rx mode setting to use workqueue")
- add a patch to disable F_STANDBY if F_MAC is not set

v2:
- remove vdpa_sim related fixes
- check virtio_has_feature(vdev, VIRTIO_NET_F_MAC) rather than
addr_assign_type

Laurent Vivier (2):
virtio_net: disable VIRTIO_NET_F_STANDBY if VIRTIO_NET_F_MAC is not
set
virtio_net: notify MAC address change on device initialization

drivers/net/virtio_net.c | 26 ++++++++++++++++++++++++++
1 file changed, 26 insertions(+)

--
2.39.1



2023-01-27 20:46:23

by Laurent Vivier

[permalink] [raw]
Subject: [PATCH v3 1/2] virtio_net: disable VIRTIO_NET_F_STANDBY if VIRTIO_NET_F_MAC is not set

failover relies on the MAC address to pair the primary and the standby
devices:

"[...] the hypervisor needs to enable VIRTIO_NET_F_STANDBY
feature on the virtio-net interface and assign the same MAC address
to both virtio-net and VF interfaces."

Documentation/networking/net_failover.rst

This patch disables the STANDBY feature if the MAC address is not
provided by the hypervisor.

Signed-off-by: Laurent Vivier <[email protected]>
---
drivers/net/virtio_net.c | 6 ++++++
1 file changed, 6 insertions(+)

diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c
index 7723b2a49d8e..7d700f8e545a 100644
--- a/drivers/net/virtio_net.c
+++ b/drivers/net/virtio_net.c
@@ -3688,6 +3688,12 @@ static int virtnet_validate(struct virtio_device *vdev)
__virtio_clear_bit(vdev, VIRTIO_NET_F_MTU);
}

+ if (virtio_has_feature(vdev, VIRTIO_NET_F_STANDBY) &&
+ !virtio_has_feature(vdev, VIRTIO_NET_F_MAC)) {
+ dev_warn(&vdev->dev, "device advertises feature VIRTIO_NET_F_STANDBY but not VIRTIO_NET_F_MAC, disabling standby");
+ __virtio_clear_bit(vdev, VIRTIO_NET_F_STANDBY);
+ }
+
return 0;
}

--
2.39.1


2023-01-27 20:46:26

by Laurent Vivier

[permalink] [raw]
Subject: [PATCH v3 2/2] virtio_net: notify MAC address change on device initialization

In virtnet_probe(), if the device doesn't provide a MAC address the
driver assigns a random one.
As we modify the MAC address we need to notify the device to allow it
to update all the related information.

The problem can be seen with vDPA and mlx5_vdpa driver as it doesn't
assign a MAC address by default. The virtio_net device uses a random
MAC address (we can see it with "ip link"), but we can't ping a net
namespace from another one using the virtio-vdpa device because the
new MAC address has not been provided to the hardware:
RX packets are dropped since they don't go through the receive filters,
TX packets go through unaffected.

Signed-off-by: Laurent Vivier <[email protected]>
---
drivers/net/virtio_net.c | 20 ++++++++++++++++++++
1 file changed, 20 insertions(+)

diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c
index 7d700f8e545a..704a05f1c279 100644
--- a/drivers/net/virtio_net.c
+++ b/drivers/net/virtio_net.c
@@ -3806,6 +3806,8 @@ static int virtnet_probe(struct virtio_device *vdev)
eth_hw_addr_set(dev, addr);
} else {
eth_hw_addr_random(dev);
+ dev_info(&vdev->dev, "Assigned random MAC address %pM\n",
+ dev->dev_addr);
}

/* Set up our device-specific information */
@@ -3933,6 +3935,24 @@ static int virtnet_probe(struct virtio_device *vdev)

virtio_device_ready(vdev);

+ /* a random MAC address has been assigned, notify the device.
+ * We don't fail probe if VIRTIO_NET_F_CTRL_MAC_ADDR is not there
+ * because many devices work fine without getting MAC explicitly
+ */
+ if (!virtio_has_feature(vdev, VIRTIO_NET_F_MAC) &&
+ virtio_has_feature(vi->vdev, VIRTIO_NET_F_CTRL_MAC_ADDR)) {
+ struct scatterlist sg;
+
+ sg_init_one(&sg, dev->dev_addr, dev->addr_len);
+ if (!virtnet_send_command(vi, VIRTIO_NET_CTRL_MAC,
+ VIRTIO_NET_CTRL_MAC_ADDR_SET, &sg)) {
+ pr_debug("virtio_net: setting MAC address failed\n");
+ rtnl_unlock();
+ err = -EINVAL;
+ goto free_unregister_netdev;
+ }
+ }
+
rtnl_unlock();

err = virtnet_cpu_notif_add(vi);
--
2.39.1


2023-01-31 09:07:36

by Paolo Abeni

[permalink] [raw]
Subject: Re: [PATCH v3 2/2] virtio_net: notify MAC address change on device initialization

On Fri, 2023-01-27 at 21:45 +0100, Laurent Vivier wrote:
> In virtnet_probe(), if the device doesn't provide a MAC address the
> driver assigns a random one.
> As we modify the MAC address we need to notify the device to allow it
> to update all the related information.
>
> The problem can be seen with vDPA and mlx5_vdpa driver as it doesn't
> assign a MAC address by default. The virtio_net device uses a random
> MAC address (we can see it with "ip link"), but we can't ping a net
> namespace from another one using the virtio-vdpa device because the
> new MAC address has not been provided to the hardware:
> RX packets are dropped since they don't go through the receive filters,
> TX packets go through unaffected.
>
> Signed-off-by: Laurent Vivier <[email protected]>
> ---
> drivers/net/virtio_net.c | 20 ++++++++++++++++++++
> 1 file changed, 20 insertions(+)
>
> diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c
> index 7d700f8e545a..704a05f1c279 100644
> --- a/drivers/net/virtio_net.c
> +++ b/drivers/net/virtio_net.c
> @@ -3806,6 +3806,8 @@ static int virtnet_probe(struct virtio_device *vdev)
> eth_hw_addr_set(dev, addr);
> } else {
> eth_hw_addr_random(dev);
> + dev_info(&vdev->dev, "Assigned random MAC address %pM\n",
> + dev->dev_addr);
> }
>
> /* Set up our device-specific information */
> @@ -3933,6 +3935,24 @@ static int virtnet_probe(struct virtio_device *vdev)
>
> virtio_device_ready(vdev);
>
> + /* a random MAC address has been assigned, notify the device.
> + * We don't fail probe if VIRTIO_NET_F_CTRL_MAC_ADDR is not there
> + * because many devices work fine without getting MAC explicitly
> + */
> + if (!virtio_has_feature(vdev, VIRTIO_NET_F_MAC) &&
> + virtio_has_feature(vi->vdev, VIRTIO_NET_F_CTRL_MAC_ADDR)) {
> + struct scatterlist sg;
> +
> + sg_init_one(&sg, dev->dev_addr, dev->addr_len);
> + if (!virtnet_send_command(vi, VIRTIO_NET_CTRL_MAC,
> + VIRTIO_NET_CTRL_MAC_ADDR_SET, &sg)) {
> + pr_debug("virtio_net: setting MAC address failed\n");
> + rtnl_unlock();
> + err = -EINVAL;
> + goto free_unregister_netdev;

Since the above is still dealing with device initialization, would it
make sense moving such init step before registering the netdevice?

Cheers,

Paolo


2023-01-31 09:33:45

by Laurent Vivier

[permalink] [raw]
Subject: Re: [PATCH v3 2/2] virtio_net: notify MAC address change on device initialization

On 1/31/23 10:01, Paolo Abeni wrote:
> On Fri, 2023-01-27 at 21:45 +0100, Laurent Vivier wrote:
>> In virtnet_probe(), if the device doesn't provide a MAC address the
>> driver assigns a random one.
>> As we modify the MAC address we need to notify the device to allow it
>> to update all the related information.
>>
>> The problem can be seen with vDPA and mlx5_vdpa driver as it doesn't
>> assign a MAC address by default. The virtio_net device uses a random
>> MAC address (we can see it with "ip link"), but we can't ping a net
>> namespace from another one using the virtio-vdpa device because the
>> new MAC address has not been provided to the hardware:
>> RX packets are dropped since they don't go through the receive filters,
>> TX packets go through unaffected.
>>
>> Signed-off-by: Laurent Vivier <[email protected]>
>> ---
>> drivers/net/virtio_net.c | 20 ++++++++++++++++++++
>> 1 file changed, 20 insertions(+)
>>
>> diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c
>> index 7d700f8e545a..704a05f1c279 100644
>> --- a/drivers/net/virtio_net.c
>> +++ b/drivers/net/virtio_net.c
>> @@ -3806,6 +3806,8 @@ static int virtnet_probe(struct virtio_device *vdev)
>> eth_hw_addr_set(dev, addr);
>> } else {
>> eth_hw_addr_random(dev);
>> + dev_info(&vdev->dev, "Assigned random MAC address %pM\n",
>> + dev->dev_addr);
>> }
>>
>> /* Set up our device-specific information */
>> @@ -3933,6 +3935,24 @@ static int virtnet_probe(struct virtio_device *vdev)
>>
>> virtio_device_ready(vdev);
>>
>> + /* a random MAC address has been assigned, notify the device.
>> + * We don't fail probe if VIRTIO_NET_F_CTRL_MAC_ADDR is not there
>> + * because many devices work fine without getting MAC explicitly
>> + */
>> + if (!virtio_has_feature(vdev, VIRTIO_NET_F_MAC) &&
>> + virtio_has_feature(vi->vdev, VIRTIO_NET_F_CTRL_MAC_ADDR)) {
>> + struct scatterlist sg;
>> +
>> + sg_init_one(&sg, dev->dev_addr, dev->addr_len);
>> + if (!virtnet_send_command(vi, VIRTIO_NET_CTRL_MAC,
>> + VIRTIO_NET_CTRL_MAC_ADDR_SET, &sg)) {
>> + pr_debug("virtio_net: setting MAC address failed\n");
>> + rtnl_unlock();
>> + err = -EINVAL;
>> + goto free_unregister_netdev;
>
> Since the above is still dealing with device initialization, would it
> make sense moving such init step before registering the netdevice?

It depends if we can send the command using the control command queue or not.
I don't think we can use a vq before virtio_device_ready().

Thanks,
Laurent


2023-01-31 11:42:00

by Paolo Abeni

[permalink] [raw]
Subject: Re: [PATCH v3 2/2] virtio_net: notify MAC address change on device initialization

On Tue, 2023-01-31 at 10:32 +0100, Laurent Vivier wrote:
> On 1/31/23 10:01, Paolo Abeni wrote:
> > On Fri, 2023-01-27 at 21:45 +0100, Laurent Vivier wrote:
> > > In virtnet_probe(), if the device doesn't provide a MAC address the
> > > driver assigns a random one.
> > > As we modify the MAC address we need to notify the device to allow it
> > > to update all the related information.
> > >
> > > The problem can be seen with vDPA and mlx5_vdpa driver as it doesn't
> > > assign a MAC address by default. The virtio_net device uses a random
> > > MAC address (we can see it with "ip link"), but we can't ping a net
> > > namespace from another one using the virtio-vdpa device because the
> > > new MAC address has not been provided to the hardware:
> > > RX packets are dropped since they don't go through the receive filters,
> > > TX packets go through unaffected.
> > >
> > > Signed-off-by: Laurent Vivier <[email protected]>
> > > ---
> > > drivers/net/virtio_net.c | 20 ++++++++++++++++++++
> > > 1 file changed, 20 insertions(+)
> > >
> > > diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c
> > > index 7d700f8e545a..704a05f1c279 100644
> > > --- a/drivers/net/virtio_net.c
> > > +++ b/drivers/net/virtio_net.c
> > > @@ -3806,6 +3806,8 @@ static int virtnet_probe(struct virtio_device *vdev)
> > > eth_hw_addr_set(dev, addr);
> > > } else {
> > > eth_hw_addr_random(dev);
> > > + dev_info(&vdev->dev, "Assigned random MAC address %pM\n",
> > > + dev->dev_addr);
> > > }
> > >
> > > /* Set up our device-specific information */
> > > @@ -3933,6 +3935,24 @@ static int virtnet_probe(struct virtio_device *vdev)
> > >
> > > virtio_device_ready(vdev);
> > >
> > > + /* a random MAC address has been assigned, notify the device.
> > > + * We don't fail probe if VIRTIO_NET_F_CTRL_MAC_ADDR is not there
> > > + * because many devices work fine without getting MAC explicitly
> > > + */
> > > + if (!virtio_has_feature(vdev, VIRTIO_NET_F_MAC) &&
> > > + virtio_has_feature(vi->vdev, VIRTIO_NET_F_CTRL_MAC_ADDR)) {
> > > + struct scatterlist sg;
> > > +
> > > + sg_init_one(&sg, dev->dev_addr, dev->addr_len);
> > > + if (!virtnet_send_command(vi, VIRTIO_NET_CTRL_MAC,
> > > + VIRTIO_NET_CTRL_MAC_ADDR_SET, &sg)) {
> > > + pr_debug("virtio_net: setting MAC address failed\n");
> > > + rtnl_unlock();
> > > + err = -EINVAL;
> > > + goto free_unregister_netdev;
> >
> > Since the above is still dealing with device initialization, would it
> > make sense moving such init step before registering the netdevice?
>
> It depends if we can send the command using the control command queue or not.
> I don't think we can use a vq before virtio_device_ready().

Sounds reasonable. @Michael: do you have any additional comments?

Thanks!

Paolo


2023-01-31 13:44:22

by Michael S. Tsirkin

[permalink] [raw]
Subject: Re: [PATCH v3 2/2] virtio_net: notify MAC address change on device initialization

On Tue, Jan 31, 2023 at 10:01:53AM +0100, Paolo Abeni wrote:
> On Fri, 2023-01-27 at 21:45 +0100, Laurent Vivier wrote:
> > In virtnet_probe(), if the device doesn't provide a MAC address the
> > driver assigns a random one.
> > As we modify the MAC address we need to notify the device to allow it
> > to update all the related information.
> >
> > The problem can be seen with vDPA and mlx5_vdpa driver as it doesn't
> > assign a MAC address by default. The virtio_net device uses a random
> > MAC address (we can see it with "ip link"), but we can't ping a net
> > namespace from another one using the virtio-vdpa device because the
> > new MAC address has not been provided to the hardware:
> > RX packets are dropped since they don't go through the receive filters,
> > TX packets go through unaffected.
> >
> > Signed-off-by: Laurent Vivier <[email protected]>
> > ---
> > drivers/net/virtio_net.c | 20 ++++++++++++++++++++
> > 1 file changed, 20 insertions(+)
> >
> > diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c
> > index 7d700f8e545a..704a05f1c279 100644
> > --- a/drivers/net/virtio_net.c
> > +++ b/drivers/net/virtio_net.c
> > @@ -3806,6 +3806,8 @@ static int virtnet_probe(struct virtio_device *vdev)
> > eth_hw_addr_set(dev, addr);
> > } else {
> > eth_hw_addr_random(dev);
> > + dev_info(&vdev->dev, "Assigned random MAC address %pM\n",
> > + dev->dev_addr);
> > }
> >
> > /* Set up our device-specific information */
> > @@ -3933,6 +3935,24 @@ static int virtnet_probe(struct virtio_device *vdev)
> >
> > virtio_device_ready(vdev);
> >
> > + /* a random MAC address has been assigned, notify the device.
> > + * We don't fail probe if VIRTIO_NET_F_CTRL_MAC_ADDR is not there
> > + * because many devices work fine without getting MAC explicitly
> > + */
> > + if (!virtio_has_feature(vdev, VIRTIO_NET_F_MAC) &&
> > + virtio_has_feature(vi->vdev, VIRTIO_NET_F_CTRL_MAC_ADDR)) {
> > + struct scatterlist sg;
> > +
> > + sg_init_one(&sg, dev->dev_addr, dev->addr_len);
> > + if (!virtnet_send_command(vi, VIRTIO_NET_CTRL_MAC,
> > + VIRTIO_NET_CTRL_MAC_ADDR_SET, &sg)) {
> > + pr_debug("virtio_net: setting MAC address failed\n");
> > + rtnl_unlock();
> > + err = -EINVAL;
> > + goto free_unregister_netdev;
>
> Since the above is still dealing with device initialization, would it
> make sense moving such init step before registering the netdevice?
>
> Cheers,
>
> Paolo

We can't really, device has to be ready otherwise
we can't send commands to it.

--
MST


2023-01-31 14:41:54

by Paolo Abeni

[permalink] [raw]
Subject: Re: [PATCH v3 2/2] virtio_net: notify MAC address change on device initialization

On Tue, 2023-01-31 at 08:43 -0500, Michael S. Tsirkin wrote:
> On Tue, Jan 31, 2023 at 10:01:53AM +0100, Paolo Abeni wrote:
> > On Fri, 2023-01-27 at 21:45 +0100, Laurent Vivier wrote:
> > > In virtnet_probe(), if the device doesn't provide a MAC address the
> > > driver assigns a random one.
> > > As we modify the MAC address we need to notify the device to allow it
> > > to update all the related information.
> > >
> > > The problem can be seen with vDPA and mlx5_vdpa driver as it doesn't
> > > assign a MAC address by default. The virtio_net device uses a random
> > > MAC address (we can see it with "ip link"), but we can't ping a net
> > > namespace from another one using the virtio-vdpa device because the
> > > new MAC address has not been provided to the hardware:
> > > RX packets are dropped since they don't go through the receive filters,
> > > TX packets go through unaffected.
> > >
> > > Signed-off-by: Laurent Vivier <[email protected]>
> > > ---
> > > drivers/net/virtio_net.c | 20 ++++++++++++++++++++
> > > 1 file changed, 20 insertions(+)
> > >
> > > diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c
> > > index 7d700f8e545a..704a05f1c279 100644
> > > --- a/drivers/net/virtio_net.c
> > > +++ b/drivers/net/virtio_net.c
> > > @@ -3806,6 +3806,8 @@ static int virtnet_probe(struct virtio_device *vdev)
> > > eth_hw_addr_set(dev, addr);
> > > } else {
> > > eth_hw_addr_random(dev);
> > > + dev_info(&vdev->dev, "Assigned random MAC address %pM\n",
> > > + dev->dev_addr);
> > > }
> > >
> > > /* Set up our device-specific information */
> > > @@ -3933,6 +3935,24 @@ static int virtnet_probe(struct virtio_device *vdev)
> > >
> > > virtio_device_ready(vdev);
> > >
> > > + /* a random MAC address has been assigned, notify the device.
> > > + * We don't fail probe if VIRTIO_NET_F_CTRL_MAC_ADDR is not there
> > > + * because many devices work fine without getting MAC explicitly
> > > + */
> > > + if (!virtio_has_feature(vdev, VIRTIO_NET_F_MAC) &&
> > > + virtio_has_feature(vi->vdev, VIRTIO_NET_F_CTRL_MAC_ADDR)) {
> > > + struct scatterlist sg;
> > > +
> > > + sg_init_one(&sg, dev->dev_addr, dev->addr_len);
> > > + if (!virtnet_send_command(vi, VIRTIO_NET_CTRL_MAC,
> > > + VIRTIO_NET_CTRL_MAC_ADDR_SET, &sg)) {
> > > + pr_debug("virtio_net: setting MAC address failed\n");
> > > + rtnl_unlock();
> > > + err = -EINVAL;
> > > + goto free_unregister_netdev;
> >
> > Since the above is still dealing with device initialization, would it
> > make sense moving such init step before registering the netdevice?
> >
> > Cheers,
> >
> > Paolo
>
> We can't really, device has to be ready otherwise
> we can't send commands to it.

Thanks, I see. Also, Laurent already clarified the above.

Should I read your comment as you are ok with the patches in the
current form?

Cheers,

Paolo


2023-01-31 22:10:15

by Michael S. Tsirkin

[permalink] [raw]
Subject: Re: [PATCH v3 2/2] virtio_net: notify MAC address change on device initialization

On Fri, Jan 27, 2023 at 09:45:00PM +0100, Laurent Vivier wrote:
> In virtnet_probe(), if the device doesn't provide a MAC address the
> driver assigns a random one.
> As we modify the MAC address we need to notify the device to allow it
> to update all the related information.
>
> The problem can be seen with vDPA and mlx5_vdpa driver as it doesn't
> assign a MAC address by default. The virtio_net device uses a random
> MAC address (we can see it with "ip link"), but we can't ping a net
> namespace from another one using the virtio-vdpa device because the
> new MAC address has not been provided to the hardware:
> RX packets are dropped since they don't go through the receive filters,
> TX packets go through unaffected.
>
> Signed-off-by: Laurent Vivier <[email protected]>

Acked-by: Michael S. Tsirkin <[email protected]>

> ---
> drivers/net/virtio_net.c | 20 ++++++++++++++++++++
> 1 file changed, 20 insertions(+)
>
> diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c
> index 7d700f8e545a..704a05f1c279 100644
> --- a/drivers/net/virtio_net.c
> +++ b/drivers/net/virtio_net.c
> @@ -3806,6 +3806,8 @@ static int virtnet_probe(struct virtio_device *vdev)
> eth_hw_addr_set(dev, addr);
> } else {
> eth_hw_addr_random(dev);
> + dev_info(&vdev->dev, "Assigned random MAC address %pM\n",
> + dev->dev_addr);
> }
>
> /* Set up our device-specific information */
> @@ -3933,6 +3935,24 @@ static int virtnet_probe(struct virtio_device *vdev)
>
> virtio_device_ready(vdev);
>
> + /* a random MAC address has been assigned, notify the device.
> + * We don't fail probe if VIRTIO_NET_F_CTRL_MAC_ADDR is not there
> + * because many devices work fine without getting MAC explicitly
> + */
> + if (!virtio_has_feature(vdev, VIRTIO_NET_F_MAC) &&
> + virtio_has_feature(vi->vdev, VIRTIO_NET_F_CTRL_MAC_ADDR)) {
> + struct scatterlist sg;
> +
> + sg_init_one(&sg, dev->dev_addr, dev->addr_len);
> + if (!virtnet_send_command(vi, VIRTIO_NET_CTRL_MAC,
> + VIRTIO_NET_CTRL_MAC_ADDR_SET, &sg)) {
> + pr_debug("virtio_net: setting MAC address failed\n");
> + rtnl_unlock();
> + err = -EINVAL;
> + goto free_unregister_netdev;
> + }
> + }
> +
> rtnl_unlock();
>
> err = virtnet_cpu_notif_add(vi);
> --
> 2.39.1


2023-01-31 22:10:32

by Michael S. Tsirkin

[permalink] [raw]
Subject: Re: [PATCH v3 1/2] virtio_net: disable VIRTIO_NET_F_STANDBY if VIRTIO_NET_F_MAC is not set

On Fri, Jan 27, 2023 at 09:44:59PM +0100, Laurent Vivier wrote:
> failover relies on the MAC address to pair the primary and the standby
> devices:
>
> "[...] the hypervisor needs to enable VIRTIO_NET_F_STANDBY
> feature on the virtio-net interface and assign the same MAC address
> to both virtio-net and VF interfaces."
>
> Documentation/networking/net_failover.rst
>
> This patch disables the STANDBY feature if the MAC address is not
> provided by the hypervisor.
>
> Signed-off-by: Laurent Vivier <[email protected]>

Acked-by: Michael S. Tsirkin <[email protected]>

> ---
> drivers/net/virtio_net.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c
> index 7723b2a49d8e..7d700f8e545a 100644
> --- a/drivers/net/virtio_net.c
> +++ b/drivers/net/virtio_net.c
> @@ -3688,6 +3688,12 @@ static int virtnet_validate(struct virtio_device *vdev)
> __virtio_clear_bit(vdev, VIRTIO_NET_F_MTU);
> }
>
> + if (virtio_has_feature(vdev, VIRTIO_NET_F_STANDBY) &&
> + !virtio_has_feature(vdev, VIRTIO_NET_F_MAC)) {
> + dev_warn(&vdev->dev, "device advertises feature VIRTIO_NET_F_STANDBY but not VIRTIO_NET_F_MAC, disabling standby");
> + __virtio_clear_bit(vdev, VIRTIO_NET_F_STANDBY);
> + }
> +
> return 0;
> }
>
> --
> 2.39.1


2023-02-02 05:00:28

by patchwork-bot+netdevbpf

[permalink] [raw]
Subject: Re: [PATCH v3 0/2] virtio_net: vdpa: update MAC address when it is generated by virtio-net

Hello:

This series was applied to netdev/net-next.git (master)
by Jakub Kicinski <[email protected]>:

On Fri, 27 Jan 2023 21:44:58 +0100 you wrote:
> When the MAC address is not provided by the vdpa device virtio_net
> driver assigns a random one without notifying the device.
> The consequence, in the case of mlx5_vdpa, is the internal routing
> tables of the device are not updated and this can block the
> communication between two namespaces.
>
> To fix this problem, use virtnet_send_command(VIRTIO_NET_CTRL_MAC)
> to set the address from virtnet_probe() when the MAC address is
> not provided by the device.
>
> [...]

Here is the summary with links:
- [v3,1/2] virtio_net: disable VIRTIO_NET_F_STANDBY if VIRTIO_NET_F_MAC is not set
https://git.kernel.org/netdev/net-next/c/7c06458c102e
- [v3,2/2] virtio_net: notify MAC address change on device initialization
https://git.kernel.org/netdev/net-next/c/9f62d221a4b0

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html