[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] vl: Adjust the place of processing '-mon'
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH] vl: Adjust the place of processing '-mon' |
Date: |
Wed, 04 Mar 2015 12:08:28 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 |
On 04/03/2015 11:59, zhanghailiang wrote:
> From: Xiangyou Xie <address@hidden>
>
> If VM is configured with large size of hugepage, when startup,
> it will consume lots of time to zero the hugepage memory in the function
> 'os_mem_prealloc'.
> Libvirtd will wait 30 seconds for qemu to establish the monitor,
> If the timeout triggers, libvirtd will send TERM signal to kill qemu.
>
> To solve the problem, adjust the processing of '-mon' to the ahead of
> '-object'.
>
> Signed-off-by: Xiangyou Xie <address@hidden>
> Signed-off-by: zhanghailiang <address@hidden>
> ---
> vl.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/vl.c b/vl.c
> index 86bdce0..d0c03fe 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -4000,6 +4000,10 @@ int main(int argc, char **argv, char **envp)
> exit(0);
> }
>
> + if (qemu_opts_foreach(qemu_find_opts("mon"), mon_init_func, NULL, 1) !=
> 0) {
> + exit(1);
> + }
> +
> if (qemu_opts_foreach(qemu_find_opts("object"),
> object_create, NULL, 0) != 0) {
> exit(1);
> @@ -4154,10 +4158,6 @@ int main(int argc, char **argv, char **envp)
>
> parse_numa_opts();
>
> - if (qemu_opts_foreach(qemu_find_opts("mon"), mon_init_func, NULL, 1) !=
> 0) {
> - exit(1);
> - }
> -
> if (foreach_device_config(DEV_SERIAL, serial_parse) < 0)
> exit(1);
> if (foreach_device_config(DEV_PARALLEL, parallel_parse) < 0)
>
Reviewed-by: Paolo Bonzini <address@hidden>