On Sat, Jun 11, 2016 at 09:35:40PM +0100, Richard W.M. Jones wrote:
On Sat, Jun 11, 2016 at 08:51:56PM +0300, Pavel Butsykin wrote:
> The appliance memory size of 500 MB is insufficient in certain scenarios. In
> partiuclar, semodule (which may be called by a guest tools uninstaller) often
> triggers OOM (see
https://bugzilla.redhat.com/426775)
>
> Increase it to 768 MB.
>
> Signed-off-by: Pavel Butsykin <pbutsykin(a)virtuozzo.com>
> ---
> src/guestfs-internal.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/guestfs-internal.h b/src/guestfs-internal.h
> index 03f1034..1d1b996 100644
> --- a/src/guestfs-internal.h
> +++ b/src/guestfs-internal.h
> @@ -93,7 +93,7 @@
>
> /* The default and minimum memory size for most users. */
> #ifndef DEFAULT_MEMSIZE
> -# define DEFAULT_MEMSIZE 500
> +# define DEFAULT_MEMSIZE 768
> #endif
> #ifndef MIN_MEMSIZE
> # define MIN_MEMSIZE 128
If only we had a more sensible way to allocate memory :-(
Anyway, this patch needs to remove the earlier lines which set
DEFAULT_MEMSIZE to 768 on __powerpc__ and __aarch64__.
When I thought about this a bit more, I think what you want is to just
increase the memory limit for virt-v2v. How about the attached?
(Only compile tested.)
Rich.
--
Richard Jones, Virtualization Group, Red Hat
http://people.redhat.com/~rjones
Read my programming and virtualization blog:
http://rwmj.wordpress.com
libguestfs lets you edit virtual machines. Supports shell scripting,
bindings from many languages.
http://libguestfs.org