Re: [PATCH] USB: fix deadlock in HCD code

From: Jiri Kosina
Date: Wed May 21 2008 - 10:22:50 EST


On Wed, 21 May 2008, Alan Stern wrote:

> > Used in the suspend code. Your patch is indeed correct, but I fear
> > there might be a second bug caused by wrong calling conditions.
> The problem in the Novell bugzilla entry was caused by the fact that the
> OHCI irq routine was invoked with interrupts enabled, owing to a missing
> IRQF_DISABLED flag. That bug has already been fixed in 2.6.25.

That indeed is 2.6.25 kernel. I guess you are talking about commit
442258e2ff69 here. If so, the reporter is definitely using the kernel
containing this commit, and the lockups still trigger.

Seems that my patch is papering over the real bug (someone enabling
interrupts somewhere) indeed, but I can't seem to be able to find such
codepath.

Thanks,

--
Jiri Kosina
SUSE Labs
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/