On Tue, 2017-08-01 at 14:33 -0400, Meng Xu wrote: > diff --git a/tools/libxl/libxl_types.idl > b/tools/libxl/libxl_types.idl > index 8a9849c..f6c3ead 100644 > --- a/tools/libxl/libxl_types.idl > +++ b/tools/libxl/libxl_types.idl > @@ -401,6 +401,7 @@ libxl_sched_params = Struct("sched_params",[ > ("period", integer, {'init_val': > 'LIBXL_DOMAIN_SCHED_PARAM_PERIOD_DEFAULT'}), > ("extratime", integer, {'init_val': > 'LIBXL_DOMAIN_SCHED_PARAM_EXTRATIME_DEFAULT'}), > ("budget", integer, {'init_val': > 'LIBXL_DOMAIN_SCHED_PARAM_BUDGET_DEFAULT'}), > + ("is_work_conserving", integer, {'init_val': > 'LIBXL_DOMAIN_SCHED_PARAM_IS_WORK_CONSERVING_DEFAULT'}), > ]) > How about, here at libxl level, we use the "extratime" field that we have as a leftover from SEDF (and which had, in that scheduler, a similar meaning)?
If we don't want to use that one, and we want a new field, I suggest thinking to a shorter name. Regards, Dario -- <<This happens because I choose it to happen!>> (Raistlin Majere) ----------------------------------------------------------------- Dario Faggioli, Ph.D, http://about.me/dario.faggioli Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)
signature.asc
Description: This is a digitally signed message part
_______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel