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.

Sign up for GitHub

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

Jump to bottom

Investigate and fix 3 AArch64 unit test regressions found when executing on QEMU #6260

Open
AssadHashmi opened this issue Aug 10, 2023 · 0 comments
Open

Investigate and fix 3 AArch64 unit test regressions found when executing on QEMU #6260

AssadHashmi opened this issue Aug 10, 2023 · 0 comments
Labels
Component-Tests OpSys-AArch64

Comments

@AssadHashmi
Copy link
Contributor

The SVE support patch #5835 resulted in the following unit tests only failing on QEMU, always passing on native AArch64 hardware:

        code_api|client.drx_buf-test 
        code_api|sample.memval_simple 
        code_api|client.drreg-test

This issue is to investigate whether these are due to the SVE patch or a QEMU defect, see #5835 (comment)
@AssadHashmi AssadHashmi mentioned this issue Aug 10, 2023
Merged
AssadHashmi added a commit that referenced this issue Aug 10, 2023
@AssadHashmi
Addressed final set of review comments:
a57fb68 
- Updated release notes to include compatibility-affecting changes due
  to dr_mcontext_t and dr_simd_t support for SVE.
- Added binary compatibility machine context size check in
  priv_mcontext_to_dr_mcontext() for DR_MC_MULTIMEDIA clients in the
  case of pre-SVE clients.
- Made AArch32 SIMD registers comment private (non-Doxygen).
- Updated ignore list in runsuite_wrapper.pl for QEMU based regressions,
  issue #6260.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Component-Tests OpSys-AArch64
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@AssadHashmi