From: "Andreas Färber" <afaerber@suse.de>
To: Nikunj A Dadhania <nikunj@linux.vnet.ibm.com>
Cc: Alexander Graf <agraf@suse.de>,
aik@au1.ibm.com, Markus Armbruster <armbru@redhat.com>,
qemu-devel@nongnu.org, Mark Wu <wudxw@linux.vnet.ibm.com>,
qemu-ppc@nongnu.org, Paolo Bonzini <pbonzini@redhat.com>
Subject: Re: [Qemu-devel] [PATCH] spapr: skip adding usb keyboard/mouse in case of -nodefaults
Date: Fri, 04 Apr 2014 15:14:54 +0200 [thread overview]
Message-ID: <533EB04E.4080205@suse.de> (raw)
In-Reply-To: <877g758pnj.fsf@abhimanyu.in.ibm.com>
Am 04.04.2014 10:28, schrieb Nikunj A Dadhania:
> diff --git a/vl.c b/vl.c
> index 017f92d..0d6c36c 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -4348,7 +4348,8 @@ int main(int argc, char **argv, char **envp)
> .kernel_filename = kernel_filename,
> .kernel_cmdline = kernel_cmdline,
> .initrd_filename = initrd_filename,
> - .cpu_model = cpu_model };
> + .cpu_model = cpu_model,
> + .has_defaults = has_defaults, };
If we do this, please put
};
on the next line so that the next person appending something doesn't
need to touch it again. :)
I do agree with the others that libvirt shouldn't be using the legacy
-usb option.
Cheers,
Andreas
> machine->init(&args);
>
> audio_init();
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
next prev parent reply other threads:[~2014-04-04 13:15 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-04-03 16:56 [Qemu-devel] [PATCH] spapr: skip adding usb keyboard/mouse in case of -nodefaults Nikunj A Dadhania
2014-04-03 17:01 ` Andreas Färber
2014-04-03 17:07 ` Nikunj A Dadhania
2014-04-03 17:06 ` Nikunj A Dadhania
2014-04-03 17:12 ` Andreas Färber
2014-04-04 6:34 ` Markus Armbruster
2014-04-04 8:28 ` Nikunj A Dadhania
2014-04-04 10:58 ` Markus Armbruster
2014-04-04 11:00 ` Paolo Bonzini
2014-04-04 11:23 ` Markus Armbruster
2014-04-04 15:08 ` Eric Blake
2014-04-04 13:14 ` Andreas Färber [this message]
2014-04-03 18:01 ` Paolo Bonzini
2014-04-03 18:32 ` Eric Blake
2014-04-03 19:24 ` Nikunj A Dadhania
2014-04-03 19:37 ` Paolo Bonzini
2014-04-04 5:28 ` Nikunj A Dadhania
2014-04-04 11:02 ` Paolo Bonzini
2014-04-04 11:40 ` Nikunj A Dadhania
2014-04-04 11:47 ` Paolo Bonzini
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=533EB04E.4080205@suse.de \
--to=afaerber@suse.de \
--cc=agraf@suse.de \
--cc=aik@au1.ibm.com \
--cc=armbru@redhat.com \
--cc=nikunj@linux.vnet.ibm.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@nongnu.org \
--cc=wudxw@linux.vnet.ibm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).