cvs commit: src/sys/dev/fdc fdc.c

[Joerg Wunsch]

As Poul-Henning Kamp wrote:

>   Modified files:
>     sys/dev/fdc          fdc.c 
>   Log:
>   Lock the flags field with the mutex.
>   
>   Improve a number of comments.

That doesn't really explain this one:

@@ -96,8 +96,6 @@
                                 * fd_drivetype; on i386 machines, if
                                 * given as 0, use RTC type for fd0
                                 * and fd1 */
-#define FD_NO_CHLINE   0x10    /* drive does not support changeline
-                                * aka. unit attention */
 #define FD_NO_PROBE    0x20    /* don't probe drive (seek test), just
                                 * assume it is there */
 
No idea about why FD_NO_CHLINE got obsolete, but either way, that
should also be updated in src/share/man/man4/fdc.4 as well since the
0x10 flag is documented there.

What I'm also missing is a documentation of the new debug flags.  Not
necessarily in fdc(4) (it's useful for developers only), but at least
as a comment on top of the file.  (Sure, Julian's old debug output
wasn't documented either, but we're here to make it better, aren't we?
;-)

Shouldn't the flag manipulation in fdc_thread() also be protected by a
mutex?  (I currently can't really test all this as long as GEOM
doesn't allow me unloading the driver...)

-- 
cheers, J"org               .-.-.   --... ...--   -.. .  DL8DTL

http://www.sax.de/~joerg/                        NIC: JW11-RIPE
Never trust an operating system you don't have sources for. ;-)