Hi Jassi I can't comment which is the best solution since I'm not familiar with that part of the kernel code. I just applied Huajuns fix and verified that dummy_hcd was working as expected. Please note that before applying the fix most likely that more tests are needed (besides dummy_hcd being functional). Best regards, Tanya Brokhman Consultant for Qualcomm Innovation Center, Inc. Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum -----Original Message----- From: Jassi Brar [mailto:jassisinghbrar@xxxxxxxxx] Sent: Monday, May 30, 2011 1:51 PM To: Tanya Brokhman Cc: Huajun Li; linux-usb@xxxxxxxxxxxxxxx; greg@xxxxxxxxx; ablay@xxxxxxxxxxxxxx Subject: Re: Potential BUG in v2.6.39 On Mon, May 30, 2011 at 3:37 PM, Tanya Brokhman <tlinder@xxxxxxxxxxxxxx> wrote: > Hi Huajun > > Thanks for the quick reply! > Your patch fixes the issue. I just verified it. How about the following patch... diff --git a/drivers/usb/core/inode.c b/drivers/usb/core/inode.c index 1b125c2..3ef9308 100644 --- a/drivers/usb/core/inode.c +++ b/drivers/usb/core/inode.c @@ -381,7 +381,6 @@ static int usbfs_rmdir(struct inode *dir, struct dentry *dentry) dont_mount(dentry); drop_nlink(dentry->d_inode); drop_nlink(dentry->d_inode); - dput(dentry); inode->i_flags |= S_DEAD; drop_nlink(dir); error = 0; -- 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
