[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v6 4/6] hmp: create a throttle initialization fu
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [Qemu-devel] [PATCH v6 4/6] hmp: create a throttle initialization function |
Date: |
Fri, 30 Jun 2017 10:14:59 +0100 |
User-agent: |
Mutt/1.8.3 (2017-05-23) |
* Pradeep Jagadeesh (address@hidden) wrote:
> This patch creates a throttle initialization function to maximize the
> code reusability. The same code is also used by fsdev.
>
> Signed-off-by: Pradeep Jagadeesh <address@hidden>
Acked-by: Dr. David Alan Gilbert <address@hidden>
> ---
> hmp.c | 21 +++++++++++++++------
> 1 file changed, 15 insertions(+), 6 deletions(-)
>
> diff --git a/hmp.c b/hmp.c
> index 8c72c58..220d301 100644
> --- a/hmp.c
> +++ b/hmp.c
> @@ -1749,20 +1749,29 @@ void hmp_change(Monitor *mon, const QDict *qdict)
> hmp_handle_error(mon, &err);
> }
>
> +static void hmp_initialize_io_throttle(IOThrottle *iot, const QDict *qdict)
> +{
> + iot->has_id = true;
> + iot->id = (char *) qdict_get_str(qdict, "id");
> + iot->bps = qdict_get_int(qdict, "bps");
> + iot->bps_rd = qdict_get_int(qdict, "bps_rd");
> + iot->bps_wr = qdict_get_int(qdict, "bps_wr");
> + iot->iops = qdict_get_int(qdict, "iops");
> + iot->iops_rd = qdict_get_int(qdict, "iops_rd");
> + iot->iops_wr = qdict_get_int(qdict, "iops_wr");
> +}
> +
> void hmp_block_set_io_throttle(Monitor *mon, const QDict *qdict)
> {
> Error *err = NULL;
> + IOThrottle *iothrottle;
> BlockIOThrottle throttle = {
> .has_device = true,
> .device = (char *) qdict_get_str(qdict, "device"),
> - .bps = qdict_get_int(qdict, "bps"),
> - .bps_rd = qdict_get_int(qdict, "bps_rd"),
> - .bps_wr = qdict_get_int(qdict, "bps_wr"),
> - .iops = qdict_get_int(qdict, "iops"),
> - .iops_rd = qdict_get_int(qdict, "iops_rd"),
> - .iops_wr = qdict_get_int(qdict, "iops_wr"),
> };
>
> + iothrottle = qapi_BlockIOThrottle_base(&throttle);
> + hmp_initialize_io_throttle(iothrottle, qdict);
> qmp_block_set_io_throttle(&throttle, &err);
> hmp_handle_error(mon, &err);
> }
> --
> 1.8.3.1
>
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK
- [Qemu-devel] [PATCH v6 0/6] fsdev: qmp interface for io throttling, Pradeep Jagadeesh, 2017/06/29
- [Qemu-devel] [PATCH v6 2/6] qmp: Create IOThrottle structure, Pradeep Jagadeesh, 2017/06/29
- [Qemu-devel] [PATCH v6 1/6] throttle: factor out duplicate code, Pradeep Jagadeesh, 2017/06/29
- [Qemu-devel] [PATCH v6 4/6] hmp: create a throttle initialization function, Pradeep Jagadeesh, 2017/06/29
- Re: [Qemu-devel] [PATCH v6 4/6] hmp: create a throttle initialization function,
Dr. David Alan Gilbert <=
- [Qemu-devel] [PATCH v6 5/6] fsdev: hmp interface for throttling, Pradeep Jagadeesh, 2017/06/29
- [Qemu-devel] [PATCH 3/6] throttle: move out function to reuse the code, Pradeep Jagadeesh, 2017/06/29
- [Qemu-devel] [PATCH v6 6/6] fsdev: QMP interface for throttling, Pradeep Jagadeesh, 2017/06/29