[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: |
Chen Gang |
Subject: |
Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator() |
Date: |
Thu, 27 Mar 2014 17:54:56 +0800 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.4.0 |
On 03/27/2014 03:55 PM, Marcel Apfelbaum wrote:
> On Thu, 2014-03-27 at 09:16 +0800, Chen Gang wrote:
>> 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".
> Hi,
> Thanks for the patch!
>
> I would re-write
> 1. The commit subject to:
> Fix wrong warning on configure_accelerator
> 2. The commit message to:
> No need to continue to iterate over the accelerators list
> after a match is found, even if it is not supported.
>
OK, thank you for your work (re-write it before commit it).
Next I will/should notice about it. :-)
>>
>>
>> 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;
> Seems like the right thing to do.
>
> Thanks,
> Marcel
>
>> }
>> *(accel_list[i].allowed) = true;
>> ret = accel_list[i].init(machine);
>
>
>
--
Chen Gang
Open, share, and attitude like air, water, and life which God blessed
- [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(),
Chen Gang <=
- Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(), Markus Armbruster, 2014/03/27
- 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