• Maurice Petallo's avatar
    thermal: Intel SoC DTS: Don't do thermal zone update inside spin_lock · 05629296
    Maurice Petallo authored
    The driver calls spin_lock_irqsave during DTS interrupt. The interrupt
    handle then calls thermal_zone_device_update which implicitly calls
    a sleep function and produce the following bug:
    
    BUG: sleeping function called from invalid context at kernel/locking/mutex.c:97
    in_atomic(): 1, irqs_disabled(): 1, pid: 920, name: irq/86-soc_dts
    CPU: 0 PID: 920 Comm: irq/86-soc_dts Tainted: G            E  3.17.0-rc2+ #1
    Hardware name: Intel Corp. VALLEYVIEW B3 PLATFORM/NOTEBOOK, BIOS BYTICRB1.86C.0092.R31.1408290850 08/29/2014
     00000000 00000000 c25dbe74 c1818cfd f3cc488c c25dbe9c c1059305 c1b4063b
     00000001 00000001 00000398 f3cc488c f6817644 f6817644 f3ecc6c0 c25dbea8
     c18208f2 f6817400 c25dbebc c159b0bb c25dbedc f6817400 f32a2300 c25dbee8
    Call Trace:
     [<c1818cfd>] dump_stack+0x48/0x60
     [<c1059305>] __might_sleep+0xec/0xf4
     [<c18208f2>] mutex_lock+0x1c/0x34
     [<c159b0bb>] thermal_zone_get_temp+0x34/0x59
     [<c159bde5>] thermal_zone_device_update+0x2d/0xcb
     [<f85da16a>] ? iosf_mbi_write+0x6c/0x74 [iosf_mbi]
     [<f7c7445d>] soc_irq_thread_fn+0x10c/0x163 [intel_soc_dts_thermal]
     [<c107b72b>] irq_thread_fn+0x18/0x2a
     [<c107bedb>] irq_thread+0x81/0x11f
     [<c107b713>] ? irq_finalize_oneshot+0x7c/0x7c
     [<c107bf79>] ? irq_thread+0x11f/0x11f
     [<c107be5a>] ? wake_threads_waitq+0x31/0x31
     [<c1054217>] kthread+0x87/0x8c
     [<c1821e41>] ret_from_kernel_thread+0x21/0x30
     [<c1054190>] ? __kthread_parkme+0x55/0x55
    Signed-off-by: default avatarMaurice Petallo <mauricex.r.petallo@intel.com>
    Acked-by: default avatarSrinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
    Acked-by: default avatarEduardo Valentin <edubezval@gmail.com>
    CC: Kweh, Hock Leong <hock.leong.kweh@intel.com>
    Signed-off-by: default avatarZhang Rui <rui.zhang@intel.com>
    05629296
intel_soc_dts_thermal.c 11 KB