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  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 poll(2) path is working.
> 



smime.p7s
Description: S/MIME cryptographic signature
___
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel

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 poll(2) path is working.
> 

Tested-by: Matthieu Herrb 

Thanks !
-- 
Matthieu Herrb


signature.asc
Description: PGP signature
___
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel

[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.

___
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel