[11u] RFR: JDK-8256287: [windows] add loop fuse to map_or_reserve_memory_aligned

Reingruber, Richard richard.reingruber at sap.com
Thu Dec 3 20:07:50 UTC 2020


Hi Thomas,

patch looks good to me.

You might want to remove the extraneous whitespace between the ++ operator and
its operand in the line below.

3086	  for (int attempt = 0; attempt < max_attempts && aligned_base == NULL; attempt ++) {

Thanks, Richard.

-----Original Message-----
From: jdk-updates-dev <jdk-updates-dev-retn at openjdk.java.net> On Behalf Of Thomas Stüfe
Sent: Mittwoch, 2. Dezember 2020 09:31
To: jdk-updates-dev <jdk-updates-dev at openjdk.java.net>
Cc: Hotspot dev runtime <hotspot-runtime-dev at openjdk.java.net>
Subject: [11u] RFR: JDK-8256287: [windows] add loop fuse to map_or_reserve_memory_aligned

Hi,

I'd like to get reviews for the backport of the following fix:

Original issue: https://bugs.openjdk.java.net/browse/JDK-8256287
Original patch: https://github.com/openjdk/jdk/commit/0357db35
Modified patch:
http://cr.openjdk.java.net/~stuefe/webrevs/backports/8256287-windows-add-loop-fuse-to-map_or_reserve_memory_aligned-for-11.diff

This patch had to be rewritten for 11 since most of the surroundings
changed since 11 due to cleanups. But the patch itself is straightforward.

Thank you, Thomas


More information about the jdk-updates-dev mailing list