[v2] virtiofsd: Few cleanups in virtio_send_data_iov()

[PATCH v2 0/7] virtiofsd: Few cleanups in virtio_send_data_iov()

Vivek Goyal posted 7 patches 2 years, 11 months ago

Diff against v1
Download series mbox

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20210518213538.693422-1-vgoyal@redhat.com

Maintainers: "Dr. David Alan Gilbert" <dgilbert@redhat.com>, Stefan Hajnoczi <stefanha@redhat.com>

tools/virtiofsd/fuse_virtio.c | 81 +++++++++++------------------------
1 file changed, 25 insertions(+), 56 deletions(-)

Expand all Fold all

[PATCH v2 0/7] virtiofsd: Few cleanups in virtio_send_data_iov()

Posted by Vivek Goyal 2 years, 11 months ago

Hi,

This is V2 of the patches. Changes since V1 are.

- Took care of coding style issue.
- Got rid of label "err" completely in last patch. (Dave, Connor). 
- Captured Reviewed-by tags from Connor and David.

Code in virtio_send_data_iov() little twisted and complicated. This
patch series just tries to simplify it a bit to make it little easier
to read this piece of code.

Thanks
Vivek


Vivek Goyal (7):
  virtiofsd: Check for EINTR in preadv() and retry
  virtiofsd: Get rid of unreachable code in read
  virtiofsd: Use iov_discard_front() to skip bytes
  virtiofsd: get rid of in_sg_left variable
  virtiofsd: Simplify skip byte logic
  virtiofsd: Check EOF before short read
  virtiofsd: Set req->reply_sent right after sending reply

 tools/virtiofsd/fuse_virtio.c | 81 +++++++++++------------------------
 1 file changed, 25 insertions(+), 56 deletions(-)

-- 
2.25.4

Re: [PATCH v2 0/7] virtiofsd: Few cleanups in virtio_send_data_iov()

Posted by Dr. David Alan Gilbert 2 years, 11 months ago

* Vivek Goyal (vgoyal@redhat.com) wrote:
> Hi,
> 
> This is V2 of the patches. Changes since V1 are.
> 
> - Took care of coding style issue.
> - Got rid of label "err" completely in last patch. (Dave, Connor). 
> - Captured Reviewed-by tags from Connor and David.
> 
> Code in virtio_send_data_iov() little twisted and complicated. This
> patch series just tries to simplify it a bit to make it little easier
> to read this piece of code.

Queued

> Thanks
> Vivek
> 
> 
> Vivek Goyal (7):
>   virtiofsd: Check for EINTR in preadv() and retry
>   virtiofsd: Get rid of unreachable code in read
>   virtiofsd: Use iov_discard_front() to skip bytes
>   virtiofsd: get rid of in_sg_left variable
>   virtiofsd: Simplify skip byte logic
>   virtiofsd: Check EOF before short read
>   virtiofsd: Set req->reply_sent right after sending reply
> 
>  tools/virtiofsd/fuse_virtio.c | 81 +++++++++++------------------------
>  1 file changed, 25 insertions(+), 56 deletions(-)
> 
> -- 
> 2.25.4
> 
> 
-- 
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK