Re: RFR: 8293282: LoadLibraryUnloadTest.java fails with "Too few cleared WeakReferences" [v3]

2022-09-12 Thread Jaikiran Pai
On Fri, 9 Sep 2022 15:48:41 GMT, Roger Riggs wrote: >> Modify the LoadLibraryUnload test to call gc() more a few times, allowing >> multiple gc cycles to queue the expected refs. >> Short the timeout on each cycle to 10 sec. > > Roger Riggs has updated the pull request incrementally with one add

Re: RFR: 8293282: LoadLibraryUnloadTest.java fails with "Too few cleared WeakReferences" [v3]

2022-09-09 Thread Mandy Chung
On Fri, 9 Sep 2022 15:48:41 GMT, Roger Riggs wrote: >> Modify the LoadLibraryUnload test to call gc() more a few times, allowing >> multiple gc cycles to queue the expected refs. >> Short the timeout on each cycle to 10 sec. > > Roger Riggs has updated the pull request incrementally with one add

Re: RFR: 8293282: LoadLibraryUnloadTest.java fails with "Too few cleared WeakReferences" [v3]

2022-09-09 Thread Mandy Chung
On Fri, 9 Sep 2022 15:48:41 GMT, Roger Riggs wrote: >> Modify the LoadLibraryUnload test to call gc() more a few times, allowing >> multiple gc cycles to queue the expected refs. >> Short the timeout on each cycle to 10 sec. > > Roger Riggs has updated the pull request incrementally with one add

Re: RFR: 8293282: LoadLibraryUnloadTest.java fails with "Too few cleared WeakReferences" [v3]

2022-09-09 Thread Roger Riggs
> Modify the LoadLibraryUnload test to call gc() more a few times, allowing > multiple gc cycles to queue the expected refs. > Short the timeout on each cycle to 10 sec. Roger Riggs has updated the pull request incrementally with one additional commit since the last revision: Remove unused Re

Re: RFR: 8293282: LoadLibraryUnloadTest.java fails with "Too few cleared WeakReferences" [v2]

2022-09-09 Thread Roger Riggs
> Modify the LoadLibraryUnload test to call gc() more a few times, allowing > multiple gc cycles to queue the expected refs. > Short the timeout on each cycle to 10 sec. Roger Riggs has updated the pull request incrementally with one additional commit since the last revision: Refactored to us

Re: RFR: 8293282: LoadLibraryUnloadTest.java fails with "Too few cleared WeakReferences"

2022-09-09 Thread Roger Riggs
On Thu, 8 Sep 2022 20:26:41 GMT, Roger Riggs wrote: > Modify the LoadLibraryUnload test to call gc() more a few times, allowing > multiple gc cycles to queue the expected refs. > Short the timeout on each cycle to 10 sec. Hi Jai, I don't think the memory size comes into play, the GC is invoked

Re: RFR: 8293282: LoadLibraryUnloadTest.java fails with "Too few cleared WeakReferences"

2022-09-09 Thread Roger Riggs
On Thu, 8 Sep 2022 20:54:28 GMT, Mandy Chung wrote: >> Modify the LoadLibraryUnload test to call gc() more a few times, allowing >> multiple gc cycles to queue the expected refs. >> Short the timeout on each cycle to 10 sec. > > test/jdk/java/lang/ClassLoader/loadLibraryUnload/LoadLibraryUnload.

Re: RFR: 8293282: LoadLibraryUnloadTest.java fails with "Too few cleared WeakReferences"

2022-09-08 Thread Jaikiran Pai
On Thu, 8 Sep 2022 20:26:41 GMT, Roger Riggs wrote: > Modify the LoadLibraryUnload test to call gc() more a few times, allowing > multiple gc cycles to queue the expected refs. > Short the timeout on each cycle to 10 sec. Hello Roger, just a few additional points that I noticed about this test.

Re: RFR: 8293282: LoadLibraryUnloadTest.java fails with "Too few cleared WeakReferences"

2022-09-08 Thread Mandy Chung
On Thu, 8 Sep 2022 20:26:41 GMT, Roger Riggs wrote: > Modify the LoadLibraryUnload test to call gc() more a few times, allowing > multiple gc cycles to queue the expected refs. > Short the timeout on each cycle to 10 sec. test/jdk/java/lang/ClassLoader/loadLibraryUnload/LoadLibraryUnload.java l

RFR: 8293282: LoadLibraryUnloadTest.java fails with "Too few cleared WeakReferences"

2022-09-08 Thread Roger Riggs
Modify the LoadLibraryUnload test to call gc() more a few times, allowing multiple gc cycles to queue the expected refs. Short the timeout on each cycle to 10 sec. - Commit messages: - Add bugid to LoadLibraryUnloadTest - Modify the wait loop to repeat the System.gc and count queue