Skip to content

Conversation

leon-db
Copy link

@leon-db leon-db commented Sep 1, 2025

…wait forever

the killer thread:
shutdown the killed thread vio, this action will close the connection fd and remove the fd from the red-black tree of the epoll_wait fd
the killed thread:
first run the start_io function, this function will bind the connection fd to the epoll_wait fd.
second the connection wait from the epoll_wait

when worker thread process a connection, first bind the connection fd to the epoll_wait fd, then killer thread close the connection fd, which will make the killed connection have no chance to run any more.

@leon-db leon-db changed the title fix: when kill a connection, it have a change make the killed thread … [PS-10139] fix: when kill a connection, it have a change make the killed thread … Sep 1, 2025
@leon-db leon-db changed the title [PS-10139] fix: when kill a connection, it have a change make the killed thread … PS-10139 fix: when kill a connection, it have a change make the killed thread … Sep 1, 2025
@leon-db leon-db force-pushed the PS-10139-fix-when-kill-a-connection-it-have-a-change-make-the-killed-thread-wait-forever branch from 694b3ec to a1f6942 Compare September 2, 2025 06:45
…wait forever

the killer thread:
   shutdown the killed thread vio, this action will close the connection fd and remove the fd from the red-black tree of the epoll_wait fd
the killed thread:
   first run the start_io function, this function will bind the connection fd to the epoll_wait fd.
   second the connection wait from the epoll_wait

when worker thread process a connection, first bind the connection fd to the epoll_wait fd, then killer thread close the connection fd, which
will make the killed connection have no change to run any more.
@leon-db leon-db force-pushed the PS-10139-fix-when-kill-a-connection-it-have-a-change-make-the-killed-thread-wait-forever branch from a1f6942 to 27df434 Compare September 2, 2025 07:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants