[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 04/29] vl: remove bogus check
From: |
Igor Mammedov |
Subject: |
Re: [PATCH 04/29] vl: remove bogus check |
Date: |
Wed, 28 Oct 2020 17:48:57 +0100 |
On Tue, 27 Oct 2020 14:21:19 -0400
Paolo Bonzini <pbonzini@redhat.com> wrote:
> There is no reason to prevent -preconfig -daemonize. Of course if
> no monitor is defined there will be no way to start the VM,
> but that is a user error.
it was related to how libvirt starts QEMU but I don't recall why anymore,
CCing Daniel
>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
> softmmu/vl.c | 6 ------
> 1 file changed, 6 deletions(-)
>
> diff --git a/softmmu/vl.c b/softmmu/vl.c
> index 7f39ebdfee..dbb72e621e 100644
> --- a/softmmu/vl.c
> +++ b/softmmu/vl.c
> @@ -4027,12 +4027,6 @@ void qemu_init(int argc, char **argv, char **envp)
> }
>
> if (is_daemonized()) {
> - if (!preconfig_exit_requested) {
> - error_report("'preconfig' and 'daemonize' options are "
> - "mutually exclusive");
> - exit(EXIT_FAILURE);
> - }
> -
> /* According to documentation and historically, -nographic redirects
> * serial port, parallel port and monitor to stdio, which does not
> work
> * with -daemonize. We can redirect these to null instead, but since
- [RFC PATCH v2 00/37] cleanup qemu_init and make sense of command line processing, Paolo Bonzini, 2020/10/27
- [PATCH 05/29] vl: split various early command line options to a separate function, Paolo Bonzini, 2020/10/27
- [PATCH 04/29] vl: remove bogus check, Paolo Bonzini, 2020/10/27
- [PATCH 06/29] vl: move various initialization routines out of qemu_init, Paolo Bonzini, 2020/10/27
- [PATCH 02/29] semihosting: fix order of initialization functions, Paolo Bonzini, 2020/10/27
- [PATCH 07/29] vl: extract qemu_init_subsystems, Paolo Bonzini, 2020/10/27
- [PATCH 10/29] vl: preconfig and loadvm are mutually exclusive, Paolo Bonzini, 2020/10/27
- [PATCH 13/29] vl: load plugins as late as possible, Paolo Bonzini, 2020/10/27
- [PATCH 03/29] vl: extract validation of -smp to machine.c, Paolo Bonzini, 2020/10/27
- [PATCH 09/29] vl: extract various command line validation snippets to a new function, Paolo Bonzini, 2020/10/27