Messages in this thread | | | From | Andy Lutomirski <> | Date | Wed, 15 May 2013 11:22:20 -0700 | Subject | Re: [PATCH v3 6/9] radeon: Switch to arch_phys_wc_add and add a missing ..._del |
| |
On Wed, May 15, 2013 at 7:49 AM, Jerome Glisse <j.glisse@gmail.com> wrote: > On Tue, May 14, 2013 at 5:35 PM, Andy Lutomirski <luto@amacapital.net> wrote: >> On Tue, May 14, 2013 at 6:37 AM, Jerome Glisse <j.glisse@gmail.com> wrote: >>> On Tue, May 14, 2013 at 8:58 AM, Alex Deucher <alexdeucher@gmail.com> wrote: >>>> On Mon, May 13, 2013 at 7:58 PM, Andy Lutomirski <luto@amacapital.net> wrote: >>>>> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> >>>>> Signed-off-by: Andy Lutomirski <luto@amacapital.net> >>>> >>>> Reviewed-by: Alex Deucher <alexander.deucher@amd.com> >>> >>> I believe it will break something but we could deal with the fallout >>> once it happens. >> >> FWIW, I'm running with this code on my machine right now using the >> radeon driver. Everything seems fine. If I build without MTRRs and >> without PAT, though, graphics are slow (as expected). So I think >> everything's okay. >> >> --Andy > > I am worried on p4 where i last see issue with that notably with agp.
Do you remember any details? It looks like PAT is enabled on Pentium 4 (i.e. famliy 0xF).
--Andy
| |