bug report/suggested temp. patch: handling bursts of sent keys

[Jon TURNEY]

On 20/03/2010 16:32, Mark Lillibridge wrote:
> On February 23, 2010, Jon Turney wrote:
>> On 13/02/2010 20:24, Mark Lillibridge wrote:
>>> Jon wrote:
>>>>    Thanks for the patch.  Have you actually tested that this resolves your problem?
>>>
>>>       Yes.  Of course, really, really large bursts will still fail, but
>>> they should be very rare.
>>>
>>
>> Perhaps you might try the attached patch, instead, and see if that helps.
>
>      Yes, your (previously) attached patch seems to solve the problem
> without wasting memory or only working for a finite number of keys.
> Nice job.  What's the next step towards getting this patch added to the
> source/sent upstream/to Xming?

Yaakov,

Can you pull this patch and include it in the next X server release, please.

Sadly this isn't quite a complete solution: there's still the problem in 
multiwindow mode that Windows uses a modal loop when resizing/moving a window, 
so the X server code gets no chance to do anything then.  You can easily 
demonstrate this by resizing the frame of an X window, moving the mouse 
continuously for a number of seconds, and then notice that X window doesn't 
react to the queued size changes until the mouse button is released.


The following changes since commit 579715f830fbbca9e1ecb17dc18176132f5969e7:
   Rami Ylimaki (1):
         os: Prevent backtrace from being stopped in noreturn functions.

are available in the git repository at:

   git://anongit.freedesktop.org/~jturney/xserver for-yaakov

Jon TURNEY (1):
       Cygwin/X: Process one Windows message per wakeup, rather than all of them.

  hw/xwin/winblock.c  |   23 ++++-------------------
  hw/xwin/winwakeup.c |    4 ++--
  2 files changed, 6 insertions(+), 21 deletions(-)



--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Problem reports:       http://cygwin.com/problems.html
Documentation:         http://x.cygwin.com/docs/
FAQ:                   http://x.cygwin.com/docs/faq/