Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

linux: ptrace: Add definition for PTRACE_SYSEMU/_SINGLESTEP #1774

Merged
merged 1 commit into from
May 24, 2020

Conversation

jabedude
Copy link
Contributor

This patch adds definitions for the PTRACE_SYSEMU and PTRACE_SYSEMU_SINGLESTEP constants. These ptrace requests are available on Linux x86 and x86_64.

@rust-highfive
Copy link

r? @JohnTitor

(rust_highfive has picked a reviewer for you, use r? to override)

@JohnTitor
Copy link
Member

Re-triggering CI...

@JohnTitor JohnTitor closed this May 23, 2020
@JohnTitor JohnTitor reopened this May 23, 2020
@jabedude
Copy link
Contributor Author

Hmm, I don't understand the CI failures. It doesn't look like it's because of my commit?

@JohnTitor
Copy link
Member

Hmm, I don't understand the CI failures. It doesn't look like it's because of my commit?

CI failures were unrelated and re-triggering resolved it.

Copy link
Member

@JohnTitor JohnTitor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@JohnTitor JohnTitor merged commit 8684b55 into rust-lang:master May 24, 2020
bors added a commit that referenced this pull request May 12, 2023
Support for `PTRACE_SYSEMU` and `PTRACE_SYSEMU_SINGLESTEP` on musl

This adds support for the `PTRACE_SYSEMU` and `PTRACE_SYSEMU_SINGLESTEP` on certain musl architectures. I grabbed these constants by doing a `git grep -rnI PTRACE_SYSEMU` in the musl repository and got a list of certain architecture directories.

See #1774

By the way: I am not sure if I should have added support for that many architectures, just because musl supports them. After all, the corresponding glibc only supports `x86(_64)` anyway.
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.

3 participants