Re: [libvirt] [PATCH 0/5] macvtap support for Qemu/KVM VMs via libvirt
Daniel Veillard <veillard(a)redhat.com> wrote on 02/15/2010 12:24:01 PM:
On Thu, Feb 11, 2010 at 09:48:07AM -0500, Stefan Berger wrote:
> Hello!
>
> This is a re-post of previously posted patches following Daniel
> Berrange's request for changes along with other fixes PLUS a rebase
> to the latest code where the conn parameter is missing along with
> changes requested by Daniel Veillard.
Okay, I have applied and pushed the 5 patch set, but I had to do a bit
of surgery. First the detection code in configure was always setting
WITH_MACVTAP , so of course thsi didn't compile on my system. I also had
to reorganize the patch order to have each patch compiling. And I also
had to fix various things popping up when running "make syntax-check"
after each patch got applied.
Thanks a lot for doing this. I posted the patches in the sequence I have
them applied
via quilt, assuming that they are all being applied.
I ran make syntax-check on the version with the patches applied and no
error was shown on
anything that the patches touch. I see reports related to storage_conf.c,
though. I am running FC12.
Could this be related to a difference in installation?
I'll follow up with a patch regarding the 'vm shutdown' issue and libnl
for communication with the
netlink sockets, but likely not use libnl for building the actual netlink
message.
Stefan
But basically that's in and with the new version of [3/5] !
thanks !
Daniel
--
Daniel Veillard | libxml Gnome XML XSLT toolkit
http://xmlsoft.org/
daniel(a)veillard.com | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library http://libvirt.org/
Attachments:
- attachment.html (text/html — 2.7 KB)