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-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-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 . > > I started looking at this, talked to Dave Airlie and he suggested to ask

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

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 d

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, t

[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; if