On Tue, 2017-03-14 at 14:58 +0100, Olivier Fourdan wrote:

> There are some cases where we don't have a fallback code path, in which
> case we'll avoid the crash in glamor_set_destination_drawable() but won't
> render properly, but this is a rare occurence and not rendering properly
> is still better than crashing the X server and the user losing his/her
> entire session...

Discarding rendering on allocation failure is fine, yeah. Or at any
rate fb and mi already have that property.

2/2 doesn't seem to check all the glamor_set_destination_drawable calls
though (see: glamor_glyphs_flush, glamor_dash_loop, glamor_text,
glamor_xv_render). Just an oversight?

- ajax
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to