RFR(M) 8150353: PPC64LE: Support RTM on linux

David Holmes david.holmes at oracle.com
Thu Feb 25 08:21:22 UTC 2016


On 25/02/2016 6:11 PM, Gustavo Romero wrote:
> Hi David,
>
> OK, I'll fix that.
>
> Should I re-send the RFR with the right URL and abandon this one?

I think you can just post the new URL to this one.

> In case a contribution is not so small, even so is it fine to include it inline?

Well it's preferable, for larger contributions, to find someone to host 
the webrev for you.

Cheers,
David

> Thank you.
>
> Regards,
> Gustavo
>
> On 25-02-2016 04:54, David Holmes wrote:
>> Hi Gustavo,
>>
>> Just a point of order. All contributions to the OpenJDK must be made through OpenJDK infrastructure. So you must either get someone to host your webrev on cr.openjdk.java.net, or include it inline in
>> email (attachments tend to get stripped).
>>
>> Sorry for the inconvenience.
>>
>> David
>>
>> On 25/02/2016 5:50 AM, Gustavo Romero wrote:
>>> Hi Martin,
>>>
>>> Both little and big endian Linux kernel contain the syscall change, so
>>> I did not include:
>>>
>>> #if defined(COMPILER2) && (defined(AIX) || defined(VM_LITTLE_ENDIAN)
>>>
>>> in globalDefinitions_ppc.hpp.
>>>
>>> Please, could you review the following change?
>>>
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8150353
>>> Webrev (hotspot): http://81.de.7a9f.ip4.static.sl-reverse.com/webrev/
>>>
>>> Summary:
>>>
>>> * Enable RTM support for Linux on PPC64 (LE and BE).
>>> * Fix C2 compiler buffer size issue.
>>>
>>> Thank you.
>>>
>>> Regards,
>>> Gustavo
>>>
>>
>


More information about the hotspot-dev mailing list