[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