This is a bug, but I sent a patch for it last week.

regards,
dan carpenter

On Tue, Jan 20, 2015 at 02:52:38AM +0800, kbuild test robot wrote:
> TO: Ram Amrani <[email protected]>
> CC: "Kalle Valo <[email protected]> Kalle Valo" <[email protected]>
> CC: Arik Nemtsov <[email protected]>
> CC: Eliad Peller <[email protected]>
> 
> tree:   git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git 
> master
> head:   a0d4287f787889e59db0fd295853a0f1f55d0699
> commit: 6d5a748d4836ddd0ca626fe4870942a0e90a5c3d [1637/3880] wlcore: add 
> ability to reduce FW interrupts during suspend
> :::::: branch date: 11 hours ago
> :::::: commit date: 10 days ago
> 
> drivers/net/wireless/ti/wlcore/main.c:1848 wl1271_op_suspend() warn: 
> inconsistent returns 'mutex:&wl->mutex'.
>   Locked on:   line 1791
>   Unlocked on: line 1782
> 
> git remote add next 
> git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
> git remote update next
> git checkout 6d5a748d4836ddd0ca626fe4870942a0e90a5c3d
> vim +1848 drivers/net/wireless/ti/wlcore/main.c
> 
> f44e58681 drivers/net/wireless/wl12xx/main.c    Eliad Peller   2011-05-13  
> 1832       /*
> f44e58681 drivers/net/wireless/wl12xx/main.c    Eliad Peller   2011-05-13  
> 1833        * set suspended flag to avoid triggering a new threaded_irq
> f44e58681 drivers/net/wireless/wl12xx/main.c    Eliad Peller   2011-05-13  
> 1834        * work. no need for spinlock as interrupts are disabled.
> f44e58681 drivers/net/wireless/wl12xx/main.c    Eliad Peller   2011-05-13  
> 1835        */
> f44e58681 drivers/net/wireless/wl12xx/main.c    Eliad Peller   2011-05-13  
> 1836       set_bit(WL1271_FLAG_SUSPENDED, &wl->flags);
> f44e58681 drivers/net/wireless/wl12xx/main.c    Eliad Peller   2011-05-13  
> 1837  
> dd5512eb6 drivers/net/wireless/ti/wlcore/main.c Luciano Coelho 2012-04-11  
> 1838       wlcore_enable_interrupts(wl);
> f44e58681 drivers/net/wireless/wl12xx/main.c    Eliad Peller   2011-05-13  
> 1839       flush_work(&wl->tx_work);
> f44e58681 drivers/net/wireless/wl12xx/main.c    Eliad Peller   2011-05-13  
> 1840       flush_delayed_work(&wl->elp_work);
> 4a859df85 drivers/net/wireless/wl12xx/main.c    Eliad Peller   2011-06-06  
> 1841  
> 9be86cf06 drivers/net/wireless/ti/wlcore/main.c Arik Nemtsov   2014-02-10  
> 1842       /*
> 9be86cf06 drivers/net/wireless/ti/wlcore/main.c Arik Nemtsov   2014-02-10  
> 1843        * Cancel the watchdog even if above tx_flush failed. We will 
> detect
> 9be86cf06 drivers/net/wireless/ti/wlcore/main.c Arik Nemtsov   2014-02-10  
> 1844        * it on resume anyway.
> 9be86cf06 drivers/net/wireless/ti/wlcore/main.c Arik Nemtsov   2014-02-10  
> 1845        */
> 9be86cf06 drivers/net/wireless/ti/wlcore/main.c Arik Nemtsov   2014-02-10  
> 1846       cancel_delayed_work(&wl->tx_watchdog_work);
> 9be86cf06 drivers/net/wireless/ti/wlcore/main.c Arik Nemtsov   2014-02-10  
> 1847  
> 402e48616 drivers/net/wireless/wl12xx/main.c    Eliad Peller   2011-05-13 
> @1848       return 0;
> 402e48616 drivers/net/wireless/wl12xx/main.c    Eliad Peller   2011-05-13  
> 1849  }
> 402e48616 drivers/net/wireless/wl12xx/main.c    Eliad Peller   2011-05-13  
> 1850  
> 402e48616 drivers/net/wireless/wl12xx/main.c    Eliad Peller   2011-05-13  
> 1851  static int wl1271_op_resume(struct ieee80211_hw *hw)
> 402e48616 drivers/net/wireless/wl12xx/main.c    Eliad Peller   2011-05-13  
> 1852  {
> 402e48616 drivers/net/wireless/wl12xx/main.c    Eliad Peller   2011-05-13  
> 1853       struct wl1271 *wl = hw->priv;
> 6e8cd3310 drivers/net/wireless/wl12xx/main.c    Eliad Peller   2011-10-10  
> 1854       struct wl12xx_vif *wlvif;
> 4a859df85 drivers/net/wireless/wl12xx/main.c    Eliad Peller   2011-06-06  
> 1855       unsigned long flags;
> ea0a3cf95 drivers/net/wireless/ti/wlcore/main.c Arik Nemtsov   2012-06-21  
> 1856       bool run_irq_work = false, pending_recovery;
> 
> :::::: The code at line 1848 was first introduced by commit
> :::::: 402e48616078c1e56f55a69d314b77f1d750d6ad wl12xx: declare 
> suspend/resume callbacks (for wowlan)
> 
> :::::: TO: Eliad Peller <[email protected]>
> :::::: CC: Luciano Coelho <[email protected]>
> 
> ---
> 0-DAY kernel test infrastructure                Open Source Technology Center
> http://lists.01.org/mailman/listinfo/kbuild                 Intel Corporation
_______________________________________________
kbuild mailing list
[email protected]
https://lists.01.org/mailman/listinfo/kbuild

Reply via email to