[15] RFR 8245240: Shenandoah: support nesting evacuation OOM scope
Aleksey Shipilev
shade at redhat.com
Wed May 20 18:00:50 UTC 2020
On 5/20/20 5:08 PM, Zhengyu Gu wrote:
> Okay, then added assertion to make sure:
> http://cr.openjdk.java.net/~zgu/JDK-8245240/webrev.01/index.html
In push_evac_oom_scope, don't we want to assert that (level != sizeof(uint8_t)*8) or something? The
overflow there would be catastrophic, right?
Actually, now that I think of it, wouldn't it be safer to make _oom_scope_nesting_level uint32_t,
for example? That would be physically harder to overflow.
Otherwise looks fine.
--
Thanks,
-Aleksey
More information about the shenandoah-dev
mailing list