[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/6] target-i386: Move error handling to end of
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 1/6] target-i386: Move error handling to end of x86_cpu_parse_featurestr() |
Date: |
Wed, 08 Apr 2015 10:31:19 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 |
On 07/04/2015 22:46, Eduardo Habkost wrote:
> Signed-off-by: Eduardo Habkost <address@hidden>
> ---
> target-i386/cpu.c | 8 ++++++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/target-i386/cpu.c b/target-i386/cpu.c
> index 03b33cf..f83d586 100644
> --- a/target-i386/cpu.c
> +++ b/target-i386/cpu.c
> @@ -1961,8 +1961,7 @@ static void x86_cpu_parse_featurestr(CPUState *cs, char
> *features,
> object_property_parse(OBJECT(cpu), "on", featurestr, &local_err);
> }
> if (local_err) {
> - error_propagate(errp, local_err);
> - return;
> + goto out;
> }
> featurestr = strtok(NULL, ",");
> }
> @@ -1978,6 +1977,11 @@ static void x86_cpu_parse_featurestr(CPUState *cs,
> char *features,
> env->features[w] |= plus_features[w];
> env->features[w] &= ~minus_features[w];
> }
> +
> +out:
> + if (local_err) {
> + error_propagate(errp, local_err);
> + }
> }
>
> /* Print all cpuid feature names in featureset
>
Reviewed-by: Paolo Bonzini <address@hidden>