Re: [et-mgmt-tools] [PATCH] virtinst: fix canonical OUI for QEMU/KVM MAC addresses

[Cole Robinson <crobinso redhat com>]

Kaspar Brand wrote:
> When support for QEMU/KVM MAC addresses was added to virtinst/util.py
> last August [1], the first two bytes were inadvertently swapped, AFAICT.
> 
> Instead of 54-52-00, the "canonical" prefix for QEMU/KVM addresses
> should be 52-54-00, as can be determined from
> 
>   http://git.savannah.gnu.org/cgit/qemu.git/tree/net.c#n2124
> 
> The attached patch changes all occurrences I found in the virtinst tree
> (hopefully I caught them all).
> 
> Kaspar
> 
> [1] https://www.redhat.com/archives/et-mgmt-tools/2008-August/msg00079.html
> 
> 

Bad timing! This was just fixed upstream last week:

http://hg.et.redhat.com/cgi-bin/hg-virt.cgi/applications/virtinst--devel/rev/a3334a88986f

However there were some occurences that the above missed, so I took
those from your patch and applied:

http://hg.et.redhat.com/cgi-bin/hg-virt.cgi/applications/virtinst--devel/rev/9f11c8f159ff

Thanks,
Cole