From owner-cvs-src Sat Mar 8 23:58:45 2003 Delivered-To: cvs-src@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 331C237B401; Sat, 8 Mar 2003 23:58:42 -0800 (PST) Received: from critter.freebsd.dk (critter.freebsd.dk [212.242.86.163]) by mx1.FreeBSD.org (Postfix) with ESMTP id 3AEDF43FB1; Sat, 8 Mar 2003 23:58:41 -0800 (PST) (envelope-from phk@phk.freebsd.dk) Received: from critter.freebsd.dk (localhost [127.0.0.1]) by critter.freebsd.dk (8.12.8/8.12.8) with ESMTP id h297weNw078339; Sun, 9 Mar 2003 08:58:40 +0100 (CET) (envelope-from phk@phk.freebsd.dk) To: Nate Lawson 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 ... From: "Poul-Henning Kamp" In-Reply-To: Your message of "Sat, 08 Mar 2003 23:46:43 PST." Date: Sun, 09 Mar 2003 08:58:40 +0100 Message-ID: <78338.1047196720@critter.freebsd.dk> Sender: owner-cvs-src@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG In message , Nate Lawson wri tes: >> 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 >> > 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. Because the blocksize is only relevant while the device is opened: no requests will happen while it is closed. All that logic was pointless. -- Poul-Henning Kamp | UNIX since Zilog Zeus 3.20 phk@FreeBSD.ORG | TCP/IP since RFC 956 FreeBSD committer | BSD since 4.3-tahoe Never attribute to malice what can adequately be explained by incompetence. To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-src" in the body of the message