<br>On Thu, Mar 15, 2012 at 5:04 PM, nobled <span dir="ltr">&lt;<a href="mailto:nobled@dreamwidth.org">nobled@dreamwidth.org</a>&gt;</span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im">On Thu, Mar 15, 2012 at 2:26 PM, Anuj Phogat &lt;<a href="mailto:anuj.phogat@gmail.com">anuj.phogat@gmail.com</a>&gt; wrote:<br>
&gt;<br>
&gt;<br>
&gt; On Wed, Mar 7, 2012 at 4:28 PM, Anuj Phogat &lt;<a href="mailto:anuj.phogat@gmail.com">anuj.phogat@gmail.com</a>&gt; wrote:<br>
&gt;&gt;<br>
&gt;&gt; On Mon 27 Feb 2012 11:45:46 AM PST, Anuj Phogat wrote:<br>
&gt;&gt; &gt; This patch handles a case when mapping a large texture fails<br>
&gt;&gt; &gt; in drm_intel_gem_bo_map_gtt(). These changes avoid assertion<br>
&gt;&gt; &gt; failure later in the driver as reported in following bugs:<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; <a href="https://bugs.freedesktop.org/show_bug.cgi?id=44970" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=44970</a><br>
&gt;&gt; &gt; <a href="https://bugs.freedesktop.org/show_bug.cgi?id=46303" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=46303</a><br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; Signed-off-by: Anuj Phogat &lt;<a href="mailto:anuj.phogat@gmail.com">anuj.phogat@gmail.com</a>&gt;<br>
<br>
</div>I&#39;ve been testing this since yesterday applied on top of the 8.0<br>
branch, and compiz is still crashing for me (bug 46303).<br></blockquote><div><br></div><div>I haven&#39;t tried compiz with this patch. looks like compiz crash (<a href="https://bugs.freedesktop.org//show_bug.cgi?id=46739">Bug 46737</a>)</div>

<div>is a different issue than assertion failure in intel_regions.c line 310 (<a href="https://bugs.freedesktop.org/show_bug.cgi?id=46303">Bug 46303</a>). </div><div>You can look at the <b style="background-color:rgb(255,255,255);color:rgb(102,51,102);font-family:Verdana,sans-serif"><a href="https://bugs.freedesktop.org//attachment.cgi?id=57785" title="View the content of the attachment" style="color:rgb(102,51,102)">ThreadStacktrcace.txt</a> . </b>Last instruction  executed on</div>

<div> Thread 1: </div><div>#0 intel_miptree_release (mt=0x220) at intel_mipmap_tree.c:290</div><div>        __FUNCTION__ = &quot;intel_miptree_release&quot;</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div><div class="h5"><br>
&gt;&gt; &gt; ---<br>
&gt;&gt; &gt;  src/mesa/drivers/dri/intel/intel_mipmap_tree.c |   26<br>
&gt;&gt; &gt; ++++++++++++++++-------<br>
&gt;&gt; &gt;  src/mesa/drivers/dri/intel/intel_regions.c     |    7 ++++-<br>
&gt;&gt; &gt;  2 files changed, 23 insertions(+), 10 deletions(-)<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; diff --git a/src/mesa/drivers/dri/intel/intel_mipmap_tree.c<br>
&gt;&gt; &gt; b/src/mesa/drivers/dri/intel/intel_mipmap_tree.c<br>
&gt;&gt; &gt; index 5290da4..507702a 100644<br>
&gt;&gt; &gt; --- a/src/mesa/drivers/dri/intel/intel_mipmap_tree.c<br>
&gt;&gt; &gt; +++ b/src/mesa/drivers/dri/intel/intel_mipmap_tree.c<br>
&gt;&gt; &gt; @@ -702,15 +702,20 @@ intel_miptree_map_gtt(struct intel_context *intel,<br>
&gt;&gt; &gt;     y /= bh;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;     base = intel_region_map(intel, mt-&gt;region, map-&gt;mode);<br>
&gt;&gt; &gt; -   /* Note that in the case of cube maps, the caller must have passed<br>
&gt;&gt; &gt; the slice<br>
&gt;&gt; &gt; -    * number referencing the face.<br>
&gt;&gt; &gt; -    */<br>
&gt;&gt; &gt; -   intel_miptree_get_image_offset(mt, level, 0, slice, &amp;image_x,<br>
&gt;&gt; &gt; &amp;image_y);<br>
&gt;&gt; &gt; -   x += image_x;<br>
&gt;&gt; &gt; -   y += image_y;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; -   map-&gt;stride = mt-&gt;region-&gt;pitch * mt-&gt;cpp;<br>
&gt;&gt; &gt; -   map-&gt;ptr = base + y * map-&gt;stride + x * mt-&gt;cpp;<br>
&gt;&gt; &gt; +   if (base == NULL)<br>
&gt;&gt; &gt; +      map-&gt;ptr = NULL;<br>
&gt;&gt; &gt; +   else {<br>
&gt;&gt; &gt; +      /* Note that in the case of cube maps, the caller must have<br>
&gt;&gt; &gt; passed the<br>
&gt;&gt; &gt; +       * slice number referencing the face.<br>
&gt;&gt; &gt; +      */<br>
&gt;&gt; &gt; +      intel_miptree_get_image_offset(mt, level, 0, slice, &amp;image_x,<br>
&gt;&gt; &gt; &amp;image_y);<br>
&gt;&gt; &gt; +      x += image_x;<br>
&gt;&gt; &gt; +      y += image_y;<br>
&gt;&gt; &gt; +<br>
&gt;&gt; &gt; +      map-&gt;stride = mt-&gt;region-&gt;pitch * mt-&gt;cpp;<br>
&gt;&gt; &gt; +      map-&gt;ptr = base + y * map-&gt;stride + x * mt-&gt;cpp;<br>
&gt;&gt; &gt; +   }<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;     DBG(&quot;%s: %d,%d %dx%d from mt %p (%s) %d,%d = %p/%d\n&quot;, __FUNCTION__,<br>
&gt;&gt; &gt;         map-&gt;x, map-&gt;y, map-&gt;w, map-&gt;h,<br>
&gt;&gt; &gt; @@ -1063,6 +1068,11 @@ intel_miptree_map(struct intel_context *intel,<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;     *out_ptr = map-&gt;ptr;<br>
&gt;&gt; &gt;     *out_stride = map-&gt;stride;<br>
&gt;&gt; &gt; +<br>
&gt;&gt; &gt; +   if (map-&gt;ptr == NULL) {<br>
&gt;&gt; &gt; +      mt-&gt;level[level].slice[slice].map = NULL;<br>
&gt;&gt; &gt; +      free(map);<br>
&gt;&gt; &gt; +   }<br>
&gt;&gt; &gt;  }<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;  void<br>
&gt;&gt; &gt; diff --git a/src/mesa/drivers/dri/intel/intel_regions.c<br>
&gt;&gt; &gt; b/src/mesa/drivers/dri/intel/intel_regions.c<br>
&gt;&gt; &gt; index bc83649..982d6cb 100644<br>
&gt;&gt; &gt; --- a/src/mesa/drivers/dri/intel/intel_regions.c<br>
&gt;&gt; &gt; +++ b/src/mesa/drivers/dri/intel/intel_regions.c<br>
&gt;&gt; &gt; @@ -124,7 +124,7 @@ intel_region_map(struct intel_context *intel, struct<br>
&gt;&gt; &gt; intel_region *region,<br>
&gt;&gt; &gt;      */<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;     _DBG(&quot;%s %p\n&quot;, __FUNCTION__, region);<br>
&gt;&gt; &gt; -   if (!region-&gt;map_refcount++) {<br>
&gt;&gt; &gt; +   if (!region-&gt;map_refcount) {<br>
&gt;&gt; &gt;        intel_flush(&amp;intel-&gt;ctx);<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;        if (region-&gt;tiling != I915_TILING_NONE)<br>
&gt;&gt; &gt; @@ -133,7 +133,10 @@ intel_region_map(struct intel_context *intel,<br>
&gt;&gt; &gt; struct intel_region *region,<br>
&gt;&gt; &gt;        drm_intel_bo_map(region-&gt;bo, true);<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;        region-&gt;map = region-&gt;bo-&gt;virtual;<br>
&gt;&gt; &gt; -      ++intel-&gt;num_mapped_regions;<br>
&gt;&gt; &gt; +      if (region-&gt;map) {<br>
&gt;&gt; &gt; +         ++intel-&gt;num_mapped_regions;<br>
&gt;&gt; &gt; +      region-&gt;map_refcount++;<br>
&gt;&gt; &gt; +      }<br>
&gt;&gt; &gt;     }<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;     return region-&gt;map;<br>
&gt;&gt;<br>
&gt;&gt; Any comments on this patch?<br>
&gt;<br>
&gt;<br>
&gt; I am planning to push this today.<br>
</div></div>&gt; _______________________________________________<br>
&gt; mesa-dev mailing list<br>
&gt; <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
&gt; <a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
&gt;<br>
</blockquote></div><br>