From mboxrd@z Thu Jan 1 00:00:00 1970 From: George Dunlap Subject: Re: [PATCH] xen/sched_credit: Use delay to control scheduling frequency Date: Mon, 9 Jan 2012 09:49:02 +0000 Message-ID: <1326102542.13599.1.camel@elijah> References: <18f40e1d027491964a88.1324889181@wsm-ep-n0> <4F075204.3040408@eu.citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: "Lv, Hui" Cc: George Dunlap , "xen-devel@lists.xensource.com" , "raistlin@linux.it" , Ian Campbell , "JBeulich@suse.com" List-Id: xen-devel@lists.xenproject.org On Sun, 2012-01-08 at 12:03 +0000, Lv, Hui wrote: > Thanks, George. > Should I send a revised version? > Can it be checked in? Yes, please send a revised version. It will be checked in if one of the maintainers thinks there's enough consensus (probably if no one has any more comments in the next day or two). Thanks, -George > > -----Original Message----- > From: George Dunlap [mailto:george.dunlap@eu.citrix.com] > Sent: Saturday, January 07, 2012 3:57 AM > To: Lv, Hui > Cc: xen-devel@lists.xensource.com; raistlin@linux.it; JBeulich@suse.com; Ian Campbell > Subject: Re: [PATCH] xen/sched_credit: Use delay to control scheduling frequency > > Sorry for the delay; just catching up after the Christmas holidays. > > On 26/12/11 03:46, Hui Lv wrote: > > @@ -1533,6 +1569,15 @@ csched_init(struct scheduler *ops) > > prv->tick_period_us = prv->tslice_ms * 1000 / prv->ticks_per_tslice; > > prv->credits_per_tslice = CSCHED_CREDITS_PER_MSEC * prv->tslice_ms; > > > > + if ( MICROSECS(sched_ratelimit_us)> MILLISECS(sched_credit_tslice_ms) ) > > + { > > + printk("WARNING: sched_ratelimit_us>" > > + "sched_credit_tslice_ms is undefined\n" > > + "ratelimit_us is set to 1000 * tslice_ms forcely\n") > The standard idiom for this kind of message would be: > WARNING [what's wrong] > [What you're doing about it] > > So the last sentence of the warning should be: > Setting ratelimit_us to 1000 * tslice_ms > > (Grammatically, you could say "Forcing ratelimit..." but I think "force" > is too strong in this case.) > > Other than that, I'm happy with it, if everyone else is: > > Acked-by: George Dunlap >