[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: Roger Pau Monné <roger.pau@xxxxxxxxxx>
- Date: Tue, 4 Oct 2022 11:33:15 +0200
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.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=ZPMVPlzMhqoLpJn5DKgly0KDoFNYA1yWPczeSKxEU8w=; b=IfEw+ztksjumj30O5HUCNJWgECxvnKtHCs2sHcdLmt7tgzgx1BriZ1TFo/SySyAqfGOgO2TZdWzDZQJE9NV+HOVhKujWDQ/LPF0EpcWb67f4LX8m4QbalMAgjWBMdhCAKuLX9eG09mWsu7E/P5I8rWxIQR8iHuBSJqIzCIj9jgrLsu8o1Ke1wsgjfEnYy/+idOgh4X/0QF9vOMk7VufOabaa27DOdNHfCsABQiPcoqLjlo2B59/jSy4+edsRdHlgIPOLF5Bg91BOrWnZt0JhgxAsTssuhHHqjALqyFmGA3eopYrnU9tpO5+nml9vGorRc7c93LYeveehQgc837OuFw==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Gs2yAQaJP4lS4eNGaOLYxaCXEkOo0TaYExwSLFEWFoYbbXnakppwgn9RdMigIsZ4bD+WAw1J0raYxCaYnRz1AyxJlZ5yf1u+/3f2g9uIFpwIoV5fWifKA7eMpNPaYpYIKrYt+WVYSXugi9rLNMxIq5d+/qvORQNLCTmokL/CCQRF/M6aCTdbO7RfV1/PXWb8/M5Y2533gH+lqIrZq29kOJd9I95t+6e/0GY6CSya3Y7NBHaX6NPb+4JQXQBvW6z/7jDue7uzwQadqqZ/jLGuSfd6QvrHk8htj+8wLkd32kLMLKUUuDDhyKIJaztfZzn7PhGgfc7sU+koUmI2quzA0A==
- Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=citrix.com;
- Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, 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: Tue, 04 Oct 2022 09:33:39 +0000
- Ironport-data: A9a23:K8eQR6zJe5kubkMXy3F6t+e9xyrEfRIJ4+MujC+fZmUNrF6WrkVVz TdLDz2AOPaCa2CnfIgladi+/U9TuZHTmtM2T1ZuryAxQypGp/SeCIXCJC8cHc8wwu7rFxs7s ppEOrEsCOhuExcwcz/0auCJQUFUjP3OHPykYAL9EngZbRd+Tys8gg5Ulec8g4p56fC0GArIs t7pyyHlEAbNNwVcbyRFsMpvlDs15K6o4WtA4gRnDRx2lAS2e0c9Xcp3yZ6ZdxMUcqEMdsamS uDKyq2O/2+x13/B3fv8z94X2mVTKlLjFVDmZkh+AsBOsTAbzsAG6Y4pNeJ0VKtio27hc+ada jl6ncfYpQ8BZsUgkQmGOvVSO3kW0aZuoNcrLZUj2CA6IoKvn3bEmp1T4E8K0YIwp91oLkwQ5 6UicRNKQR6np8Htn63mRbw57igjBJGD0II3nFhFlW2cJ9B2BJfJTuPN+MNS2yo2ioZWB/HCa sEFaD1pKhPdfxlIPVRRA5U79AuqriCnL3sE9xTK+exruAA/zyQouFTpGMDSddGQA91cg26Tp 37c/nS/CRYfXDCa4WrZri713bGT9c/9cKsrLPqf1O9OuX209GdNDjs6TEa9ptDs3yZSXPoac ST44BEGr6I/6UiqRdnVRACjrTiPuRt0c8pdFag25R+AzoLQ4h2FHS4UQzhZctskucQqAzsw2 TehgNfBFTFp9rqPRhq19KqQrD60ETgYKykFfyBsZRAe/9DprYU3jxTOZtVuCqi4ipvyAz6Y6 y6NqC8ylrACl/kB3q+w/U3Eqz+0r52PRQkwji3dU3i59Ap/aMigbpax9FnAxf9aKcCSSVzpl FgJndKPqt8HC56lnTaIBu4KGdmUC+2tNTTdhRtlGckn/jH1p3q7J9gMu3d5OVtjNdsCdXnxe kjPtAhN5ZhVeny3catwZIH3AMMvpUT9KenYujnvRoImSvBMmMWvpkmCuWb4M7jRrXUR
- Ironport-hdrordr: A9a23:W3mhjq6GHDfDA7eAyQPXwVOBI+orL9Y04lQ7vn2ZFiY5TiXIra qTdaogviMc6Ax/ZJjvo6HkBEClewKlyXcT2/hrAV7CZniehILMFu1fBOTZowEIdxeOldK1kJ 0QCZSWa+eAcmSS7/yKhzVQeuxIqLfnzEnrv5a5854Ed3AXV0gK1XYcNu/0KDwVeOEQbqBJaa Z0q/A30QaISDAyVICWF3MFV+/Mq5nik4/nWwcPA1oC5BOVhT2lxbbmG1zAty1uGw9n8PMHyy zoggb57qKsv7WSzQLd7Xba69BzlMH6wtVOKcSQgow+KynqiCyveIN9Mofy9QwdkaWK0hIHgd PMqxAvM4Ba7G7QRHi8pV/X1wzpwF8Vmgrf4G7dpUGmjd3yRTo8BcYEr5leaAHl500pu8w5+L 5X3kqC3qAnQS/orWDY3ZzlRhtqnk27rT4JiugIlUFSVoMYdft4sZEfxkVIC50NdRiKpbzPKN MeQv002cwmMG9zNxvizylSKZ2XLz4O9y69Mwc/Upf/6UkUoJh7p3FotvD30E1wtq7VcKM0l9 gsAp4Y6o2mcfVmHJ6VJN1xNfdfWVa9Ni7kASa1HWnNMp0hFjbkl6PXiY9Fl91CPqZ4h6cPpA ==
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On Tue, Oct 04, 2022 at 10:06:36AM +0200, Jan Beulich wrote:
> On 30.09.2022 16:28, Roger Pau Monné wrote:
> > On Fri, Sep 30, 2022 at 09:50:40AM +0200, Jan Beulich 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.
> >
> > What about dom0? Should it be translated to E820_RESERVED so that
> > dom0 doesn't try to use it either?
>
> I'm afraid I don't understand the questions. Not the least because I
> think "it" can't really mean "dom0" from the earlier sentence.
Sorry, let me try again:
The memory map provided to dom0 will contain E820_ACPI entries for
memory ranges with the EFI_MEMORY_RUNTIME attributes in the EFI memory
map. Is there a risk from dom0 reclaiming such E820_ACPI ranges,
overwriting the data needed for runtime services?
Thanks, Roger.
|
