On Wed, 10 Dec 2008, Andreas Bombe wrote: > Apart from sleep_on() calls that could be easily converted to > wait_event() and completion calls amiflop also used a flag in ms_delay() > and ms_isr() as a custom mutex for ms_delay() without a need for > explicit unlocking. I converted that to a standard mutex. > > The replacement for the unconditional sleep_on() in fd_motor_on() is a > complete_all() together with a INIT_COMPLETION() before the mod_timer() > call. It appears to me that fd_motor_on() might be called concurrently > and fd_select() does not guarantee mutual exclusivity in the case the > same drive gets selected again. > > Signed-off-by: Andreas Bombe <[email protected]> > --- > > This version changes the struct semaphore used as mutex into a struct > mutex. Still only compile tested. > > I just tried out my Amiga and it appears to be in better shape than > expected. So I might get to actually test it when I find the time. My > Amiga has no Linux installation so I need to whip up a ramdisk or > something else sufficient to test it.
I think the good old ELF ramdisk at ftp://ftp.uni-erlangen.de/pub/unix/Linux/680x0/v2.0/filesys/filesys-ELF-2.0.x-1400K-2.gz should be sufficient to give it a try... Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected] In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe linux-m68k" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
