[PATCH] Lock scd(4): test or the driver will be removed
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.
More information about the freebsd-current