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

Re: [PATCH][4.17] EFI: don't convert memory marked for runtime use to ordinary RAM


  • To: Jan Beulich <jbeulich@xxxxxxxx>
  • From: Luca Fancellu <Luca.Fancellu@xxxxxxx>
  • Date: Fri, 30 Sep 2022 12:47:57 +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=armh.onmicrosoft.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=2TnTgGY9U3j/bL/Zn2HiBAWv15w+rYlWg7GYB2jmkKo=; b=ZVVU3QGUqbYzZrZjEh0EREKTvM3p3E8pcuvf1M1ZvjYFGTMBTLdkTbNzLMCar6xe35InvjZCZL1VuxmjklLbbW0GU+mpdAD5FOv6KBBSdezZqYcUCJVfdd9Oj/il0XLoU+eK5AId2FWGygLwsZ6T7co7XW/03JjlMAR/ghVm6n54dypqGErA6xr/rkRD/GSztR4eQVJQdlYl1n2TMWAmgvugqitwDlpKDuFxqg49N2VDx9I9qMor8cUm9Vko6SSVA86YoL0Bx1CLaBtruxlwIO9A5Jb4T2KIwLk4ygEDFuh9U8KOd0rJqY0qOErTbMJn2erLnPsPQqFhkqfdE4jZhg==
  • 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=2TnTgGY9U3j/bL/Zn2HiBAWv15w+rYlWg7GYB2jmkKo=; b=hC/WFOKTxdzE0/H7ZM+gkCRSCQ1y0X0dYKDr9DyAZ91uSWYFpxSL/oxvVg5eoanAmmnxoE1MLCIk6JY0TpF9PrKHUlOBnYZU7EUnWv3CNX1f9G+e0fj2uKhcZeXDLMxd8UfrOPW2mnzrAMk4rmV2hP4f7Utgf7oiCm3eWJiXCMe3Ow/WU53EfPZqxtMoNRiXKb0eDxtpuAGe6qOmqJrvuNyeynfPCogYaZKhUyej3DG7cbYE3A5HWspZmDkrWPp/X0G7RtZFVcstL7zeveNIwXgzKwVcN57xO73wcpdgBn/OGuYyL8mdkJGN9Dg5vyotjiO7vMdHb6K9qhm5+1PVsA==
  • Arc-seal: i=2; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=pass; b=O/lWuoDZS/+9lTTFcokIAProszSZwkX2QsUSIJ0gHKSERQAP+4Moh/7de1F0yqsGMPVxA84FP5MLuHFEep+CAUR0+TuLsv628hQBxa7HPz2uZXFE4zQmnYfNSbuCyVaMY6KZU1KxVOHZMGx+DWVWn/0EqFf8yrAvz3QNL8HSMsfUZ7IwblUL4Gl4NFYU2TyVsShJK//QMZdSVhht1RtsEvHO5U7eUcl7KWXCCdBCftPUd3dcOpvaYJGBW4J1VutLdx5hplduyWTFORaNLDTsN8HcW6RkWmjq2/IWOpdkr4KCr8dgEYDc8cMbP5VYxB18JmK4MmdrIAY8JCYzZHPn1Q==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=kfflKIS3xQBIxo4bZdDjn1qXWIiWL8FhqEjcNetYKVFAUMWEz6AXPWDDI27Xxo1+1gelMonLO+FSIK+uGOw1XYphGetcIlDBSnz5NpimeNucDlSVgm7NdfcrR+biL9l3UxWOM6Q10EqRER6dLnzZ9sCjvevSZW4GNOlgQ/hN7sBAX59AXRg2tokJKEvMiytqKn1MbwH3Pr/P1lM6Zn/iOIt5dfUy81+nchtgXTBCLtgAWdLmI286+3sG2GnwOTK67v2X/CWgmK+HIQbJyse420214RQQHH2C+Vt6BgT6YXGVoDZ0xx44LmQuuG3lpdikH+QDbdSMEUomKvMJCtg7tg==
  • Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Volodymyr Babchuk <volodymyr_babchuk@xxxxxxxx>, Bertrand Marquis <Bertrand.Marquis@xxxxxxx>, Henry Wang <Henry.Wang@xxxxxxx>
  • Delivery-date: Fri, 30 Sep 2022 12:48:34 +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: AQHY1KFr9QM2MLmHw02Oz3qGbH+jNq337HYA
  • Thread-topic: [PATCH][4.17] EFI: don't convert memory marked for runtime use to ordinary RAM


> On 30 Sep 2022, at 08:50, Jan Beulich <jbeulich@xxxxxxxx> wrote:
> 
> efi_init_memory() in both relevant places is treating EFI_MEMORY_RUNTIME
> higher priority than the type of the range. To avoid accessing memory at
> runtime which was re-used for other purposes, make
> efi_arch_process_memory_map() follow suit. While on x86 in theory the
> same would apply to EfiACPIReclaimMemory, we don't actually "reclaim"
> E820_ACPI memory there and hence that type's handling can be left alone.
> 
> Fixes: bf6501a62e80 ("x86-64: EFI boot code")
> Fixes: facac0af87ef ("x86-64: EFI runtime code")
> Fixes: 6d70ea10d49f ("Add ARM EFI boot support")
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
> ---

Hi Jan,

For the Arm part:

Reviewed-by: Luca Fancellu <luca.fancellu@xxxxxxx>

I’ve also tested the EFI+ACPI boot on two arm boards

Tested-By: Luca Fancellu <luca.fancellu@xxxxxxx>





 


Rackspace

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