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

port to ARM master issue #1551

Closed
derekbruening opened this issue Nov 28, 2014 · 4 comments
Closed

port to ARM master issue #1551

derekbruening opened this issue Nov 28, 2014 · 4 comments

Comments

@derekbruening
Copy link
Contributor

From bruen...@google.com on October 08, 2014 00:19:33

xref issue #1550 xref issue #269 This is a container case for references in the code for tasks we're delaying while we're porting.

Original issue: http://code.google.com/p/dynamorio/issues/detail?id=1551

@derekbruening
Copy link
Contributor Author

After a massive amount of work and hundreds of commits, AArch32 support is pretty much complete. We can run large applications now (including Chrome with simple workloads, though some bugs are seen there). There are still some known missing features (IT splits, some cases of fault handling) but we'll treat those as corner cases and file separate issues on them from here on out.

@algr
Copy link

algr commented Jul 27, 2015

The ARM port is looking great. It is already very functional, and it seems right that this issue is now closed. The source still has several comments "FIXME i#1551: NYI on ARM". Perhaps these could be made specific to open issues for individual features?

@zhaoqin
Copy link
Contributor

zhaoqin commented Jul 29, 2015

> git grep "FIXME i#1551: NYI on ARM" | wc -l
74

We put FIXME comment back then and thought those are much less common cases. Some are small fixes and some need more efforts. ASSERT will be triggered if any is hit, so we can file issue or fix it then.

@derekbruening
Copy link
Contributor Author

The source still has several comments "FIXME i#1551: NYI on ARM". Perhaps these could be made
specific to open issues for individual features?

Yes, these should be cleaned up. Some should be removed as they are in code we do not plan to ever port. Likely some should be updated to already-filed issues. There may be some that need new issues: someone will need to go through them. Any volunteers?

derekbruening added a commit that referenced this issue Dec 12, 2019
Implements dr_app_start() and dr_running_under_dynamorio() for ARM.
This is initial support for the start/stop interface.
There is more to be done before we can enable the start/stop tests.

Issue: #1551
derekbruening added a commit that referenced this issue Dec 13, 2019
Implements dr_app_start() and dr_running_under_dynamorio() for ARM.
This is initial support for the start/stop interface.
There is more to be done before we can enable the start/stop tests.

Issue: #1551
ksco added a commit that referenced this issue Jan 13, 2024
…e_rt_t (#6554)

The `retcode` field in `sigframe_rt_t` is only available in x86 and
AArch32, this fixes a buffer overflow in `memcpy_rt_frame()` on RISC-V.

Issue: #3544 #1551 #1569
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants