Adam Jackson <[email protected]> writes:

> Signed-off-by: Adam Jackson <[email protected]>

I don't remember much about DRI1, but looking at the code, I'm not
seeing how this manages to suspend DRI1 clients who are in the middle of
rendering something? Are they supposed to be regularly calling GetDrawableInfo?

> @@ -521,6 +522,9 @@ ProcXF86DRIDispatch(register ClientPtr client)
>  {
>      REQUEST(xReq);
>  
> +    if (!xf86VTOwner())
> +        return xf86SetAndSleepVTClient(client);
> +

Might move this after the client->local check so you don't suspend
clients just asking for the version of the extension.

-- 
-keith

Attachment: signature.asc
Description: PGP signature

_______________________________________________
[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