On Sun, Aug 5, 2012 at 12:16 AM, Laine Stump <laine@xxxxxxxxx> 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 -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list