Re: [Spice-devel] Further thoughts on the write polling in the xf86_spice_xql driver

2012-06-20 Thread Jeremy White
I've started a conversation on the xorg-devel mailing list[1], and I'll follow up directly in irc with Adam and/or Keith if I don't get any answers. Keith very kindly put together a patch to add write block handlers for us: http://lists.x.org/archives/xorg-devel/2012-June/031783.html It not

Re: [Spice-devel] Further thoughts on the write polling in the xf86_spice_xql driver

2012-06-20 Thread Alon Levy
On Wed, Jun 20, 2012 at 08:59:20AM -0500, Jeremy White wrote: I've started a conversation on the xorg-devel mailing list[1], and I'll follow up directly in irc with Adam and/or Keith if I don't get any answers. Keith very kindly put together a patch to add write block handlers for us:

Re: [Spice-devel] Further thoughts on the write polling in the xf86_spice_xql driver

2012-06-14 Thread Jeremy White
Okay; I unimaginatively decided that would be Hard. Wouldn't that break the ABI, and therefore be rejected unilaterally? I'm willing to try to craft a patch and submit it; but maybe I just need encouragement grin. I started looking at this, talked to Dave Airlie and he suggested to ask on

Re: [Spice-devel] Further thoughts on the write polling in the xf86_spice_xql driver

2012-06-05 Thread Alon Levy
On Mon, Jun 04, 2012 at 08:42:17AM -0500, Jeremy White wrote: However, while the X server main loop has a facility that lets us add fds to it's select read mask, it has no such facility to let us add fds to it's *write* select mask. Right. This is the problem. I think the right

Re: [Spice-devel] Further thoughts on the write polling in the xf86_spice_xql driver

2012-06-04 Thread Jeremy White
However, while the X server main loop has a facility that lets us add fds to it's select read mask, it has no such facility to let us add fds to it's *write* select mask. Right. This is the problem. I think the right solution is to patch the X server. Okay; I unimaginatively decided

Re: [Spice-devel] Further thoughts on the write polling in the xf86_spice_xql driver

2012-06-03 Thread Alon Levy
On Sat, Jun 02, 2012 at 01:24:29PM -0500, Jeremy White wrote: I spent a fair amount of energy trying to understand the issue with EAGAIN and the xspice server. I've sent in one patch which I believe to be correct. I had some further thoughts I wanted to share as well. To reprise, the

[Spice-devel] Further thoughts on the write polling in the xf86_spice_xql driver

2012-06-02 Thread Jeremy White
I spent a fair amount of energy trying to understand the issue with EAGAIN and the xspice server. I've sent in one patch which I believe to be correct. I had some further thoughts I wanted to share as well. To reprise, the issue occurs when we send the large ping test packet from the server;