Hi David,
I will incorporate changes suggested by you. Let's wait for few more
review comments and then I will send consolidated webrev.
Regards
Harsha
On Wednesday 17 August 2016 09:02 AM, David Holmes wrote:
On 16/08/2016 11:33 PM, Harsha Wardhana B wrote:
Hi David,
Agreed that we could
On 16/08/2016 11:33 PM, Harsha Wardhana B wrote:
Hi David,
Agreed that we could fix WaitingThread the way you have said, but in
recent past, there aren't any issues reported w.r.t WaitingThread.
Nor are there likely to be - that's what makes spurious wakeup bugs so
difficult to detect!
Thi
On 17/08/2016 6:57 AM, Leela Mohan wrote:
Your thoughts on this will be helpful.
Thanks,
Leela
[ Removed "serviceability-...@openjdk.net" mailing alias added
"hotspot-dev" since i am getting failed mail delivery notification ]
Added back - it is openjdk.java.net, not openjdk.net
David
-
Hi David,
Agreed that we could fix WaitingThread the way you have said, but in
recent past, there aren't any issues reported w.r.t WaitingThread.
This test has been fixed several times (3-4) for intermittent failures
and hence I would not like to meddle with code that is not causing any
prob
Hi all,
Have you checked changes for this issue?
http://cr.openjdk.java.net/~ysuenaga/JDK-8163581/webrev.01/hotspot/
http://cr.openjdk.java.net/~ysuenaga/JDK-8163581/webrev.01/jdk/
For example, ParallelScavengeHeap.java overrides printOn() method.
This method shows addresses of each memory
Thanks for the Review, Staffan.
-Harsha
On Tuesday 16 August 2016 01:17 PM, Staffan Larsen wrote:
Ok. Reviewed.
/Staffan
On 16 aug. 2016, at 08:15, Harsha Wardhana B
wrote:
Yes. assertEQorGTE can be used. But using assertEQorLTE is better suited as it
increase code readability. So I wou
Hi Harsha,
On 16/08/2016 4:08 PM, Harsha Wardhana B wrote:
Hello,
Please review and provide comments for fix for issue,
https://bugs.openjdk.java.net/browse/JDK-8152589
having webrev located at
http://cr.openjdk.java.net/~hb/8152589/webrev.00/
These changes look quite good (though I have t
Looks fine to me.
Thanks
Max
On 8/16/2016 14:19, Mallikarjuna Avaluri wrote:
Hi,
Reminder, please help me with this review.
Regards,
Mallikarjuna Avaluri
On 8/9/2016 11:11 AM, Mallikarjuna Avaluri wrote:
Hi all,
Could you please review fix for following issue.
Bug id: https://bugs.ope
Thanks David!
The matching logic is not that complicated (but it took me the better part of a
day to figure all of this out.) The debugger sets a filter for which exceptions
should be reported. Part of that filter includes a class reference. If the
exception class IsInstanceOf() the class in th
Ok. Reviewed.
/Staffan
> On 16 aug. 2016, at 08:15, Harsha Wardhana B
> wrote:
>
> Yes. assertEQorGTE can be used. But using assertEQorLTE is better suited as
> it increase code readability. So I would like to leave that in place.
>
>
> On Saturday 13 August 2016 12:59 AM, Staffan Larsen wr
10 matches
Mail list logo