From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:46860) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1V0ByA-0001hl-MK for qemu-devel@nongnu.org; Fri, 19 Jul 2013 10:47:47 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1V0By3-0005Pb-NK for qemu-devel@nongnu.org; Fri, 19 Jul 2013 10:47:42 -0400 Received: from relay1.mentorg.com ([192.94.38.131]:37485) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1V0By3-0005PM-HY for qemu-devel@nongnu.org; Fri, 19 Jul 2013 10:47:35 -0400 From: Kwok Cheung Yeung Date: Fri, 19 Jul 2013 07:47:13 -0700 Message-ID: <1374245233-14732-1-git-send-email-kcy@codesourcery.com> MIME-Version: 1.0 Content-Type: text/plain Subject: [Qemu-devel] [PATCH v2] linux-user: Handle microMIPS encoding when processing trap exceptions List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: peter.maydell@linaro.org, qemu-devel@nongnu.org Cc: Kwok Cheung Yeung , riku.voipio@iki.fi, aurelien@aurel32.net Decode trap instructions during the handling of an EXCP_TRAP according to the current ISA mode. Signed-off-by: Kwok Cheung Yeung --- linux-user/main.c | 20 ++++++++++++++++++-- 1 file changed, 18 insertions(+), 2 deletions(-) v2: Read microMIPS instructions sequentially as 16-bit values to avoid endianess issues. Add braces to if statement to conform to formatting standards. diff --git a/linux-user/main.c b/linux-user/main.c index 7f15d3d..7faa945 100644 --- a/linux-user/main.c +++ b/linux-user/main.c @@ -2372,14 +2372,30 @@ done_syscall: abi_ulong trap_instr; unsigned int code = 0; - ret = get_user_ual(trap_instr, env->active_tc.PC); + if (env->hflags & MIPS_HFLAG_M16) { + /* microMIPS mode */ + abi_ulong instr[2]; + + ret = get_user_u16(instr[0], env->active_tc.PC) || + get_user_u16(instr[1], env->active_tc.PC + 2); + + trap_instr = (instr[0] << 16) | instr[1]; + } else { + ret = get_user_ual(trap_instr, env->active_tc.PC); + } + if (ret != 0) { goto error; } /* The immediate versions don't provide a code. */ if (!(trap_instr & 0xFC000000)) { - code = ((trap_instr >> 6) & ((1 << 10) - 1)); + if (env->hflags & MIPS_HFLAG_M16) { + /* microMIPS mode */ + code = ((trap_instr >> 12) & ((1 << 4) - 1)); + } else { + code = ((trap_instr >> 6) & ((1 << 10) - 1)); + } } if (do_break(env, &info, code) != 0) { -- 1.8.3.3