Re: [PATCH v3 3/4] libxl: add rtds scheduler

[Ian Campbell <ian.campbell@citrix.com>]

On Tue, 2014-09-16 at 10:04 +0200, Dario Faggioli wrote:
> On Sun, 2014-09-14 at 17:37 -0400, Meng Xu wrote:
> > Add libxl functions to set/get domain's parameters for rtds scheduler
> > Note: VCPU's information (period, budget) is in microsecond (us).
> > 
> > Signed-off-by: Meng Xu <mengxu@cis.upenn.edu>
> > Signed-off-by: Sisu Xi <xisisu@gmail.com>
> >
> With the proper LIBXL_HAVE_XXX added:
> 
> Reviewed-by: Dario Faggioli <dario.faggioli@citrix.com>
> 
> As Ian said, look at libxl.h. You're adding a valid enum value ("rtds")
> in libxl_scheduler, and a "budget" field in libxl_domain_sched_params.
> 
> I'm not sure whether I'd go for something like
> 
> #define LIBXL_HAVE_SCHED_RTDS 1
> 
> or
> 
> #define LIBXL_HAVE_SCHEDPARAMS_BUDGET 1
> 
> If it were me, I probably would have added both, but perhaps both is too
> much...

Just the former is sufficient, having advertised the generic feature it
can then be assumed that the associated set of initially avaiable fields
and interfaces are tere, which is what the application developer cares
about.

If you like you can spell out exactly which fields that involves in the
associated comment (most of them do today, although in many cases IMHO
it isn't strictly needed)