Romain Malmain
Romain Malmain
yeah that's fine, I can apply it. I think it's more a qemu patch than a libafl-qemu one tho
i'll merge the PR in its current state. There could be other things to add, but i believe it's already big enough like this. let's address the other points in...
indeed, thanks for the proposition. i think it would be nice to expose this in some way. do you mind opening a pr with the changes you did? i can...
good, it would be better to upstream this to QEMU directly. in that case, i think it's better to see how it goes on their side before merging your PR....
hey @vringar, do you have any news regarding the patch? if not, i think we can merge it there while they decide to merge it or not in mainline.
great, thanks for the update. i'll close this issue and the PR once we merge back the changes with the next qemu release.
it's good to go, sorry for the late review
we do not need a thread-safe hash table for ram snapshotting for our use case (at least it's not supported). each instance is intended to be run with one core....
it's true. i remember doing tests with complex targets and getting snapshotting errors if not commenting this out. ideally, we should check for those flags. i can try to re-run...
hey @enisrat, sorry for the late reply, very busy times. thanks for the PR, I'll check now