<html>
    <head>
      <base href="https://bugs.freedesktop.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - X server crashes in i965 on desktop startup when DRI3 v1.2 / modifier support is enabled"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=106642#c22">Comment # 22</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - X server crashes in i965 on desktop startup when DRI3 v1.2 / modifier support is enabled"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=106642">bug 106642</a>
              from <span class="vcard"><a class="email" href="mailto:lionel.g.landwerlin@linux.intel.com" title="Lionel Landwerlin <lionel.g.landwerlin@linux.intel.com>"> <span class="fn">Lionel Landwerlin</span></a>
</span></b>
        <pre>(In reply to Eero Tamminen from <a href="show_bug.cgi?id=106642#c19">comment #19</a>)
<span class="quote">> (In reply to Lionel Landwerlin from <a href="show_bug.cgi?id=106642#c14">comment #14</a>)
> > Created <span class=""><a href="attachment.cgi?id=140055" name="attach_140055" title="dri: add missing 16bits formats mapping">attachment 140055</a> <a href="attachment.cgi?id=140055&action=edit" title="dri: add missing 16bits formats mapping">[details]</a></span> <a href='page.cgi?id=splinter.html&bug=106642&attachment=140055'>[review]</a> [review] [review]
> > dri: add missing 16bits formats mapping
> > 
> > Here is a v2 for the new dri_format (G16R16 this time).
> > I've gone through all the dri image formats, so hopefully we're not missing
> > anything anymore.

> A quick try of this patch alone -> still get the same crash?

> ---------------------------------------------------
> Thread 1 "X" received signal SIGSEGV, Segmentation fault.
> 0x00007f4666cd5c2a in isl_format_supports_ccs_e
> (devinfo=devinfo@entry=0x1aa72e8, 
>     format=ISL_FORMAT_UNSUPPORTED) at isl/isl_format.c:558
> 558        isl/isl_format.c: No such file or directory.
> (gdb) bt
> #0  0x00007f4666cd5c2a in isl_format_supports_ccs_e
> (devinfo=devinfo@entry=0x1aa72e8, 
>     format=ISL_FORMAT_UNSUPPORTED) at isl/isl_format.c:558
> #1  0x00007f4666ba1fac in modifier_is_supported
> (devinfo=devinfo@entry=0x1aa72e8, 
>     fmt=fmt@entry=0x7f4666ed6f98 <intel_image_formats+216>, dri_format=4112,
> dri_format@entry=0, 
>     modifier=modifier@entry=72057594037927940) at intel_screen.c:345</span >

I'm really disturbed by the value of dri_format for this function now.
In the commit you pointed at for mesa, the caller should be this :

<a href="https://cgit.freedesktop.org/mesa/mesa/tree/src/mesa/drivers/dri/i965/intel_screen.c?id=66c61797ada12e0e2b396affcc2dc495b6cc04ed#n1304">https://cgit.freedesktop.org/mesa/mesa/tree/src/mesa/drivers/dri/i965/intel_screen.c?id=66c61797ada12e0e2b396affcc2dc495b6cc04ed#n1304</a>

So dri_format should be 0, but the backtrace implies it's not...</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the QA Contact for the bug.</li>
      </ul>
    </body>
</html>