-
Notifications
You must be signed in to change notification settings - Fork 13
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
Checkpoint after restore not working #21
Comments
cc @rvansa |
I agree that it could be very useful, but you're right, it does not work ATM. The problem is actually in |
Thanks @rvansa !
I tried adding the arg
From your CRIU fork readme:
Is a fork really inevitable? It makes me really worried for the project.
Cheers |
Hmm, what was the issue? You can enable more verbose log on restore using
Not sure what's the question.
CRIU tries to restore the process in exactly the same state as it was before, and having some pages mmapped from disk rather than being anonymous memory is a significant difference that might cause trouble here and there (such as this inability to do repeated checkpoint). We have other fixes in place, e.g. related to standart FDs - again something that doesn't make the process an exact copy, but is very practical for our usecase. So, having it in upstream is not really likely as it breaks the goal of the project.
If you got the restore working, most likely there's no 'catch' that you'd experience later during runtime. OpenJ9 has something similar to CRaC and they use unpatched CRIU AFAIK. But even besides optimization, we've identified use-cases where upstream CRIU was less practical.
This is fine, if you want to ask more interactively you're welcome to |
Hi, I tried several different ways to checkpoint a restored VM. Nothing worked.
Is it supposed to? If not, that should be clearly documented.
Repeated checkpoints/restores of the same process are extremely useful for in-memory object databases. Regular serialization is orders of magnitude slower than a memory dump.
The text was updated successfully, but these errors were encountered: