[PATCH 1/6][SRU][Jammy] x86/PCI: Eliminate remove_e820_regions() common subexpressions

AceLan Kao acelan.kao at canonical.com
Mon Aug 1 13:38:26 UTC 2022


From: Bjorn Helgaas <bhelgaas at google.com>

BugLink: https://bugs.launchpad.net/bugs/1884232

Add local variables to reduce repetition later.  No functional change
intended.

Link: https://lore.kernel.org/r/20220304035110.988712-2-helgaas@kernel.org
Signed-off-by: Bjorn Helgaas <bhelgaas at google.com>
Reviewed-by: Hans de Goede <hdegoede at redhat.com>
Reviewed-by: Mika Westerberg <mika.westerberg at linux.intel.com>
Acked-by: Rafael J. Wysocki <rafael.j.wysocki at intel.com>
(cherry picked from commit 93d256cd3c1e93c4093e8015b371e832de4c4146)
Signed-off-by: Chia-Lin Kao (AceLan) <acelan.kao at canonical.com>
---
 arch/x86/kernel/resource.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/arch/x86/kernel/resource.c b/arch/x86/kernel/resource.c
index 9b9fb7882c20..8ffe68437744 100644
--- a/arch/x86/kernel/resource.c
+++ b/arch/x86/kernel/resource.c
@@ -27,12 +27,14 @@ static void remove_e820_regions(struct resource *avail)
 {
 	int i;
 	struct e820_entry *entry;
+	u64 e820_start, e820_end;
 
 	for (i = 0; i < e820_table->nr_entries; i++) {
 		entry = &e820_table->entries[i];
+		e820_start = entry->addr;
+		e820_end = entry->addr + entry->size - 1;
 
-		resource_clip(avail, entry->addr,
-			      entry->addr + entry->size - 1);
+		resource_clip(avail, e820_start, e820_end);
 	}
 }
 
-- 
2.25.1




More information about the kernel-team mailing list