2016-04-26 18:42 GMT+03:00 Laine Stump <laine@laine.org>:
The simplest fix would be to just swap peer and address in the set-address call for QEMU, but the fact that the problem didn't leap out at anyone indicates that the current name "peer" may not be the best choice (even though that's what it really is in the netlink message that sets the addresses; of course that *won't* be the case after it's fixed - "address" will be set as "peer", and "peer" as "address"). So, as I suggested in my earlier message, maybe we can come up with a better name for what is now called "peer" ("address" must stay the same, since it has already been in several releases). Does "hostAddress" sound okay?
Either of these patches (the one to fix the missing attribute in the XML or the one to rename the attribute) could be pushed even after DV begins the freeze period for the release - they qualify as bug fixes.
(I actually already have a patch to fix the missing peer attribute in the XML, and am testing it, so the big question is whether to change the name from "peer" to something more accurate, and if so, what to change it to).
Thanks! May be combine Host and Peer to something like HostPeer ? =) -- Vasiliy Tolstov, e-mail: v.tolstov@selfip.ru