[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v6 2/2] iothread: let aio_epoll_disable fit to a
From: |
WangJie (Pluto) |
Subject: |
Re: [Qemu-block] [PATCH v6 2/2] iothread: let aio_epoll_disable fit to aio_context_destroy |
Date: |
Thu, 17 May 2018 14:48:53 +0800 |
User-agent: |
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 |
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:
>> 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 <address@hidden>
>> ---
>> util/aio-posix.c | 10 ++++------
>> 1 file changed, 4 insertions(+), 6 deletions(-)
>>
>> diff --git a/util/aio-posix.c b/util/aio-posix.c
>> index 0ade2c7..118bf57 100644
>> --- a/util/aio-posix.c
>> +++ b/util/aio-posix.c
>> @@ -45,11 +45,11 @@ struct AioHandler
>>
>> static void aio_epoll_disable(AioContext *ctx)
>> {
>> - ctx->epoll_available = false;
>> - if (!ctx->epoll_enabled) {
>> + ctx->epoll_enabled = false;
>> + if (!ctx->epoll_available) {
>> return;
>> }
>> - ctx->epoll_enabled = false;
>> + ctx->epoll_available = false;
>> close(ctx->epollfd);
>> }
>>
>> @@ -716,9 +716,7 @@ void aio_context_setup(AioContext *ctx)
>> void aio_context_destroy(AioContext *ctx)
>> {
>> #ifdef CONFIG_EPOLL_CREATE1
>> - if (ctx->epollfd >= 0) {
>> - close(ctx->epollfd);
>> - }
>> + aio_epoll_disable(ctx);
>
> Hmm... I think this patch should be the first if to split.
>
> Anyway, IMHO version 5 is already good enough and has got r-bs, no
> need to bother reposting a version 7. Maintainer could possibly touch
> up the commit message if necessary.
>
> Thanks,
>
>> #endif
>> }
>>
>> --
>> 1.8.3.1
>>
>