Johan Sjölén
Johan Sjölén
The goal should be to replace tty with UL in as many cases as possible. So to me this change is great, because in the far future we can reduce...
> > > The goal should be to replace tty with UL > > > > > > UL has the same problems. UL is configured by argument parsing, so...
/label add hotspot
>Also did you establish this RW-lock will in fact be usable in this form for UL? I have rewritten the relevant code (`LogOutputList` class) using this RW-lock and it can...
>How does it compare with, for example, Terekhov's algorithm? I'm not familiar with this algorithm, do you have a link? >What is the source of the algorithm being used here?...
>What's proposed by this change has some similarities. This moves some state manipulation out from under the lock by using atomic operations on the state. It only has one condvar,...
I just pushed my first clean up of this code based on your comments. Here's a short summary: 1. Rename to ReadWriteLock 2. Move to utilities 3. Remove template, use...
Thank you for the additional review, @dholmes-ora . I fixed those and added some more documentation. The changes to `threadHelper.inline.hpp` is being merged through a separate patch (see JDK-8292679, https://github.com/openjdk/jdk/pull/9962),...
As no current review comments were alive I rebased onto master and force pushed. This reduces the PR a bit, as parts of it (additions to `threadHelper.inline.hpp`) was merged in...
Latest commit addresses most of Kim's review comments. It also makes the `current` thread optional, allowing it to be null. This is mainly useful if the lock is used very...