RFR: JDK-8302736: Major performance regression in Math.log on aarch64

David Holmes dholmes at openjdk.org
Thu May 11 01:14:45 UTC 2023


On Wed, 10 May 2023 13:05:00 GMT, Tobias Holenstein <tholenstein at openjdk.org> wrote:

>> Nice analysis, Toby. This point fix looks good to me.
>> 
>> As @theRealAph mentioned in the bug comments, and since there are other coarse-grained usages of `ThreadWXEnable` in the code (for example, in the `VM/JTR_ENTRY` macros), please file a follow-up RFE to improve this situation. The `ThreadWXEnable` should be as close as possible to the code that does the actual write access to the code cache.
>
>> 
> 
> 
> 
>> Nice analysis, Toby. This point fix looks good to me.
>> 
>> As @theRealAph mentioned in the bug comments, and since there are other coarse-grained usages of `ThreadWXEnable` in the code (for example, in the `VM/JTR_ENTRY` macros), please file a follow-up RFE to improve this situation. The `ThreadWXEnable` should be as close as possible to the code that does the actual write access to the code cache.
> 
> Thanks! I filed https://bugs.openjdk.org/browse/JDK-8307817

@tobiasholenstein what testing has been done on this? We may need to run all tiers (1-8) to ensure we have completely covered all the code paths affected. Thanks.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/13606#issuecomment-1543031742


More information about the hotspot-dev mailing list