RFR (XS): 8007901 SA: Don't read flag values as constants
David Holmes
david.holmes at oracle.com
Mon Feb 11 16:20:51 PST 2013
On 10/02/2013 5:06 AM, Staffan Larsen wrote:
> Please review this small patch to avoid reading flag values in SA as constants. Reading them as constants means SA will only see the default value for these flags. Instead the infrastructure in SA to read command line flags should be used. In addition the value if EnableInvokeDynamic is never used, so I removed that from SA.
Isn't the real problem in the way db is initialized with the values for
these flags? ie shouldn't db.lookupIntConstant("UseTLAB").intValue() be
returning the actual value of UseTLAB as it occurs in the VM ?
Also we need a regression test for this as obviously it ain't being
tested! :(
Thanks,
David
> webrev: http://cr.openjdk.java.net/~sla/8007901/webrev.00/
> bug: http://bugs.sun.com/view_bug.do?bug_id=8007901 (not yet visible)
>
> Thanks,
> /Staffan
>
More information about the serviceability-dev
mailing list