From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:36673) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1V4QTE-0008Od-SF for qemu-devel@nongnu.org; Wed, 31 Jul 2013 03:05:22 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1V4QT9-0002vB-Pd for qemu-devel@nongnu.org; Wed, 31 Jul 2013 03:05:16 -0400 Received: from mx1.redhat.com ([209.132.183.28]:38419) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1V4QT9-0002us-Hc for qemu-devel@nongnu.org; Wed, 31 Jul 2013 03:05:11 -0400 Received: from int-mx02.intmail.prod.int.phx2.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id r6V75AVv011596 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Wed, 31 Jul 2013 03:05:10 -0400 Message-ID: <51F8B725.20803@redhat.com> Date: Wed, 31 Jul 2013 09:05:09 +0200 From: Michal Novotny MIME-Version: 1.0 References: <0e2ed0a9f48b8e6bbfbfa127e2e92c3d01b591fd.1375251985.git.minovotn@redhat.com> <87siyvns66.fsf@blackfin.pond.sub.org> In-Reply-To: <87siyvns66.fsf@blackfin.pond.sub.org> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v4] vl.c: Output error on invalid machine type provided List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Markus Armbruster Cc: qemu-devel@nongnu.org On 07/31/2013 08:45 AM, Markus Armbruster wrote: > Michal Novotny writes: > >> Output error message to stderr when user provides the invalid machine >> type on the command line. This also saves time to find what issue is >> when you downgrade from one version of qemu to another that doesn't >> support required machine type yet (the version user downgraded to have >> to have this patch applied too, of course). >> >> Signed-off-by: Michal Novotny >> --- >> vl.c | 5 +++++ >> 1 file changed, 5 insertions(+) >> >> diff --git a/vl.c b/vl.c >> index f422a1c..6ee1a03 100644 >> --- a/vl.c >> +++ b/vl.c >> @@ -2671,6 +2671,11 @@ static QEMUMachine *machine_parse(const char *name) >> if (machine) { >> return machine; >> } >> + >> + if (name && !is_help_option(name)) { >> + fprintf(stderr, "Error: Unsupported machine type '%s'\n", name); >> + } >> + >> printf("Supported machines are:\n"); >> for (m = first_machine; m != NULL; m = m->next) { >> if (m->alias) { > Sorry, should've looked more closely. Please use error_report() like > this: > > error_report("Unsupported machine type"); > > Improves the message from > > Error: Unsupported machine type 'HAL-9000' > > to the standard form > > qemu-system-x86_64: -M HAL-9000: Unsupported machine type FYI: v5 has been sent using the error_report() Michal > -- Michal Novotny , RHCE, Red Hat Virtualization | libvirt-php bindings | php-virt-control.org