RFR: 8252093: formula used to calculate decaying variance in numberSeq
stefan.johansson at oracle.com
stefan.johansson at oracle.com
Thu Aug 27 15:23:46 UTC 2020
Hi Albert,
On 2020-08-26 15:59, Albert Yang wrote:
> Hi,
>
> I am sending this to both runtime-dev and gc-dev, since the changed code
> lives in runtime, but it's used by GC.
>
> https://bugs.openjdk.java.net/browse/JDK-8252093
> http://cr.openjdk.java.net/~ayang/webrevs/8252093/webrev.1/
Looks good, nice to have the comment as reference. As Kim, I wonder if
this will cause any visible change because of changed predictions. Have
you run any benchmarks and compared scores as well as GC-behavior?
>
> Both the code and the literature use `alpha`, but its meaning is
> actually inverted. I could include this refactoring in later revision if
> people feel that it's desirable.
Is there a good reason for keeping it inverted, I see that we have a few
places where we specify an alpha when creating the seq, so I guess there
are a few places to change. Otherwise maybe adding a comment to the
defined default might be helpful.
Thanks,
Stefan
>
> Tested: tier1 (locally) and tier1-3 (mach5)
>
More information about the hotspot-runtime-dev
mailing list