Re: [Xen-devel] [PATCH v2 03/11] xen: credit2: rework load tracking logic

[Dario Faggioli <dario.faggioli@xxxxxxxxxx>]

On Mon, 2016-07-18 at 15:46 +0100, George Dunlap wrote:
> On 15/07/16 15:49, Dario Faggioli wrote:
> > 
> > This may look handy, but it introduced a (not especially well
> > documented) dependency between the lenght of the window and
> > the precision of the calculations, which really should be
> > two independent things. Especially if treating them as such
> > (like it is done in this patch) does not lead to more
> > complex maths (same number of multiplications and shifts, and
> > there is still room for some optimization).
> > 
> > Therefore, in this patch, we:
> >  - split length of the window and precision (and, since there
> >    is already a command line parameter for length of window,
> >    introduce one for precision too),
> >  - align the math with one proper incarnation of exponential
> >    smoothing (at no added cost),
> >  - add comments, about the details of the algorithm and the
> >    math used.
> > 
> > While there fix a couple of style issues as well (pointless
> > initialization, long lines, comments).
> > 
> > Signed-off-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
> > ---
> > Changes from v1:
> >  * reconciled comments and actual code about load_window_shift
> > handling;
> And this apparently means changing the code to match the comments
> (not
> the other way around), so that the actual load_window_shift default
> is
> now 30 (as the comments say) instead of 20 (as it was in the previous
> patch). :-)
> 
Indeed. I should have made it more clear that this is what's
happening... Sorry! :-)

> Reviewed-by: George Dunlap <george.dunlap@xxxxxxxxxx>
> 
Thanks,
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