[<prev] [next>] [day] [month] [year] [list]
Message-Id: <20200622014857.17829-1-gaurav1086@gmail.com>
Date: Sun, 21 Jun 2020 21:48:33 -0400
From: Gaurav Singh <gaurav1086@...il.com>
To: gaurav1086@...il.com, Dave Hansen <dave.hansen@...ux.intel.com>,
Andy Lutomirski <luto@...nel.org>,
Peter Zijlstra <peterz@...radead.org>,
Thomas Gleixner <tglx@...utronix.de>,
Ingo Molnar <mingo@...hat.com>, Borislav Petkov <bp@...en8.de>,
x86@...nel.org (maintainer:X86 ARCHITECTURE (32-BIT AND 64-BIT)),
"H. Peter Anvin" <hpa@...or.com>,
Logan Gunthorpe <logang@...tatee.com>,
Benjamin Thiel <b.thiel@...teo.de>,
linux-kernel@...r.kernel.org (open list:X86 MM)
Subject: [PATCH] [x86] overlap: cleanup redundant logic
In overlap check, same expression is repeated twice.
Remove one of them.
Signed-off-by: Gaurav Singh <gaurav1086@...il.com>
---
arch/x86/mm/pat/set_memory.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/arch/x86/mm/pat/set_memory.c b/arch/x86/mm/pat/set_memory.c
index 77e04304a2a7..7d44b4863ee5 100644
--- a/arch/x86/mm/pat/set_memory.c
+++ b/arch/x86/mm/pat/set_memory.c
@@ -388,8 +388,7 @@ static void cpa_flush(struct cpa_data *data, int cache)
static bool overlaps(unsigned long r1_start, unsigned long r1_end,
unsigned long r2_start, unsigned long r2_end)
{
- return (r1_start <= r2_end && r1_end >= r2_start) ||
- (r2_start <= r1_end && r2_end >= r1_start);
+ return (r1_start <= r2_end && r1_end >= r2_start);
}
#ifdef CONFIG_PCI_BIOS
--
2.17.1
Powered by blists - more mailing lists