lkml.org 
[lkml]   [2012]   [Feb]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH] x86-64: Fix CFI data for common_interrupt
    Date
    Commit eab9e6 "x86-64: Fix CFI data for interrupt frames" introduced
    a DW_CFA_def_cfa_expression in the SAVE_ARGS_IRQ macro. To later define
    the CFA using a simple register+offset rule both register and offset
    need to be supplied. Just using CFI_DEF_CFA_REGISTER leaves the offset
    undefined. So use CFI_DEF_CFA with reg+off explicitly at the end of
    common_interrupt.

    Signed-off-by: Mark Wielaard <mjw@redhat.com>
    ---
    arch/x86/kernel/entry_64.S | 2 +-
    1 files changed, 1 insertions(+), 1 deletions(-)

    diff --git a/arch/x86/kernel/entry_64.S b/arch/x86/kernel/entry_64.S
    index 3fe8239..e00ef55 100644
    --- a/arch/x86/kernel/entry_64.S
    +++ b/arch/x86/kernel/entry_64.S
    @@ -813,7 +813,7 @@ ret_from_intr:

    /* Restore saved previous stack */
    popq %rsi
    - CFI_DEF_CFA_REGISTER rsi
    + CFI_DEF_CFA rsi,0 /* needed after def_cfa_expression */
    leaq ARGOFFSET-RBP(%rsi), %rsp
    CFI_DEF_CFA_REGISTER rsp
    CFI_ADJUST_CFA_OFFSET RBP-ARGOFFSET
    --
    1.7.7.6


    \
     
     \ /
      Last update: 2012-02-21 15:41    [W:0.027 / U:0.104 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site