Skip to content

os/exec: use pidfd for waiting and signaling of processes#60461

Draft
mitar wants to merge 1 commit intogolang:masterfrom
mitar:pidfd_open
Draft

os/exec: use pidfd for waiting and signaling of processes#60461
mitar wants to merge 1 commit intogolang:masterfrom
mitar:pidfd_open

Conversation

@mitar
Copy link
Contributor

@mitar mitar commented May 26, 2023

Using pidfd allows us to have a handle on the process and poll the handle to non-blocking wait for the process to exit.

Fixes #34396
Fixes #60321
Fixes #60320

@gopherbot
Copy link
Contributor

This PR (HEAD: 14ffe54) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/go/+/498615 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

@gopherbot
Copy link
Contributor

This PR (HEAD: 86ae506) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/go/+/498615 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

Using pidfd allows us to have a handle on the process and
poll the handle to non-blocking wait for the process to
exit.

Fixes golang#34396
Fixes golang#60321
Fixes golang#60320
@gopherbot
Copy link
Contributor

This PR (HEAD: 9459756) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/go/+/498615 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

@gopherbot
Copy link
Contributor

Message from Mitar:

Patch Set 2:

(14 comments)


Please don’t reply on this GitHub thread. Visit golang.org/cl/498615.
After addressing review feedback, remember to publish your drafts!

@seankhliao
Copy link
Member

stale?

@seankhliao seankhliao closed this Apr 15, 2025
@mitar
Copy link
Contributor Author

mitar commented Apr 15, 2025

It is not stale, but no feedback was given on the CR. :-(

So I would be really sad if this is just closed. Please review the CR instead.

@seankhliao seankhliao reopened this Apr 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants