[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC PATCH 1/2] xen/arm: Add DT reserve map regions to bootinfo.reserved_mem
- To: Julien Grall <julien@xxxxxxx>
- From: Luca Fancellu <Luca.Fancellu@xxxxxxx>
- Date: Tue, 14 May 2024 07:53:25 +0000
- Accept-language: en-GB, en-US
- Arc-authentication-results: i=2; mx.microsoft.com 1; spf=pass (sender ip is 63.35.35.123) smtp.rcpttodomain=lists.xenproject.org smtp.mailfrom=arm.com; dmarc=pass (p=none sp=none pct=100) action=none header.from=arm.com; dkim=pass (signature was verified) header.d=arm.com; arc=pass (0 oda=1 ltdi=1 spf=[1,1,smtp.mailfrom=arm.com] dkim=[1,1,header.d=arm.com] dmarc=[1,1,header.from=arm.com])
- 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=2; 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=DERBBdoj0rOaglpD1IoOe+rXZPnkfSDKmKH3vSPMDQ0=; b=hr6koaKMspYWWXBmP2xanTBxDluxpsnrBkYgfoz6qD8e6YanC9Vr0yEQXFC1su+NzA/fGDqGmDYgMs+qE8vWe33NvtQsWwcDjiLdkYDp5xtt0IP0XZ3u4uCpx99436o+921H3dRHzDM/9FVOTlYFy7YJo8sN3K78ePOCW8T1H3bdOabrVputbbsmXMRbi0jc3B/LLJeRvnvLT5gTUGseVf2yEGemsD1lhZdfDtBZR+ycT6ICfsAeqvGmPdly9c+jdBjecN7l4UVl7mVpCHHtwXhgXDj5/q/ptEPSkMQf8qSjVJlwoDRPIBCXOMDG+WffuS7aOQdQfa9hs0I9F/tOtQ==
- 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=DERBBdoj0rOaglpD1IoOe+rXZPnkfSDKmKH3vSPMDQ0=; b=ZFO/BOxW/muvEJ2shs5ijqmJCAd+Y9p8SBKTJ3B+JYIaz3wHXBMFe4AaiZDZi2XUfXUcBhzYjHdzylPLoYJmcXlkx3DtMeHcotWs1mjSsvhmhALn72Ynzf7vdWoo6PMq8UYkLe+hAywFZ07eyWTWX+yDHzG81eCQWGp5cYagwFQrTgJzfbUSR0RzEYLD6HCReXeeJCBk+AswGfCHMQjQh3PqOSXR2qShfLUAYpSFo20V8ehpdR+hdnYZq1ZB4E1q4KHqx1MqJdCJyhBX+i/k7Z3WjvZVi/Zm909Y/aK2f+J90k1nNdAgvH2LYEGUbarebfinmxarKIx8drLKmbGzcg==
- Arc-seal: i=2; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=pass; b=UwWLz3powiR/h0Zc8Q+iUoO6al6BDbD84Bj3cLhbaRCGGs3J1JN31p2VbKraVY/Vm42C5DVUNq+X09IVzI/N8iz0kRd7cuUe/FKUPjMMMpoJJ+85Tj4acNY6Agnp+0/b0HXgU6iku2hFgPU1lIGVekVIFCuoXp1FlRwogDijLrFiJYhDv06Pcgar8hOZV8bjAYOXW4v5YOfz2m6YEt2mp3K+Z8iGEJvsfBfVv89wD6RoIKxyPxTaFWj/vYrw+Pv+3rWrrtyFdLx9KdKrNGMwWILbCt11dd524nTDXTNS8LIXl0bMpp/N9Cju7pdraNFtlHw3rKxRA3mS/K4IEkg9Fg==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=DCGRiinJqNTgTPLBV1XUSjs+HhWlSEv6qGi7z1KQSGvChM5Niu1tRbLNzHQll64e08TAP88egeHc9tDyT407nvi3ltJgPaYE2gaDcnwxMizTT1+Ow7zWST2Vfwcb7FnZA7opJwGv+mYHzm+EGk8rXZcKiDhud7qlNi/kwy53aJOcEQORXoXLushuSQ90bYi0nTbSeF0CI8GkDCrzSnVqvms6Ov/jIy+V3gsE0zqn9D4Z05M9hdTEd6Z0GcU5CM6FSCxGRTq8fAOA3qzoDoGzEd7LTHE/yZDq+/6Jo1+HdDAqNW1M8RrTxt4moWWxvdIJ5vDYUCdI0bX6KbycKqYzXA==
- Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
- Cc: Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Bertrand Marquis <Bertrand.Marquis@xxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>
- Delivery-date: Tue, 14 May 2024 07:53:43 +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: AQHalxIi36RR5t8uSEeoUslJ6d4QpLGV0j4AgACnKQA=
- Thread-topic: [RFC PATCH 1/2] xen/arm: Add DT reserve map regions to bootinfo.reserved_mem
Hi Julien,
Thanks for having a look on the patch,
> On 13 May 2024, at 22:54, Julien Grall <julien@xxxxxxx> wrote:
>
> Hi Luca,
>
> On 25/04/2024 14:11, Luca Fancellu wrote:
>> Currently the code is listing device tree reserve map regions
>> as reserved memory for Xen, but they are not added into
>> bootinfo.reserved_mem and they are fetched in multiple places
>> using the same code sequence, causing duplication. Fix this
>> by adding them to the bootinfo.reserved_mem at early stage.
>
> Do we have enough space in bootinfo.reserved_mem for them?
So we have 255 banks, in my experience I would say I’ve never saw too many
reserved regions
in the DT, maybe a couple, but I’ve always had to deal with embedded platforms.
I’ve tested this one with ADLINK AVA board, n1sdp, Juno, raspberry pi, qemu,
fvp.
In your experience, have you seen any numbers that could be concerning?
Cheers,
Luca
|
