From: Pau Cabre <pau.cabre@metempsy.com>
Date: Fri, 1 Sep 2017 12:06:33 +0000 (+0200)
Subject: cpu-minor: Fix for addr range coverage calculation
X-Git-Tag: v19.0.0.0~2640
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=29d7c72b89949199653ab05f5bbb298c733b08b7;p=gem5.git

cpu-minor: Fix for addr range coverage calculation

Coverage was wrongly set to PartialAddrRangeCoverage in the case of
disjoint adjacent ranges

Change-Id: I29aaf5145e6cdcf5f0b8f4e009d57ee57bd4c944
Signed-off-by: Pau Cabre <pau.cabre@metempsy.com>
Reviewed-on: https://gem5-review.googlesource.com/4640
Reviewed-by: Andreas Sandberg <andreas.sandberg@arm.com>
Maintainer: Andreas Sandberg <andreas.sandberg@arm.com>
---

diff --git a/src/cpu/minor/lsq.cc b/src/cpu/minor/lsq.cc
index bf2612071..b7d5360ac 100644
--- a/src/cpu/minor/lsq.cc
+++ b/src/cpu/minor/lsq.cc
@@ -97,7 +97,7 @@ LSQ::LSQRequest::containsAddrRangeOf(
 
     AddrRangeCoverage ret;
 
-    if (req1_addr > req2_end_addr || req1_end_addr < req2_addr)
+    if (req1_addr >= req2_end_addr || req1_end_addr <= req2_addr)
         ret = NoAddrRangeCoverage;
     else if (req1_addr <= req2_addr && req1_end_addr >= req2_end_addr)
         ret = FullAddrRangeCoverage;