[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [dpdk-dev] [ovs-dev] [PATCH RFC] netdev-dpdk: Fix devic

From: Tan, Jianfeng
Subject: Re: [Qemu-devel] [dpdk-dev] [ovs-dev] [PATCH RFC] netdev-dpdk: Fix device obtain mac address when received first packet in vhost type
Date: Mon, 27 Nov 2017 23:34:10 +0800
User-agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0

On 11/27/2017 10:27 PM, Yuanhan Liu wrote:
On Fri, Nov 24, 2017 at 05:59:09PM +0800, Chen Hailin wrote:
Hi Aaron Conole && Jianfeng,

The stp could not work in ovs-dpdk vhostuser.
Because the attached vhost device doesn't have MAC address.

Now we have two ways to solve this problem.
1. The vhost learns MAC address from packet like as my first patch.
I do agree with Aaron this is not the right way.

I do think it should be the vswitch's responsibility to learn mac of vhost port.

Except that it's the only feasible way without modifying the spec (yuanhan already makes it very clear below), we can treat the vswitch as a phsical switch, VM as a physical server, virtio/vhost port as a back-to-back connected NICs, the only way of the physical switch to know the mac of the NIC on the other side is ARP learning.

Might I ask why you don't think it's a right way?


2. The virtio notifies MAC address actively to vhost user .
Unfortunately, AFAIK, there is no way to achieve that so far. we could
either let virtio/QEMU to expose the CQ to vhost or add a new VHOST_USER
message to carry the mac address. While vhost-user is a generic interface
adding a virtio-net specific message also doesn't seem quite right.
Exposing CQ is probably the best we can do.

Anyway, both need spec change.

In my opinions,  if we treat it as a device,  we should allocate
MAC address for the device when the VM started.

Which one do you think better?

Best Regards,
Chen Hailin
From: Aaron Conole
Date: 2017-11-18 10:00
To: Hailin Chen
CC: address@hidden; Maxime Coquelin; address@hidden
Subject: Re: [ovs-dev] [PATCH RFC] netdev-dpdk: Fix device obtain mac address 
when received first packet in vhost type
Hi Hailin,
Hailin Chen <address@hidden> writes:
The stp could not work on netdev-dpdk if network is loop.
Because the stp protocol negotiates designate port by sending
BPDU packets which contains MAC address.
However the device doesn't have MAC address in vhostuser type.
Thus, function send_bpdu_cb would not send BPDU packets.

This patch will set the MAC for device when received first packet.

Signed-off-by: Hailin Chen <address@hidden>
Thanks for the patch. In general, I don't think this is the right approach to deal with this
type of issue.  I believe the problem statement is that OvS bridge is
unaware of the guest MAC address - did I get it right?  In that case, I
would think that a better way to solve this would be to have virtio tell
the mac address of the guest.  I don't recall right now if that's
allowed in the virtio spec, but I do remember some kind of negotiation
I've CC'd Maxime, who is one of the maintainers of the virtio code from
DPDK side.  Perhaps there is an alternate way to solve this.
dev mailing list

reply via email to

[Prev in Thread] Current Thread [Next in Thread]