Re: [Xen-devel] [PATCH v3 16/19] mini-os: map page allocator's bitmap to virtual kernel area for ballooning

2016-08-11 Thread Juergen Gross
On 11/08/16 12:21, Wei Liu wrote: > On Thu, Aug 11, 2016 at 12:19:20PM +0200, Juergen Gross wrote: >> On 11/08/16 11:40, Samuel Thibault wrote: >>> Juergen Gross, on Thu 11 Aug 2016 11:18:19 +0200, wrote: +extern unsigned long *mm_bitmap; +extern unsigned long mm_bitmap_size; >>> >>> Ah I

Re: [Xen-devel] [PATCH v3 16/19] mini-os: map page allocator's bitmap to virtual kernel area for ballooning

2016-08-11 Thread Samuel Thibault
Juergen Gross, on Thu 11 Aug 2016 12:19:20 +0200, wrote: > On 11/08/16 11:40, Samuel Thibault wrote: > > Juergen Gross, on Thu 11 Aug 2016 11:18:19 +0200, wrote: > >> +extern unsigned long *mm_bitmap; > >> +extern unsigned long mm_bitmap_size; > > > > Ah I was thinking to have these use mm_malloc_

Re: [Xen-devel] [PATCH v3 16/19] mini-os: map page allocator's bitmap to virtual kernel area for ballooning

2016-08-11 Thread Wei Liu
On Thu, Aug 11, 2016 at 12:19:20PM +0200, Juergen Gross wrote: > On 11/08/16 11:40, Samuel Thibault wrote: > > Juergen Gross, on Thu 11 Aug 2016 11:18:19 +0200, wrote: > >> +extern unsigned long *mm_bitmap; > >> +extern unsigned long mm_bitmap_size; > > > > Ah I was thinking to have these use mm_m

Re: [Xen-devel] [PATCH v3 16/19] mini-os: map page allocator's bitmap to virtual kernel area for ballooning

2016-08-11 Thread Juergen Gross
On 11/08/16 11:40, Samuel Thibault wrote: > Juergen Gross, on Thu 11 Aug 2016 11:18:19 +0200, wrote: >> +extern unsigned long *mm_bitmap; >> +extern unsigned long mm_bitmap_size; > > Ah I was thinking to have these use mm_malloc_ too. "mm_bitmap" seems > short for namespace pollution. Sorry, you

Re: [Xen-devel] [PATCH v3 16/19] mini-os: map page allocator's bitmap to virtual kernel area for ballooning

2016-08-11 Thread Samuel Thibault
Juergen Gross, on Thu 11 Aug 2016 11:18:19 +0200, wrote: > +extern unsigned long *mm_bitmap; > +extern unsigned long mm_bitmap_size; Ah I was thinking to have these use mm_malloc_ too. "mm_bitmap" seems short for namespace pollution. Samuel ___ Xen-dev

[Xen-devel] [PATCH v3 16/19] mini-os: map page allocator's bitmap to virtual kernel area for ballooning

2016-08-11 Thread Juergen Gross
In case of CONFIG_BALLOON the page allocator's bitmap needs some space to be able to grow. Remap it to kernel virtual area if the preallocated area isn't large enough. Signed-off-by: Juergen Gross --- V3: - add assertion as requested by Samuel Thibault - rename functions to have mm_ prefix as