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

Re: [Xen-devel] [PATCH 2/5] xen: ARM: suspend the tick (if in use) when going idle.



On Tue, 2017-08-01 at 09:53 +0100, Julien Grall wrote:
> On 31/07/2017 21:59, Stefano Stabellini wrote:
> > On Thu, 27 Jul 2017, Dario Faggioli wrote:
> > > 
> > > Signed-off-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
> > 
> > Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
> 
> This patch looks standalone, but please don't commit this patch
> without 
> the rest of the series. Otherwise, we will introduce regression in
> Xen 
> with credit1 as well.
> 
I'm not sure I'd call that a "regression". In fact, the current
situation is that the code has a bug, but the absence of this power-
efficiency optimization, on ARM, is covering the bug itself. :-D

Anyway, zero intention to turn this into a terminology bunfight, and,
overall, I even agree with you that it's probably better that this
series go in all-together. :-)

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)

Attachment: signature.asc
Description: This is a digitally signed message part

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

 


Rackspace

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