[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in conf
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator() |
Date: |
Thu, 27 Mar 2014 09:59:56 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.2 (gnu/linux) |
Chen Gang <address@hidden> writes:
> At present, each 'opt_name' of 'accel_list' is uniq with each other, so
> 'buf' can only match one 'opt_name'.
>
> When drop into the matching code block, can 'break' outside related
> 'for' looping after finish processing it (just like the other 'break'
> within the matching block).
>
> After print "... not support for this target", it can avoid to print
> "... accelerator does not exist".
>
>
> Signed-off-by: Chen Gang <address@hidden>
> ---
> vl.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/vl.c b/vl.c
> index 842e897..b4f98fa 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -2709,7 +2709,7 @@ static int configure_accelerator(QEMUMachine *machine)
> if (!accel_list[i].available()) {
> printf("%s not supported for this target\n",
> accel_list[i].name);
> - continue;
> + break;
> }
> *(accel_list[i].allowed) = true;
> ret = accel_list[i].init(machine);
Works, because the opt_name in accel_list[] are distinct, as you
explained in your commit message.
Apropos commit message. You first explain what you do, and only then
state the problem you're trying to solve. That's backwards. Start with
the problem. Only then explain your solution, if it needs explaining.
This one, I think, doesn't.
Suggested commit message:
vl: Report accelerator not supported for target more nicely
When you ask for an accelerator not supported for your target, you
get a bogus "accelerator does not exist" message:
$ qemu-system-arm -machine none,accel=kvm
KVM not supported for this target
"kvm" accelerator does not exist.
No accelerator found!
Suppress it.
- [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(), Chen Gang, 2014/03/26
- Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(), Marcel Apfelbaum, 2014/03/27
- Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(),
Markus Armbruster <=
- Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(), Chen Gang, 2014/03/27
- Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(), Chen Gang, 2014/03/30
- Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(), Markus Armbruster, 2014/03/31
- Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(), Chen Gang, 2014/03/31
- Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(), Peter Maydell, 2014/03/31
- Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(), Chen Gang, 2014/03/31
- Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(), Peter Maydell, 2014/03/31
- Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(), Chen Gang, 2014/03/31
- Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(), Peter Maydell, 2014/03/31
- Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(), Chen Gang, 2014/03/31