[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v6 13/18] block/sheepdog: drop have_co_req() and
From: |
Wenchao Xia |
Subject: |
Re: [Qemu-devel] [PATCH v6 13/18] block/sheepdog: drop have_co_req() and aio_flush_request() |
Date: |
Mon, 29 Jul 2013 16:24:45 +0800 |
User-agent: |
Mozilla/5.0 (Windows NT 5.1; rv:17.0) Gecko/20130620 Thunderbird/17.0.7 |
Reviewed-by: Wenchao Xia <address@hidden>
> .io_flush() is no longer called so drop have_co_req() and
> aio_flush_request().
>
> Signed-off-by: Stefan Hajnoczi <address@hidden>
> ---
> block/sheepdog.c | 25 +++++--------------------
> 1 file changed, 5 insertions(+), 20 deletions(-)
>
> diff --git a/block/sheepdog.c b/block/sheepdog.c
> index 6a41ad9..e216047 100644
> --- a/block/sheepdog.c
> +++ b/block/sheepdog.c
> @@ -509,13 +509,6 @@ static void restart_co_req(void *opaque)
> qemu_coroutine_enter(co, NULL);
> }
>
> -static int have_co_req(void *opaque)
> -{
> - /* this handler is set only when there is a pending request, so
> - * always returns 1. */
> - return 1;
> -}
> -
> typedef struct SheepdogReqCo {
> int sockfd;
> SheepdogReq *hdr;
> @@ -538,14 +531,14 @@ static coroutine_fn void do_co_req(void *opaque)
> unsigned int *rlen = srco->rlen;
>
> co = qemu_coroutine_self();
> - qemu_aio_set_fd_handler(sockfd, NULL, restart_co_req, have_co_req, co);
> + qemu_aio_set_fd_handler(sockfd, NULL, restart_co_req, NULL, co);
>
> ret = send_co_req(sockfd, hdr, data, wlen);
> if (ret < 0) {
> goto out;
> }
>
> - qemu_aio_set_fd_handler(sockfd, restart_co_req, NULL, have_co_req, co);
> + qemu_aio_set_fd_handler(sockfd, restart_co_req, NULL, NULL, co);
>
> ret = qemu_co_recv(sockfd, hdr, sizeof(*hdr));
> if (ret < sizeof(*hdr)) {
> @@ -796,14 +789,6 @@ static void co_write_request(void *opaque)
> qemu_coroutine_enter(s->co_send, NULL);
> }
>
> -static int aio_flush_request(void *opaque)
> -{
> - BDRVSheepdogState *s = opaque;
> -
> - return !QLIST_EMPTY(&s->inflight_aio_head) ||
> - !QLIST_EMPTY(&s->pending_aio_head);
> -}
> -
> /*
> * Return a socket discriptor to read/write objects.
> *
> @@ -819,7 +804,7 @@ static int get_sheep_fd(BDRVSheepdogState *s)
> return fd;
> }
>
> - qemu_aio_set_fd_handler(fd, co_read_response, NULL, aio_flush_request,
> s);
> + qemu_aio_set_fd_handler(fd, co_read_response, NULL, NULL, s);
> return fd;
> }
>
> @@ -1070,7 +1055,7 @@ static int coroutine_fn
> add_aio_request(BDRVSheepdogState *s, AIOReq *aio_req,
> qemu_co_mutex_lock(&s->lock);
> s->co_send = qemu_coroutine_self();
> qemu_aio_set_fd_handler(s->fd, co_read_response, co_write_request,
> - aio_flush_request, s);
> + NULL, s);
> socket_set_cork(s->fd, 1);
>
> /* send a header */
> @@ -1092,7 +1077,7 @@ static int coroutine_fn
> add_aio_request(BDRVSheepdogState *s, AIOReq *aio_req,
>
> socket_set_cork(s->fd, 0);
> qemu_aio_set_fd_handler(s->fd, co_read_response, NULL,
> - aio_flush_request, s);
> + NULL, s);
> qemu_co_mutex_unlock(&s->lock);
>
> return 0;
>
--
Best Regards
Wenchao Xia
- [Qemu-devel] [PATCH v6 16/18] thread-pool: drop thread_pool_active(), (continued)
- [Qemu-devel] [PATCH v6 16/18] thread-pool: drop thread_pool_active(), Stefan Hajnoczi, 2013/07/25
- [Qemu-devel] [PATCH v6 12/18] block/rbd: drop qemu_rbd_aio_flush_cb(), Stefan Hajnoczi, 2013/07/25
- [Qemu-devel] [PATCH v6 14/18] block/ssh: drop return_true(), Stefan Hajnoczi, 2013/07/25
- [Qemu-devel] [PATCH v6 10/18] block/linux-aio: drop qemu_laio_completion_cb(), Stefan Hajnoczi, 2013/07/25
- [Qemu-devel] [PATCH v6 13/18] block/sheepdog: drop have_co_req() and aio_flush_request(), Stefan Hajnoczi, 2013/07/25
- [Qemu-devel] [PATCH v6 09/18] block/iscsi: drop iscsi_process_flush(), Stefan Hajnoczi, 2013/07/25
- [Qemu-devel] [PATCH v6 07/18] block/curl: drop curl_aio_flush(), Stefan Hajnoczi, 2013/07/25
- [Qemu-devel] [PATCH v6 05/18] tests: adjust test-thread-pool to new aio_poll() semantics, Stefan Hajnoczi, 2013/07/25