Re: RFR: JDK-8306441: Two phase segmented heap dump [v26]

2023-08-08 Thread Yi Yang
On Tue, 8 Aug 2023 07:41:23 GMT, Kevin Walls wrote: > It's not too bad to change the test to launch a LingeredApp and test the > output, I made an attempt at that. I'll share a testcase suggestion here, see > what you think. Thank you for the share! It makes sense to me. This test will start

Re: RFR: JDK-8306441: Two phase segmented heap dump [v26]

2023-08-08 Thread Kevin Walls
On Fri, 4 Aug 2023 13:04:28 GMT, Yi Yang wrote: >> ### Motivation and proposal >> Hi, heap dump brings about pauses for application's execution(STW), this is >> a well-known pain. JDK-8252842 have added parallel support to heapdump in an >> attempt to alleviate this issue. However, all

Re: RFR: JDK-8306441: Two phase segmented heap dump [v26]

2023-08-08 Thread Kevin Walls
On Fri, 4 Aug 2023 13:04:28 GMT, Yi Yang wrote: >> ### Motivation and proposal >> Hi, heap dump brings about pauses for application's execution(STW), this is >> a well-known pain. JDK-8252842 have added parallel support to heapdump in an >> attempt to alleviate this issue. However, all

Re: RFR: JDK-8306441: Two phase segmented heap dump [v26]

2023-08-04 Thread Yi Yang
On Fri, 4 Aug 2023 16:31:28 GMT, Kevin Walls wrote: >> Yi Yang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> new can_parallel_dump > > test/hotspot/jtreg/serviceability/dcmd/gc/HeapDumpParallelTest.java line 59: > >> 57:

Re: RFR: JDK-8306441: Two phase segmented heap dump [v26]

2023-08-04 Thread Kevin Walls
On Fri, 4 Aug 2023 13:04:28 GMT, Yi Yang wrote: >> ### Motivation and proposal >> Hi, heap dump brings about pauses for application's execution(STW), this is >> a well-known pain. JDK-8252842 have added parallel support to heapdump in an >> attempt to alleviate this issue. However, all

Re: RFR: JDK-8306441: Two phase segmented heap dump [v26]

2023-08-04 Thread Yi Yang
> ### Motivation and proposal > Hi, heap dump brings about pauses for application's execution(STW), this is a > well-known pain. JDK-8252842 have added parallel support to heapdump in an > attempt to alleviate this issue. However, all concurrent threads > competitively write heap data to the