lkml.org 
[lkml]   [2014]   [Oct]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 3.16 097/127] sparc64: Do not disable interrupts in nmi_cpu_busy()
    Date
    3.16-stable review patch.  If anyone has any objections, please let me know.

    ------------------

    From: "David S. Miller" <davem@davemloft.net>

    [ Upstream commit 58556104e9cd0107a7a8d2692cf04ef31669f6e4 ]

    nmi_cpu_busy() is a SMP function call that just makes sure that all of the
    cpus are spinning using cpu cycles while the NMI test runs.

    It does not need to disable IRQs because we just care about NMIs executing
    which will even with 'normal' IRQs disabled.

    It is not legal to enable hard IRQs in a SMP cross call, in fact this bug
    triggers the BUG check in irq_work_run_list():

    BUG_ON(!irqs_disabled());

    Because now irq_work_run() is invoked from the tail of
    generic_smp_call_function_single_interrupt().

    Signed-off-by: David S. Miller <davem@davemloft.net>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
    ---
    arch/sparc/kernel/nmi.c | 1 -
    1 file changed, 1 deletion(-)

    --- a/arch/sparc/kernel/nmi.c
    +++ b/arch/sparc/kernel/nmi.c
    @@ -130,7 +130,6 @@ static inline unsigned int get_nmi_count

    static __init void nmi_cpu_busy(void *data)
    {
    - local_irq_enable_in_hardirq();
    while (endflag == 0)
    mb();
    }



    \
     
     \ /
      Last update: 2014-10-28 08:41    [W:3.163 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site