Re: mcdx -- do_request(): non-read command to cd!!

Pekka J Enberg
2007-04-03T18:32:05Z

On Tue, 3 Apr 2007, Rene Herman wrote:
> That is, the exact same oops the tar test showed which I expect is progress --
> the dd is now in fact doing something :-)

Yes, that's expected. I think we fixed dd now.

On Tue, 3 Apr 2007, Rene Herman wrote:
> When I now switched the monitor to 5va2 itself it was happily generating more
> oopses and supposedly "fixing up recursive faults", while advicing me to
> reboot. 2 seconds later it didn't leave me a choice as it dropped dead :-)

Please enable CONFIG_DEBUG_SLAB now and reproduce. It should tell us
what's going wrong there.

On Tue, 3 Apr 2007, Rene Herman wrote:
> Just to keep track, I'm now using the patches (minus 03) at:
>
> http://members.home.nl/rene.herman/mcdx/
>
> (for lkml: with 03, the machine locks itself hard each and every time without
> information).

Yes, because it is totally broken. So please just drop the patch. AFAICT
the driver makes the process sleep in mcdex_transfer() until an interrupt
comes to acknowledge that the I/O has completed but the patch makes the
code sleep while holding the mutex and once the interrupt comes and tries
to grab it, the machine will deadlock.

We probably need to fix it somehow but lets see what CONFIG_DEBUG_SLAB
tells us first. Btw, if anyone has a datasheet for this beast, I would
like a copy.

Pekka