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