From owner-cvs-all Sun Apr 2 17:15:55 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 27F0437BC1A; Sun, 2 Apr 2000 17:15:46 -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 CAA17317; Mon, 3 Apr 2000 02:15:41 +0200 (CEST) (envelope-from phk@critter.freebsd.dk) To: Mike Smith 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 17:06:21 PDT." <200004030006.RAA03339@mass.cdrom.com> Date: Mon, 03 Apr 2000 02:15:41 +0200 Message-ID: <17315.954720941@critter.freebsd.dk> From: Poul-Henning Kamp Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG In message <200004030006.RAA03339@mass.cdrom.com>, Mike Smith writes: >b_data is mostly an abomination from a device-driver's point of view, as >I think has already been discussed. We should probably not bother >putting it in struct bio at all; rather have a function that will map the >physical pages for a transfer temporarily for those few drivers that >actually need it. I could maybe agree with this, but for now it is part of our interface and it has to go where it is needed as part of that interface. >This requires attention in the busdma code first. ...and not all drivers can do it with busdma. vinum/RAID5 for instance needs to be able to calculate the parity. -- 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