[8u] RFR (XS) 8255734: VM should ignore SIGXFSZ on ppc64, s390 too

Aleksey Shipilev shade at redhat.com
Tue Apr 27 13:11:30 UTC 2021


On 4/27/21 3:09 PM, Severin Gehwolf wrote:
> On Mon, 2021-04-19 at 16:23 +0200, Aleksey Shipilev wrote:
>> 8u variant:
>>
>> diff -r 7ab0ec535776 src/os_cpu/aix_ppc/vm/os_aix_ppc.cpp
>> --- a/src/os_cpu/aix_ppc/vm/os_aix_ppc.cpp      Thu Oct 24 16:28:51 2019 +0200
>> +++ b/src/os_cpu/aix_ppc/vm/os_aix_ppc.cpp      Mon Apr 19 10:54:12 2021 +0200
>> @@ -178,7 +178,7 @@
>>      // avoid unnecessary crash when libjsig is not preloaded, try handle signals
>>      // that do not require siginfo/ucontext first.
>>
>> -  if (sig == SIGPIPE) {
>> +  if (sig == SIGPIPE || sig == SIGXFSZ) {
>>        if (os::Aix::chained_handler(sig, info, ucVoid)) {
>>          return 1;
>>        } else {
>> diff -r 7ab0ec535776 src/os_cpu/linux_ppc/vm/os_linux_ppc.cpp
>> --- a/src/os_cpu/linux_ppc/vm/os_linux_ppc.cpp  Thu Oct 24 16:28:51 2019 +0200
>> +++ b/src/os_cpu/linux_ppc/vm/os_linux_ppc.cpp  Mon Apr 19 10:54:12 2021 +0200
>> @@ -185,7 +185,7 @@
>>      // avoid unnecessary crash when libjsig is not preloaded, try handle signals
>>      // that do not require siginfo/ucontext first.
>>
>> -  if (sig == SIGPIPE) {
>> +  if (sig == SIGPIPE || sig == SIGXFSZ) {
>>        if (os::Linux::chained_handler(sig, info, ucVoid)) {
>>          return true;
>>        } else {
> 
> This looks fine.

Thank you, already tagged for approvals.

-- 
-Aleksey



More information about the jdk8u-dev mailing list