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>