From: Matt Redfearn <matt.redfearn@mips.com>
To: Ralf Baechle <ralf@linux-mips.org>, James Hogan <jhogan@kernel.org>
Cc: <linux-mips@linux-mips.org>,
Matt Redfearn <matt.redfearn@mips.com>,
"Paul Burton" <paul.burton@mips.com>,
James Hogan <james.hogan@mips.com>,
"stable # v4 . 9+" <stable@vger.kernel.org>,
Huacai Chen <chenhc@lemote.com>, <linux-kernel@vger.kernel.org>
Subject: [PATCH 3/3] MIPS: Add barrier between icache flush and execution hazard barrier
Date: Thu, 21 Dec 2017 11:16:04 +0000 [thread overview]
Message-ID: <1513854965-3880-3-git-send-email-matt.redfearn@mips.com> (raw)
In-Reply-To: <1513854965-3880-1-git-send-email-matt.redfearn@mips.com>
Hit-based icache operations may complete before the CM completes
intervention with the local L1. Thus code which invalidates the icache
and then attempts to execute those addresses must include a barrier to
prevent the scenario which:
- icache instruction completes
- icache fetch occurs
- core executes icache data
- CM completes icache invalidate
If the above were allowed to happen then the core would execute stale
instructions from the icache.
A barrier is required to prevent the core i-fetching before the icache
operation has completed. This goes together with the instruction_hazard
to ensure that the pipeline is stalled until the icache operation is
completed and the core will fetch the new instructions.
Suggested-by: Leonid Yegoshin <Leonid.Yegoshin@mips.com>
Signed-off-by: Matt Redfearn <matt.redfearn@mips.com>
Cc: Paul Burton <paul.burton@mips.com>
Cc: James Hogan <james.hogan@mips.com>
Cc: stable <stable@vger.kernel.org> # v4.9+
---
arch/mips/mm/c-r4k.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/mips/mm/c-r4k.c b/arch/mips/mm/c-r4k.c
index b7186d47184b..844685e51109 100644
--- a/arch/mips/mm/c-r4k.c
+++ b/arch/mips/mm/c-r4k.c
@@ -763,6 +763,8 @@ static inline void __local_r4k_flush_icache_range(unsigned long start,
break;
}
}
+ /* Ensure icache operation has completed */
+ mb();
/* Hazard to force new i-fetch */
instruction_hazard();
}
--
2.7.4
next prev parent reply other threads:[~2017-12-21 11:17 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-21 11:16 [PATCH 1/3] MIPS: c-r4k: instruction_hazard should immediately follow cache op Matt Redfearn
2017-12-21 11:16 ` [PATCH 2/3] MIPS: Add barrier between dcache & icache flushes Matt Redfearn
2017-12-21 14:58 ` James Hogan
2017-12-21 11:16 ` Matt Redfearn [this message]
2017-12-21 15:14 ` [PATCH 1/3] MIPS: c-r4k: instruction_hazard should immediately follow cache op James Hogan
2017-12-21 15:19 ` Matt Redfearn
2017-12-21 15:30 ` James Hogan
2017-12-21 16:59 ` Matt Redfearn
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1513854965-3880-3-git-send-email-matt.redfearn@mips.com \
--to=matt.redfearn@mips.com \
--cc=chenhc@lemote.com \
--cc=james.hogan@mips.com \
--cc=jhogan@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mips@linux-mips.org \
--cc=paul.burton@mips.com \
--cc=ralf@linux-mips.org \
--cc=stable@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®