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

Thomas Stüfe thomas.stuefe at gmail.com
Thu Dec 3 20:26:59 UTC 2020


Thank you Richard. Will do.

.:Thomas

On Thu, Dec 3, 2020 at 9:07 PM Reingruber, Richard <
richard.reingruber at sap.com> wrote:

> 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