On 13.09.21 16:52, Michal Privoznik wrote:
v4 of:
https://listman.redhat.com/archives/libvir-list/2021-June/msg00679.html
diff to v4: - Rebased onto current master - Worked in David's suggestions, e.g. rename from <actual/> to <current/>, implemented offline memory update, implemented --node argument to virsh update-memory-device, prealloc is OFF and reserve is ON for virtio-mem
Some suggestions are left as future work. For instance: - Don't require memory slots because virtio-mem lives on PCI bus anyway - Allow path backed backend for virtio-mem
Just a note that <memoryBacking> <source type='file'/> <access mode='shared'/> </memoryBacking> is doing what it's supposed to do. So only explicit file paths are not supported yet.
- support .prealloc for virtio-mem object (not memory-backend-* !)
I keep occasionally rebased version on my gitlab:
https://gitlab.com/MichalPrivoznik/libvirt/-/commits/virtio_mem_v5/
I just played with it and "virsh update-memory-device" is working like a charm now: a) with "--node" b) with "--alias", including manually specified alias like "<alias name='ua-virtiomem1'/>" c) with --config, --live, --current I see that "aliases" prefixed with "ua-" are an existing concept. Maybe we want to cross-reference that in the virtio-mem documentation? Nothing unusual found during my testing. I did not play with huge pages, as it's initially not supported. Thanks a bunch! -- Thanks, David / dhildenb