Re: [Qemu-devel] [PATCH v6 2/2] iothread: let aio_epoll_disable fit to aio_context_destroy

2018-05-17 Thread WangJie (Pluto)
I enjoyed the great benefit of your suggestions, and I will improve next time. :) This time, I ask maintainers to touch up the commit message base on version 5 and merge it, thanks very much. On 2018/5/17 14:22, Peter Xu wrote: > On Thu, May 17, 2018 at 10:26:17AM +0800, Jie Wang wrote: >>

Re: [Qemu-devel] [PATCH v6 2/2] iothread: let aio_epoll_disable fit to aio_context_destroy

2018-05-17 Thread Peter Xu
On Thu, May 17, 2018 at 10:26:17AM +0800, Jie Wang wrote: > epoll_available will only be set if epollfd != -1, os we > can swap the two variables in aio_epoll_disable, and > aio_context_destroy can call aio_epoll_disable directly. > > Signed-off-by: Jie Wang > --- >

Re: [Qemu-devel] [PATCH v6 2/2] iothread: let aio_epoll_disable fit to aio_context_destroy

2018-05-16 Thread Fam Zheng
On Thu, 05/17 10:26, Jie Wang wrote: > epoll_available will only be set if epollfd != -1, os we s/os/so/ > can swap the two variables in aio_epoll_disable, and > aio_context_destroy can call aio_epoll_disable directly. If you put this as 1/2 in v7, you do not want to mention the

[Qemu-devel] [PATCH v6 2/2] iothread: let aio_epoll_disable fit to aio_context_destroy

2018-05-16 Thread Jie Wang
epoll_available will only be set if epollfd != -1, os we can swap the two variables in aio_epoll_disable, and aio_context_destroy can call aio_epoll_disable directly. Signed-off-by: Jie Wang --- util/aio-posix.c | 10 -- 1 file changed, 4 insertions(+), 6