[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

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@xxxxxxxxxxxxx>
> > Signed-off-by: Sisu Xi <xisisu@xxxxxxxxx>
> >
> With the proper LIBXL_HAVE_XXX added:
> Reviewed-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
> 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
> or
> 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

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)

Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.