Re: RFR: 8268857: Merge VM_PrintJNI and VM_PrintThreads and remove the unused field 'is_deadlock' of DeadlockCycle [v7]

2021-06-17 Thread Denghui Dong
> Hi, > > Could I have a review of this change that merges three vm > operations(VM_PrintThreads, VM_PrintJNI, VM_FindDeadlocks) in thread_dump and > signal_thread_entry. > > `jstack` is a very common command, even in the production environment. > > In addition to reduce the cost of entering s

Re: RFR: 8268857: Merge VM_PrintJNI and VM_PrintThreads and remove the unused field 'is_deadlock' of DeadlockCycle [v5]

2021-06-17 Thread Denghui Dong
On Fri, 18 Jun 2021 06:17:24 GMT, David Holmes wrote: >> Denghui Dong has updated the pull request incrementally with one additional >> commit since the last revision: >> >> polish > > src/hotspot/share/runtime/os.cpp line 402: > >> 400: // the output stream (e.g. tty->flush()) after

Re: Monitoring Java Safepoint Time in JDK16+

2021-06-17 Thread David Holmes
On 18/06/2021 4:35 pm, Alan Bateman wrote: On 17/06/2021 22:44, David Holmes wrote: I must admit I'm a bit confused about these implementation-specific MBeans. They are implementation-specific, so no part of the primary java.management namespace, but they are provided so that they can be use

Re: RFR: 8268857: Merge VM_PrintJNI and VM_PrintThreads and remove the unused field 'is_deadlock' of DeadlockCycle [v6]

2021-06-17 Thread Denghui Dong
> Hi, > > Could I have a review of this change that merges three vm > operations(VM_PrintThreads, VM_PrintJNI, VM_FindDeadlocks) in thread_dump and > signal_thread_entry. > > `jstack` is a very common command, even in the production environment. > > In addition to reduce the cost of entering s

Re: Monitoring Java Safepoint Time in JDK16+

2021-06-17 Thread Alan Bateman
On 17/06/2021 22:44, David Holmes wrote: I must admit I'm a bit confused about these implementation-specific MBeans. They are implementation-specific, so no part of the primary java.management namespace, but they are provided so that they can be used - so shutting them behind the modular door

Re: RFR: 8268857: Merge VM_PrintJNI and VM_PrintThreads and remove the unused field 'is_deadlock' of DeadlockCycle [v5]

2021-06-17 Thread David Holmes
On Fri, 18 Jun 2021 06:08:48 GMT, Denghui Dong wrote: >> Hi, >> >> Could I have a review of this change that merges three vm >> operations(VM_PrintThreads, VM_PrintJNI, VM_FindDeadlocks) in thread_dump >> and signal_thread_entry. >> >> `jstack` is a very common command, even in the production

Re: RFR: 8268698: Use Objects.check{Index, FromToIndex, FromIndexSize} where possible [v2]

2021-06-17 Thread Yi Yang
On Thu, 17 Jun 2021 15:45:47 GMT, Paul Sandoz wrote: >> src/java.base/share/classes/java/util/Base64.java line 934: >> >>> 932: if (closed) >>> 933: throw new IOException("Stream is closed"); >>> 934: Preconditions.checkFromIndexSize(len, off, b.length, (x

Re: RFR: 8268698: Use Objects.check{Index, FromToIndex, FromIndexSize} where possible [v2]

2021-06-17 Thread Yi Yang
On Fri, 18 Jun 2021 05:54:01 GMT, Yi Yang wrote: >> After JDK-8265518(#3615), it's possible to replace all variants of >> checkIndex by >> Objects.checkIndex/Objects.checkFromToIndex/Objects.checkFromIndexSize in >> the whole JDK codebase. > > Yi Yang has updated the pull request incrementally

Re: RFR: 8268857: Merge VM_PrintJNI and VM_PrintThreads and remove the unused field 'is_deadlock' of DeadlockCycle [v5]

2021-06-17 Thread Denghui Dong
> Hi, > > Could I have a review of this change that merges three vm > operations(VM_PrintThreads, VM_PrintJNI, VM_FindDeadlocks) in thread_dump and > signal_thread_entry. > > `jstack` is a very common command, even in the production environment. > > In addition to reduce the cost of entering s

Re: RFR: 8268698: Use Objects.check{Index, FromToIndex, FromIndexSize} where possible [v2]

2021-06-17 Thread Yi Yang
> After JDK-8265518(#3615), it's possible to replace all variants of checkIndex > by Objects.checkIndex/Objects.checkFromToIndex/Objects.checkFromIndexSize in > the whole JDK codebase. Yi Yang has updated the pull request incrementally with one additional commit since the last revision: rest

Re: RFR: 8268698: Use Objects.check{Index, FromToIndex, FromIndexSize} where possible [v2]

2021-06-17 Thread Yi Yang
On Thu, 17 Jun 2021 10:19:43 GMT, Alan Bateman wrote: >> Yi Yang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> restore IndexOfOufBoundsException; split exception line > > src/java.base/share/classes/java/lang/AbstractStringBuilder.java

Re: RFR: 8268857: Merge VM_PrintJNI and VM_PrintThreads and remove the unused field 'is_deadlock' of DeadlockCycle [v4]

2021-06-17 Thread Denghui Dong
> Hi, > > Could I have a review of this change that merges three vm > operations(VM_PrintThreads, VM_PrintJNI, VM_FindDeadlocks) in thread_dump and > signal_thread_entry. > > `jstack` is a very common command, even in the production environment. > > In addition to reduce the cost of entering s

Re: RFR: 8268857: Merge three vm operations in thread_dump [v3]

2021-06-17 Thread David Holmes
On 18/06/2021 12:29 pm, Denghui Dong wrote: On Thu, 17 Jun 2021 23:12:04 GMT, David Holmes wrote: Sorry still not a fan even with the hacked in deadline support for find_deadlocks. It is too arbitrary - you have no idea what constitutes a reasonable deadline, nor by how much you may overshoo

Re: RFR: 8268857: Merge three vm operations in thread_dump [v3]

2021-06-17 Thread Denghui Dong
On Thu, 17 Jun 2021 23:12:04 GMT, David Holmes wrote: > Sorry still not a fan even with the hacked in deadline support for > find_deadlocks. It is too arbitrary - you have no idea what constitutes a > reasonable deadline, nor by how much you may overshoot that deadline. > > David Hi David, T

Re: RFR: 8268857: Merge three vm operations in thread_dump [v3]

2021-06-17 Thread David Holmes
On Thu, 17 Jun 2021 15:32:01 GMT, Denghui Dong wrote: >> Hi, >> >> Could I have a review of this change that merges three vm >> operations(VM_PrintThreads, VM_PrintJNI, VM_FindDeadlocks) in thread_dump >> and signal_thread_entry. >> >> `jstack` is a very common command, even in the production

Re: Monitoring Java Safepoint Time in JDK16+

2021-06-17 Thread David Holmes
On 17/06/2021 11:44 pm, Carter Kozak wrote: On 17/06/2021 3:15 pm, Alan Bateman wrote: > On 17/06/2021 02:39, David Holmes wrote: >> In what way does this no longer work? We have a test in >> jdk/sun/management/HotspotRuntimeMBean/GetTotalSafepointTime.java that >> uses this API and doesn't s

Re: RFR: 8264941: Remove CodeCache::mark_for_evol_deoptimization() method

2021-06-17 Thread Vladimir Ivanov
On Wed, 16 Jun 2021 12:52:46 GMT, Coleen Phillimore wrote: > This change removes the mark_for_evol_deoptimization method and removes the > flag that all dependencies are recorded. Before the change to walk the > entire nmethod looking for "old" (redefined) methods with metadata_do(), we > use

Re: [jdk17] RFR: 8268433: serviceability/dcmd/framework/VMVersionTest.java fails with Unable to send object throw not established PipeIO Listener Thread connection

2021-06-17 Thread Daniel D . Daugherty
On Fri, 11 Jun 2021 19:40:09 GMT, Alex Menkov wrote: > The fix adds check that IOPipe is connected before sending "quit" command ProblemListings that are integrated in JDK17 will get sync'ed forward to JDK18. The thing to do is remove the test from the ProblemList in your PR so that your test jo

Re: RFR: 8264941: Remove CodeCache::mark_for_evol_deoptimization() method

2021-06-17 Thread Vladimir Kozlov
On Wed, 16 Jun 2021 12:52:46 GMT, Coleen Phillimore wrote: > This change removes the mark_for_evol_deoptimization method and removes the > flag that all dependencies are recorded. Before the change to walk the > entire nmethod looking for "old" (redefined) methods with metadata_do(), we > use

Re: RFR: 8268698: Use Objects.check{Index, FromToIndex, FromIndexSize} where possible

2021-06-17 Thread Paul Sandoz
On Thu, 17 Jun 2021 10:21:35 GMT, Alan Bateman wrote: >> After JDK-8265518(#3615), it's possible to replace all variants of >> checkIndex by >> Objects.checkIndex/Objects.checkFromToIndex/Objects.checkFromIndexSize in >> the whole JDK codebase. > > src/java.base/share/classes/java/util/Base64.

Re: RFR: 8268857: Merge three vm operations in thread_dump [v3]

2021-06-17 Thread Denghui Dong
> Hi, > > Could I have a review of this change that merges three vm > operations(VM_PrintThreads, VM_PrintJNI, VM_FindDeadlocks) in thread_dump and > signal_thread_entry. > > `jstack` is a very common command, even in the production environment. > > In addition to reduce the cost of entering s

Re: Monitoring Java Safepoint Time in JDK16+

2021-06-17 Thread Carter Kozak
On 17/06/2021 3:15 pm, Alan Bateman wrote: > On 17/06/2021 02:39, David Holmes wrote: >> In what way does this no longer work? We have a test in >> jdk/sun/management/HotspotRuntimeMBean/GetTotalSafepointTime.java that >> uses this API and doesn't seem to need to do anything to allow access. ?? > T

Re: RFR: 8268698: Use Objects.check{Index, FromToIndex, FromIndexSize} where possible

2021-06-17 Thread David Holmes
On 17/06/2021 8:50 pm, Alan Bateman wrote: On Thu, 17 Jun 2021 05:16:14 GMT, David Holmes wrote: There are a lot more tests than just tier1. :) I don't expect many, if any, tests to be looking for a specific IOOBE message, and I can't see an easy way to find such tests without running them.

Re: RFR: 8178287: AsyncGetCallTrace fails to traverse valid Java stacks [v2]

2021-06-17 Thread Jaroslav Bachorik
On Thu, 10 Jun 2021 07:50:57 GMT, Ludovic Henry wrote: >> When the signal sent for AsyncGetCallTrace or JFR would land on a runtime >> stub (like arraycopy), a vtable stub, or the prolog of a compiled method, >> it wouldn't be able to detect the sender (caller) frame for multiple >> reasons.

Re: RFR: 8268698: Use Objects.check{Index, FromToIndex, FromIndexSize} where possible

2021-06-17 Thread Alan Bateman
On Thu, 17 Jun 2021 05:16:14 GMT, David Holmes wrote: > There are a lot more tests than just tier1. :) I don't expect many, if any, > tests to be looking for a specific IOOBE message, and I can't see an easy way > to find such tests without running them. If core-libs folk are okay with this >

Re: RFR: 8268698: Use Objects.check{Index, FromToIndex, FromIndexSize} where possible

2021-06-17 Thread Alan Bateman
On Wed, 16 Jun 2021 08:08:47 GMT, Yi Yang wrote: > After JDK-8265518(#3615), it's possible to replace all variants of checkIndex > by Objects.checkIndex/Objects.checkFromToIndex/Objects.checkFromIndexSize in > the whole JDK codebase. I looked through the changes in java.base and only spotted o

Re: RFR: JDK-8268893: jcmd to trim the glibc heap [v2]

2021-06-17 Thread Thomas Stuefe
> The glibc is somewhat notorious for retaining released C Heap memory: calling > free(3) returns memory to the glibc, and most libc variants will return at > least a portion of it back to the Operating System, but the glibc often does > not. > > This depends on the granularity of the allocatio

Re: RFR: JDK-8268893: jcmd to trim the glibc heap

2021-06-17 Thread Thomas Stuefe
On Thu, 17 Jun 2021 08:25:23 GMT, Severin Gehwolf wrote: >> The glibc is somewhat notorious for retaining released C Heap memory: >> calling free(3) returns memory to the glibc, and most libc variants will >> return at least a portion of it back to the Operating System, but the glibc >> often

Re: RFR: JDK-8268893: jcmd to trim the glibc heap

2021-06-17 Thread Severin Gehwolf
On Wed, 16 Jun 2021 12:57:44 GMT, Thomas Stuefe wrote: > The glibc is somewhat notorious for retaining released C Heap memory: calling > free(3) returns memory to the glibc, and most libc variants will return at > least a portion of it back to the Operating System, but the glibc often does > n