RFR (XS) 8219574: Minimal VM build failure after JDK-8219414

[Chris Plummer]

Apologies. I had two separate threads for this review, and replied on 
the thread that no one else replied to, then later saw the thread with 
all the activity.

Chris

On 2/22/19 8:48 AM, Chris Plummer wrote:
> Looks good.
>
> Chris
>
> On 2/22/19 1:51 AM, Aleksey Shipilev wrote:
>> Bug:
>>    https://bugs.openjdk.java.net/browse/JDK-8219574
>>
>> Fix:
>>
>> diff -r e94ed0236046 src/hotspot/os/linux/os_linux.cpp
>> --- a/src/hotspot/os/linux/os_linux.cpp Fri Feb 22 09:23:37 2019 +0100
>> +++ b/src/hotspot/os/linux/os_linux.cpp Fri Feb 22 10:51:11 2019 +0100
>> @@ -1357,11 +1357,11 @@
>>   // called from signal handler. Before adding something to 
>> os::abort(), make
>>   // sure it is async-safe and can handle partially initialized VM.
>>   void os::abort(bool dump_core, void* siginfo, const void* context) {
>>     os::shutdown();
>>     if (dump_core) {
>> -#ifndef ZERO
>> +#ifndef INCLUDE_CDS
>>       if (DumpPrivateMappingsInCore) {
>>         ClassLoader::close_jrt_image();
>>       }
>>   #endif
>>   #ifndef PRODUCT
>>
>>
>> JDK-8219414 incorrectly removed INCLUDE_CDS guard, while 
>> ClassLoader::close_jrt_image() is still
>> only defined under INCLUDE_CDS.
>>
>> Testing: Linux x86_64 {server, minimal, zero} builds
>>
>> Thanks,
>> -Aleksey
>>
>