[PATCH] libxl: Normalize MAC address in device conf when hotplugging a netdev

Jim Fehlig jfehlig at suse.com
Wed Jun 3 14:33:45 UTC 2020


On 6/2/20 9:58 PM, Laine Stump wrote:
> On 5/28/20 1:09 PM, Jim Fehlig wrote:
>> Similar to commit 6c17606b7cc, normalize the MAC addresses in persistent
>> and live device config to avoid a different MAC address for the device
>> once the VM is rebooted and persistent config takes affect.
> 
> 
> Well...
> 
> 
> This has a bigger change than commit 6c17606b7cc! :-)
> 
> 
> For those who don't feel like digging back through the git blame history, commit 
> 6c17606b7cc just added a call to qemuDomainAttachDeviceLiveAndConfigHomogenize() 
> in the QEMU driver version of AttachDeviceFlags() to make the MAC addresses of 
> the *already-existing* "live" and "config" copies of the device object match 
> each other - qemu's function had previously been changed back in commit 55ce6564 
> to create separate copies of the device object for config and live (although it 
> still had to save an extra pointer to the config copy, which was being consumed 
> by the the qemuDomainAttachDeviceConfig(), with the pointer NULLed
> 
> 
> The libxl driver hadn't gotten that change though, so up until now it's only had 
> a single device object pointer, and although it separately parsed the XML for 
> live and config, this was done sequentially into the same object, so the two 
> never existed at the same time. Since both objects must exist at the same time 
> to copy anything from one to the other (without "extra steps"), this new patch 
> is effectively the libxl equivalent of commit 55ce6564 and commit 6c17606b7cc 
> put together.
> 
> That was a long prelude to "maybe mention commit 55ce6564 in the commit log as 
> well" :-)

Agreed, I'll add it. Thanks for the details.

>> Signed-off-by: Jim Fehlig <jfehlig at suse.com>
>> ---
>>   src/libxl/libxl_driver.c | 56 +++++++++++++++++++++++++++++++++-------
>>   1 file changed, 46 insertions(+), 10 deletions(-)
>>
>> diff --git a/src/libxl/libxl_driver.c b/src/libxl/libxl_driver.c
>> index 63ec0a2188..a80bc3fe3a 100644
>> --- a/src/libxl/libxl_driver.c
>> +++ b/src/libxl/libxl_driver.c
>> @@ -4096,6 +4096,31 @@ libxlDomainUpdateDeviceConfig(virDomainDefPtr vmdef, 
>> virDomainDeviceDefPtr dev)
>>   }
>> +static void
>> +libxlDomainAttachDeviceNormalize(const virDomainDeviceDef *devConf,
> 
> 
> What? You didn't like my verb in the QEMU version? :-)

It made me think of milk :-).

> (I can't remember why I picked Homogenize instead of Normalize, but I must have 
> had some reason. Maybe I was just thinking back to the dairy farm next door to 
> my childhood home (true story!))

Haha, that explains it! I think I would have preferred a dairy farm over the 
steel mills and refineries of my childhood :-).

> At any rate, it all looks good
> 
> 
> Reviewed-by: Laine Stump <laine at redhat.com>

Thanks for the review!

Regards,
Jim





More information about the libvir-list mailing list