RFR(S): 8132242: LogTouchedMethods (8025692) asserts if TieredCompilation is off.

Volker Simonis volker.simonis at gmail.com
Wed Jul 29 08:21:11 UTC 2015


Hi Goetz, Yumin,

the change looks good!
You can go ahead and push it.

Thanks,
Volker


On Tue, Jul 28, 2015 at 5:48 PM, Yumin Qi <yumin.qi at oracle.com> wrote:
> Goetz,
>
>   Thanks for taking care of this issue. It looks good. I am not a "R"eviewer
> and I think Ioi is not either, we still need a 'R'eviewer to give a pass.
>   I can sponsor for the push.
>
> Thanks
> Yumin
>
> On 7/28/2015 1:58 AM, Lindenmaier, Goetz wrote:
>>
>> Hi,
>>
>> Yumin, could you have a look at this, please?  It's trivial.
>> Or could someone else sponsor this?
>> http://cr.openjdk.java.net/~goetz/webrevs/8132242-jtregPTM/webrev.01/
>>
>> ... I added the bugID I forgot before to the subject.
>>
>> Best regards,
>>    Goetz.
>>
>> -----Original Message-----
>> From: Daniel D. Daugherty [mailto:daniel.daugherty at oracle.com]
>> Sent: Donnerstag, 23. Juli 2015 15:49
>> To: Lindenmaier, Goetz
>> Cc: hotspot-runtime-dev at openjdk.java.net
>> Subject: Re: RFR(S): LogTouchedMethods (8025692) asserts if
>> TieredCompilation is off.
>>
>> Yumin,
>>
>> Since you did the work on:
>>
>> JDK-8025692 Log what methods are touched at run-time
>> https://bugs.openjdk.java.net/browse/JDK-8025692
>>
>> can you help Goetz with this tweak?
>>
>> Dan
>>
>>
>> On 7/23/15 7:30 AM, Lindenmaier, Goetz wrote:
>>>
>>> Hi,
>>>
>>> LogTouchedMethods causes the template interpreter to generate profiling
>>> code even
>>> if no compiler is used.  If TieredCompilation is off, code containing an
>>> assertion
>>> that checks that UseCompiler is set, is reached.
>>> This assertion exists on the sparc and ppc platforms.
>>> Fix:
>>> Check for LogTouchedMethods in the assertion.
>>> Also added testcase with -XX:-TieredCompilation.
>>>
>>> The problem can be reproduced with the new test case on solaris sparc.
>>>
>>> Please review this change.  I please need a sponsor.
>>> http://cr.openjdk.java.net/~goetz/webrevs/8132242-jtregPTM/webrev.01/
>>>
>>> Best regards,
>>>     Goetz Lindenmaier
>
>


More information about the hotspot-runtime-dev mailing list