Re: [libvirt] [PATCH] add the check of memory size in xenXMDriver forsetmem of Xen domain

On Fri, Oct 31, 2008 at 05:34:24PM +0900, Atsushi SAKAI wrote:

Hi, Daniel,

This patch itself is OK.

Yes except for the extra ; after the if making it return -1 all the time, but once fixed sure :-) Patch applied and commited [...]

...

+ if (memory < 1024 * MIN_XEN_GUEST_SIZE);

if (memory < 1024 * MIN_XEN_GUEST_SIZE)

...

+ return (-1);

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/