Re: [Spice-devel] Is it possible to put spice channels into different threads?

2020-06-15 Thread 陈炤
Hi, After debugging, I think my problem is probably not accociated with cooperative multitasking. Here is where spice-gtk read data: /* treat all incoming data (block on message completion) */ while (!c->has_error && c->state != SPICE_CHANNEL_STATE_MIGRATING && g

Re: [Spice-devel] Review of the C++ patches

2020-06-15 Thread Frediano Ziglio
> commit 7b3637417062fe657c0bf1fced5ef59a489dbdc3 > Author: Frediano Ziglio > Date: Mon Mar 9 10:04:24 2020 + > > Adjust some warnings > > Remove -Werror and add -fpermissive, this will allow to compile C code with > a GNU C++ compiler. > > Ignore warnings as our code use some featur

Re: [Spice-devel] Review of the C++ patches

2020-06-15 Thread Frediano Ziglio
> > commit 61affed2a2e36b59e6935e608d7d80242f976c7e > Author: Frediano Ziglio > Date: Mon May 20 16:49:17 2019 +0100 > > red-channel-client: Better private initialisation > > Initialise RedChannelClientPrivate fields from the new > constructor instead from RedChannelClient. > Also change

Re: [Spice-devel] Review of the C++ patches

2020-06-15 Thread Frediano Ziglio
> > commit 43c6bf91b7c53ee9f93f7ea1cead5bba94c61f88 > Author: Frediano Ziglio > Date: Thu Mar 5 13:01:27 2020 + > > reds: Remove a weak pointer usage > > RedCharDevice can all be removed just calling unref, beside > the agent that needs special threatment. > > Signed-off-by: Frediano

Re: [Spice-devel] Is it possible to put spice channels into different threads?

2020-06-15 Thread Jakub Janku
Hi, yes, I think this is a real issue. I feel like I've stumbled across something similar in the past when working on webdav. So yeah, it seems like the channel might get blocked in one direction -- meaning data isn't sent until you read all available data. However, that doesn't mean that other

Re: [Spice-devel] Is it possible to put spice channels into different threads?

2020-06-15 Thread 陈炤
Hi, For now I simply used a counter to work around this issue. while (... g_pollable_input_stream_is_readable ) { if (loop > 10) break; } Ugly but seems useful. BR Don At 2020-06-15 20:59:00, "Jakub Janku" wrote: >Hi, > >yes, I think this is a real issue. I feel like