qemu-block
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-block] [PATCH v3 2/3] nbd: Split nbd.c


From: Daniel P. Berrange
Subject: Re: [Qemu-block] [PATCH v3 2/3] nbd: Split nbd.c
Date: Thu, 14 Jan 2016 17:26:10 +0000
User-agent: Mutt/1.5.24 (2015-08-30)

On Thu, Jan 14, 2016 at 04:41:02PM +0800, Fam Zheng wrote:
> We have NBD server code and client code, all mixed in a file. Now split
> them into separate files under nbd/, and update MAINTAINERS.
> 
> filter_nbd for iotest 083 is updated to keep the log filtered out.
> 
> Signed-off-by: Fam Zheng <address@hidden>
> ---
>  MAINTAINERS                  |   5 +-
>  Makefile.objs                |   3 +-
>  include/block/nbd-internal.h | 113 +++++++++++
>  nbd/Makefile.objs            |   1 +
>  nbd/client.c                 | 361 ++++++++++++++++++++++++++++++++++
>  nbd/common.c                 |  64 ++++++
>  nbd.c => nbd/server.c        | 451 
> +------------------------------------------
>  tests/qemu-iotests/083       |   2 +-
>  8 files changed, 547 insertions(+), 453 deletions(-)
>  create mode 100644 include/block/nbd-internal.h
>  create mode 100644 nbd/Makefile.objs
>  create mode 100644 nbd/client.c
>  create mode 100644 nbd/common.c
>  rename nbd.c => nbd/server.c (68%)

Reviewed-by: Daniel P. Berrange <address@hidden>

Splitting this up is well overdue. Though this will conflict
horribly with my TLS series, that's not the end of the world
if Paolo wishes to merge this quickly.


Regards,
Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|



reply via email to

[Prev in Thread] Current Thread [Next in Thread]