Re: [Qemu-devel] [PATCH v1 0/3] fsdev-throttle-qmp: refactor code for qmp interface

[Greg Kurz <groug@kaod.org>]

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 <xiezhide@huawei.com> 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
>