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&gt; --- 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

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&gt; >> --- >> 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.

On Sat, Jun 11, 2016 at 08:51:57PM +0300, Pavel Butsykin wrote: > Guest tools for Linux in Parallels / Virtuozzo Server 6 come with a script > that can be used to uninstall them. > > Signed-off-by: Pavel Butsykin <pbutsykin(a)virtuozzo.com&gt; > --- > v2v/convert_linux.ml | 15 +++++++++++++++ > 1 file changed, 15 insertions(+) > > diff --git a/v2v/convert_linux.ml b/v2v/convert_linux.ml > index 7eef2ab..681466e 100644 > --- a/v2v/convert_linux.ml > +++ b/v2v/convert_linux.ml > @@ -694,6 +694,20 @@ let rec convert ~keep_serial_console (g : G.guestfs) inspect source rcaps = > ignore (g#command [| "/sbin/chkconfig"; "kudzu"; "off" |]) > ) > > + and unconfigure_prltools () = > + let prltools_path = "/usr/lib/parallels-tools/install" in > + if g#is_file ~followsymlinks:true prltools_path then ( > + try > + ignore (g#command [| prltools_path; "-r" |]); > + > + (* Reload Augeas to detect changes made by prltools uninst. *) > + Linux.augeas_reload g > + with > + G.Error msg -> > + warning (f_"Parallels tools was detected, but uninstallation failed. The error message was: %s (ignored)") > + msg > + ) > + > and configure_kernel () = > (* Previously this function would try to install kernels, but we > * don't do that any longer. > @@ -1414,6 +1428,7 @@ let rec convert ~keep_serial_console (g : G.guestfs) inspect source rcaps = > unconfigure_vmware (); > unconfigure_citrix (); > unconfigure_kudzu (); > + unconfigure_prltools (); > > let kernel, virtio = configure_kernel () in I have pushed this patch. Thanks for your contribution. I will look at the other more complex patch series later -- tomorrow hopefully. My apologies for the mailing list delays/filtering -- this is because spammers try to sign up for the list to spam it, so we have to moderate initial messages. Every mail from should go though without any delays now.

Rich.