Skip to content

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

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

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

Conversation

@pitrou

@pitrou pitrou commented Jun 12, 2017

Copy link
Copy Markdown
Member

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

pitrou commented Jun 12, 2017

Copy link
Copy Markdown
Member Author

@njsmith

@njsmith

njsmith commented Jun 12, 2017

Copy link
Copy Markdown
Contributor

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

pitrou commented Jun 13, 2017

Copy link
Copy Markdown
Member Author

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

njsmith commented Jun 13, 2017

Copy link
Copy Markdown
Contributor

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