[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH for-5.2 1/4] ivshmem-test: do not use short-form boolean opti
From: |
Markus Armbruster |
Subject: |
Re: [PATCH for-5.2 1/4] ivshmem-test: do not use short-form boolean option |
Date: |
Wed, 04 Nov 2020 10:12:32 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) |
Paolo Bonzini <pbonzini@redhat.com> writes:
> This QemuOpts idiom will be deprecated, so get rid of it in the tests.
>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
> tests/qtest/ivshmem-test.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/qtest/ivshmem-test.c b/tests/qtest/ivshmem-test.c
> index d5c8b9f128..dfa69424ed 100644
> --- a/tests/qtest/ivshmem-test.c
> +++ b/tests/qtest/ivshmem-test.c
> @@ -135,7 +135,7 @@ static void setup_vm_cmd(IVState *s, const char *cmd,
> bool msix)
> static void setup_vm(IVState *s)
> {
> char *cmd = g_strdup_printf("-object memory-backend-file"
> - ",id=mb1,size=1M,share,mem-path=/dev/shm%s"
> +
> ",id=mb1,size=1M,share=on,mem-path=/dev/shm%s"
> " -device ivshmem-plain,memdev=mb1", tmpshm);
>
> setup_vm_cmd(s, cmd, false);
Reviewed-by: Markus Armbruster <armbru@redhat.com>
[PATCH for-5.2 2/4] qemu-option: move help handling to get_opt_name_value, Paolo Bonzini, 2020/11/03