Open vanhauser-thc opened 6 years ago
Unfortunately I do not think you are missing anything. It looks like this has never been implemented for ARM or AArch64: https://github.com/DynamoRIO/dynamorio/blob/ce3a9f40970cd53a8f29c00bbff38d5140673870/core/lib/instrument.c#L6133
Issues #1551
@vanhauser-thc from a quick look at the X86 implementation of dr_insert_mbr_instrumentation, it does not look like there is a huge amount of code to port to get it working on AArch64/ARM.
Let me know if you are interested in having a go at porting it, I am happy to help with any questions
Hi Florian, I would not mind at all to implemend that, it should be not too hard. However I am drowned in programming work at the moment so it will take several weeks until I can/could start with it. If it is still undone when I have the time for it then I will have a go
That would be great, just let us know when you start looking into this.
See also #2297. @AssadHashmi is working on implementing dr_insert_cbr_instrumentation
for AArch64
The code below works find on intel x64 but on on arm7/AARCHXX:
on ARM7/AARCH:
where the mbr instrumentation function is never called.
on INTEL X64 its what I would except:
or did I forget something important which is necessary for ARM?