arch-arm: Do not use _flushMva for TLBI IPA
authorGiacomo Travaglini <giacomo.travaglini@arm.com>
Wed, 23 Sep 2020 15:05:46 +0000 (16:05 +0100)
committerGiacomo Travaglini <giacomo.travaglini@arm.com>
Tue, 3 Nov 2020 09:55:37 +0000 (09:55 +0000)
This is just a cosmetic change

Change-Id: If9ea1114ed7e20d5c952f401935532cf3335c501
Signed-off-by: Giacomo Travaglini <giacomo.travaglini@arm.com>
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/35246
Reviewed-by: Nikos Nikoleris <nikos.nikoleris@arm.com>
Tested-by: kokoro <noreply+kokoro@google.com>
src/arch/arm/tlb.cc
src/arch/arm/tlbi_op.hh

index 04b5cd409bbe6806232b1ae028ec00cdf8778e94..5d2ed902d2eb1e63cb3b983a8e63e74175e03bf7 100644 (file)
@@ -463,8 +463,9 @@ void
 TLB::flush(const TLBIIPA &tlbi_op)
 {
     assert(!isStage2);
-    stage2Tlb->_flushMva(tlbi_op.addr, 0xbeef, tlbi_op.secureLookup,
-        true, tlbi_op.targetEL, false);
+
+    // Note, TLBIIPA::makeStage2 will generare a TLBIMVAA
+    stage2Tlb->flush(tlbi_op.makeStage2());
 }
 
 void
index cab0e52ae5e6e8ad486420c85e81023fc3f651f1..ce72dfbca3c13fcff2f4c4a8aae4c68d2d363b91 100644 (file)
@@ -292,6 +292,13 @@ class TLBIIPA : public TLBIOp
 
     void operator()(ThreadContext* tc) override;
 
+    /** TLBIIPA is basically a TLBIMVAA for stage2 TLBs */
+    TLBIMVAA
+    makeStage2() const
+    {
+        return TLBIMVAA(EL1, secureLookup, addr);
+    }
+
     Addr addr;
 };