[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 2/2] xen/arm: Skip loops in init_pdx() when no PDX compression is used
- To: <xen-devel@xxxxxxxxxxxxxxxxxxxx>
- From: Michal Orzel <michal.orzel@xxxxxxx>
- Date: Fri, 4 Jul 2025 09:54:28 +0200
- 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 (0)
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; 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=i0IBfdC6Tm1Qv/CJzlHXWpGpzVGi7GarsMLS4AsB0Ws=; b=ElrQSfzre4Taz1GCc9I6ACDVnxJfqFKw45+qQ2HTW6gHR5cDzBJI5s7eZFQmEX9C1tspsInMmrer6kk2vkURca9CdFewiEyQRJgBkJjY4VhqCFDaeRDp5IZkTvNs2mUpy7Dj66R/e9esdo/nryLbCFEvLStgwIhl4I9wg4dJouY6Fuq1iMdkZpCPh82fHocQ6yEQPTLMQx3uPuSqHS5bAsuSzbNg5GQ4DKbmFemDOP6s22HWuobZOfBzxGh7OLuQR8LYJJGltNdOaHRC4PDnuSyBMtjCoisHgDsvwn6txlIo6QCTXRRS+7h3xqjFd8O9NqrSSFDlzI4MKseVXeDVog==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=QRVnDUpDE8Mtk7kkD5w2Ok4Mn74RFYaCYMahDsAC7em7AxkCFOppdzyDCr6W/yY+xOFKi2d8p71HWofTfq1yHyxByHZsAIts4TFMeIxs6Z9pfAvrq4YK/V3iKOPoLwEHlbg33fawQTasnZHEZMtyAOA8gssZ2PNjp4Wc7KxIWhDMpRdswHK5g8IV602AktncWyOWbDTt5uWVMorHJHKUsg7KTkvivr1r3Vrti11gnwFoKN5sIhJZcp42Y0QuSrs68+64fvOt1U+LeFi1N4rbPqx4LM9i14E/WaOp+VHonHFsa2uyyB4aXmszgdyFghZRLX5eUQjN3v+P9FTJD8cGnQ==
- Cc: Michal Orzel <michal.orzel@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Bertrand Marquis <bertrand.marquis@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>
- Delivery-date: Fri, 04 Jul 2025 07:54:42 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
When CONFIG_PDX_COMPRESSION=n, pdx_init_mask(), pdx_region_mask() and
pfn_pdx_hole_setup() are just stubs doing nothing. It does not make
sense to keep the two loops iterating over all the memory banks.
Signed-off-by: Michal Orzel <michal.orzel@xxxxxxx>
---
xen/arch/arm/setup.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/xen/arch/arm/setup.c b/xen/arch/arm/setup.c
index 93b730ffb5fb..12b76a0a9837 100644
--- a/xen/arch/arm/setup.c
+++ b/xen/arch/arm/setup.c
@@ -255,7 +255,9 @@ void __init init_pdx(void)
{
const struct membanks *mem = bootinfo_get_mem();
paddr_t bank_start, bank_size, bank_end, ram_end = 0;
+ int bank;
+#ifdef CONFIG_PDX_COMPRESSION
/*
* Arm does not have any restrictions on the bits to compress. Pass 0 to
* let the common code further restrict the mask.
@@ -264,7 +266,6 @@ void __init init_pdx(void)
* update this function too.
*/
uint64_t mask = pdx_init_mask(0x0);
- int bank;
for ( bank = 0 ; bank < mem->nr_banks; bank++ )
{
@@ -284,6 +285,7 @@ void __init init_pdx(void)
}
pfn_pdx_hole_setup(mask >> PAGE_SHIFT);
+#endif
for ( bank = 0 ; bank < mem->nr_banks; bank++ )
{
--
2.25.1
|