Re: RFR: 8319376: ParallelGC: Forwarded objects found during heap inspection [v6]

2023-11-07 Thread Albert Mingkun Yang
On Tue, 7 Nov 2023 18:12:45 GMT, Roman Kennke wrote: >> See JBS issue for details. >> >> Testing: >> - [x] gc/logging/TestUnifiedLoggingSwitchStress.java -XX:+UseParallelGC >> - [x] tier1 -XX:+UseParallelGC >> - [ ] tier2 -XX:+UseParallelGC >> - [ ] hotspot_gc > > Roman Kennke has updated th

Re: RFR: 8319376: ParallelGC: Forwarded objects found during heap inspection [v6]

2023-11-07 Thread Roman Kennke
> See JBS issue for details. > > Testing: > - [x] gc/logging/TestUnifiedLoggingSwitchStress.java -XX:+UseParallelGC > - [x] tier1 -XX:+UseParallelGC > - [ ] tier2 -XX:+UseParallelGC > - [ ] hotspot_gc Roman Kennke has updated the pull request incrementally with one additional commit since th

Re: RFR: 8318706: Implementation of JDK-8276094: JEP 423: Region Pinning for G1 [v15]

2023-11-07 Thread Thomas Schatzl
> The JEP covers the idea very well, so I'm only covering some implementation > details here: > > * regions get a "pin count" (reference count). As long as it is non-zero, we > conservatively never reclaim that region even if there is no reference in > there. JNI code might have references to i

Re: RFR: JDK-8318636: Add jcmd to print annotated process memory map [v6]

2023-11-07 Thread Johan Sjölen
On Wed, 1 Nov 2023 10:04:30 GMT, Johan Sjölen wrote: >> Thomas Stuefe has updated the pull request incrementally with one additional >> commit since the last revision: >> >> fix various builds > > OK, went through the cache. Will continue later. > @jdksjolen anything more needed from my side

Re: RFR: 8319376: ParallelGC: Forwarded objects found during heap inspection [v5]

2023-11-07 Thread Albert Mingkun Yang
On Tue, 7 Nov 2023 14:59:48 GMT, Roman Kennke wrote: >> See JBS issue for details. >> >> Testing: >> - [x] gc/logging/TestUnifiedLoggingSwitchStress.java -XX:+UseParallelGC >> - [x] tier1 -XX:+UseParallelGC >> - [ ] tier2 -XX:+UseParallelGC >> - [ ] hotspot_gc > > Roman Kennke has updated th

Re: RFR: 8319376: ParallelGC: Forwarded objects found during heap inspection [v5]

2023-11-07 Thread Roman Kennke
> See JBS issue for details. > > Testing: > - [x] gc/logging/TestUnifiedLoggingSwitchStress.java -XX:+UseParallelGC > - [x] tier1 -XX:+UseParallelGC > - [ ] tier2 -XX:+UseParallelGC > - [ ] hotspot_gc Roman Kennke has updated the pull request incrementally with one additional commit since th

Re: RFR: 8319244: implement JVMTI handshakes support for virtual threads [v4]

2023-11-07 Thread Patricio Chilano Mateo
On Mon, 6 Nov 2023 23:22:03 GMT, Serguei Spitsyn wrote: >> The handshakes support for virtual threads is needed to simplify the JVMTI >> implementation for virtual threads. There is a significant duplication in >> the JVMTI code to differentiate code intended to support platform, virtual >> th

Re: RFR: 8319376: ParallelGC: Forwarded objects found during heap inspection [v4]

2023-11-07 Thread Albert Mingkun Yang
On Tue, 7 Nov 2023 14:31:49 GMT, Roman Kennke wrote: >> See JBS issue for details. >> >> Testing: >> - [x] gc/logging/TestUnifiedLoggingSwitchStress.java -XX:+UseParallelGC >> - [x] tier1 -XX:+UseParallelGC >> - [ ] tier2 -XX:+UseParallelGC >> - [ ] hotspot_gc > > Roman Kennke has updated th

Re: RFR: 8319376: ParallelGC: Forwarded objects found during heap inspection [v4]

2023-11-07 Thread Roman Kennke
> See JBS issue for details. > > Testing: > - [x] gc/logging/TestUnifiedLoggingSwitchStress.java -XX:+UseParallelGC > - [x] tier1 -XX:+UseParallelGC > - [ ] tier2 -XX:+UseParallelGC > - [ ] hotspot_gc Roman Kennke has updated the pull request incrementally with one additional commit since th

Re: RFR: 8318706: Implementation of JDK-8276094: JEP 423: Region Pinning for G1 [v14]

2023-11-07 Thread Thomas Schatzl
> The JEP covers the idea very well, so I'm only covering some implementation > details here: > > * regions get a "pin count" (reference count). As long as it is non-zero, we > conservatively never reclaim that region even if there is no reference in > there. JNI code might have references to i

Re: RFR: 8319376: ParallelGC: Forwarded objects found during heap inspection [v3]

2023-11-07 Thread Albert Mingkun Yang
On Tue, 7 Nov 2023 12:38:14 GMT, Roman Kennke wrote: > all references to original copies of promoted objects are updated, and > therefore the original copies are dead (and can safely be skipped) Yes; after scavenging, all pointers are correct (same value for promotion-fail and new value otherw

Re: RFR: 8319376: ParallelGC: Forwarded objects found during heap inspection [v3]

2023-11-07 Thread Roman Kennke
On Tue, 7 Nov 2023 10:02:04 GMT, Albert Mingkun Yang wrote: > > Would the failed-promotion slide directly into full-GC which repairs it? > > No, promotion-fail objects are specially handled, as documented in > `PSPromotionManager::oop_promotion_failed`. Other successfully forwarded > objects a

Re: RFR: 8314502: Change the comparator taking version of GrowableArray::find to be a template method [v8]

2023-11-07 Thread Afshin Zafari
> The `find` method now is > ```C++ > template > int find(T* token, bool f(T*, E)) const { > ... > > Any other functions which use this are also changed. > Local linux-x64-debug hotspot:tier1 passed. Mach5 tier1 build on linux and > Windows passed. Afshin Zafari has updated the pull request inc

Re: RFR: 8319376: ParallelGC: Forwarded objects found during heap inspection [v3]

2023-11-07 Thread Thomas Schatzl
On Mon, 6 Nov 2023 19:46:07 GMT, Roman Kennke wrote: > I'm curious (and puzzled), though. How would this situation normally have > been repaired without this explicit header-resetting code? Would the > failed-promotion slide directly into full-GC which repairs it? OTOH, I don't > even see code

Integrated: 8319238: JMX ThreadPoolAccTest.java is too verbose and should fail before timeout

2023-11-07 Thread Kevin Walls
On Wed, 1 Nov 2023 17:10:34 GMT, Kevin Walls wrote: > Discovered while testing changes that made this test fail. The test failure > is hard to diagnose as it logs and retries at full speed, possibly forever, > until timeout. This can hit a log file limit. We can save thousands of > lines of

Re: RFR: 8319376: ParallelGC: Forwarded objects found during heap inspection [v3]

2023-11-07 Thread Albert Mingkun Yang
On Mon, 6 Nov 2023 19:46:07 GMT, Roman Kennke wrote: > Would the failed-promotion slide directly into full-GC which repairs it? No, promotion-fail objects are specially handled, as documented in `PSPromotionManager::oop_promotion_failed`. Other successfully forwarded objects are essentially de

Re: RFR: 8315149: Add hsperf counters for CPU time of internal GC threads [v35]

2023-11-07 Thread Stefan Johansson
On Tue, 7 Nov 2023 01:06:12 GMT, Man Cao wrote: > I think it looks great. It is mainly refactoring that consolidates the > declarations/definitions of the hsperf counters in to a single file. Would it > be better to name that class `CPUTimeCounters`, so we could move > `sun.threads.cpu_time.vm

Re: RFR: 8318706: Implementation of JDK-8276094: JEP 423: Region Pinning for G1 [v13]

2023-11-07 Thread Thomas Schatzl
> The JEP covers the idea very well, so I'm only covering some implementation > details here: > > * regions get a "pin count" (reference count). As long as it is non-zero, we > conservatively never reclaim that region even if there is no reference in > there. JNI code might have references to i