Verified that with your patch things still work as expected.<br><br>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.<br>
<br>This is not something we can solve easily at the agent level I am afraid, but something to add to our TODO list.<br><br>Ack both patches for upstream?<br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Nov 22, 2012 at 5:29 PM, Hans de Goede <span dir="ltr"><<a href="mailto:hdegoede@redhat.com" target="_blank">hdegoede@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<div class="im"><br>
<br>
On 11/22/2012 05:07 PM, Marc-André Lureau wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
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.<br>
</blockquote>
<br></div>
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)<br>

<br>
Regards,<br>
<br>
Hans<br>
</blockquote></div><br><br clear="all"><br>-- <br>Marc-André Lureau<br>
</div>