Re: [PATCH xserver 0/2] os: Fix write block handling in poll(2) path

2016-09-22 Thread Jeremy Huddleston Sequoia
Series is: Reviewed-by: Jeremy Huddleston Sequoia Tested-by: Jeremy Huddleston Sequoia I'd still like to get rid of the large NewOutputPending hammer, but this is good for now. Thanks, Jeremy > On Sep 21, 2016, at 16:53, Keith Packard

Re: [PATCH xserver 0/2] os: Fix write block handling in poll(2) path

2016-09-22 Thread Matthieu Herrb
On Thu, Sep 22, 2016 at 02:53:44AM +0300, Keith Packard wrote: > My previous attempt to fix this just hit the wrong variable to fix > edge triggering; it should have reset the POLLOUT bit in > ospoll->osfds[pos].revents instead of ospoll->fds[pos].revents. With > that fixed, it looks like the

[PATCH xserver 0/2] os: Fix write block handling in poll(2) path

2016-09-22 Thread Keith Packard
My previous attempt to fix this just hit the wrong variable to fix edge triggering; it should have reset the POLLOUT bit in ospoll->osfds[pos].revents instead of ospoll->fds[pos].revents. With that fixed, it looks like the poll(2) path is working. ___