Skip to content

Commit

Permalink
block/sheepdog: drop have_co_req() and aio_flush_request()
Browse files Browse the repository at this point in the history
.io_flush() is no longer called so drop have_co_req() and
aio_flush_request().

Signed-off-by: Stefan Hajnoczi <[email protected]>
  • Loading branch information
stefanhaRH committed Aug 19, 2013
1 parent 5d289cc commit d6d94c6
Showing 1 changed file with 5 additions and 20 deletions.
25 changes: 5 additions & 20 deletions block/sheepdog.c
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand All @@ -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)) {
Expand Down Expand Up @@ -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.
*
Expand All @@ -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;
}

Expand Down Expand Up @@ -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 */
Expand All @@ -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;
Expand Down

0 comments on commit d6d94c6

Please sign in to comment.