Re: [PATCH 1/1] xen/balloon: Enforce various limits on target

2013-03-07 Thread Daniel Kiper
On Thu, Mar 07, 2013 at 12:07:09PM +, David Vrabel wrote: > On 07/03/13 11:28, Daniel Kiper wrote: > > On Wed, Mar 06, 2013 at 05:52:28PM +, David Vrabel wrote: > > > >> If you set the target above d->max_pages you won't be able to populate > >> them. > >> > >> So, using the maximum_reserv

Re: [PATCH 1/1] xen/balloon: Enforce various limits on target

2013-03-07 Thread David Vrabel
On 07/03/13 11:28, Daniel Kiper wrote: > On Wed, Mar 06, 2013 at 05:52:28PM +, David Vrabel wrote: > >> If you set the target above d->max_pages you won't be able to populate them. >> >> So, using the maximum_reservation call seems like the right thing to me. > > Please look above. If you use

Re: [PATCH 1/1] xen/balloon: Enforce various limits on target

2013-03-07 Thread Daniel Kiper
On Wed, Mar 06, 2013 at 05:52:28PM +, David Vrabel wrote: > On 06/03/13 16:47, Daniel Kiper wrote: [...] > >> I think reserve_additional_memory() should check the current resource > >> map and the e820 map to find a large enough unused region. This can be > >> done as an additional patch at

Re: [PATCH 1/1] xen/balloon: Enforce various limits on target

2013-03-06 Thread David Vrabel
On 06/03/13 16:47, Daniel Kiper wrote: > On Wed, Mar 06, 2013 at 11:05:03AM +, David Vrabel wrote: >> On 04/03/13 21:14, Daniel Kiper wrote: >>> This patch enforces on target limit statically defined in Linux Kernel >>> source and limit defined by hypervisor or host. >>> >>> Particularly this p

Re: [PATCH 1/1] xen/balloon: Enforce various limits on target

2013-03-06 Thread Daniel Kiper
On Wed, Mar 06, 2013 at 11:05:03AM +, David Vrabel wrote: > On 04/03/13 21:14, Daniel Kiper wrote: > > This patch enforces on target limit statically defined in Linux Kernel > > source and limit defined by hypervisor or host. > > > > Particularly this patch fixes bug which led to flood > > of d

Re: [PATCH 1/1] xen/balloon: Enforce various limits on target

2013-03-06 Thread David Vrabel
On 04/03/13 21:14, Daniel Kiper wrote: > This patch enforces on target limit statically defined in Linux Kernel > source and limit defined by hypervisor or host. > > Particularly this patch fixes bug which led to flood > of dom0 kernel log with messages similar to: > > System RAM resource [mem 0x

Re: [PATCH 1/1] xen/balloon: Enforce various limits on target

2013-03-05 Thread Konrad Rzeszutek Wilk
On Mon, Mar 04, 2013 at 10:14:51PM +0100, Daniel Kiper wrote: > This patch enforces on target limit statically defined in Linux Kernel > source and limit defined by hypervisor or host. > > Particularly this patch fixes bug which led to flood > of dom0 kernel log with messages similar to: > > Syst

[PATCH 1/1] xen/balloon: Enforce various limits on target

2013-03-04 Thread Daniel Kiper
This patch enforces on target limit statically defined in Linux Kernel source and limit defined by hypervisor or host. Particularly this patch fixes bug which led to flood of dom0 kernel log with messages similar to: System RAM resource [mem 0x1b800-0x1bfff] cannot be added xen_balloon: r