[v3] nbd client refactoring and fixing

[Qemu-devel] [PATCH v3 0/3] nbd client refactoring and fixing

Vladimir Sementsov-Ogievskiy posted 3 patches 6 years, 7 months ago

Diff against v1 v2
Download series mbox

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

Test checkpatch passed

Test docker passed

Test s390x passed

block/nbd-client.c | 33 ++++++++++++++++++---------------
1 file changed, 18 insertions(+), 15 deletions(-)

Expand all Fold all

[Qemu-devel] [PATCH v3 0/3] nbd client refactoring and fixing

Posted by Vladimir Sementsov-Ogievskiy 6 years, 7 months ago

v3: dropped all controversial things. I'll try to implement minimal
structured reply and block status over this small residue.

Vladimir Sementsov-Ogievskiy (3):
  block/nbd-client: refactor nbd_co_receive_reply
  block/nbd-client: simplify check in nbd_co_receive_reply
  block/nbd-client: nbd_co_send_request: fix return code

 block/nbd-client.c | 33 ++++++++++++++++++---------------
 1 file changed, 18 insertions(+), 15 deletions(-)

-- 
2.11.1

Re: [Qemu-devel] [PATCH v3 0/3] nbd client refactoring and fixing

Posted by Eric Blake 6 years, 7 months ago

On 09/20/2017 07:45 AM, Vladimir Sementsov-Ogievskiy wrote:
> v3: dropped all controversial things. I'll try to implement minimal
> structured reply and block status over this small residue.
> 
> Vladimir Sementsov-Ogievskiy (3):
>   block/nbd-client: refactor nbd_co_receive_reply
>   block/nbd-client: simplify check in nbd_co_receive_reply
>   block/nbd-client: nbd_co_send_request: fix return code

Thanks; queued on my NBD tree, will send pull request after a bit more
time to see if it collects further reviews:
git://repo.or.cz/qemu/ericb.git nbd

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org

Re: [Qemu-devel] [PATCH v3 0/3] nbd client refactoring and fixing

Posted by Vladimir Sementsov-Ogievskiy 6 years, 7 months ago

20.09.2017 20:22, Eric Blake wrote:
> On 09/20/2017 07:45 AM, Vladimir Sementsov-Ogievskiy wrote:
>> v3: dropped all controversial things. I'll try to implement minimal
>> structured reply and block status over this small residue.
>>
>> Vladimir Sementsov-Ogievskiy (3):
>>    block/nbd-client: refactor nbd_co_receive_reply
>>    block/nbd-client: simplify check in nbd_co_receive_reply
>>    block/nbd-client: nbd_co_send_request: fix return code
> Thanks; queued on my NBD tree, will send pull request after a bit more
> time to see if it collects further reviews:
> git://repo.or.cz/qemu/ericb.git nbd
>

Thank you, Eric!


-- 
Best regards,
Vladimir