On Tue, Apr 29, 2014 at 12:44 PM, Chris Wilson <ch...@chris-wilson.co.uk> wrote: > On Tue, Apr 29, 2014 at 11:53:58AM +0200, Daniel Vetter wrote: >> Otherwise we'll end up spamming dmesg on every context creation on snb >> with vt-d enabled. This regression was introduced in >> >> commit 246cbfb5fb9a1ca0997fbb135464c1ff5bb9c549 >> Author: Ben Widawsky <benjamin.widaw...@intel.com> >> Date: Fri Dec 6 14:11:14 2013 -0800 >> >> drm/i915: Reorganize intel_enable_ppgtt >> >> As the i915.enable_ppgtt is read-only it cannot be changed after the >> module is loaded and so we can perform an early sanitization of the >> values. >> >> v2: >> - Add comment and pimp commit message (Chris) >> - Use the param consistently (Jani) >> >> v3: >> - Fix init sequence on pre-gen6 by moving the sanitize_ppgtt call to >> gtt_init. Fixes boot hangs on pre-gen6. >> - Add a debug output for the sanitize ppgtt mode. >> >> References: https://lkml.org/lkml/2014/4/17/599 >> Rererences: https://bugs.freedesktop.org/show_bug.cgi?id=77916 >> Cc: Alessandro Suardi <alessandro.sua...@gmail.com> >> Cc: Ben Widawsky <b...@bwidawsk.net> >> Cc: Chris Wilson <ch...@chris-wilson.co.uk> >> Signed-off-by: Daniel Vetter <daniel.vet...@ffwll.ch> > > I am pretty sure we only call i915_gem_gtt_init() once during driver > load... So let's try that r-b again. > Reviewed-by: Chris Wilson <ch...@chris-wilson.co.uk> > -Chris
I'd like to add my Tested-By, but this patch doesn't apply cleanly to 3.15-rc3 - is there any prerequisite patch on top of -rc3 or should I use a git tree to test this fix ? Offtopic, this email has a tiny typo (Rererences in the freedesktop bug ID line), just in case git actually stores that kind of metadata :) thanks, --alessandro "don't underestimate the things that I will do" (Adele, "Rolling In The Deep") _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx