Hi,

On 3/15/19 11:57 PM, Feng Kan OS wrote:
Thanks Julien.

On 3/15/19 4:21 AM, Julien Grall wrote:
Hi,

Thank you for posting the patch.

Title: No need for full stop in the commit title.

On 15/03/2019 08:34, Vishnu Pajjuri OS wrote:
I can't remember which other platforms support 42-bits PA. I think at
that time it was X-Gene. As long as no current embedded platform we
support use 42-bit PA, this change may be ok. Stefano do you recall what
was the platform?
Ampere eMAG platform is essentially the continuation of X-Gene. These
systems are targeted as servers with upto 1TB of RAM.

In any case, the new behavior (and consequence) needs to be clearly
explained in the commit message.
Got it, we will resubmit if Stefano is okay with the change.

To make progress on this patch, I would suggest to resubmit it with the changes requested.

Cheers,

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

Reply via email to