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

Re: [PATCH v4 2/2] xen/arm: Enlarge identity map space to 10TB


  • To: Bertrand Marquis <Bertrand.Marquis@xxxxxxx>
  • From: Luca Fancellu <Luca.Fancellu@xxxxxxx>
  • Date: Mon, 16 Oct 2023 14:13:30 +0000
  • Accept-language: en-GB, en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=0l2iLTh1w+3tqfm+dfdNDd84vd/fBqCZF7spkLjYXao=; b=TQxZ75Rdu2fyS3huK0mtDxBbO5sRqzRvQSz4AY2EYiOpo9IwqUpWbB7TcUv0I+oJ6+CKA+2WtfWgacieQEH/V8bUDm5OYavJJ9loQp6sADN5cbgBMX0in3sZ+Gh4hx8Y+RFfl22Ti91g478B6K+xA6LrYkeOUVYZZgBVrEjymCzF8lXlCR/8SykhmLdWFEy/5QMEmKduDU0xqF1M4pjNlQrMpfSJHe5/Q5toj4c69dGA13NuWnoY1zQ9svDXMkP/iY02yRkzPk+HCHsn7avKka86IQPQ+j5G0BU8OjTFYpr14k+NJIjuOctlzZbQvMc7pjoRTOguxc7lf2n5Pp4AVA==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ItORtWpcUnCys1psKUKmALkxRKWkUIhK9fNu7vGiqIWDC7izc+SY9QobpG2bl6ym6fAK7VkVP3dajvJkFEj5KfzDqCvuqHU2gzbJ3irQ/Iv3AuuThIRMT2yTKLC6qbW/ENDQYcA4UDw46rdjERAiCjwFWVxdzlLh6rH3ZZNXaG25bhN4z7+HbX60vMA04uSOreqWNNcFMLrzh9kAQFFrtOZgDtZKs979q5B+K2ac503Hih6iR9t4N7Js+nfto1+kcwLHnEr3SVDcdZOJHvo2DBf4HCQy46R5ZHTudfzyLsNPfLGQ73RmVwiAzU3kvyJEdgGIWnV+aQjvXzn/LSDgvA==
  • Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Cc: Julien Grall <julien@xxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>, "leo.yan@xxxxxxxxxx" <leo.yan@xxxxxxxxxx>, Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>, Alexey Klimov <alexey.klimov@xxxxxxxxxx>, Rahul Singh <Rahul.Singh@xxxxxxx>
  • Delivery-date: Mon, 16 Oct 2023 14:14:01 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Nodisclaimer: true
  • Original-authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Thread-index: AQHaADhLetLGz4GvX0qAhZnpE9HtkrBMcc4AgAADlIA=
  • Thread-topic: [PATCH v4 2/2] xen/arm: Enlarge identity map space to 10TB


> On 16 Oct 2023, at 15:00, Bertrand Marquis <Bertrand.Marquis@xxxxxxx> wrote:
> 
> Hi
> 
> +Luca and Rahul
> 
>> On 16 Oct 2023, at 15:54, Julien Grall <julien@xxxxxxx> wrote:
>> 
>> 
>> 
>> On 16/10/2023 09:44, Michal Orzel wrote:
>>> Hi,
>> 
>> Hi,
>> 
>>> On 13/10/2023 14:26, Leo Yan wrote:
>>>> 
>>>> 
>>>> On ADLink AVA platform (Ampere Altra SoC with 32 Arm Neoverse N1 cores),
>>>> the physical memory regions are:
>>>> 
>>>>  DRAM memory regions:
>>>>    Node[0] Region[0]: 0x000080000000 - 0x0000ffffffff
>>>>    Node[0] Region[1]: 0x080000000000 - 0x08007fffffff
>>>>    Node[0] Region[2]: 0x080100000000 - 0x0807ffffffff
>>>> 
>>>> The UEFI loads Xen hypervisor and DTB into the high memory, the kernel
>>>> and ramdisk images are loaded into the low memory space:
>>>> 
>>>>  (XEN) MODULE[0]: 00000807f6df0000 - 00000807f6f3e000 Xen
>>>>  (XEN) MODULE[1]: 00000807f8054000 - 00000807f8056000 Device Tree
>>>>  (XEN) MODULE[2]: 00000000fa834000 - 00000000fc5de1d5 Ramdisk
>>>>  (XEN) MODULE[3]: 00000000fc5df000 - 00000000ffb3f810 Kernel
>>>> 
>>>> In this case, the Xen binary is loaded above 8TB, which exceeds the
>>>> maximum supported identity map space of 2TB in Xen. Consequently, the
>>>> system fails to boot.
>>>> 
>>>> This patch enlarges identity map space to 10TB, allowing module loading
>>>> within the range of [0x0 .. 0x000009ff_ffff_ffff].
>>>> 
>>>> Fixes: 1c78d76b67 ("xen/arm64: mm: Introduce helpers to 
>>>> prepare/enable/disable")
>>> I don't think a fixes tag applies here given that 2TB was just a number we 
>>> believed is enough
>>> and all of this is platform dependent.
>>> This can be dropped on commit if committer agrees
>> Xen may have booted on that platform before hand. So this would be 
>> considered a regression and therefore a tag would be warrant.
>> 
>> AFAICT, the commit is only present on the upcoming 4.18. So the question is 
>> whether Xen 4.17 booted out-of-the-box on ADLink? If the answer is yes, then 
>> we need to add a Fixes tag. But the correct one would be
>> 
> 
> @Rahul or Luca: could you give an answer here ?
> I know you used Xen on an AVA platform but was it booting out of the box ?

I can’t say for Xen 4.17, but our nightly job has run successfully on AVA for 
the commit 730406ab81094115d9fb5ca00ba8d53cec1279b3
(docs/misra: add deviations.rst to document additional deviations.)

We are not applying any patch for it to run on AVA.

> 
>> 1c78d76b67e1 ("xen/arm64: mm: Introduce helpers to prepare/enable/disable 
>> the identity mapping").
>> 
>> We would also need to consider it as a candidate for Xen 4.18 because we 
>> would regress boot on ADLink.
> 
> Ack
> 
> Cheers
> Bertrand
> 
>> 
>> Cheers,
>> 
>> --
>> Julien Grall
> 
> 


 


Rackspace

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