[PATCH xserver 3/3] present: Only requeue if target MSC is not reached after an unflip

[Martin Peres]

On 28/03/16 05:24, Michel Dänzer wrote:
> On 01.03.2016 20:26, Martin Peres wrote:
>> On 25/02/16 17:28, Adam Jackson wrote:
>>> On Thu, 2016-02-25 at 09:49 +0000, Chris Wilson wrote:
>>>> On Wed, Feb 24, 2016 at 04:52:59PM +0900, Michel Dänzer wrote:
>>>>> From: Michel Dänzer <michel.daenzer at amd.com>
>>>>>
>>>>> While present_pixmap decrements target_msc by 1 for
>>>>> present_queue_vblank,
>>>>> it leaves the original vblank->target_msc intact. So incrementing the
>>>>> latter for requeueing resulted in the requeued presentation being
>>>>> executed too late.
>>>> My mistake. Yes, the local target_msc is decremented but after
>>>> vblank->target_msc is assigned.
>>>>
>>>>> Also, no need to requeue if the target MSC is already reached.
>>>>>
>>>>> This further reduces stutter when a popup menu appears on top of a
>>>>> flipping fullscreen window.
>>>>>
>>>>> Signed-off-by: Michel Dänzer <michel.daenzer at amd.com>
>>>> Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
>>>> -Chris
>>> Series merged:
>>>
>>> remote: I: patch #74919 updated using rev
>>> 1a9f8c4623c4e6b6955cb6d5f44d29c244dfd32a.
>>> remote: I: patch #74915 updated using rev
>>> e7a35b9e16aa12970908f5d55371bb1b862f8f24.
>>> remote: I: patch #74910 updated using rev
>>> b4ac7b142fa3c536e9b283cfd34b94d82c03aac6.
>>> remote: I: 3 patch(es) updated to state Accepted.
>>> To ssh://git.freedesktop.org/git/xorg/xserver
>>>      0461bca..b4ac7b1  master -> master
>>>
>>
>> For some reason, this patch prevents kde from starting.
>
> Does this still happen with 3b385105 ("present: Only requeue for next
> MSC after flip failure")?

Sorry again for the delay.

I verified that this particular commits fixes it. Now I can run the the 
upstream HEAD on both modesetting and the intel ddx.

Thanks!