RFR: JDK-8288556: VM crashes if it gets sent SIGUSR2 from outside
Thomas Stuefe
stuefe at openjdk.org
Mon Jun 20 13:07:51 UTC 2022
On Mon, 20 Jun 2022 12:48:49 GMT, David Holmes <dholmes at openjdk.org> wrote:
>
> On the si_pid part ... I have no prior knowledge of this (didn't even know it existed), so have no idea whether it is reliable or not.
>
> Seems to me we have far greater risk of breaking something unexpectedly with changing this code than we potentially benefit from making the change.
>
> So I'd vote for doing nothing.
"Nothing" as in I should withdraw this patch? Surely not?
The behavioral difference my patch brings would be:
debug: assert with useless information -> assert with useful information
release: crash with useless report -> print useful information, continue
As I wrote, I can compromise the second part to:
release: crash with useless report -> print useful information, exit VM
-------------
PR: https://git.openjdk.org/jdk/pull/9181
More information about the hotspot-runtime-dev
mailing list