<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - SIGSEGV "get_fb: failed to add fb" / sna_block_handler"
href="https://bugs.freedesktop.org/show_bug.cgi?id=91467#c17">Comment # 17</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - SIGSEGV "get_fb: failed to add fb" / sna_block_handler"
href="https://bugs.freedesktop.org/show_bug.cgi?id=91467">bug 91467</a>
from <span class="vcard"><a class="email" href="mailto:chris@chris-wilson.co.uk" title="Chris Wilson <chris@chris-wilson.co.uk>"> <span class="fn">Chris Wilson</span></a>
</span></b>
<pre>(In reply to Andreas Reis from <a href="show_bug.cgi?id=91467#c16">comment #16</a>)
<span class="quote">> Created <span class=""><a href="attachment.cgi?id=117387" name="attach_117387" title="Xorg.0.log with sna: Add a small pixmap sanity check">attachment 117387</a> <a href="attachment.cgi?id=117387&action=edit" title="Xorg.0.log with sna: Add a small pixmap sanity check">[details]</a></span>
> Xorg.0.log with sna: Add a small pixmap sanity check
>
> With sna: Add a small pixmap sanity check the server won't even start as it
> instantly segfaults.</span >
Oops,
commit d11dc75fb5a95ba410fedd86d9e1dd50260af979
Author: Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>>
Date: Sun Jul 26 19:07:45 2015 +0100
sna: Only check non-NULL Pixmaps
check_pixmap() can be called very early in the Window setup proceeding,
before a pixmap is even assigned to a Window. There we expect the Window
to be NULL, so be more careful in our check_pixmap.
Bugzilla: <a class="bz_bug_link
bz_status_NEW "
title="NEW - SIGSEGV "get_fb: failed to add fb" / sna_block_handler"
href="show_bug.cgi?id=91467#c16">https://bugs.freedesktop.org/show_bug.cgi?id=91467#c16</a>
Signed-off-by: Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>>
I think I understand the freeze,
commit e5f8f90f686879950766babbe805cd9d2412aca3
Author: Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>>
Date: Sun Jul 26 19:03:46 2015 +0100
sna: Stall for outstanding TearFree flips when taking over with Present
When juggling Present and TearFree, we have to hide the extra flips from
Present as it cannot account for them. Preferrably we want to schedule
the Present flip following completion of the TearFree flip, but for the
moment simply block and wait until TearFree completes before starting
Present.
Reported-by: Andreas Reis <<a href="mailto:andreas.reis@gmail.com">andreas.reis@gmail.com</a>>
Bugzilla: <a class="bz_bug_link
bz_status_NEW "
title="NEW - SIGSEGV "get_fb: failed to add fb" / sna_block_handler"
href="show_bug.cgi?id=91467">https://bugs.freedesktop.org/show_bug.cgi?id=91467</a>
Signed-off-by: Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>>
should prevent the freeze at the cost of a small stall everytime Present stops
and restarts flipping.
However, not convinced if that is related to the earlier crash - for which as
you can tell I've added some more debugging to hopefully catch in action.</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>