There is no reason to set it in label "err". We should be able to set it right after sending reply. It is easier to read.
Signed-off-by: Vivek Goyal <[email protected]> --- tools/virtiofsd/fuse_virtio.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/tools/virtiofsd/fuse_virtio.c b/tools/virtiofsd/fuse_virtio.c index aa53808ef9..b1767dd5b9 100644 --- a/tools/virtiofsd/fuse_virtio.c +++ b/tools/virtiofsd/fuse_virtio.c @@ -446,12 +446,9 @@ int virtio_send_data_iov(struct fuse_session *se, struct fuse_chan *ch, vu_queue_notify(dev, q); pthread_mutex_unlock(&qi->vq_lock); vu_dispatch_unlock(qi->virtio_dev); + req->reply_sent = true; err: - if (ret == 0) { - req->reply_sent = true; - } - return ret; } -- 2.25.4 _______________________________________________ Virtio-fs mailing list [email protected] https://listman.redhat.com/mailman/listinfo/virtio-fs
