Re: [Qemu-devel] [PATCH 3/3] iothread: delay the context release to finalize

2017-09-24 Thread Fam Zheng
On Mon, 09/25 13:50, Peter Xu wrote: > On Mon, Sep 25, 2017 at 01:30:02PM +0800, Fam Zheng wrote: > > On Mon, 09/25 13:23, Peter Xu wrote: > > > On Fri, Sep 22, 2017 at 09:09:22PM +0800, Fam Zheng wrote: > > > > On Fri, 09/22 16:56, Peter Xu wrote: > > > > > When gcontext is used with iothread,

Re: [Qemu-devel] [PATCH 3/3] iothread: delay the context release to finalize

2017-09-24 Thread Peter Xu
On Mon, Sep 25, 2017 at 01:30:02PM +0800, Fam Zheng wrote: > On Mon, 09/25 13:23, Peter Xu wrote: > > On Fri, Sep 22, 2017 at 09:09:22PM +0800, Fam Zheng wrote: > > > On Fri, 09/22 16:56, Peter Xu wrote: > > > > When gcontext is used with iothread, the context will be destroyed > > > > during

Re: [Qemu-devel] [PATCH 3/3] iothread: delay the context release to finalize

2017-09-24 Thread Fam Zheng
On Mon, 09/25 13:23, Peter Xu wrote: > On Fri, Sep 22, 2017 at 09:09:22PM +0800, Fam Zheng wrote: > > On Fri, 09/22 16:56, Peter Xu wrote: > > > When gcontext is used with iothread, the context will be destroyed > > > during iothread_stop(). That's not good since sometimes we would like > > > to

Re: [Qemu-devel] [PATCH 3/3] iothread: delay the context release to finalize

2017-09-24 Thread Peter Xu
On Fri, Sep 22, 2017 at 09:09:22PM +0800, Fam Zheng wrote: > On Fri, 09/22 16:56, Peter Xu wrote: > > When gcontext is used with iothread, the context will be destroyed > > during iothread_stop(). That's not good since sometimes we would like > > to keep the resources until iothread is destroyed,

Re: [Qemu-devel] [PATCH 3/3] iothread: delay the context release to finalize

2017-09-22 Thread Fam Zheng
On Fri, 09/22 16:56, Peter Xu wrote: > When gcontext is used with iothread, the context will be destroyed > during iothread_stop(). That's not good since sometimes we would like > to keep the resources until iothread is destroyed, but we may want to > stop the thread before that point. Would be

[Qemu-devel] [PATCH 3/3] iothread: delay the context release to finalize

2017-09-22 Thread Peter Xu
When gcontext is used with iothread, the context will be destroyed during iothread_stop(). That's not good since sometimes we would like to keep the resources until iothread is destroyed, but we may want to stop the thread before that point. Delay the destruction of gcontext to iothread