Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 03 Apr 2000 02:15:41 +0200
From:      Poul-Henning Kamp <phk@critter.freebsd.dk>
To:        Mike Smith <msmith@FreeBSD.ORG>
Cc:        Matthew Dillon <dillon@apollo.backplane.com>, 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 
Message-ID:  <17315.954720941@critter.freebsd.dk>
In-Reply-To: Your message of "Sun, 02 Apr 2000 17:06:21 PDT." <200004030006.RAA03339@mass.cdrom.com> 

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




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