Commit cdfd2c5c authored by Ivo van Doorn's avatar Ivo van Doorn Committed by John W. Linville

rt2x00: Move watchdog work to kernel work_queue

The watchdog function must run on a work_queue
which is independent of any other work inside rt2x00.

The main reasons, being that a broken work on the mac80211
work_queue can otherwise prevent the watchdog to run (while
in fact the watchdog could fix the issue). And on the other
hand because the watchdog relies on the completion of the
completion handlers for RX/TX which for the USB case, occur
on the mac80211 workqueue.

This fixes some "Queue %d failed to flush" errors, which were
caused by the watchdog function waiting on the completion
handler which was scheduled to run right after the watchdog.
Signed-off-by: default avatarIvo van Doorn <IvDoorn@gmail.com>
Acked-by: default avatarHelmut Schaa <helmut.schaa@googlemail.com>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent fa69560f
...@@ -338,6 +338,11 @@ struct link { ...@@ -338,6 +338,11 @@ struct link {
/* /*
* Work structure for scheduling periodic watchdog monitoring. * Work structure for scheduling periodic watchdog monitoring.
* This work must be scheduled on the kernel workqueue, while
* all other work structures must be queued on the mac80211
* workqueue. This guarantees that the watchdog can schedule
* other work structures and wait for their completion in order
* to bring the device/driver back into the desired state.
*/ */
struct delayed_work watchdog_work; struct delayed_work watchdog_work;
}; };
......
...@@ -417,8 +417,7 @@ void rt2x00link_start_watchdog(struct rt2x00_dev *rt2x00dev) ...@@ -417,8 +417,7 @@ void rt2x00link_start_watchdog(struct rt2x00_dev *rt2x00dev)
!test_bit(DRIVER_SUPPORT_WATCHDOG, &rt2x00dev->flags)) !test_bit(DRIVER_SUPPORT_WATCHDOG, &rt2x00dev->flags))
return; return;
ieee80211_queue_delayed_work(rt2x00dev->hw, schedule_delayed_work(&link->watchdog_work, WATCHDOG_INTERVAL);
&link->watchdog_work, WATCHDOG_INTERVAL);
} }
void rt2x00link_stop_watchdog(struct rt2x00_dev *rt2x00dev) void rt2x00link_stop_watchdog(struct rt2x00_dev *rt2x00dev)
...@@ -442,8 +441,7 @@ static void rt2x00link_watchdog(struct work_struct *work) ...@@ -442,8 +441,7 @@ static void rt2x00link_watchdog(struct work_struct *work)
rt2x00dev->ops->lib->watchdog(rt2x00dev); rt2x00dev->ops->lib->watchdog(rt2x00dev);
if (test_bit(DEVICE_STATE_PRESENT, &rt2x00dev->flags)) if (test_bit(DEVICE_STATE_PRESENT, &rt2x00dev->flags))
ieee80211_queue_delayed_work(rt2x00dev->hw, schedule_delayed_work(&link->watchdog_work, WATCHDOG_INTERVAL);
&link->watchdog_work, WATCHDOG_INTERVAL);
} }
void rt2x00link_register(struct rt2x00_dev *rt2x00dev) void rt2x00link_register(struct rt2x00_dev *rt2x00dev)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment