Date: Wed, 02 Jan 2008 21:15:01 +0000 From: "Poul-Henning Kamp" <phk@phk.freebsd.dk> To: Nate Lawson <nate@root.org> Cc: cvs-src@freebsd.org, src-committers@freebsd.org, cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/dev/ata ata-all.h Message-ID: <1429.1199308501@critter.freebsd.dk> In-Reply-To: Your message of "Wed, 02 Jan 2008 13:01:17 PST." <477BFB9D.8070201@root.org>
next in thread | previous in thread | raw e-mail | index | archive | help
In message <477BFB9D.8070201@root.org>, Nate Lawson writes: >Poul-Henning Kamp wrote: >Ok, but this still doesn't address the performance loss fixed by this >3-year-old patch: I think your patch is wrong because it moves the responsibility for the performance in the wrong direction: We shouldn't have to care about this in every single device driver. When I created bus_{read|write}_* that was mostly to reduce the risk of repetive strain injuries for device driver writers, but it was also an attempt to move the responsibility in the right direction. rman and bus_space was sort of conceived separately, but it is time they realize that they are indeed a couple so everybody don't have to pretend how surprised they are to see them together at this instant. -- 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.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1429.1199308501>