From: David Daney <ddaney.cavm@gmail.com>
To: linux-mips@linux-mips.org, ralf@linux-mips.org
Cc: Leonid Yegoshin <Leonid.Yegoshin@imgtec.com>,
Zubair.Kakakhel@imgtec.com, geert+renesas@glider.be,
peterz@infradead.org, paul.gortmaker@windriver.com,
macro@linux-mips.org, chenhc@lemote.com, cl@linux.com,
mingo@kernel.org, richard@nod.at, zajec5@gmail.com,
james.hogan@imgtec.com, keescook@chromium.org, tj@kernel.org,
alex@alex-smith.me.uk, pbonzini@redhat.com, blogic@openwrt.org,
paul.burton@imgtec.com, qais.yousef@imgtec.com,
linux-kernel@vger.kernel.org, markos.chandras@imgtec.com,
dengcheng.zhu@imgtec.com, manuel.lauss@gmail.com,
lars.persson@axis.com, David Daney <david.daney@cavium.com>
Subject: [PATCH 3/3] MIPS: Use full instruction emulation for FPU emulator delay slot emulation.
Date: Wed, 3 Dec 2014 15:44:18 -0800 [thread overview]
Message-ID: <1417650258-2811-4-git-send-email-ddaney.cavm@gmail.com> (raw)
In-Reply-To: <1417650258-2811-1-git-send-email-ddaney.cavm@gmail.com>
From: David Daney <david.daney@cavium.com>
Current delay slot handling does eXecute Out of Line (XOL) on the
stack, which prevents a non-executable stack. Use the instruction
emulator instead.
Tested by booting 32-bit Debian on OCTEON. More than 1700
instructions emulated to login to command line.
Signed-off-by: David Daney <david.daney@cavium.com>
---
arch/mips/math-emu/cp1emu.c | 13 +++++++++++--
1 file changed, 11 insertions(+), 2 deletions(-)
diff --git a/arch/mips/math-emu/cp1emu.c b/arch/mips/math-emu/cp1emu.c
index cac529a..787de7a 100644
--- a/arch/mips/math-emu/cp1emu.c
+++ b/arch/mips/math-emu/cp1emu.c
@@ -694,11 +694,12 @@ do { \
* Emulate the single floating point instruction pointed at by EPC.
* Two instructions if the instruction is in a branch delay slot.
*/
-
+int mips_insn_emul(struct pt_regs *regs, mips_instruction ir, void *__user *fault_addr);
static int cop1Emulate(struct pt_regs *xcp, struct mips_fpu_struct *ctx,
struct mm_decoded_insn dec_insn, void *__user *fault_addr)
{
unsigned long contpc = xcp->cp0_epc + dec_insn.pc_inc;
+ unsigned long origpc = xcp->cp0_epc;
unsigned int cond, cbit;
mips_instruction ir;
int likely, pc_inc;
@@ -1038,7 +1039,15 @@ emul:
* Single step the non-cp1
* instruction in the dslot
*/
- return mips_dsemul(xcp, ir, contpc);
+ sig = mips_insn_emul(xcp, ir, fault_addr);
+ if (sig == 0) {
+ xcp->cp0_epc = contpc;
+ MIPS_FPU_EMU_INC_STATS(insn_emul);
+ } else {
+ xcp->cp0_epc = origpc;
+ pr_err("mips_insn_emul: %08x ->%d\n", (unsigned)ir, sig);
+ }
+ return sig;
} else if (likely) { /* branch not taken */
/*
* branch likely nullifies
--
1.7.11.7
prev parent reply other threads:[~2014-12-03 23:44 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-12-03 23:44 [PATCH 0/3] MIPS: Get ready for non-executable stack David Daney
2014-12-03 23:44 ` [PATCH 1/3] MIPS: Add FPU emulator counter for non-FPU instructions emulated David Daney
2014-12-03 23:44 ` [PATCH 2/3] MIPS: Add full ISA emulator David Daney
2014-12-03 23:55 ` Leonid Yegoshin
2014-12-04 0:20 ` David Daney
2014-12-04 0:52 ` Leonid Yegoshin
2014-12-04 1:29 ` David Daney
[not found] ` <547FBF63.70802@imgtec.com>
2014-12-04 2:21 ` David Daney
2014-12-04 10:16 ` Paul Burton
2014-12-04 10:45 ` Qais Yousef
2014-12-04 11:49 ` Maciej W. Rozycki
2014-12-04 17:40 ` David Daney
2014-12-04 20:32 ` Maciej W. Rozycki
2014-12-03 23:44 ` David Daney [this message]
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=1417650258-2811-4-git-send-email-ddaney.cavm@gmail.com \
--to=ddaney.cavm@gmail.com \
--cc=Leonid.Yegoshin@imgtec.com \
--cc=Zubair.Kakakhel@imgtec.com \
--cc=alex@alex-smith.me.uk \
--cc=blogic@openwrt.org \
--cc=chenhc@lemote.com \
--cc=cl@linux.com \
--cc=david.daney@cavium.com \
--cc=dengcheng.zhu@imgtec.com \
--cc=geert+renesas@glider.be \
--cc=james.hogan@imgtec.com \
--cc=keescook@chromium.org \
--cc=lars.persson@axis.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mips@linux-mips.org \
--cc=macro@linux-mips.org \
--cc=manuel.lauss@gmail.com \
--cc=markos.chandras@imgtec.com \
--cc=mingo@kernel.org \
--cc=paul.burton@imgtec.com \
--cc=paul.gortmaker@windriver.com \
--cc=pbonzini@redhat.com \
--cc=peterz@infradead.org \
--cc=qais.yousef@imgtec.com \
--cc=ralf@linux-mips.org \
--cc=richard@nod.at \
--cc=tj@kernel.org \
--cc=zajec5@gmail.com \
/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