[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] block: split block/qapi.c to avoid linking util
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH] block: split block/qapi.c to avoid linking utilities with qapi |
Date: |
Mon, 28 Jan 2019 11:52:48 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.4.0 |
On 1/28/19 11:39 AM, Anton Kuchin wrote:
> Only part of block/qapi.c is used by qemu-io qemu-nbd and qemu-img
> and its not realy about QAPI, so move it to separate file to reduce
s/realy/really/
> amount of unused code linked to utilities and avoid unnecessary
> dependencies.
That's a decent start for a good commit message; although giving even
more details about which interfaces are candidates for moving (to make
it easier for future patches to know which of the two files to add to)
would be helpful.
Also, do you have any before/after comparisons, perhaps such as size of
the executables or of 'nm | wc' output, proving that the split makes an
obvious difference?
>
> Signed-off-by: Anton Kuchin <address@hidden>
> ---
> block.c | 2 +-
> block/Makefile.objs | 3 +-
> block/bdrv_info.c | 582 ++++++++++++++++++++++++++
> block/qapi.c | 553 +-----------------------
> hmp.c | 2 +-
> include/block/{qapi.h => bdrv_info.h} | 6 +-
> monitor.c | 2 +-
> qemu-img.c | 2 +-
> qemu-io-cmds.c | 2 +-
> 9 files changed, 593 insertions(+), 561 deletions(-)
> create mode 100644 block/bdrv_info.c
> rename include/block/{qapi.h => bdrv_info.h} (97%)
>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3226
Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature