X Tutup
Skip to content

bpo-30644: Mention how to disable signal fd wakeup#2140

Merged
pitrou merged 1 commit intopython:masterfrom
pitrou:wakeup_fd_doc_nit
Jun 13, 2017
Merged

bpo-30644: Mention how to disable signal fd wakeup#2140
pitrou merged 1 commit intopython:masterfrom
pitrou:wakeup_fd_doc_nit

Conversation

@pitrou
Copy link
Copy Markdown
Member

@pitrou pitrou commented Jun 12, 2017

No description provided.

@mention-bot
Copy link
Copy Markdown

@pitrou, thanks for your PR! By analyzing the history of the files in this pull request, we identified @birkenfeld, @loewis and @taleinat to be potential reviewers.

@pitrou
Copy link
Copy Markdown
Member Author

pitrou commented Jun 12, 2017

@njsmith

@njsmith
Copy link
Copy Markdown
Contributor

njsmith commented Jun 12, 2017

Yeah, good idea and lgtm

If you want to get really fancy, I guess you could also mention that on unixes it's an arbitrary FD, but on Windows it should be a socket handle (not an FD at all).

@pitrou
Copy link
Copy Markdown
Member Author

pitrou commented Jun 13, 2017

If I'm reading signalmodule.c correctly (especially the functions signal_set_wakeup_fd and trip_signal), we support both regular file descriptors and sockets on Windows (but not native Windows handles).

@pitrou pitrou merged commit d79c1d4 into python:master Jun 13, 2017
@pitrou pitrou deleted the wakeup_fd_doc_nit branch June 13, 2017 08:14
pitrou added a commit to pitrou/cpython that referenced this pull request Jun 13, 2017
pitrou added a commit to pitrou/cpython that referenced this pull request Jun 13, 2017
pitrou added a commit to pitrou/cpython that referenced this pull request Jun 13, 2017
@vstinner vstinner changed the title Mention how to disable signal fd wakeup bpo-30644: Mention how to disable signal fd wakeup Jun 13, 2017
pitrou added a commit that referenced this pull request Jun 13, 2017
pitrou added a commit that referenced this pull request Jun 13, 2017
pitrou added a commit that referenced this pull request Jun 13, 2017
@njsmith
Copy link
Copy Markdown
Contributor

njsmith commented Jun 13, 2017

we support both regular file descriptors and sockets on Windows (but not native Windows handles).

Ah, looks like you're right. In practice though I don't think there are any uses for passing in a regular file descriptor, because only sockets can be set in non-blocking mode, and python sockets are always represented as socket handles and not wrapped into an fd.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation in the Doc dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

X Tutup