RFR: 8240370: Provide Intel JCC Erratum opt-out
Vladimir Ivanov
vladimir.x.ivanov at oracle.com
Tue Mar 3 15:29:15 UTC 2020
Hi Erik,
> New webrev: http://cr.openjdk.java.net/~eosterlund/8240370/webrev.01/
IMO it makes sense to support both forcibly turning the flag on and off
from the command-line.
So, I prefer:
bool VM_Version::compute_has_intel_jcc_erratum() {
+ if (!FLAG_IS_DEFAULT(IntelJccErratumMitigation)) {
+ return IntelJccErratumMitigation;
+ }
Or even:
if (FLAG_IS_DEFAULT(IntelJccErratumMitigation)) {
_has_intel_jcc_erratum = compute_has_intel_jcc_erratum();
} else {
_has_intel_jcc_erratum = IntelJccErratumMitigation;
}
Best regards,
Vladimir Ivanov
>> /Claes
>>
>> On 2020-03-03 13:23, Erik Österlund wrote:
>>> Hi,
>>>
>>> Today, the Intel JCC Erratum mitigation is automatically enabled when
>>> running on an affected machine.
>>> While it is strongly recommended that customers run code in
>>> environments with the micro code updates
>>> provided by Intel to mitigate the erratum, and hence are probably
>>> almost always better off with the
>>> mitigation, there could be value in being able to opt out to analyze
>>> the performance characteristics
>>> of an application, as a diagnostic option.
>>>
>>> This patch adds a diagnostic option that allows you to opt out with:
>>> -XX:-IntelJccErratum
>>>
>>> Webrev:
>>> http://cr.openjdk.java.net/~eosterlund/8240370/webrev.00/
>>>
>>> Bug:
>>> https://bugs.openjdk.java.net/browse/JDK-8240370
>>>
>>> Thanks,
>>> /Erik
>
More information about the hotspot-compiler-dev
mailing list