8246648: issue with OperatingSystemImpl getFreeSwapSpaceSize in docker after 8242480

Baesken, Matthias matthias.baesken at sap.com
Fri Jun 5 10:17:28 UTC 2020


Hi Severin,

>In summary: Patch looks OK to me.

Thanks for the review. 
May I have a second review ?



>This comment --^ is wrong. Should be "we see a limit == -1 ..."
>
>  65                 return (limit < memLimit) ? 0 : limit - memLimit;
>  66             }
>
>Not sure if you want to change that as part of this patch.

Better not in this change .

Best regards, Matthias


More information about the hotspot-dev mailing list