On Sun, Aug 5, 2012 at 12:16 AM, Laine Stump <laine(a)laine.org> wrote:
This function was overlooked when openvswitch support was
added. Fortunately it's only use for update-device, which is
relatively new and seldom-used.
---
src/util/virnetdevvportprofile.c | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/src/util/virnetdevvportprofile.c b/src/util/virnetdevvportprofile.c
index 506240b..ac5aa24 100644
--- a/src/util/virnetdevvportprofile.c
+++ b/src/util/virnetdevvportprofile.c
@@ -107,6 +107,12 @@ virNetDevVPortProfileEqual(virNetDevVPortProfilePtr a,
virNetDevVPortProfilePtr
return false;
break;
+ case VIR_NETDEV_VPORT_PROFILE_OPENVSWITCH:
+ if (STRNEQ(a->profileID, b->profileID) ||
+ memcmp(a->interfaceID, b->interfaceID, VIR_UUID_BUFLEN) != 0)
+ return false;
+ break;
+
default:
break;
}
--
1.7.11.2
--
If I understand your commit message correctly, it appears this was
omitted in a previous patchset and needed to be part of the
switch/case. The change looks good so ACK.
--
Doug Goldstein