This is a note to let you know that I've just added the patch titled
Subject: usb: cdc-wdm: Fix deadlock between write and resume
to my gregkh-2.6 tree. Its filename is
usb-cdc-wdm-fix-deadlock-between-write-and-resume.patch
This tree can be found at
http://www.kernel.org/pub/linux/kernel/people/gregkh/gregkh-2.6/patches/
>From oliver@xxxxxxxxxx Mon Mar 15 16:15:34 2010
From: Oliver Neukum <oliver@xxxxxxxxxx>
Date: Sat, 27 Feb 2010 20:57:12 +0100
Subject: usb: cdc-wdm: Fix deadlock between write and resume
To: Greg KH <greg@xxxxxxxxx>
Cc: Rickard Bellini <rickard.bellini@xxxxxxxxxxxx>, "linux-usb@xxxxxxxxxxxxxxx" <linux-usb@xxxxxxxxxxxxxxx>
Message-ID: <201002272057.12494.oliver@xxxxxxxxxx>
From: Oliver Neukum <oliver@xxxxxxxxxx>
The new runtime PM scheme allows resume() to have no locks.
This fixes the deadlock.
Signed-off-by: Oliver Neukum <neukum@xxxxxxxxxxxxx>
Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxx>
---
drivers/usb/class/cdc-wdm.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
--- a/drivers/usb/class/cdc-wdm.c
+++ b/drivers/usb/class/cdc-wdm.c
@@ -839,10 +839,10 @@ static int wdm_resume(struct usb_interfa
int rv;
dev_dbg(&desc->intf->dev, "wdm%d_resume\n", intf->minor);
- mutex_lock(&desc->lock);
+
clear_bit(WDM_SUSPENDING, &desc->flags);
rv = recover_from_urb_loss(desc);
- mutex_unlock(&desc->lock);
+
return rv;
}
#endif
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
