Re: [PATCH spice-gtk v2 9/8] main: don't update display timer for unchanged config

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Jonathon

On Wed, Mar 23, 2016 at 8:58 PM, Jonathon Jongsma <jjongsma@xxxxxxxxxx> wrote:
>> It also solves/avoids problems with a guest running on wayland when the
>> "resize-guest" property is TRUE.
>> See bug https://bugzilla.redhat.com/show_bug.cgi?id=1266484 ;
>> It avoids destroying the primary surface when the display configuration
>> has not changed
>
> hmm.  so, the suggestion from my previous email (only compare with previous
> config and return early from update_display() if there's currently a pending
> config message scheduled) would probably break this case again (if I understand
> the situation correctly). Do you know *why* we're repeatedly updating to the
> same monitor config under wayland?

I don't fully understand your hypotetical issue. You said in practice
it cannot happen. Do you want to investigate it further before we
apply this fix? I'd apply now, as it fixes 2 existing problems ;)

thanks


-- 
Marc-André Lureau
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]