Review request: Add barrier set kind for G1 throughput barrier

David Holmes david.holmes at oracle.com
Mon Jan 5 01:03:05 UTC 2015


On 31/12/2014 6:02 AM, Joseph Provino wrote:
> Jon, you are right about the bug id being incorrect.
>
> I think to completely fix the problem described in the real bug report
> I also have to add a protected destructor to barrierSet.hpp.
>
> I'll submit to jprt then send another webrev with the right bug id.

Also note that hotspot reviews go on the appropriate hotspot team 
mailing list (or the general hotspot-dev) not the jdk9-dev alias.

David

> thanks.
>
> joe
>
> PS I'm planning to fix https://bugs.openjdk.java.net/browse/JDK-8067191
> next.
>
>
> On 12/30/2014 12:48 PM, Jon Masamitsu wrote:
>> Joe,
>>
>> Change looks good as long as the bug for this change
>> is really
>>
>> https://bugs.openjdk.java.net/browse/JDK-8064947
>>
>> Or maybe the webrev link is not right?
>>
>> Jon
>>
>> On 12/29/2014 09:51 AM, Joseph Provino wrote:
>>> Can I get reviews for this very small change?  I also need a sponsor
>>> to push the change.
>>>
>>> Bug report is here:
>>>
>>> https://bugs.openjdk.java.net/browse/JDK-8067191
>>>
>>> Webrev is here:
>>>
>>> http://cr.openjdk.java.net/~jprovino/8067191/webrev.00
>>>
>>> Testing:  jprt
>>
>


More information about the jdk9-dev mailing list