[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 2/4] 9pfs: drop pdu_push_and_notify()
From: |
Stefano Stabellini |
Subject: |
Re: [Qemu-devel] [PATCH v2 2/4] 9pfs: drop pdu_push_and_notify() |
Date: |
Thu, 27 Apr 2017 11:17:06 -0700 (PDT) |
User-agent: |
Alpine 2.10 (DEB 1266 2009-07-14) |
On Thu, 27 Apr 2017, Greg Kurz wrote:
> Only pdu_complete() needs to notify the client that a request has completed.
>
> Signed-off-by: Greg Kurz <address@hidden>
> ---
> hw/9pfs/9p.c | 7 +------
> 1 file changed, 1 insertion(+), 6 deletions(-)
>
> diff --git a/hw/9pfs/9p.c b/hw/9pfs/9p.c
> index c80ba67389ce..01deffa0c3b5 100644
> --- a/hw/9pfs/9p.c
> +++ b/hw/9pfs/9p.c
> @@ -65,11 +65,6 @@ ssize_t pdu_unmarshal(V9fsPDU *pdu, size_t offset, const
> char *fmt, ...)
> return ret;
> }
>
> -static void pdu_push_and_notify(V9fsPDU *pdu)
I would probably turn this into a static inline and keep it around as
syntactic sugar. Regardless:
Reviewed-by: Stefano Stabellini <address@hidden>
> -{
> - pdu->s->transport->push_and_notify(pdu);
> -}
> -
> static int omode_to_uflags(int8_t mode)
> {
> int ret = 0;
> @@ -668,7 +663,7 @@ static void coroutine_fn pdu_complete(V9fsPDU *pdu,
> ssize_t len)
> pdu->size = len;
> pdu->id = id;
>
> - pdu_push_and_notify(pdu);
> + pdu->s->transport->push_and_notify(pdu);
>
> /* Now wakeup anybody waiting in flush for this request */
> if (!qemu_co_queue_next(&pdu->complete)) {
>
>
- [Qemu-devel] [PATCH v2 0/4] 9pfs: handle transport errors, Greg Kurz, 2017/04/27
- [Qemu-devel] [PATCH v2 1/4] fsdev: don't allow unknown format in marshal/unmarshal, Greg Kurz, 2017/04/27
- [Qemu-devel] [PATCH v2 2/4] 9pfs: drop pdu_push_and_notify(), Greg Kurz, 2017/04/27
- Re: [Qemu-devel] [PATCH v2 2/4] 9pfs: drop pdu_push_and_notify(),
Stefano Stabellini <=
- [Qemu-devel] [PATCH v2 3/4] virtio-9p: factor out virtio_9p_error_err(), Greg Kurz, 2017/04/27
- [Qemu-devel] [PATCH v2 4/4] 9pfs: handle broken transport, Greg Kurz, 2017/04/27
- Re: [Qemu-devel] [PATCH v2 0/4] 9pfs: handle transport errors, no-reply, 2017/04/27