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

feat(riscv64,gnu): add some const PTRACE_* #2668

Merged
merged 1 commit into from
Feb 6, 2022

Conversation

CoelacanthusHex
Copy link
Contributor

  • PTRACE_GETFPREGS
  • PTRACE_SETFPREGS
  • PTRACE_GETFPXREGS
  • PTRACE_SETFPXREGS
  • PTRACE_GETREGS
  • PTRACE_SETREGS

get from sys/ptrace.h of riscv64 glibc

Signed-off-by: Coelacanthus coelacanthus@outlook.com

@rust-highfive
Copy link

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @Amanieu (or someone else) soon.

Please see the contribution instructions for more information.

* PTRACE_GETFPREGS
* PTRACE_SETFPREGS
* PTRACE_GETFPXREGS
* PTRACE_SETFPXREGS
* PTRACE_GETREGS
* PTRACE_SETREGS

get from sys/ptrace.h of riscv64 glibc

Signed-off-by: Coelacanthus <coelacanthus@outlook.com>
@Amanieu
Copy link
Member

Amanieu commented Feb 6, 2022

@bors r+

@bors
Copy link
Contributor

bors commented Feb 6, 2022

📌 Commit a08edb4 has been approved by Amanieu

@bors
Copy link
Contributor

bors commented Feb 6, 2022

⌛ Testing commit a08edb4 with merge d5f6f7a...

@bors
Copy link
Contributor

bors commented Feb 6, 2022

☀️ Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: Amanieu
Pushing d5f6f7a to master...

@bors bors merged commit d5f6f7a into rust-lang:master Feb 6, 2022
Avimitin added a commit to Avimitin/archriscv-packages that referenced this pull request Apr 21, 2022
This patch fix three error:

1. The [target specification issue].

2. Crate libc can not be compiled due to symbol missing.

This is fixed in [rust-lang/libc PR felixonmars#2668], and release in 0.2.118
version.

3. Test failure

Some tests of diskonaut depend on the previous events. It take a
snapshot of the TUI pixels output, then compare those snapshot pixel by
pixel to determine if the cli is working correctly.

However those test are not reliable. If the previous event took too
long to finish, the TUI output will not match what is expected.

The current workaround is to increase the event interval. One second
interval is enough for all event to finish its jobs.

Tested in 5950x QEMU environment.

Ref:
* [target specification issue]: felixonmars#670 (comment)
* [rust-lang/libc PR felixonmars#2668]: rust-lang/libc#2668

Signed-off-by: Avimitin <avimitin@gmail.com>
felixonmars pushed a commit to felixonmars/archriscv-packages that referenced this pull request Apr 22, 2022
This patch fix three error:

1. The [target specification issue].

2. Crate libc can not be compiled due to symbol missing.

This is fixed in [rust-lang/libc PR #2668], and release in 0.2.118
version.

3. Test failure

Some tests of diskonaut depend on the previous events. It take a
snapshot of the TUI pixels output, then compare those snapshot pixel by
pixel to determine if the cli is working correctly.

However those test are not reliable. If the previous event took too
long to finish, the TUI output will not match what is expected.

The current workaround is to increase the event interval. One second
interval is enough for all event to finish its jobs.

Tested in 5950x QEMU environment.

Ref:
* [target specification issue]: #670 (comment)
* [rust-lang/libc PR #2668]: rust-lang/libc#2668

Signed-off-by: Avimitin <avimitin@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants