Re: [PATCH vdagent-linux] randr: do not set crtc mode to NULL when changing resolution

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

 



Verified that with your patch things still work as expected.

The minor issue remaining is that when switching users, the client window get resized for a brief moment to previous resolution before the agent actually adjust to current window resolution, and then the client window is restored.

This is not something we can solve easily at the agent level I am afraid, but something to add to our TODO list.

Ack both patches for upstream?


On Thu, Nov 22, 2012 at 5:29 PM, Hans de Goede <hdegoede@xxxxxxxxxx> wrote:
Hi,


On 11/22/2012 05:07 PM, Marc-André Lureau wrote:
Note with this patch, the previous patch is unnecessary: the configuration can be sent to all agents (gdm & user) the creation/deletion races will be ignored.

Cool, good way to test the race robustness of the code. Still I think it would be better to just not race at all, I know that cannot be completely avoided. But lets at least fix the agents racing amongst themselves. The attached patch  (untested)

Regards,

Hans



--
Marc-André Lureau
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://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]