On Thu, Jul 25, 2013 at 01:16:48PM -0700, Jesse Barnes wrote: > On Thu, 25 Jul 2013 22:05:51 +0200 > Ingo Molnar <mingo@xxxxxxxxxx> wrote: > > > Chris has some patches on top to add a new E820 type so > > > we can look up the region later, which removes some > > > redundant code in the i915 driver at least. > > > > > > Any comments? I assume no one likes this, but maybe it's > > > just another early quirk we'll have to live with... > > > > No strong feelings against it - my only suggestion would be > > to make this more visible - right now it's added as e820 > > reserved which hides amongst other areas already marked > > reserved - would a low-key printk() of the range added make > > it more apparent that a kernel quirk activated here? > > Sounds good, I think Chris's patches should satisfy there. They make > it a new E820 type so it's clear in /proc/iomem too. I think it'd be good to get it all in in one go, since with Chris' patches i915 will also use the detection logic from the quirk code, so more testing for it. And imo the i915 cleanups shouldn't conflict really with ongoing work in drm/i915, so could all go in through x86 trees. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
