Move to an external snapshot file.#12
Draft
LindirQuenya wants to merge 1 commit intoFlashpointProject:masterfrom
Draft
Move to an external snapshot file.#12LindirQuenya wants to merge 1 commit intoFlashpointProject:masterfrom
LindirQuenya wants to merge 1 commit intoFlashpointProject:masterfrom
Conversation
Member
Author
|
Hm, Windows support doesn't seem to be as good as I thought. No matter what I put into the |
Member
Author
|
Here's what I've found so far:
I'll probably have a bit more time to investigate this evening. |
Collaborator
|
@LindirQuenya Awesome work on this, it looks like you're almost there. This would be great to have. P.S. Definitely submit the patch to QEMU when you do. |
Collaborator
Member
Author
|
With those changes, the patch became a bit more complicated. I'll try to submit it to QEMU tomorrow. |
Member
Author
|
From my minimal testing of my debug build, it works. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Improved variant on #11, using a read-only external state file with a read-only disk to achieve complete hash stability. All writes are performed to the in-memory tmpfs, and don't get written back to the state file. The state file can also be compressed, for extra space savings. The QEMU command line should be changed to:
qemu-system-i386 -machine pc-i440fx-5.2 -m 128 -net nic,model=virtio-net-pci -net user,hostfwd=tcp:127.0.0.1:22500-:80 -qmp tcp:127.0.0.1:22501,server,nowait -qmp tcp:127.0.0.1:22502,server,nowait -drive file=alpine.qcow2,if=virtio,readonly=on -serial stdio -incoming "exec:7za x -so snapshot.7z" -display noneThe corresponding
argumentslist inservices.jsonwould be:A built image and snapshot may be found here. Note that Windows will require some extra steps: you'll need to copy
7za.exetoServer, and update QEMU to at least 7.2.0.I'll mark this as ready for review after I've tested it on Windows.