On Fri, 24 Nov 2023 09:58:17 GMT, Daniel Jeliński <djelin...@openjdk.org> wrote:
>> The recent cdb versions do not support `.dump /f`: >> >> ***************************************************************************** >> * .dump /f is not supported on a user mode process. * >> * * >> * .dump /ma creates a complete memory dump of a user mode process. * >> ***************************************************************************** >> >> and after printing that message, cdb ignores the rest of the command line >> and never quits. >> >> This PR updates the dump command to use the recommended `/ma` parameter. >> This allows the command to produce a dump and complete in a timely manner. > > Daniel Jeliński has updated the pull request incrementally with one > additional commit since the last revision: > > Address review comments The timeout change looks good to me. ------------- Marked as reviewed by jpai (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/16806#pullrequestreview-1747793376
