[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <21d7e9970801100144x63b02e1bga548acd3c5bed441@mail.gmail.com>
Date: Thu, 10 Jan 2008 19:44:03 +1000
From: "Dave Airlie" <airlied@...il.com>
To: "Ingo Molnar" <mingo@...e.hu>
Cc: "Andi Kleen" <ak@...e.de>, linux-kernel@...r.kernel.org,
"Thomas Gleixner" <tglx@...utronix.de>,
"H. Peter Anvin" <hpa@...or.com>,
"Venki Pallipadi" <venkatesh.pallipadi@...el.com>,
suresh.b.siddha@...el.com,
"Arjan van de Ven" <arjan@...radead.org>,
"Dave Jones" <davej@...hat.com>
Subject: Re: CPA patchset
>
> finally managed to get the time to review your CPA patchset, and i
> fundamentally agree with most of the detail changes done in it. But here
> are a few structural high-level observations:
>
> - firstly, there's no rationale given. So we'll change ioremap()/etc.
> from doing a cflush-range instruction instead of a WBINVD. But why?
> WBINVD isnt particular fast (takes a few msecs), but why is that a
> problem? Drivers dont do high-frequency ioremap-ing. It's typically
> only done at driver/device startup and that's it. Whether module load
> time takes 1254 msecs instead of 1250 msecs is no big deal.
read graphics drivers, even though I think we may avoid the whole path
if we can and end up doing some of this in the drivers
when they know more about the situation so can avoid safeties..
but I still see this being used in AGP a fair bit at some point on
some drivers..
Dave.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists