On 12/14/2017 03:06 AM, Kevin Wolf wrote: > Am 14.12.2017 um 00:20 hat Philippe Mathieu-Daudé geschrieben: >> Use Base64 to serialize the binary blobs in JSON. >> So far at most 512 bytes will be transfered, which result
s/transfered/transferred/ >> in a 684 bytes payload. >> Since this command is intented for qtesting, this is acceptable. s/intented/intended/ >> >> Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> > > Doing this kind of thing over QMP doesn't look right to me. qtests > should access hardware the same way as real guests access the hardware > (i.e. MMIO and I/O ports). > > But if for some reason the QMP maintainers were to think that this is > acceptable in QMP, I'd argue it should at least get an x-debug- prefix > to avoid making it a stable API that management tools may rely on. I'm not convinced the command is needed, but agree that if we want it, it should have an 'x-debug-' prefix. -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3266 Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
