RFR: 8291569: Consider removing JNI checks for signals SIGPIPE and SIGXFSZ [v2]
    David Holmes 
    dholmes at openjdk.org
       
    Mon Jan 23 00:37:01 UTC 2023
    
    
  
> Simple enhancement to skip checking of the handlers for SIGPIPE and SIGXFSZ as we really don't care if they change, and we expect that they can.
> 
> The signal checking code might seem to have a redundancy as we skip these signals in two places, but the primary use of the `do_check_signal_periodically` array is to allow for only issuing one warning per signal. I could have skipped them in either place and get the same effect but it seemed inappropriate to set the array entry and not do the check; and vice-versa.
> 
> Testing: just basic sanity test tiers 1-3. This doesn't affect any application behaviour.
> 
> There are no regression tests in this specific area and it did not seem worth the effort creating one just for this.
> 
> Thanks.
David Holmes has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision:
 - Merge branch 'master' into 8291569-signals
 - 8291569: Consider removing JNI checks for signals SIGPIPE and SIGXFSZ
-------------
Changes:
  - all: https://git.openjdk.org/jdk/pull/12062/files
  - new: https://git.openjdk.org/jdk/pull/12062/files/b5442e7d..1a6ac606
Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=12062&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=12062&range=00-01
  Stats: 12251 lines in 535 files changed: 6663 ins; 1265 del; 4323 mod
  Patch: https://git.openjdk.org/jdk/pull/12062.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/12062/head:pull/12062
PR: https://git.openjdk.org/jdk/pull/12062
    
    
More information about the hotspot-runtime-dev
mailing list