qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [QEMU-1.6 PATCH] vl.c: Output error on invalid machine


From: Eric Blake
Subject: Re: [Qemu-devel] [QEMU-1.6 PATCH] vl.c: Output error on invalid machine type provided
Date: Mon, 12 Aug 2013 10:47:43 -0600
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130805 Thunderbird/17.0.8

On 08/12/2013 10:28 AM, Michal Novotny wrote:
> Output error message using qemu's error_report() function 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).

s/have to have/has to have/

> 
> Signed-off-by: Michal Novotny <address@hidden>
> ---
>  vl.c | 5 +++++
>  1 file changed, 5 insertions(+)

Reviewed-by: Eric Blake <address@hidden>

(It helps to add a v6 or repost nomenclature in the subject line, when
pointing out that this is a repost of a v5 [1]; also, when the patch is
unchanged from an already-reviewed earlier version, it is recommended
that you amend the message to include Reviewed-by lines carried forward,
rather than waiting for reviewers to repeat their comments [2])

[1] https://lists.gnu.org/archive/html/qemu-devel/2013-08/msg01589.html
[2] http://wiki.qemu.org/Contribute/SubmitAPatch

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]