qemu-trivial
[Top][All Lists]
Advanced

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

Re: [Qemu-trivial] [Qemu-devel] [PULL 00/15] trivial patches for 2013-09


From: Stefan Weil
Subject: Re: [Qemu-trivial] [Qemu-devel] [PULL 00/15] trivial patches for 2013-09-14
Date: Tue, 17 Sep 2013 18:38:52 +0200
User-agent: Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/20130803 Thunderbird/17.0.8

Am 17.09.2013 17:06, schrieb Anthony Liguori:
> On Sat, Sep 14, 2013 at 2:47 PM, Michael Tokarev <address@hidden> wrote:
>> Just a bit late but here's the next trivial-patches pull request.
>>
>> Thanks,
> Breaks the build:
>
> /home/anthony/git/qemu/cpu-exec.c: In function ‘cpu_alpha_exec’:
> /home/anthony/git/qemu/cpu-exec.c:204:15: error: variable ‘cpu’ might
> be clobbered by ‘longjmp’ or ‘vfork’ [-Werror=clobbered]
> /home/anthony/git/qemu/cpu-exec.c:202:28: error: argument ‘env’ might
> be clobbered by ‘longjmp’ or ‘vfork’ [-Werror=clobbered]
> cc1: all warnings being treated as errors
>
> I think it's one of the warning patches from Stefan.  I cannot bisect
> here.  My gcc info is below:

Yes, it's the new -Wextra compiler (see "configure: Enable extra
compiler warnings").
Some versions of gcc complain about this code in cpu-exec.c:

            /* Reload env after longjmp - the compiler may have smashed all
             * local variables as longjmp is marked 'noreturn'. */
            cpu = current_cpu;
            env = cpu->env_ptr;

I think that lines can be simply removed (which also fixes the warning).
Please apply the trivial patches without my -Wextra patch or wait until
I have sent a patch to fix cpu-exec.c.

Regards
Stefan




reply via email to

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