Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 13 Feb 2003 22:17:08 -0800
From:      Marcel Moolenaar <marcel@xcllnt.net>
To:        Eric Anholt <eta@lclark.edu>
Cc:        src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/modules Makefile
Message-ID:  <20030214061708.GA2109@athlon.pn.xcllnt.net>
In-Reply-To: <1045200753.84507.54.camel@leguin>
References:  <20030213223058.769DA2A8C1@canning.wemm.org> <1045185451.11981.17.camel@leguin> <20030214023218.GA1573@athlon.pn.xcllnt.net> <1045194133.11981.87.camel@leguin> <20030214043028.GA1797@athlon.pn.xcllnt.net> <1045200753.84507.54.camel@leguin>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Feb 13, 2003 at 09:32:33PM -0800, Eric Anholt wrote:
> > 
> > In that case, we'd better make sure there's cache coherency. Do we
> > actually have the code structured in a way that allows having the
> > flushing chipset dependent (not to mention dependent on the address)?
> 
> No, currently all the cache flushes (four in agp.c, three in i810 and
> amd code) are unconditional agp_flush_cache calls after modifying the
> gatt entries.  They aren't tied to a specific memory range, but could be
> pretty easily, if not the most efficiently, by pushing some of them into
> the (un)bind_pages.  There's probably a better way.

I wonder: do we actually need to flush at all? GART updates are PCI/AGP
writes and should be coherent, right?
Isn't updating the SGM (system graphics memory) itself that needs
cache flushes to make sure the AGP device gets the right data?

Also, on ia64 bus I/O is done with a virtual address that has the
non-cacheable property. Flushing would not be required irrespective.

Anyway: the nit was about having conditional compilation based on the
architecture, not whether the code was actually required. When in doubt,
commit the code :-)

-- 
 Marcel Moolenaar	  USPA: A-39004		 marcel@xcllnt.net

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?20030214061708.GA2109>