On Tue, Dec 23, 2008 at 04:04:21PM +0100, Markus Armbruster wrote: > This patch series attempts to make libvirt just work on RHEL-5. Right > now it doesn't, mostly because libvirt relies on version number checks > in a couple of places, and RHEL-5's version numbers aren't the whole > truth due to various backports of later stuff. Based on the earlier discussion, I think we should apply those patches: > The first two patches are plain bug fixes: > > [PATCH 1/7] Fix network device inconsistency between xm and sxpr > [PATCH 2/7] Fix graphics configuration inconsistency between xm and sxpr because those are just plain bug fixes > The next three add a few special cases for RHEL-5: > > [PATCH 3/7] New configure option --with-rhel5-api > [PATCH 4/7] Fix HVM network device configuration for RHEL-5 > [PATCH 5/7] Fix PVFB device configuration for RHEL-5 > [PATCH 6/7] Enable NUMA support for RHEL-5 because they are really no-ops if one didn't configured --with-rhel5-api (assuming the slight change I pointed out in 4/7) > The last one drops a counter-productive error check: > > [PATCH 7/7] Don't treat missing topology information as error That's really a cleanup IMHO, > See the patches for more detailed descriptions. > > I'm not proposing this for immediate commit, as I'm still testing. > But I'd appreciate review: is this the right way to do it? That looks fine to me, the only point is whether the configure option should be added, I guess it benefits more than just the RHEL users but also CentOS users so those should be commited. I plan to do this today but I just want to do a bit of testing on it first and check my 4/7 suggested change. Daniel -- Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ daniel@xxxxxxxxxxxx | Rpmfind RPM search engine http://rpmfind.net/ http://veillard.com/ | virtualization library http://libvirt.org/ -- Libvir-list mailing list Libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list