-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
unicorn 2.1.1: SMC doesn't work. #2029
Milestone
Comments
Ah, it seems we leave out invalid mem hooks? Need to investigate. |
PhilippTakacs
added a commit
to PhilippTakacs/unicorn
that referenced
this issue
Oct 31, 2024
When SMC access the memory region more then once the tb must be rebuild multible times. fixes unicorn-engine#2029
wtdcode
pushed a commit
that referenced
this issue
Oct 31, 2024
* enable notdirty_write for snapshots when possible Snapshots only happens when the priority of the memory region is smaller then the snapshot_level. After a snapshot notdirty can be set. * disable notdirty_write for self modifying code When SMC access the memory region more then once the tb must be rebuild multible times. fixes #2029 * notdirty_write better hook check Check all relevant memory hooks before enabling notdirty write. This also checks if the memory hook is registered for the affected region. So it is possible to use notdirty write and have some hooks on different addresses. * notdirty_write check for addr_write in snapshot case * self modifying code clear recursive mem access when self modifying code does unaligned memory accese sometimes uc->size_recur_mem is changed but for notdirty write not changed back. This causes mem_hooks to be missed. To fix this uc->size_recur_mem is set to 0 before each cpu_exec() call.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
hi,
This code will get a UC_ERR_READ_UNMAPPED error.
It seems that the code memory cache has not been updated.
But when I add memory read/write hooks it works fine.
The text was updated successfully, but these errors were encountered: