Re: [libvirt] [PATCHv2 00/13] IPv6 support for virtual networks using bridge driver
On 12/29/2010 10:05 AM, Stefan Berger wrote:
On 12/23/2010 04:16 PM, Laine Stump wrote:
> I squashed in all of Eric's recommendations and pushed this entire
> series.
The networkApplyTest.sh of the libvirt tck test suite does not pass
anymore. I assume it's due to the changes in this part of the code. I
see the following change on the broadcast address of the interface:
sh ./networkApplyTest.sh --wait --verbose --force
FAIL networkxml2xmlin/tck-testnet-2.xml : ifconfig tck-testbr | grep
':10\.1\.2\.'
1c1
< inet addr:10.1.2.1 Bcast:0.0.0.0 Mask:255.255.255.0
---
> inet addr:10.1.2.1 Bcast:10.1.2.255 Mask:255.255.255.0
The Bcast address is now set to 0.0.0.0 and is expected to be 10.1.2.255.
Thanks for catching this, Stefan! Apparently "ip addr add" doesn't set
the broadcast address unless it's specified, where the old method of
using ioctls must have set it implicitly when the netmask was set.
I'll post a patch for this later today.