Re: [Qemu-devel] [PATCH 3/3] chardev: introduce qemu_chr_timeout_add() and use

2018-01-02 Thread Peter Xu
On Tue, Jan 02, 2018 at 05:10:01PM +0100, Marc-André Lureau wrote: > Hi > > On Thu, Dec 28, 2017 at 8:29 AM, Peter Xu wrote: > > It's a replacement of g_timeout_add[_seconds]() for chardevs. Chardevs > > now can have dedicated gcontext, we should always bind chardev tasks > > onto those gcontext

Re: [Qemu-devel] [PATCH 3/3] chardev: introduce qemu_chr_timeout_add() and use

2018-01-02 Thread Marc-André Lureau
Hi On Thu, Dec 28, 2017 at 8:29 AM, Peter Xu wrote: > It's a replacement of g_timeout_add[_seconds]() for chardevs. Chardevs > now can have dedicated gcontext, we should always bind chardev tasks > onto those gcontext rather than the default main context. Since there > are quite a few of g_time

[Qemu-devel] [PATCH 3/3] chardev: introduce qemu_chr_timeout_add() and use

2017-12-27 Thread Peter Xu
It's a replacement of g_timeout_add[_seconds]() for chardevs. Chardevs now can have dedicated gcontext, we should always bind chardev tasks onto those gcontext rather than the default main context. Since there are quite a few of g_timeout_add[_seconds]() callers, a new function qemu_chr_timeout_a