lkml.org 
[lkml]   [2017]   [Mar]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH v2 2/5] MIPS: microMIPS: Fix decoding of addiusp instruction
    Date
    Commit 34c2f668d0f6 ("MIPS: microMIPS: Add unaligned access support.")
    added handling of microMIPS instructions to manipulate the stack
    pointer. Unfortunately the decoding of the addiusp instruction was
    incorrect, and performed a left shift by 2 bits to the raw immediate,
    rather than decoding the immediate and then performing the shift, as
    documented in the ISA.

    This led to incomplete stack traces, due to incorrect frame sizes being
    calculated. For example the instruction:
    801faee0 <do_sys_poll>:
    801faee0: 4e25 addiu sp,sp,-952

    As decoded by objdump, would be interpreted by the existing code as
    having manipulated the stack pointer by +1096.

    Fix this by changing the order of decoding the immediate and applying
    the left shift.

    Fixes: 34c2f668d0f6 ("MIPS: microMIPS: Add unaligned access support.")
    Signed-off-by: Matt Redfearn <matt.redfearn@imgtec.com>

    ---

    Changes in v2:
    - Replace conditional with xor and subtract

    arch/mips/kernel/process.c | 5 +++--
    1 file changed, 3 insertions(+), 2 deletions(-)

    diff --git a/arch/mips/kernel/process.c b/arch/mips/kernel/process.c
    index df69ebd361fc..799273d45d21 100644
    --- a/arch/mips/kernel/process.c
    +++ b/arch/mips/kernel/process.c
    @@ -386,8 +386,9 @@ static int get_frame_info(struct mips_frame_info *info)

    if (ip->halfword[0] & mm_addiusp_func)
    {
    - tmp = (((ip->halfword[0] >> 1) & 0x1ff) << 2);
    - info->frame_size = -(signed short)(tmp | ((tmp & 0x100) ? 0xfe00 : 0));
    + tmp = (ip->halfword[0] >> 1) & 0x1ff;
    + tmp = (tmp ^ 0x100) - 0x100;
    + info->frame_size = -(signed short)(tmp << 2);
    } else {
    tmp = (ip->halfword[0] >> 1);
    info->frame_size = -(signed short)(tmp & 0xf);
    --
    2.7.4
    \
     
     \ /
      Last update: 2017-03-01 16:33    [W:5.542 / U:0.176 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site