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

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

2022-01-27 Thread Alan Bateman
On Wed, 26 Jan 2022 20:03:21 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 done.

RFR: 8280553: resourcehogs/serviceability/sa/TestHeapDumpForLargeArray.java can fail if GC occurs

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 local