Hi! Thanks for the reply, Aleksey and Alan. I had submitted the patch after viewing the first response but have now withdrawn it as PR is already open. Regards, Kartik
On Thu, Sep 9, 2021 at 6:32 PM Alan Bateman <alan.bate...@oracle.com> wrote: > On 09/09/2021 13:36, Kartik Ohri wrote: > > Hi all! > > I would like to work on a fix for JDK-8273541 > > <https://bugs.openjdk.java.net/browse/JDK-8273541>. I think the > regression > > was introduced by the change here > > < > https://github.com/openjdk/jdk/pull/2380/files#diff-d62f4055445b40f5d9862e02f8be6da2d547ac77ffae362d7034595d58c7389aL220-R217 > >. > > It seems that this was accidentally changed from Thread.MAX_PRIORITY to > > Thread.MIN_PRIORITY. As Thread.MIN_PRIORITY - 2 is negative and because > of > > the check here > > < > https://github.com/openjdk/jdk/pull/2380/files#diff-9f140a5980d51de2a5f392a6ffb91f51d2303760b699d5366125fb66f55b70f0R120-R122 > >, > > the thread priority is not set. Changing this back to > > Thread.MAX_PRIORITY should fix the problem. I tried the patch > > < > https://github.com/amCap1712/jdk/commit/b894bda67f9f4f8f48ba97679247a8218f6b95dc > > > > locally and it seems to fix the issue for me. If it is fine, I would like > > to submit this fix. > > > Heinz Kabutz is working on a PR here: > https://github.com/openjdk/jdk/pull/5439 > > -Alan > > >
