[Spice-devel] [PATCH spice 13/17] spicec-x11: Add XFlush calls were needed

Gerd Hoffmann kraxel at redhat.com
Tue Oct 5 02:08:35 PDT 2010


   Hi,

>  > is it the right way to do it?
>
> I believe so :) I had issues with this in the agent which also
> does a select on the server fd for read (as it needs to monitor
> the fd for vdagent messages as well). I had a case where I would
> not get an answer to a XConvertSelection, as I did not flush it.

The alternative approach would be to call xflush before entering 
select() / poll().  Advantage is that you have only one place where you 
call xflush.  Should work fine for vdagent.  spicec might be more tricky 
due to the abstraction layers we have there for x11/win portability.

cheers,
   Gerd



More information about the Spice-devel mailing list