Message ID | 20220908095247.2582144-1-liuhaiwei9699@126.com |
---|---|
State | New |
Headers | show |
Series | Use QMP command object-add instead of object_add for memory hotplugin | expand |
why the hmp using the object_add , qmp using the object-add command? can't we use the same command ? At 2022-09-08 19:41:33, "Markus Armbruster" <armbru@redhat.com> wrote: >liuhaiwei <liuhaiwei9699@126.com> writes: > >> From: liuhaiwei <liuhaiwei@inspur.com> >> >> Signed-off-by: liuhaiwei <liuhaiwei@inspur.com> >> --- >> docs/memory-hotplug.txt | 8 ++++---- >> 1 file changed, 4 insertions(+), 4 deletions(-) >> >> diff --git a/docs/memory-hotplug.txt b/docs/memory-hotplug.txt >> index 6aa5e17e26..85ed4d8f3d 100644 >> --- a/docs/memory-hotplug.txt >> +++ b/docs/memory-hotplug.txt >> @@ -34,15 +34,15 @@ hotplugged by using any combination of the available memory slots. >> >> Two monitor commands are used to hotplug memory: >> >> - - "object_add": creates a memory backend object >> + - "object-add": creates a memory backend object >> - "device_add": creates a front-end pc-dimm device and inserts it >> into the first empty slot >> >> For example, the following commands add another 1GB to the guest >> discussed earlier: >> >> - (qemu) object_add memory-backend-ram,id=mem1,size=1G >> - (qemu) device_add pc-dimm,id=dimm1,memdev=mem1 >> + (qemu) object-add qom-type=memory-backend-ram id=mem1 size=1073741824 >> + (qemu) device_add driver=pc-dimm id=dimm1 memdev=mem1 > >This is HMP, where the command is spelled object_add. Your patch is >wrong. > >> >> Using the file backend >> ---------------------- >> @@ -55,7 +55,7 @@ For example, assuming that the host has 1GB hugepages available in >> the /mnt/hugepages-1GB directory, a 1GB hugepage could be hotplugged >> into the guest from the previous section with the following commands: >> >> - (qemu) object_add memory-backend-file,id=mem1,size=1G,mem-path=/mnt/hugepages-1GB > >Likewise. > >> + (qemu) object-add qom-type=memory-backend-file id=mem1 size=1073741824 mem-path=/mnt/hugepages-1GB >> (qemu) device_add pc-dimm,id=dimm1,memdev=mem1 >> >> It's also possible to start a guest with memory cold-plugged into the
* Markus Armbruster (armbru@redhat.com) wrote: > liuhaiwei9699 <liuhaiwei9699@126.com> writes: > > > why the hmp using the object_add , qmp using the object-add command? > > can't we use the same command ? > > Command names differ between HMP and QMP for historical reasons. > > QMP is a stable interface, and changing names there is no go. > > HMP is not a stable interface, but changing names would still > inconvenience users. We don't do that without really compelling > reasons. > > I think HMP could fold '_' and '-' together in command names, so that > both object_add and object-add work. Best to check with the HMP > maintainer before you start coding. Yes, I'd be up for having folding on _/- - I never remember which I need in any case. Dave
diff --git a/docs/memory-hotplug.txt b/docs/memory-hotplug.txt index 6aa5e17e26..85ed4d8f3d 100644 --- a/docs/memory-hotplug.txt +++ b/docs/memory-hotplug.txt @@ -34,15 +34,15 @@ hotplugged by using any combination of the available memory slots. Two monitor commands are used to hotplug memory: - - "object_add": creates a memory backend object + - "object-add": creates a memory backend object - "device_add": creates a front-end pc-dimm device and inserts it into the first empty slot For example, the following commands add another 1GB to the guest discussed earlier: - (qemu) object_add memory-backend-ram,id=mem1,size=1G - (qemu) device_add pc-dimm,id=dimm1,memdev=mem1 + (qemu) object-add qom-type=memory-backend-ram id=mem1 size=1073741824 + (qemu) device_add driver=pc-dimm id=dimm1 memdev=mem1 Using the file backend ---------------------- @@ -55,7 +55,7 @@ For example, assuming that the host has 1GB hugepages available in the /mnt/hugepages-1GB directory, a 1GB hugepage could be hotplugged into the guest from the previous section with the following commands: - (qemu) object_add memory-backend-file,id=mem1,size=1G,mem-path=/mnt/hugepages-1GB + (qemu) object-add qom-type=memory-backend-file id=mem1 size=1073741824 mem-path=/mnt/hugepages-1GB (qemu) device_add pc-dimm,id=dimm1,memdev=mem1 It's also possible to start a guest with memory cold-plugged into the