Re: [Xen-devel] [PATCH] memory: don't suppress P2M update in populate_physmap()

2017-06-20 Thread Julien Grall
On 06/20/2017 02:03 PM, Jan Beulich wrote: On 20.06.17 at 14:51, wrote: On 06/20/2017 01:40 PM, Andrew Cooper wrote: On 20/06/17 13:39, Julien Grall wrote: On 06/20/2017 09:37 AM, Andrew Cooper wrote: On 20/06/17 07:19, Jan Beulich wrote: Commit d18627583d ("memory: don't hand MFN info to

Re: [Xen-devel] [PATCH] memory: don't suppress P2M update in populate_physmap()

2017-06-20 Thread Jan Beulich
>>> On 20.06.17 at 14:51, wrote: > On 06/20/2017 01:40 PM, Andrew Cooper wrote: >> On 20/06/17 13:39, Julien Grall wrote: >>> On 06/20/2017 09:37 AM, Andrew Cooper wrote: On 20/06/17 07:19, Jan Beulich wrote: > Commit d18627583d ("memory: don't hand MFN info to translated guests") > w

Re: [Xen-devel] [PATCH] memory: don't suppress P2M update in populate_physmap()

2017-06-20 Thread Julien Grall
Hi, On 06/20/2017 01:40 PM, Andrew Cooper wrote: On 20/06/17 13:39, Julien Grall wrote: Hi, On 06/20/2017 09:37 AM, Andrew Cooper wrote: On 20/06/17 07:19, Jan Beulich wrote: Commit d18627583d ("memory: don't hand MFN info to translated guests") wrongly added a null-handle check there - just

Re: [Xen-devel] [PATCH] memory: don't suppress P2M update in populate_physmap()

2017-06-20 Thread Andrew Cooper
On 20/06/17 13:39, Julien Grall wrote: > Hi, > > On 06/20/2017 09:37 AM, Andrew Cooper wrote: >> On 20/06/17 07:19, Jan Beulich wrote: >>> Commit d18627583d ("memory: don't hand MFN info to translated guests") >>> wrongly added a null-handle check there - just like stated in its >>> description for

Re: [Xen-devel] [PATCH] memory: don't suppress P2M update in populate_physmap()

2017-06-20 Thread Julien Grall
Hi, On 06/20/2017 09:37 AM, Andrew Cooper wrote: On 20/06/17 07:19, Jan Beulich wrote: Commit d18627583d ("memory: don't hand MFN info to translated guests") wrongly added a null-handle check there - just like stated in its description for memory_exchange(), the array is also an input for popul

Re: [Xen-devel] [PATCH] memory: don't suppress P2M update in populate_physmap()

2017-06-20 Thread Andrew Cooper
On 20/06/17 07:19, Jan Beulich wrote: > Commit d18627583d ("memory: don't hand MFN info to translated guests") > wrongly added a null-handle check there - just like stated in its > description for memory_exchange(), the array is also an input for > populate_physmap() (and hence can't reasonably be