BigW Consortium Gitlab

Commit d10c7e44 by Shikai Chen

fixed a potential crash bug caused by invoking mutex in interrupt context

parent 0b15385e
...@@ -241,13 +241,9 @@ static void _on_display_transfer_finished(struct urb *urb) ...@@ -241,13 +241,9 @@ static void _on_display_transfer_finished(struct urb *urb)
static void _on_status_query_finished(struct urb *urb) static void _on_status_query_finished(struct urb *urb)
{ {
struct rpusbdisp_dev *dev = urb->context; struct rpusbdisp_dev *dev = urb->context;
mutex_lock(&dev->op_locker);
if (!dev->is_alive) { if (!dev->is_alive) {
mutex_unlock(&dev->op_locker);
return; return;
} }
...@@ -273,8 +269,6 @@ static void _on_status_query_finished(struct urb *urb) ...@@ -273,8 +269,6 @@ static void _on_status_query_finished(struct urb *urb)
if (dev->urb_status_fail_count < RPUSBDISP_STATUS_QUERY_RETRY_COUNT) { if (dev->urb_status_fail_count < RPUSBDISP_STATUS_QUERY_RETRY_COUNT) {
_status_start_querying(dev); _status_start_querying(dev);
} }
mutex_unlock(&dev->op_locker);
} }
......
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