]> git.karo-electronics.de Git - karo-tx-linux.git/commit
ARM: SAMSUNG: Use spin_lock_{irqsave,irqrestore} in clk_set_rate
authorTushar Behera <tushar.behera@linaro.org>
Tue, 18 Sep 2012 01:05:34 +0000 (10:05 +0900)
committerKukjin Kim <kgene.kim@samsung.com>
Tue, 18 Sep 2012 01:05:34 +0000 (10:05 +0900)
commitd6838a62b4d36d3e2791cffe155586973b20a381
tree7eda3f38919335cd1e06873259ee2d1b21b30792
parentdbc5e1e89aafd484ad279ec6b9267dcd2c67df3f
ARM: SAMSUNG: Use spin_lock_{irqsave,irqrestore} in clk_set_rate

The spinlock clocks_lock can be held during ISR, hence it is not safe to
hold that lock with disabling interrupts.

It fixes following potential deadlock.

=========================================================
[ INFO: possible irq lock inversion dependency detected ]
3.6.0-rc4+ #2 Not tainted
---------------------------------------------------------
swapper/0/1 just changed the state of lock:
 (&(&host->lock)->rlock){-.....}, at: [<c027fb0d>] sdhci_irq+0x15/0x564
but this lock took another, HARDIRQ-unsafe lock in the past:
 (clocks_lock){+.+...}

and interrupts could create inverse lock ordering between them.

other info that might help us debug this:
 Possible interrupt unsafe locking scenario:

       CPU0                    CPU1
       ----                    ----
  lock(clocks_lock);
                               local_irq_disable();
                               lock(&(&host->lock)->rlock);
                               lock(clocks_lock);
  <Interrupt>
    lock(&(&host->lock)->rlock);

 *** DEADLOCK ***

Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
Signed-off-by: Kukjin Kim <kgene.kim@samsung.com>
arch/arm/plat-samsung/clock.c