RFR: 8169529: AArch64: Revert old JDK-8167595 changes after JDK-8159035 fix is pushed

[Tobias Hartmann]

Hi,

this looks good to me. Here are the changes for JDK-8159035 (should have been reviewed in the open):
http://hg.openjdk.java.net/jdk9/hs/jdk/rev/0e98c765ce9b

Best regards,
Tobias

On 17.11.2016 12:52, Andrew Dinn wrote:
> 
> 
> On 17/11/16 11:45, Andrew Haley wrote:
>> On 17/11/16 11:08, Rahul Raghavan wrote:
>>> <webrev> - http://cr.openjdk.java.net/~rraghavan/8169529/webrev.00/
>>> <jbs> - https://bugs.openjdk.java.net/browse/JDK-8169529
>>>
>>> Old JDK-8167595 changes was done in stubGenerator_aarch64.cpp, 
>>>   to fix crash in cipherBlockChaining_decryptAESCrypt due to unhandled case of input cipher length value as 0.
>>> With JDK-8159035 push the same issue is fixed in core-libs Java wrapper method side.
>>> So now old 8167595 changes may be reverted.
>>
>> OK.  I can't see the bug and I can't see the JDK changes, but I'm
>> prepared to believe what you say.
> 
> It seems this is an Oracle-specific bug. I assume the JDk patch will
> include some changes to open code. Could we at least see the relevant
> part of the JDK change set?
> 
> regards,
> 
> 
> Andrew Dinn
> -----------
> Senior Principal Software Engineer
> Red Hat UK Ltd
> Registered in England and Wales under Company Registration No. 03798903
> Directors: Michael Cunningham, Michael ("Mike") O'Neill, Eric Shander
>