[PATCH xserver] present: Only send PresentCompleteNotify events to the presenting client
Michel Dänzer
michel at daenzer.net
Mon Oct 23 14:26:16 UTC 2017
On 20/10/17 09:58 PM, Keith Packard wrote:
> Michel Dänzer <michel at daenzer.net> writes:
>
>> From: Michel Dänzer <michel.daenzer at amd.com>
>>
>> We were sending the events to all clients listening for them on the
>> window. But clients can get confused by events from another client, and
>> I can't imagine any case where reciving events from other clients would
>> be required.
>
> While I agree that it's unlikely to be useful to send the event to all
> listening clients, there is a 'PRESENTEVENTID' in the CompleteNotify
> which clients "should" be using to check to see if the event being
> delivered is the one associated with their action.
>
> Did we mess up in some client library and not do this? If so, we might
> want to fix that library?
Looking at the Mesa code, it uses xcb_generate_id to generate the event
ID. AFAICT the XCB code is properly checking and only queuing special
events with a matching event ID, so the only possible explanation seems
that xcb_generate_id ends up generating the same ID for multiple
clients. Is that plausible?
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Mesa and X developer
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 224 bytes
Desc: OpenPGP digital signature
URL: <https://lists.x.org/archives/xorg-devel/attachments/20171023/3f029de7/attachment.sig>
More information about the xorg-devel
mailing list