"Jason J. Herne" <jjhe...@linux.vnet.ibm.com> wrote: > @@ -344,6 +359,18 @@ void qmp_migrate_set_parameters(bool has_compress_level, > "is invalid, it should be in the range of 1 to 255"); > return; > } > + if (has_x_cpu_throttle_initial && > + (x_cpu_throttle_initial < 1 || x_cpu_throttle_initial > 99)) { > + error_set(errp, QERR_INVALID_PARAMETER_VALUE, > + "x_cpu_throttle_initial", > + "an integer in the range of 1 to 99"); > + } > + if (has_x_cpu_throttle_increment && > + (x_cpu_throttle_increment < 1 || x_cpu_throttle_increment > 99)) > { > + error_set(errp, QERR_INVALID_PARAMETER_VALUE, > + "x_cpu_throttle_increment", > + "an integer in the range of 1 to 99"); > + }
s/error_set/error_setg/ the same than the rest of the file, and without that change it don't even compile against Today master. Done by me, no need to do anything. Later, Juan.