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