[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v4 00/21] nbd: add qemu-nbd --list
From: |
Vladimir Sementsov-Ogievskiy |
Subject: |
Re: [Qemu-block] [PATCH v4 00/21] nbd: add qemu-nbd --list |
Date: |
Fri, 18 Jan 2019 08:15:20 +0000 |
17.01.2019 22:36, Eric Blake wrote:
> I got tired of debugging whether a server was advertising the
> correct things during negotiation by inspecting the trace
> logs of qemu-io as client - not to mention that without SOME
> sort of client tracing particular commands, we can't easily
> regression test the server for correct behavior. The final
> straw was at KVM Forum, when Nir asked me to make sure there
> was a way to easily determine if an NBD server is exposing what
> we really want (and fixing x-dirty-bitmap to behave saner fell
> out as a result of answering that question).
>
> I note that upstream NBD has 'nbd-client -l $host' for querying
> just export names (with no quoting, so you have to know that
> a blank line means the default export), but it wasn't powerful
> enough, so I implemented 'qemu-nbd -L' to document everything.
> Upstream NBD has separate 'nbd-client' and 'nbd-server' binaries,
> while we only have 'qemu-nbd' (which is normally just a server,
> but 'qemu-nbd -c' also operates a second thread as a client).
> Our other uses of qemu as NBD client are for consuming a block
> device (as in qemu-io, qemu-img, or a drive to qemu) - but those
> binaries are less suited to something so specific to the NBD
> protocol.
>
> Bonus: As a result of my work on this series, nbdkit now supports
> NBD_OPT_INFO (my interoperability testing between server
> implementations has been paying off, both at fixing server bugs,
> and at making this code more reliable across difference in valid
> servers).
>
> Also available at:
> https://repo.or.cz/qemu/ericb.git qemu-nbd-list-v4
>
> Currently based on master.
>
> Since v3:
> - 1 new patch (1/21)
> - split old patch 15/19 into two (16,17/21)
> - retitle two patches (git backport-diff doesn't do too well at showing
> the diff on a retitled patch; 5,11/21)
> - fix review comments from Rich, Vladimir
>
> 001/21:[down] 'iotests: Make 233 output more reliable'
> 002/21:[----] [--] 'maint: Allow for EXAMPLES in texi2pod'
> 003/21:[----] [--] 'qemu-nbd: Enhance man page'
Interesting, I don't get it again. Searched in outlook online, I found only v2
of it,
checked spam folder and filters. Magic.
anyway, I can look at
https://lists.gnu.org/archive/html/qemu-devel/2019-01/msg04328.html
--
Best regards,
Vladimir
- [Qemu-block] [PATCH v4 18/21] nbd/client: Add meta contexts to nbd_receive_export_list(), (continued)
- [Qemu-block] [PATCH v4 18/21] nbd/client: Add meta contexts to nbd_receive_export_list(), Eric Blake, 2019/01/17
- [Qemu-block] [PATCH v4 16/21] nbd/client: Refactor nbd_opt_go() to support NBD_OPT_INFO, Eric Blake, 2019/01/17
- [Qemu-block] [PATCH v4 12/21] nbd/client: Split out nbd_receive_one_meta_context(), Eric Blake, 2019/01/17
- [Qemu-block] [PATCH v4 17/21] nbd/client: Add nbd_receive_export_list(), Eric Blake, 2019/01/17
- [Qemu-block] [PATCH v4 19/21] qemu-nbd: Add --list option, Eric Blake, 2019/01/17
- [Qemu-block] [PATCH v4 14/21] nbd/client: Split handshake into two functions, Eric Blake, 2019/01/17
- [Qemu-block] [PATCH v4 13/21] nbd/client: Refactor return of nbd_receive_negotiate(), Eric Blake, 2019/01/17
- Re: [Qemu-block] [PATCH v4 00/21] nbd: add qemu-nbd --list,
Vladimir Sementsov-Ogievskiy <=
- Re: [Qemu-block] [PATCH v4 00/21] nbd: add qemu-nbd --list, Vladimir Sementsov-Ogievskiy, 2019/01/18
- Re: [Qemu-block] [PATCH v4 00/21] nbd: add qemu-nbd --list, Eric Blake, 2019/01/18
- Re: [Qemu-block] [PATCH v4 00/21] nbd: add qemu-nbd --list, Eric Blake, 2019/01/18
- Re: [Qemu-block] [PATCH v4 00/21] nbd: add qemu-nbd --list, Richard W.M. Jones, 2019/01/19
- Re: [Qemu-block] [PATCH v4 00/21] nbd: add qemu-nbd --list, Richard W.M. Jones, 2019/01/19
- Re: [Qemu-block] [PATCH v4 00/21] nbd: add qemu-nbd --list, Richard W.M. Jones, 2019/01/20
Re: [Qemu-block] [Qemu-devel] [PATCH v4 00/21] nbd: add qemu-nbd --list, Eric Blake, 2019/01/18