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

[XEN v2 08/11] xen/arm: guest_walk: LPAE specific bits should be enclosed within "ifndef CONFIG_ARM_PA_32"


  • To: <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Ayan Kumar Halder <ayan.kumar.halder@xxxxxxx>
  • Date: Tue, 17 Jan 2023 17:43:55 +0000
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=lists.xenproject.org smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); 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=mTuCjvySdNRIh7I98ybx4BVBMGxho8OVLY6od2Qthjw=; b=hFDq7byjPtDsdp+YU25UvxF9NcSn6X60FTXX1bSR9p40/rA4MZ1tHWoAF51KiaBgKlxwHIvIra0yTzyEunp+LGeelTfqsgWhCg51kb52Ka//99Sf1jaUccrBDB0jDPw9vTL+CE8MaPNJoSmZ1H9Gv42e1G+YpIFfMiH2vK6ji7/UOVNFUZIGNcM6D0GQ3Wd11+c/AX9IsfkE//oD4YhK2MrlIhgzhn3LqbrIhAMJtqkXjE0q0AH3jpS4UBvwsODj6nzwryOptY6QD/Bds+zZ0QULYhAGLDUz6W2eM/F7KYJRID7YIxs1/Ov4P6+2Zl2nZsosnuipUvpQ2XTkasAHUQ==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=aKFcRBIoSL8hFvCF6230um6e8PhO6Yx8sUL5w75R809q0lwztfKZmhqJ4jJy6sXPxox5ncT9X2ONi6Kj7rUivZ5twsYJW7BRkBo4GZ0+HLA3yrL7SMmwIrBUAX0rGGviZMl0pDLCwRsAEKkzTmxm0kHFTtGHS5+NbiKZLPNPqFj0T76IYjH+mi4b5FFhiaAfxiMFEJJY2qxb48Sj7GUL4UFkynFdImZy8LgDIxXAG+gnTRo41sZ6kVS6QBj6miRy8ny9MXg3Ul0CHaV739Gd4ep51Gglkj5lKpN5vnsyy8CdLCoISGI8ChVVwTcX8uKzrNCC5NaJ1XlKbxjuyaxV0w==
  • Cc: <sstabellini@xxxxxxxxxx>, <stefano.stabellini@xxxxxxx>, <julien@xxxxxxx>, <Volodymyr_Babchuk@xxxxxxxx>, <bertrand.marquis@xxxxxxx>, Ayan Kumar Halder <ayan.kumar.halder@xxxxxxx>
  • Delivery-date: Tue, 17 Jan 2023 17:45:45 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

In the subsequent patch, we introduce "CONFIG_ARM_PA_32" to support
32 bit physical addresses. Thus, the code specific to
"Large Page Address Extension" (ie LPAE) should be enclosed within
"ifndef CONFIG_ARM_PA_32".

Refer xen/arch/arm/include/asm/short-desc.h, "short_desc_l1_supersec_t"
unsigned int extbase1:4;    /* Extended base address, PA[35:32] */
unsigned int extbase2:4;    /* Extended base address, PA[39:36] */

Thus, extbase1 and extbase2 are not valid when 32 bit physical addresses
are supported.

Signed-off-by: Ayan Kumar Halder <ayan.kumar.halder@xxxxxxx>
---

Changes from -
v1 - 1. Extracted from "[XEN v1 8/9] xen/arm: Other adaptations required to 
support 32bit paddr".

 xen/arch/arm/guest_walk.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/xen/arch/arm/guest_walk.c b/xen/arch/arm/guest_walk.c
index 43d3215304..0feb7b76ec 100644
--- a/xen/arch/arm/guest_walk.c
+++ b/xen/arch/arm/guest_walk.c
@@ -154,8 +154,10 @@ static bool guest_walk_sd(const struct vcpu *v,
             mask = (1ULL << L1DESC_SUPERSECTION_SHIFT) - 1;
             *ipa = gva & mask;
             *ipa |= (paddr_t)(pte.supersec.base) << L1DESC_SUPERSECTION_SHIFT;
+#ifndef CONFIG_ARM_PA_32
             *ipa |= (paddr_t)(pte.supersec.extbase1) << 
L1DESC_SUPERSECTION_EXT_BASE1_SHIFT;
             *ipa |= (paddr_t)(pte.supersec.extbase2) << 
L1DESC_SUPERSECTION_EXT_BASE2_SHIFT;
+#endif /* CONFIG_ARM_PA_32 */
         }
 
         /* Set permissions so that the caller can check the flags by herself. 
*/
-- 
2.17.1




 


Rackspace

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