On Mon, 28 Nov 2011 22:24:52 +0100, Daniel Vetter <daniel.vetter at ffwll.ch> wrote: > +void i915_gem_init_ppgtt(struct drm_device *dev) > +{ > + drm_i915_private_t *dev_priv = dev->dev_private; > + uint32_t pd_offset; > + struct intel_ring_buffer *ring; > + int i; > + > + if (!HAS_ALIASING_PPGTT(dev)) > + return; This wants to be if (dev_priv->mm.aliasing_ppgtt == NULL) return; instead. Slightly safer against the clumsy ;-) -Chris -- Chris Wilson, Intel Open Source Technology Centre