[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v10 6/6] fsdev: hmp interface for throttling
From: |
Alberto Garcia |
Subject: |
Re: [Qemu-devel] [PATCH v10 6/6] fsdev: hmp interface for throttling |
Date: |
Tue, 05 Sep 2017 09:53:18 +0200 |
User-agent: |
Notmuch/0.18.2 (http://notmuchmail.org) Emacs/24.4.1 (i586-pc-linux-gnu) |
On Mon 04 Sep 2017 06:07:47 PM CEST, Pradeep Jagadeesh wrote:
> +void hmp_info_fsdev_iothrottle(Monitor *mon, const QDict *qdict)
> +{
> + Error *err = NULL;
> + IOThrottleList *fsdev_list, *info;
> + fsdev_list = qmp_query_fsdev_io_throttle(&err);
> +
> + for (info = fsdev_list; info; info = info->next) {
> + print_fsdev_throttle_config(mon, info->value);
> + }
> + qapi_free_IOThrottleList(fsdev_list);
> +}
You're passing an Error to qmp_query_fsdev_io_throttle() but then you
don't handle it. Use hmp_handle_error() as I said in my previous e-mail.
I know that with the current code qmp_query_fsdev_io_throttle() is never
going to fail, but that's no reason to declare an Error and then ignore
it.
Berto
- [Qemu-devel] [PATCH v10 0/6] fsdev: qmp interface for io throttling, Pradeep Jagadeesh, 2017/09/04
- [Qemu-devel] [PATCH v10 2/6] qmp: Create IOThrottle structure, Pradeep Jagadeesh, 2017/09/04
- [Qemu-devel] [PATCH v10 3/6] throttle: move out function to reuse the code, Pradeep Jagadeesh, 2017/09/04
- [Qemu-devel] [PATCH v10 1/6] throttle: factor out duplicate code, Pradeep Jagadeesh, 2017/09/04
- [Qemu-devel] [PATCH v10 6/6] fsdev: hmp interface for throttling, Pradeep Jagadeesh, 2017/09/04
- Re: [Qemu-devel] [PATCH v10 6/6] fsdev: hmp interface for throttling,
Alberto Garcia <=
- Re: [Qemu-devel] [PATCH v10 6/6] fsdev: hmp interface for throttling, Pradeep Jagadeesh, 2017/09/05
- Re: [Qemu-devel] [PATCH v10 6/6] fsdev: hmp interface for throttling, Greg Kurz, 2017/09/05
- Re: [Qemu-devel] [PATCH v10 6/6] fsdev: hmp interface for throttling, Alberto Garcia, 2017/09/05
- Re: [Qemu-devel] [PATCH v10 6/6] fsdev: hmp interface for throttling, Pradeep Jagadeesh, 2017/09/05
- Re: [Qemu-devel] [PATCH v10 6/6] fsdev: hmp interface for throttling, Alberto Garcia, 2017/09/05
- Re: [Qemu-devel] [PATCH v10 6/6] fsdev: hmp interface for throttling, Pradeep Jagadeesh, 2017/09/05
Re: [Qemu-devel] [PATCH v10 6/6] fsdev: hmp interface for throttling, Dr. David Alan Gilbert, 2017/09/05
[Qemu-devel] [PATCH v10 5/6] fsdev: QMP interface for throttling, Pradeep Jagadeesh, 2017/09/04