Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 8 Mar 2003 23:46:43 -0800 (PST)
From:      Nate Lawson <nate@root.org>
To:        Poul-Henning Kamp <phk@FreeBSD.org>
Cc:        src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/alpha/alpha autoconf.c src/sys/cam cam_periph.h cam_xpt.c src/sys/cam/scsi scsi_da.c scsi_low.c scsi_ses.c src/sys/contrib/dev/fla fla.c src/sys/dev/aac aac.c aac_cam.c aac_debug.c aac_disk.c aac_pci.c aacvar.h ...
Message-ID:  <Pine.BSF.4.21.0303082323370.90870-100000@root.org>
In-Reply-To: <F9337CAB-5187-11D7-ABBA-000393754B1C@vangelderen.org>

next in thread | previous in thread | raw e-mail | index | archive | help
> On Saturday, Mar 8, 2003, at 03:01 US/Eastern, Poul-Henning Kamp wrote:
> 
> > phk         2003/03/08 00:01:31 PST
> >
> >   FreeBSD src repository
> >
> >   Modified files:
> >     sys/alpha/alpha      autoconf.c
> > [...]
> >   Log:
> >   Centralize the devstat handling for all GEOM disk device drivers
> >   in geom_disk.c.
> >
> >   As a side effect this makes a lot of #include <sys/devicestat.h>
> >   lines not needed and some biofinish() calls can be reduced to
> >   biodone() again.

In cam/scsi/scsi_da.c, please explain to me why you have removed all
checking of the DEVSTAT_BS_UNAVAILABLE flag as well as comments
explaining why this is useful.

-Nate


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.BSF.4.21.0303082323370.90870-100000>