smsc911x.c driver and SMP (was Re: Kernel related (?) user space crash at ARM11 MPCore)
Catalin Marinas
catalin.marinas at arm.com
Fri Sep 11 12:06:29 EDT 2009
On Fri, 2009-09-11 at 10:20 -0500, Bill Gatliff wrote:
> Catalin Marinas wrote:
> > On Fri, 2009-09-11 at 07:55 -0500, Bill Gatliff wrote:
> >> Why take the lock in smsc911x_tx_writefifo? It looks like you take the
> >> same lock again in smsc911x_reg_write.
> >
> > Ah, ok, I haven't tried the 16-bit mode, only the 32-bit one and was
> > working fine, no deadlocks. But I can't say whether that's the right
> > fix, it's better for SMSC to look into it.
>
> I certainly won't object to SMSC taking a look at it. :) But your
> changes pass the common-sense test AFAICT, apart from my question above.
Fixed it with this new commit (I'm not posting the patch here as it's
not meant for review on this list):
http://www.linux-arm.org/git?p=linux-2.6.git;a=commitdiff;h=3cc8ee538f6727d1f6773fff6d8e31bc5522bfee
Thanks for spotting the problem.
--
Catalin
More information about the linux-arm-kernel
mailing list