Re: RFR: 8280553: resourcehogs/serviceability/sa/TestHeapDumpForLargeArray.java can fail if GC occurs [v2]

2022-01-29 Thread Chris Plummer
On Fri, 28 Jan 2022 07:46:40 GMT, Chris Plummer wrote: >> This test is failing in the loom repo when using -Xcomp. The reason is >> because loom introduced doing a full GC in the codecache sweeper, which >> causes the large array that the test allocates to be GC'd before the heap >> dump is

Re: RFR: 8280553: resourcehogs/serviceability/sa/TestHeapDumpForLargeArray.java can fail if GC occurs [v2]

2022-01-28 Thread Alex Menkov
On Fri, 28 Jan 2022 07:46:40 GMT, Chris Plummer wrote: >> This test is failing in the loom repo when using -Xcomp. The reason is >> because loom introduced doing a full GC in the codecache sweeper, which >> causes the large array that the test allocates to be GC'd before the heap >> dump is

Re: RFR: 8280553: resourcehogs/serviceability/sa/TestHeapDumpForLargeArray.java can fail if GC occurs [v2]

2022-01-27 Thread Chris Plummer
On Fri, 28 Jan 2022 06:08:55 GMT, Alan Bateman wrote: >> Chris Plummer has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Use Reference.reachabilityFence() instead of static field. > >

Re: RFR: 8280553: resourcehogs/serviceability/sa/TestHeapDumpForLargeArray.java can fail if GC occurs [v2]

2022-01-27 Thread Alan Bateman
On Fri, 28 Jan 2022 07:43:45 GMT, Chris Plummer wrote: >> This test is failing in the loom repo when using -Xcomp. The reason is >> because loom introduced doing a full GC in the codecache sweeper, which >> causes the large array that the test allocates to be GC'd before the heap >> dump is

Re: RFR: 8280553: resourcehogs/serviceability/sa/TestHeapDumpForLargeArray.java can fail if GC occurs [v2]

2022-01-27 Thread Chris Plummer
> This test is failing in the loom repo when using -Xcomp. The reason is > because loom introduced doing a full GC in the codecache sweeper, which > causes the large array that the test allocates to be GC'd before the heap > dump is done. The fix is to make the large array static rather than a