On Mon, Aug 24, 2009 at 02:13:07PM +0100, Daniel P. Berrange wrote:
On Sat, Aug 22, 2009 at 10:04:30PM +0200, Matthias Bolte wrote:
Hi,
The commit "Generic shared impls of all NUMA apis" (b0b968efd56f6c66bfa23eebbecd491ea993f99b) changed the UML driver struct to use the shared NUMA API. But now the UML driver struct refers to the public API functions:
virNodeGetCellsFreeMemory virNodeGetFreeMemory
instead of the shared NUMA API functions
nodeGetCellsFreeMemory nodeGetFreeMemory
This results in an infinite recursion, if someone's going to call virNodeGetCellsFreeMemory with an UML connection.
Opps, that's a bit of a nasty bug. Clearly need to add these APis to the libvirt-TCK tests
Apparently the bug seems fixed in git: static virDriver umlDriver = { ... NULL, /* domainMemoryPeek */ nodeGetCellsFreeMemory, /* nodeGetCellsFreeMemory */ nodeGetFreeMemory, /* getFreeMemory */ NULL, /* domainEventRegister */ so issue seems solved now. Daniel -- Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ daniel@veillard.com | Rpmfind RPM search engine http://rpmfind.net/ http://veillard.com/ | virtualization library http://libvirt.org/