Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 9 Oct 2014 14:02:22 -0400
From:      John Baldwin <jhb@freebsd.org>
To:        Alfred Perlstein <alfred@freebsd.org>
Cc:        freebsd-current@freebsd.org
Subject:   Re: [PATCH] Lock scd(4): test or the driver will be removed
Message-ID:  <201410091402.22466.jhb@freebsd.org>
In-Reply-To: <5435D59A.9090506@freebsd.org>
References:  <3885426.QchWJeH1gV@ralph.baldwin.cx> <5435D59A.9090506@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
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



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201410091402.22466.jhb>