RFR: 8222825: ARM32 SIGILL issue on single core CPU (not supported PLDW instruction)
Boris Ulasevich
boris.ulasevich at bell-sw.com
Thu May 16 12:37:46 UTC 2019
Hi David, Dean,
Yes. Thanks for waiting. I think two arm.ad rules are definitely better
than one with incorrect print. Now arm.ad looks very similar to the
state before removing obsolete AssumeMP option:
http://cr.openjdk.java.net/~bulasevich/8222825/webrev.04.jdk-jdk
thanks,
Boris
On 16.05.2019 12:00, David Holmes wrote:
> Hi Boris,
>
> Waiting to see if you have any response to Dean's suggestion before I
> push this for you.
>
> David
>
> On 14/05/2019 4:28 pm, dean.long at oracle.com wrote:
>> This looks good. In arm.ad, you could use two rules and a predicate,
>> so that the PrintOptoAssembly output is correct.
>>
>> dl
>>
>> On 5/13/19 11:04 PM, Boris Ulasevich wrote:
>>> Hi David,
>>>
>>> It took a time to test the change :)
>>> Yes, please. Thank you!
>>>
>>> Boris
>>>
>>> 14.05.2019 8:00, David Holmes пишет:
>>>> Hi Boris,
>>>>
>>>> I was wondering what had happened to this one :)
>>>>
>>>> Looks good! I can sponsor for you.
>>>>
>>>> Thanks,
>>>> David
>>>>
>>>> On 14/05/2019 2:45 pm, Boris Ulasevich wrote:
>>>>> Hi all,
>>>>>
>>>>> Please review the fix for SIGILL that happens on ARM32 CPU without
>>>>> Multiprocessing Extensions support (this applies to single-core
>>>>> arm32 processors, which are not very common this days). With the
>>>>> fix we check PLDW instruction availability on JVM start-up to
>>>>> correct further instructions generation.
>>>>>
>>>>> https://bugs.openjdk.java.net/browse/JDK-8222825
>>>>> http://cr.openjdk.java.net/~bulasevich/8222825/webrev.03.jdk-jdk
>>>>>
>>>>> thanks,
>>>>> Boris
>>>
>>>
>>
More information about the hotspot-dev
mailing list