add missing macro USE_DISCONNECTEX switch #705
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Owner
|
windows上现在默认用的wepoll，没有编译使用这个文件了 |
Contributor
Author
我之前以为开了USE IOCP会启用重叠IO这个文件实现。。这个重叠IO为什么不打算用了？据说iocp+overlapio才是socket通讯性能极限。 |
Owner
因为select、poll、epoll、kqueue这些都是reactor模型，事件到来然后进行读写，而iocp+overlapio是proactor模型，先投递读写，然后通知完成，这两种模型不太好统一，所以目前在windows上是使用了wepoll来调用iocp实现reactor模型，没有使用到overlapio |
Contributor
Author
我看后续规划中有支持linux的io_uring考虑，这个应该和iocp+overlapio类似吧 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
event/overlapio.c 行397存在这个宏开关