<div class="gmail_quote">On Thu, Feb 9, 2012 at 14:39, Chris Wilson <span dir="ltr"><<a href="mailto:chris@chris-wilson.co.uk" target="_blank">chris@chris-wilson.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



<div>On Thu,  9 Feb 2012 17:15:50 +0100, Daniel Vetter <<a href="mailto:daniel.vetter@ffwll.ch" target="_blank">daniel.vetter@ffwll.ch</a>> wrote:<br>
> v3: Pimp commit message and make Chris Wilson less grumpy by adding a<br>
> module option.<br>
<br>
</div><div>> +void i915_gem_init_ppgtt(struct drm_device *dev)<br>
> +{<br>
> +     drm_i915_private_t *dev_priv = dev->dev_private;<br>
> +     uint32_t pd_offset;<br>
> +     struct intel_ring_buffer *ring;<br>
> +     int i;<br>
> +<br>
> +     if (i915_enable_ppgtt && !HAS_ALIASING_PPGTT(dev))<br>
> +             return;<br>
> +<br>
<br>
</div>This is quite a late check since we have already partitioned the<br>
aperture and run i915_gem_init_aliasing_ppgtt. I think the enable check<br>
should be done earlier and this check reduced to<br>
  if (!dev_priv->mm.aliasing_ppgtt)<br>
    return;<br></blockquote><div><br>I think we need to keep the enable check here, because this can be called from within i915_reset as well.<br><br>To fix this and prevent clash between i915_gem_init_ppgtt and i915_gem_init_ppgtt, I guess you could add the 'i915_enable_ppgtt && ' check to the i915_gem_init_ppgtt call in Patch3 as well.<br>

</div></div><br>-- <br>Eugeni Dodonov<a href="http://eugeni.dodonov.net/" target="_blank"><br></a><br>