[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <1671727226.207651253175241980.JavaMail.root@zmail06.collab.prod.int.phx2.redhat.com>
Date: Thu, 17 Sep 2009 04:14:01 -0400 (EDT)
From: Miroslav Rezanina <mrezanin@...hat.com>
To: Jeremy Fitzhardinge <jeremy@...p.org>
Cc: Konrad Rzeszutek Wilk <konrad.wilk@...cle.com>,
xen-devel@...ts.xensource.com, linux-kernel@...r.kernel.org
Subject: Re: [Xen-devel] Re: [PATCH][v2.6.29][XEN] Return unused memory to
hypervisor
Hi Jeremy,
I check your application. It won't work correctly. You are calling return_unused_memory in xen_memory_setup that is too soon.
When you are calling it, there is no hole in map in any case. Calling should be in xen_post_allocator_init function (in arch/x86/xen/mmu.c).
Mirek
----- "Jeremy Fitzhardinge" <jeremy@...p.org> wrote:
> From: "Jeremy Fitzhardinge" <jeremy@...p.org>
> To: "Miroslav Rezanina" <mrezanin@...hat.com>
> Cc: "Konrad Rzeszutek Wilk" <konrad.wilk@...cle.com>, xen-devel@...ts.xensource.com, linux-kernel@...r.kernel.org
> Sent: Thursday, September 17, 2009 6:54:56 AM GMT +01:00 Amsterdam / Berlin / Bern / Rome / Stockholm / Vienna
> Subject: Re: [Xen-devel] Re: [PATCH][v2.6.29][XEN] Return unused memory to hypervisor
>
> On 09/16/09 21:43, Miroslav Rezanina wrote:
> > Ups.....This should not be commented out. I will resent the patch
> with
> > correct (uncommented) line.
>
> I've already applied a variant of your patch. Check the
> rebase/core-freemem branch in xen.git.
>
> J
>
> --
> To unsubscribe from this list: send the line "unsubscribe
> linux-kernel" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
--
Miroslav Rezanina
Software Engineer - Virtualization Team - XEN kernel
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists