[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 4/7] block: convert ThrottleGroup to object w
From: |
Alberto Garcia |
Subject: |
Re: [Qemu-devel] [PATCH v4 4/7] block: convert ThrottleGroup to object with QOM |
Date: |
Thu, 10 Aug 2017 16:44:36 +0200 |
User-agent: |
Notmuch/0.18.2 (http://notmuchmail.org) Emacs/24.4.1 (i586-pc-linux-gnu) |
On Wed 09 Aug 2017 12:07:31 PM CEST, Manos Pitsidianakis wrote:
> + /* unfix buckets to check validity */
> + throttle_get_config(&tg->ts, cfg);
> + if (!throttle_is_valid(cfg, errp)) {
> + return;
> + }
> + /* fix buckets again */
> + throttle_config(&tg->ts, tg->clock_type, cfg);
Ugh, this whole fix/unfix buckets should be transparent to outside users
of throttle.c.
You can leave this here for now but I'll think of a way to eliminate the
need to do this.
Berto
- [Qemu-devel] [PATCH v4 0/7] add throttle block driver filter, Manos Pitsidianakis, 2017/08/09
- [Qemu-devel] [PATCH v4 2/7] block: add aio_context field in ThrottleGroupMember, Manos Pitsidianakis, 2017/08/09
- [Qemu-devel] [PATCH v4 1/7] block: move ThrottleGroup membership to ThrottleGroupMember, Manos Pitsidianakis, 2017/08/09
- [Qemu-devel] [PATCH v4 3/7] block: tidy ThrottleGroupMember initializations, Manos Pitsidianakis, 2017/08/09
- [Qemu-devel] [PATCH v4 5/7] block: add throttle block filter driver, Manos Pitsidianakis, 2017/08/09
- [Qemu-devel] [PATCH v4 6/7] block: add BlockDevOptionsThrottle to QAPI, Manos Pitsidianakis, 2017/08/09
- [Qemu-devel] [PATCH v4 4/7] block: convert ThrottleGroup to object with QOM, Manos Pitsidianakis, 2017/08/09
- Re: [Qemu-devel] [PATCH v4 4/7] block: convert ThrottleGroup to object with QOM,
Alberto Garcia <=
- [Qemu-devel] [PATCH v4 7/7] block: add throttle block filter driver interface tests, Manos Pitsidianakis, 2017/08/09