dragonwell-project / dragonwell8

Alibaba Dragonwell8 JDK
http://dragonwell-jdk.io
GNU General Public License v2.0
4.21k stars 496 forks source link

[Backport] 8225690: Multiple AttachListener threads can be created #261

Closed XHao closed 2 years ago

XHao commented 3 years ago

Description: VM can't be attached If someone clean up the /tmp directory. JDK-8225690 Multiple AttachListener threads can be created can fix this.

D-D-H commented 3 years ago

Hi @XHao ,

I agree with this backport, but I think that we should discuss this backport in the OpenJDK 8u community, which will also benefit other users using JDK 8.

y1yang0 commented 3 years ago

I will try to backport this to OpenJDK 8u upstream and also backport to Dragonwell 8 once upstream OpenJDK 8u upstream is merged.

luchsh commented 3 years ago

I do not see any 8u backport requests in the upstream mailing list, can anybody post a link of where the discussion happens? thanks.

y1yang0 commented 3 years ago

@XHao @luchsh Sorry for confusion before. I've prepared a backport changeset to wait for approval.

https://mail.openjdk.java.net/pipermail/jdk8u-dev/2021-October/014334.html https://mail.openjdk.java.net/pipermail/jdk8u-dev/2021-October/014375.html https://mail.openjdk.java.net/pipermail/jdk8u-dev/2021-October/014376.html

y1yang0 commented 2 years ago

Patches in upstream are merged, I'd like to backport them to downstream(dragonwell8), will get back soon.

D-D-H commented 2 years ago

Patches in upstream are merged, I'd like to backport them to downstream(dragonwell8), will get back soon.

Thanks, but I think this backport will be auto included in the next release of Alibaba dragonwell 8, we don't need to backport it again, right?

D-D-H commented 2 years ago

@XHao Hi, This issue has been backported to OpenJDK 8u (https://bugs.openjdk.java.net/browse/JDK-8274953). And the next Alibaba Dragonwell 8 release will include this fix. So I close this issue and please feel free to reopen it if you need.