[PATCH] Lock scd(4): test or the driver will be removed

John Baldwin jhb at freebsd.org
Thu Oct 9 21:21:18 UTC 2014


On Wednesday, October 08, 2014 8:23:54 pm Alfred Perlstein wrote:
> 
> On 10/8/14 11:53 AM, John Baldwin wrote:
> > This patch adds locking to scd(4) and marks it MPSAFE.  It also uses bus_*()
> > instead of bus_space_*().  The patch is against HEAD but probably applies to 9
> > and 10 as well.
> >
> > http://people.freebsd.org/~jhb/patches/scd_locking.patch
> >
> > Note that this driver is using a deprecated API that will be removed in 11.
> > If no one tests updates to this driver then it is not feasible to continue
> > maintaining it in the tree.  In that case, it will be removed from HEAD one
> > month from today.
> >
> Hey John, which APIs are being used?

timeout(9).  However, we are eventually going to start requiring drivers to
not use Giant either (and retiring timeout(9) is a step in that direction).

Also, most of these drivers are for ancient hardware.  scd(4) is support for
a relatively rare pre-ATA CD controller.  The much-more common matcd(4) driver
(which is the driver used by my old SoundBlaster 16 card that provided a CD 
controller (not just sound, but data as well)) was removed back in 2002.

-- 
John Baldwin


More information about the freebsd-current mailing list