Re: [Xen-devel] [PATCH 2 of 2 v3] xl: check for meaningful combination of sedf config file parameters

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Fri, 2012-06-08 at 15:26 +0100, Ian Jackson wrote:
> George Dunlap writes ("Re: [Xen-devel] [PATCH 2 of 2 v3] xl: check for 
> meaningful combination of sedf config file parameters"):
> > On 08/06/12 12:10, Ian Jackson wrote:
> > > Dario Faggioli writes ("[PATCH 2 of 2 v3] xl: check for meaningful 
> > > combination of sedf config file parameters"):
> > >> As it happens in the implementation of `xl sched-sedf -d ...', some
> > >> consistency checking is needed for the scheduling parameters when
> > >> they come from the config file.
> > > Thanks, I am happy with this from a libxl point of view.  I would like
> > > to see an ack from George from a scheduler point of view.
> > I think Dario knows more about what sedf wants than I do; and since it 
> > only does anything when the domain is using sedf, I think it's fine.
> > 
> > So FWIW:
> > 
> > Acked-by: George Dunlap <george.dunlap@xxxxxxxxxxxxx>
> 
> Great, thanks.
> 
> Committed-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>

Was it deliberate that you took this one but not the other patch from
this series of two, namely "libxl: propagete down the error from
libxl_domain_sched_params_set"?

Ian.



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel