From owner-cvs-all Sun Apr 2 22:17:21 2000 Delivered-To: cvs-all@freebsd.org Received: from critter.freebsd.dk (critter.freebsd.dk [212.242.40.131]) by hub.freebsd.org (Postfix) with ESMTP id 034AA37B62E; Sun, 2 Apr 2000 22:17:18 -0700 (PDT) (envelope-from phk@critter.freebsd.dk) Received: from critter.freebsd.dk (localhost.freebsd.dk [127.0.0.1]) by critter.freebsd.dk (8.9.3/8.9.3) with ESMTP id HAA18202; Mon, 3 Apr 2000 07:16:36 +0200 (CEST) (envelope-from phk@critter.freebsd.dk) To: Peter Wemm Cc: Matthew Dillon , cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/kern subr_devstat.c src/sys/ufs/ufs ufs_disksubr.c src/sys/sys buf.h devicestat.h disklabel.h In-reply-to: Your message of "Sun, 02 Apr 2000 18:16:27 PDT." <20000403011627.6E7AE1CD7@overcee.netplex.com.au> Date: Mon, 03 Apr 2000 07:16:36 +0200 Message-ID: <18200.954738996@critter.freebsd.dk> From: Poul-Henning Kamp Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG In message <20000403011627.6E7AE1CD7@overcee.netplex.com.au>, Peter Wemm writes : >I think this is probably the better thing to do - ie: duplicate it and also >the B_ERROR flag. As you point out, brelse() uses it. However, it's also >part of the driver<->bio interaction. There is no more need to duplicated anything now than there were before. The "built-in" bio in struct buf is an integral part of struct buf, all the fields except for the "_driver[12]" fields (and the pblkno and iodone_chain which will be going away) are owned by the struct buf. -- Poul-Henning Kamp FreeBSD coreteam member phk@FreeBSD.ORG "Real hackers run -current on their laptop." FreeBSD -- It will take a long time before progress goes too far! To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message