On Mar 23, 2012, at 7:21 PM, Laine Stump wrote:
Related to this patch - while looking at the uses of
virNetDevVPortProfileAssociate(), I found the function
qemuMigrationVPAssociatePortProfiles() in src/qemu/qemu_migration.c.
It
calls virNetDevVPortProfileAssociate() without ever calling
virNetDevMacVLanVPortProfileRegisterCallback(). I'm wondering if
either
1) the normal setup of port profiles (is
virNetDevMacVLanCreateWithVPortProfile) is missed during migration,
and
this function needs to be changed/enhanced, or 2) this associate is
redundant. I don't have the hardware to test either a migration *or*
vepa mode, but this seems like something important to determine (and
fix
if it's broken).
I'm currently looking at migration, and I will follow this up ASAP,
because
of the complication of having to go through pre-association there are
2 calls
to virNetDevVPortProfileAssociate() and only when migration has
completed
successfully the callback should be registered.
The combined patches check out ok, but I haven't covered all tests yet.
DirkH