RFR(M) 8072383: resolve conflicts between open and closed ports

Volker Simonis volker.simonis at gmail.com
Fri Feb 20 13:32:10 UTC 2015


Hi Dean,

I've only looked at the 8u60 changes until now but it's actually a
good idea to also check the aarch-stage repo on our platforms.
I'll run the builds and let you know.

Regards,
Volker


On Thu, Feb 19, 2015 at 10:40 PM, Dean Long <dean.long at oracle.com> wrote:
> Thanks Volker, I will list you as a reviewer for 8u60.  Did you also look at
> the 9
> changes?  If so I will list you as a reviewer for 9 as well.
>
> dl
>
>
> On 2/19/2015 2:27 AM, Volker Simonis wrote:
>>
>> Hi Dean,
>>
>> I've just checked webrev.8u60.01 on Linux/PPC64. It cleanly builds and
>> runs so thumbs up from me.
>>
>> Regards,
>> Volker
>>
>>
>> On Thu, Feb 19, 2015 at 5:12 AM, Dean Long <dean.long at oracle.com> wrote:
>>>
>>> Yes, good catch.  I've update the webrev to webrev.8u60.01.  Thanks for
>>> the
>>> review.
>>>
>>> dl
>>>
>>>
>>> On 2/18/2015 4:45 PM, Vladimir Kozlov wrote:
>>>>
>>>> Did you missed to remove #elif ppc_32 in 8u60 changes in
>>>> src/share/vm/interpreter/templateTable.hpp?
>>>>
>>>> Otherwise both versions look good.
>>>>
>>>> Thanks,
>>>> Vlaidmir
>>>>
>>>> On 2/18/15 4:26 PM, Dean Long wrote:
>>>>>
>>>>> Just to avoid confusion, let me clarify that the jdk9 changes below
>>>>> can't be pushed until after the open aarch64 port is merged into the
>>>>> main jdk9, as they are based on the staging repo.
>>>>>
>>>>> dl
>>>>>
>>>>> On 2/18/2015 4:02 PM, Dean Long wrote:
>>>>>>
>>>>>> These changes resolve some issues with references to closed ports in
>>>>>> open hotspot code,
>>>>>> primarily by removing those references completely.  I have included
>>>>>> the 8u60 backport
>>>>>> as well because it won't apply cleanly, and I may push it first
>>>>>> because the 9 changes are
>>>>>> blocked by JEP 237: Linux/AArch64 Port.
>>>>>>
>>>>>> http://cr.openjdk.java.net/~dlong/8072383/webrev.8u60.00/
>>>>>> http://cr.openjdk.java.net/~dlong/8072383/webrev.9.00/
>>>>>>
>>>>>> dl
>>>>>
>>>>>
>


More information about the hotspot-compiler-dev mailing list