Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Bug] WispEventPump epollWait IllegalArgumentException #208

Closed
joeyleeeeeee97 opened this issue Feb 20, 2021 · 0 comments
Closed

[Bug] WispEventPump epollWait IllegalArgumentException #208

joeyleeeeeee97 opened this issue Feb 20, 2021 · 0 comments
Assignees
@joeyleeeeeee97
Milestone
Dragonwell 8.6.6 ...

Comments

@joeyleeeeeee97
Copy link
Contributor

Description
In wisp Monolithic Epoll, we clear Epoll fd after the park.
But at shutdown, an interruption may break this and leave an uncleared epoll array on wise task, IllegalArgumentException may happen when reuse this wispTask.

Steps to Reproduce
Shutdown an epoll wait task then reuse it.

Expected behavior
NO exception.

JDK version
8.6.5 ga

Execution environment

  • OS and version:
  • CPU model:
  • Number of CPU cores:
  • Size of physical memory:
  • Inside Linux container?
    • Linux container name (docker, pouch, etc):
    • Linux container version:
@joeyleeeeeee97 joeyleeeeeee97 self-assigned this Feb 20, 2021
joeyleeeeeee97 pushed a commit to joeyleeeeeee97/dragonwell8_jdk that referenced this issue Feb 20, 2021
[Wisp] Fix WispEventPump epollWait IllegalArgumentException
dd4479a 
Summary: In wisp Monolithic Epoll, we clear Epoll fd after the park.
But at shutdown, an interruption may break this and leave an uncleared
epoll array on wise task, IllegalArgumentException may happen when reuse
this wispTask.

Test Plan: jtreg test/wisp2

Reviewed-by: yuleil, zhengxiaolinX

Issue: dragonwell-project/dragonwell8#208
@joeyleeeeeee97 joeyleeeeeee97 mentioned this issue Feb 20, 2021
Merged
joeyleeeeeee97 pushed a commit to joeyleeeeeee97/dragonwell8_jdk that referenced this issue Feb 20, 2021
[Wisp] Fix WispEventPump epollWait IllegalArgumentException
86bc194 
Summary: In wisp Monolithic Epoll, we clear Epoll fd after the park.
But at shutdown, an interruption may break this and leave an uncleared
epoll array on wise task, IllegalArgumentException may happen when reuse
this wispTask.

Test Plan: jtreg test/wisp2

Reviewed-by: yuleil, zhengxiaolinX

Issue: dragonwell-project/dragonwell8#208
joeyleeeeeee97 pushed a commit to dragonwell-project/dragonwell8_jdk that referenced this issue Mar 9, 2021
@joeyleeeeeee97
[Wisp] Fix WispEventPump epollWait IllegalArgumentException
4e51c2b 
Summary: In wisp Monolithic Epoll, we clear Epoll fd after the park.
But at shutdown, an interruption may break this and leave an uncleared
epoll array on wise task, IllegalArgumentException may happen when reuse
this wispTask.

Test Plan: jtreg test/wisp2

Reviewed-by: yuleil, zhengxiaolinX

Issue: dragonwell-project/dragonwell8#208
Sere-Fu pushed a commit to Sere-Fu/dragonwell11 that referenced this issue Aug 13, 2021
@Sere-Fu
[Wisp] Port from jdk8u
011d836 
Summary:
Port changes from Dragonwell8 and AJDK 8
Wisp Thread-based asynchronous IO implementation
- https://code.aone.alibaba-inc.com/ajdk/jdk8u_jdk/codereview/5309632
Wisp WispTask memory leak in shutdown
- dragonwell-project/dragonwell8#211
Wisp ResourceContainerMXBean
- dragonwell-project/dragonwell8#206
Wisp Fix WispEventPump epollWait IllegalArgumentException
- dragonwell-project/dragonwell8#208
Wisp Shutdown Enhancement
- https://code.aone.alibaba-inc.com/ajdk/jdk8u_jdk/codereview/5165311

Test Plan: wisp rcm tests

Reviewers: lei.yul, sanhong.lsh

Issue: https://aone.alibaba-inc.com/task/34331904

CR: https://code.aone.alibaba-inc.com/xcode/jdk11/codereview/5350241
Sere-Fu pushed a commit to Sere-Fu/dragonwell11 that referenced this issue Aug 30, 2021
@Sere-Fu
[Wisp] Port from jdk8u
e9c3902 
Summary:
Port changes from Dragonwell8 and AJDK 8
Wisp Thread-based asynchronous IO implementation
- https://code.aone.alibaba-inc.com/ajdk/jdk8u_jdk/codereview/5309632
Wisp WispTask memory leak in shutdown
- dragonwell-project/dragonwell8#211
Wisp ResourceContainerMXBean
- dragonwell-project/dragonwell8#206
Wisp Fix WispEventPump epollWait IllegalArgumentException
- dragonwell-project/dragonwell8#208
Wisp Shutdown Enhancement
- https://code.aone.alibaba-inc.com/ajdk/jdk8u_jdk/codereview/5165311

Test Plan: wisp rcm tests

Reviewers: lei.yul, sanhong.lsh

Issue: https://aone.alibaba-inc.com/task/34331904

CR: https://code.aone.alibaba-inc.com/xcode/jdk11/codereview/5350241
@Sere-Fu Sere-Fu mentioned this issue Sep 1, 2021
Closed
Sere-Fu pushed a commit to Sere-Fu/dragonwell11 that referenced this issue Sep 2, 2021
@Sere-Fu
[Wisp] Port from jdk8u
da13e47 
Summary:
Port changes from Dragonwell8 and AJDK 8
Wisp Thread-based asynchronous IO implementation
- https://code.aone.alibaba-inc.com/ajdk/jdk8u_jdk/codereview/5309632
Wisp WispTask memory leak in shutdown
- dragonwell-project/dragonwell8#211
Wisp ResourceContainerMXBean
- dragonwell-project/dragonwell8#206
Wisp Fix WispEventPump epollWait IllegalArgumentException
- dragonwell-project/dragonwell8#208
Wisp Shutdown Enhancement
- https://code.aone.alibaba-inc.com/ajdk/jdk8u_jdk/codereview/5165311

Test Plan: wisp rcm tests

Reviewers: lei.yul, sanhong.lsh

Issue: https://aone.alibaba-inc.com/task/34331904

CR: https://code.aone.alibaba-inc.com/xcode/jdk11/codereview/5350241
yuleil pushed a commit to yuleil/dragonwell11 that referenced this issue Jul 22, 2022
[Wisp] Port from jdk8u
a940716 
Summary:
Port changes from Dragonwell8 and AJDK 8
Wisp Thread-based asynchronous IO implementation
- https://code.aone.alibaba-inc.com/ajdk/jdk8u_jdk/codereview/5309632
Wisp WispTask memory leak in shutdown
- dragonwell-project/dragonwell8#211
Wisp ResourceContainerMXBean
- dragonwell-project/dragonwell8#206
Wisp Fix WispEventPump epollWait IllegalArgumentException
- dragonwell-project/dragonwell8#208
Wisp Shutdown Enhancement
- https://code.aone.alibaba-inc.com/ajdk/jdk8u_jdk/codereview/5165311

Test Plan: wisp rcm tests

Reviewers: lei.yul, sanhong.lsh

Issue: https://aone.alibaba-inc.com/task/34331904

CR: https://code.aone.alibaba-inc.com/xcode/jdk11/codereview/5350241
ZhaiMo15 added a commit to kuaishou-syslab/dragonwell17 that referenced this issue Jun 26, 2023
@ZhaiMo15
[Wisp] Port from jdk8u.
b172b99 
Summary:
Port changes from Dragonwell8 and AJDK 8
Wisp Thread-based asynchronous IO implementation
- https://code.aone.alibaba-inc.com/ajdk/jdk8u_jdk/codereview/5309632
Wisp WispTask memory leak in shutdown
- dragonwell-project/dragonwell8#211
Wisp ResourceContainerMXBean
- dragonwell-project/dragonwell8#206
Wisp Fix WispEventPump epollWait IllegalArgumentException
- dragonwell-project/dragonwell8#208
Wisp Shutdown Enhancement
- https://code.aone.alibaba-inc.com/ajdk/jdk8u_jdk/codereview/5165311

Test Plan: wisp rcm tests
@ZhaiMo15 ZhaiMo15 mentioned this issue Aug 28, 2023
Closed
ZhaiMo15 added a commit to kuaishou-syslab/dragonwell17 that referenced this issue Aug 28, 2023
@ZhaiMo15
[Wisp] Port from jdk8u.
3562c7a 
Summary:
Port changes from Dragonwell8 and AJDK 8
Wisp Thread-based asynchronous IO implementation
Wisp WispTask memory leak in shutdown
- dragonwell-project/dragonwell8#211
Wisp ResourceContainerMXBean
- dragonwell-project/dragonwell8#206
Wisp Fix WispEventPump epollWait IllegalArgumentException
- dragonwell-project/dragonwell8#208
Wisp Shutdown Enhancement
- https://code.aone.alibaba-inc.com/ajdk/jdk8u_jdk/codereview/5165311

Test Plan: wisp rcm tests

Reviewed-by: yulei

Issue:
dragonwell-project#142
@ZhaiMo15 ZhaiMo15 mentioned this issue Aug 28, 2023
Merged
ZhaiMo15 added a commit to kuaishou-syslab/dragonwell17 that referenced this issue Aug 28, 2023
@ZhaiMo15
[Wisp] Port from jdk8u.
89f4f14 
Summary:
Port changes from Dragonwell8 and AJDK 8
Wisp Thread-based asynchronous IO implementation
Wisp WispTask memory leak in shutdown
- dragonwell-project/dragonwell8#211
Wisp ResourceContainerMXBean
- dragonwell-project/dragonwell8#206
Wisp Fix WispEventPump epollWait IllegalArgumentException
- dragonwell-project/dragonwell8#208
Wisp Shutdown Enhancement

Test Plan: wisp rcm tests

Reviewed-by: yulei

Issue:
dragonwell-project#142
yuleil pushed a commit to dragonwell-project/dragonwell17 that referenced this issue Aug 29, 2023
@ZhaiMo15 @yuleil
[Wisp] Port from jdk8u.
8bbc879 
Summary:
Port changes from Dragonwell8 and AJDK 8
Wisp Thread-based asynchronous IO implementation
Wisp WispTask memory leak in shutdown
- dragonwell-project/dragonwell8#211
Wisp ResourceContainerMXBean
- dragonwell-project/dragonwell8#206
Wisp Fix WispEventPump epollWait IllegalArgumentException
- dragonwell-project/dragonwell8#208
Wisp Shutdown Enhancement

Test Plan: wisp rcm tests

Reviewed-by: yulei

Issue:
#142
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@joeyleeeeeee97 joeyleeeeeee97

Labels
None yet
Projects
None yet
Milestone
Dragonwell 8.6.6 Release
Development

No branches or pull requests
1 participant
@joeyleeeeeee97