RFR: 8253742: POSIX signal code cleanup [v7]

[David Holmes]

On Mon, 16 Nov 2020 17:18:29 GMT, Gerard Ziemski <gziemski at openjdk.org> wrote:

>> hi all,
>> 
>> Please review this followup to [JDK-8252324 Signal related code should be shared among POSIX platforms](https://bugs.openjdk.java.net/browse/JDK-8252324), where several issues were identified for a cleanup. This change addresses them all:
>> 
>> #1 David's feedback - removed non POSIX SIGNIFICANT_SIGNAL_MASK code
>> 
>> #2 David's feedback - used unblock_program_error_signals() on all platforms (reverted for JDK-8252533)
>> 
>> #3 David's feedback - used single JVM_handle_posix_signal API for all POSIX platforms (reverted for JDK-8255711)
>> 
>> #4 Coleen's feedback - cleanup header files in src/hotspot/os/posix/signals_posix.hpp
>> 
>> #5 Coleen's feedback - hid SR_signum assignment in src/hotspot/os/posix/signals_posix.hpp to avoid having to include <signal.h>
>> 
>> #6 Coleen's feedback - factored out print_signal_handlers()
>> 
>> #7 Thomas' feedback - factored out common POSIX os::SuspendedThreadTask::internal_do_task()
>> 
>> #8 Thomas's feedback - factored out common POSIX signal initialization code
>> 
>> #9 YaSuenag's feedback - used JVM_handle_posix_signal for the common API
>> 
>> #10 YaSuenag's feedback - unified logging out of the scope for this fix
>> 
>> #11 YaSuenag's feedback - memset usage in PosixSignals::jdk_misc_signal_init() correct?
>
> Gerard Ziemski has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Fix missing os::print_signal_handlers on Windows, white space on Linux

I'm seeing unexpected changes relating to os::fetch_compiled_frame_from_context seemingly due to the "Merge from master".

Otherwise the actual described changes since last commit seem fine.

Once Thomas's last couple of comments are addressed this is done.

Thanks,
David

-------------

Marked as reviewed by dholmes (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/636