]> git.ipfire.org Git - people/arne_f/kernel.git/commit - drivers/gpu/drm/i915/i915_gem_gtt.c
drm/i915/bdw: Initialize the PDEs
authorBen Widawsky <benjamin.widawsky@intel.com>
Tue, 5 Nov 2013 05:20:14 +0000 (21:20 -0800)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 8 Nov 2013 17:09:46 +0000 (18:09 +0100)
commitb1fe6673298113636f356dce8e8ac2a93a579882
tree8bed132f496dc936f8053cbb248e90441bfa6ecf
parent37aca44ad5b0fa30d4a9cd77b492b45f2b6a4643
drm/i915/bdw: Initialize the PDEs

The upcoming clear and insert routines will expect that PDEs all point
to valid Page Directories. Doing that lazily doesn't really buy us
anything.

The page allocation is done regardless earlier in init so it shouldn't
hurt set the PDEs.

v2: Squash in patches to implement fixed PDE write function:

- If I had done this in the first place, the bug that's going to be
  fixed in an upcoming patch would have been much easier to find.

- Use WB for PDEs.

  The PAT bit is used for page size. 2ME PDEs aren't even supported in
  BDW, so this was completely invalid. The solution is to make our
  PDEs WB+LLC instead of the pervious WB+eLLC. As far as I can guess,
  this change won't matter for performance.

  Thanks to Ville for the quick correction when discussing on IRC.

v3: Return the pde type for pde encoding (Damien)

Reviewed-by: Imre Deak <imre.deak@intel.com>
Reviewed-by: Damien Lespiau <damien.lespiau@intel.com>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/i915_gem_gtt.c