[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

To: Jan Beulich <JBeulich@xxxxxxxx>, xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>
From: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
Date: Tue, 20 Jun 2017 09:37:13 +0100
Cc: Stefano Stabellini <sstabellini@xxxxxxxxxx>, Wei Liu <wei.liu2@xxxxxxxxxx>, George Dunlap <George.Dunlap@xxxxxxxxxxxxx>, Tim Deegan <tim@xxxxxxx>, Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>, Julien Grall <julien.grall@xxxxxxx>
Delivery-date: Tue, 20 Jun 2017 08:37:34 +0000
List-id: Xen developer discussion <xen-devel.lists.xen.org>

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 null). I have no idea
> how I've managed to overlook this.
>
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>

Acked-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

Follow-Ups:
- Re: [Xen-devel] [PATCH] memory: don't suppress P2M update in populate_physmap()
  - From: Julien Grall

References:
- [Xen-devel] [PATCH] memory: don't suppress P2M update in populate_physmap()
  - From: Jan Beulich

Prev by Date: Re: [Xen-devel] [PATCH v7 11/36] x86/mm: Add SME support for read_cr3_pa()
Next by Date: Re: [Xen-devel] [PATCH v2 01/16] xen/mm: Don't use _{g, m}fn for defining INVALID_{G, M}FN
Previous by thread: [Xen-devel] [PATCH] memory: don't suppress P2M update in populate_physmap()
Next by thread: Re: [Xen-devel] [PATCH] memory: don't suppress P2M update in populate_physmap()
Index(es):
- Date
- Thread

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.