[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH 2/4] qmp/hmp: Add getfd_file monitor command
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [RFC PATCH 2/4] qmp/hmp: Add getfd_file monitor command |
Date: |
Tue, 22 May 2012 10:18:22 +0100 |
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!).
> + 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?
> 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": {} }
> +
> +
> +EQMP
QMP commands should be added to qapi-schema.json as described in
docs/writing-qmp-commands.txt.
Stefan
- Re: [Qemu-devel] [RFC PATCH 1/4] qemu-options: Add -filefd command line option, (continued)
[Qemu-devel] [RFC PATCH 4/4] Example -filefd and getfd_file server, Corey Bryant, 2012/05/21
[Qemu-devel] [RFC PATCH 3/4] block: Enable QEMU to retrieve passed fd before attempting open, Corey Bryant, 2012/05/21
[Qemu-devel] [RFC PATCH 2/4] qmp/hmp: Add getfd_file monitor command, Corey Bryant, 2012/05/21
- Re: [Qemu-devel] [RFC PATCH 2/4] qmp/hmp: Add getfd_file monitor command, Eric Blake, 2012/05/21
- Re: [Qemu-devel] [RFC PATCH 2/4] qmp/hmp: Add getfd_file monitor command,
Stefan Hajnoczi <=
- Re: [Qemu-devel] [RFC PATCH 2/4] qmp/hmp: Add getfd_file monitor command, Corey Bryant, 2012/05/22
- Re: [Qemu-devel] [RFC PATCH 2/4] qmp/hmp: Add getfd_file monitor command, Luiz Capitulino, 2012/05/22
- Re: [Qemu-devel] [RFC PATCH 2/4] qmp/hmp: Add getfd_file monitor command, Corey Bryant, 2012/05/22
- Re: [Qemu-devel] [RFC PATCH 2/4] qmp/hmp: Add getfd_file monitor command, Luiz Capitulino, 2012/05/22
- Re: [Qemu-devel] [RFC PATCH 2/4] qmp/hmp: Add getfd_file monitor command, Corey Bryant, 2012/05/22
- Re: [Qemu-devel] [RFC PATCH 2/4] qmp/hmp: Add getfd_file monitor command, Luiz Capitulino, 2012/05/23
- Re: [Qemu-devel] [RFC PATCH 2/4] qmp/hmp: Add getfd_file monitor command, Corey Bryant, 2012/05/23
Re: [Qemu-devel] [RFC PATCH 0/4] block: file descriptor passing using -filefd and getfd_file, Kevin Wolf, 2012/05/22