RFR (S): 8242036: G1 HeapRegionRemSet::_n_coarse_entries could be a bool

Erik Österlund erik.osterlund at oracle.com
Thu Jul 23 10:46:11 UTC 2020


Hi Albert,

Looks good.

Thanks,
/Erik

On 2020-07-23 11:05, Albert Yang wrote:
> Additionally, in following the guideline for concurrent variables, it is marked `volatile` to signal potential concurrent access, and all access to it use explicit atomic operations now.
>
> https://bugs.openjdk.java.net/browse/JDK-8242036
>
> the complete patch:
>
> http://cr.openjdk.java.net/~lkorinth/albert/8242036/1/
>
> Tested: hotspot-gc
>
> /Albert
>
> ----- Original Message -----
> From: albert.m.yang at oracle.com
> To: hotspot-gc-dev at openjdk.java.net
> Sent: Wednesday, July 22, 2020 11:51:57 AM GMT +01:00 Amsterdam / Berlin / Bern / Rome / Stockholm / Vienna
> Subject: RFR (S): 8242036: G1 HeapRegionRemSet::_n_coarse_entries could be a bool
>
> Hi,
>
> Changed the type from `size_t` to `bool`, since we only care whether if it's zero or not.
>
> https://bugs.openjdk.java.net/browse/JDK-8242036
>
> http://cr.openjdk.java.net/~lkorinth/albert/8242036/0/
>
> /Albert



More information about the hotspot-gc-dev mailing list