C2 compiler gets SIGILL/ILL_ILLOPC

[Simon Nash]

Hi Andrey,
Actually I was more concerned about this comment in [1]

We propose to only build for MP platforms. AssumeMP (which defaults to true since 18.3)
is hardwired to true; os::is_MP is always true, and we can elide all code and code
generation that provides for a non-MP path.

There is mothing there that says this applies to the server VM only.

Best regards,
Simon

[1] https://bugs.openjdk.java.net/browse/JDK-8188764

On 04/05/2018 09:58, Andrey Petushkov wrote:
> Hi Simon,
> 
> The discussed problem only concerns server VM. There is no use of pldw in the client VM at all. 
> Although I cannot tell if there are incorrect usages of the os::is_MP() in the client VM code for arm32 port
> 
> Regards,
> Andrey
> 
>> On 3 May 2018, at 23:21, Simon Nash <simon at cjnash.com> wrote:
>>
>> On 03/05/2018 17:00, Andrey Petushkov wrote:
>>> Hi John,
>>>> On 3 May 2018, at 18:51, John Neffenger <john at status6.com> wrote:
>>>>
>>>> On 05/03/2018 02:36 AM, Andrey Petushkov wrote:
>>>>> The -XX:-AssumeMP command line option should help in your case
>>>> Magic! That helps a lot. Thank you, Andrey.
>>> Glad to hear :)
>>>> The warning message makes me a bit nervous, though.
>>> You can safely ignore it for a meanwhile. It will strike in Java 12 though (https://bugs.openjdk.java.net/browse/JDK-8188764 <https://bugs.openjdk.java.net/browse/JDK-8188764>). (And I find it strange to decide that Java from now on only runs on multiprocessor devices. Anyway..)
>> Is this MP-only assumption only for the server VM or also for the 32-bit client VM?
>> The latter would prevent my application from being able to run on Java 12.
>>
>> Simon
>>
>>>> $ ./opt/jre-10/bin/java -XX:-AssumeMP -jar lib/hellojdk.jar
>>>> OpenJDK Server VM warning: Option AssumeMP was deprecated in version 10.0 and will likely be removed in a future release.
>>>> Hello World!
>>>> Hello World!
>>>> Hello World!
>>>>
>>>> $ ./opt/jre-11/bin/java -XX:-AssumeMP -jar lib/hellojdk.jar
>>>> OpenJDK Server VM warning: Option AssumeMP was deprecated in version 10.0 and will likely be removed in a future release.
>>>> Hello World!
>>>> Hello World!
>>>> Hello World!
>>>>
>>>> Should I open a bug report?
>>> I’m pretty sure the problem deserves it
>>> Thank you,
>>> Andrey
>>>> Thank you,
>>>> John
> 
>