Skip site navigation (1)Skip section navigation (2)
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>