RFR: 8282295: SymbolPropertyEntry::set_method_type fails with assert

David Holmes dholmes at openjdk.java.net
Wed Mar 9 07:18:05 UTC 2022


On Mon, 7 Mar 2022 16:53:38 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

> This is somewhat trivial change to zero out OopHandle in fullgc_alot_dummy_array.  Other OopHandle releases are for data that is deallocated or not reused so zeroing _obj is not needed in those places.  I didn't want to zero _obj in OopHandle::release anyway without zeroing it in the analogous WeakHandle::release. WeakHandle release code operates on const objects, also which are deallocated after release.
> Tested with tier1-4.

src/hotspot/share/memory/universe.cpp line 1251:

> 1249:       // No more dummies to release, release entire array instead
> 1250:       _fullgc_alot_dummy_array.release(Universe::vm_global());
> 1251:       _fullgc_alot_dummy_array = OopHandle(); // NULL out OopStorage pointer.

Seems odd that there is not a way to NULL the pointer directly - why is it not done as part of release()?

-------------

PR: https://git.openjdk.java.net/jdk/pull/7730


More information about the hotspot-runtime-dev mailing list