[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v1 0/3] fsdev-throttle-qmp: refactor code for qm
From: |
Greg Kurz |
Subject: |
Re: [Qemu-devel] [PATCH v1 0/3] fsdev-throttle-qmp: refactor code for qmp interface |
Date: |
Tue, 13 Nov 2018 11:28:55 +0100 |
Hi Kidd,
This posting is still broken: no threading and missing leading blanks on
unchanged lines. Also is x00390961 really your name of choice ?
In some other mail, you mentioned copy-paste to an outlook mail client...
please don't do that to post series. Use git format-patch and git send-email
instead and you'll get the formatting and threading right.
Cheers,
--
Greg
On Tue, 13 Nov 2018 09:59:54 +0000
xiezhide <address@hidden> wrote:
> This patches provide qmp interface to query/set io throttle parameters of a
> fsdev.
> Some of patches also refactor the code and structure that was present in
> block and fsdev files.
>
> x00390961 (3):
> fsdev-throttle-qmp: refactor code for qmp interface for io throttling
> fsdev-throttle-qmp: qmp interface for fsdev io throttling
> fsdev-throttle-qmp: hmp interface for fsdev io throttling
>
> Makefile | 20 +++-
> Makefile.objs | 8 ++
> block/throttle.c | 6 +-
> blockdev.c | 96 +----------------
> fsdev/qemu-fsdev-dummy.c | 11 ++
> fsdev/qemu-fsdev-throttle.c | 144 +++++++++++++++++---------
> fsdev/qemu-fsdev-throttle.h | 6 +-
> fsdev/qemu-fsdev.c | 29 ++++++
> hmp-commands-info.hx | 15 +++
> hmp-commands.hx | 15 +++
> hmp.c | 83 +++++++++++++--
> hmp.h | 4 +
> include/qemu/throttle-options.h | 3 +-
> include/qemu/throttle.h | 4 +-
> include/qemu/typedefs.h | 1 +
> monitor.c | 41 ++++----
> qapi/block-core.json | 122 +---------------------
> qapi/fsdev.json | 96 +++++++++++++++++
> qapi/qapi-schema.json | 1 +
> qapi/tlimits.json | 89 ++++++++++++++++
> qmp.c | 12 +++
> util/throttle.c | 224
> ++++++++++++++++++++++++++--------------
> 22 files changed, 654 insertions(+), 376 deletions(-)
> create mode 100644 qapi/fsdev.json
> create mode 100644 qapi/tlimits.json
>
> v0 -> v1:
> Addressed comments from Eric Blake and Greg Kurz.
> Fix patch corrupt issue due to email client change the patch format with
> copy-to-paster
> Break patch to patches
>
> --
> 1.8.3.1
>