On Fri, 2009-05-15 at 13:18 +0200, Christian Lamparter wrote:
> > Don't cancel a work under a lock that it needs!
> >
> > johannes
> >
> do you have the module this time?
Yes, why? The bug is trivial:
mutex_lock(&ar->mutex);
cancel_delayed_work_sync(&ar->tx_status_janitor);
cancel_work_sync(&ar->filter_config_work);
cancel_work_sync(&ar->beacon_work);
vs.
static void ar9170_tx_status_janitor(struct work_struct *work)
{
...
mutex_lock(&ar->mutex);
johannes
Attachment:
signature.asc
Description: This is a digitally signed message part
