Re: [Patch V2 2/2] xen: before ballooning hotplugged memory, set frames to invalid

2015-03-23 Thread David Vrabel
On 20/03/15 13:46, Daniel Kiper wrote: > On Fri, Mar 20, 2015 at 01:55:39PM +0100, Juergen Gross wrote: >> Commit 25b884a83d487fd62c3de7ac1ab5549979188482 ("x86/xen: set >> regions above the end of RAM as 1:1") introduced a regression. >> > > Should not we fill everything above "maxmem" with > INV

Re: [Patch V2 2/2] xen: before ballooning hotplugged memory, set frames to invalid

2015-03-20 Thread Juergen Gross
On 03/20/2015 02:44 PM, Boris Ostrovsky wrote: On 03/20/2015 08:55 AM, Juergen Gross wrote: Commit 25b884a83d487fd62c3de7ac1ab5549979188482 ("x86/xen: set regions above the end of RAM as 1:1") introduced a regression. To be able to add memory pages which were added via memory hotplug to a pv do

Re: [Patch V2 2/2] xen: before ballooning hotplugged memory, set frames to invalid

2015-03-20 Thread Juergen Gross
On 03/20/2015 02:46 PM, Daniel Kiper wrote: On Fri, Mar 20, 2015 at 01:55:39PM +0100, Juergen Gross wrote: Commit 25b884a83d487fd62c3de7ac1ab5549979188482 ("x86/xen: set regions above the end of RAM as 1:1") introduced a regression. To be able to add memory pages which were added via memory hot

Re: [Patch V2 2/2] xen: before ballooning hotplugged memory, set frames to invalid

2015-03-20 Thread Daniel Kiper
On Fri, Mar 20, 2015 at 01:55:39PM +0100, Juergen Gross wrote: > Commit 25b884a83d487fd62c3de7ac1ab5549979188482 ("x86/xen: set > regions above the end of RAM as 1:1") introduced a regression. > > To be able to add memory pages which were added via memory hotplug to > a pv domain, the pages must be

Re: [Patch V2 2/2] xen: before ballooning hotplugged memory, set frames to invalid

2015-03-20 Thread Boris Ostrovsky
On 03/20/2015 08:55 AM, Juergen Gross wrote: Commit 25b884a83d487fd62c3de7ac1ab5549979188482 ("x86/xen: set regions above the end of RAM as 1:1") introduced a regression. To be able to add memory pages which were added via memory hotplug to a pv domain, the pages must be "invalid" instead of "id