|
From: | Corey Bryant |
Subject: | Re: [Qemu-devel] [RFC PATCH 2/4] qmp/hmp: Add getfd_file monitor command |
Date: | Tue, 22 May 2012 10:13:28 -0400 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:12.0) Gecko/20120424 Thunderbird/12.0 |
On 05/22/2012 05:18 AM, Stefan Hajnoczi wrote:
On Mon, May 21, 2012 at 9:19 PM, Corey Bryant<address@hidden> wrote: I think Eric has raised the main questions about duplicating getfd and rules regarding canonical file names (QEMU mashes filenames together if the backing filename is relative!).
Ok, good so it sounds like we this covered in the other threads then.
+ if (qemu_isdigit(filename[0])) { + qerror_report(QERR_INVALID_PARAMETER_VALUE, "filename", + "a name not starting with a digit"); + return -1; + }What is the reason for this filename restriction?
The reason is that I copied this from 'getfd'. :) I'll remove it.
diff --git a/qmp-commands.hx b/qmp-commands.hx index db980fa..1825a91 100644 --- a/qmp-commands.hx +++ b/qmp-commands.hx @@ -891,6 +891,36 @@ Example: EQMP { + .name = "getfd_file", + .args_type = "filename:s", + .params = "getfd_file filename", + .help = "receive a file descriptor via SCM rights and assign it a filename", + .user_print = monitor_user_noop, + .mhandler.cmd_new = do_getfd_file, + }, + + +SQMP + +getfd_file +-------------- + +Receive a file descriptor via SCM rights and assign it a filename. + +Arguments: + +- "filename": filename (json-string) + +Example: + +-> { "execute": "getfd_file", + "arguments": { "filename": "/var/lib/libvirt/images/tst.img" } } +<- { "return": {} } + + +EQMPQMP commands should be added to qapi-schema.json as described in docs/writing-qmp-commands.txt. Stefan
Ok thanks! -- Regards, Corey
[Prev in Thread] | Current Thread | [Next in Thread] |