[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Overoptimization
From: |
Herbert Poetzl |
Subject: |
Re: [Qemu-devel] Overoptimization |
Date: |
Fri, 28 May 2004 12:13:51 +0200 |
User-agent: |
Mutt/1.4.1i |
On Fri, May 28, 2004 at 07:19:35AM +0600, Alexander E. Patrakov wrote:
> The CFLAGS in Makefile.target are bad for gcc 3.3.2. This is not a my
> problem with my gcc, because it passes the "make check" testsuite except
> some known errors in libstdc++, which cannot affect qemu.
>
> The victim is the tcp_input function in slirp. When compiling with the
> decault CFLAGS, for some reason all packets are dropped as having a
> wrong TCP checksum. I tried debugging this by placing some debug printfs
> in tcp_input.c file, but the problem disappeared after I inserted them.
> I concluded that it is very strange and compiled this tcp_input.c file
> without my debug statements, but with -O1 instead of -O2. The problem
> went away.
gcc isn't supposed to change the semantics of
a program based on the optimization level.
nevertheless several cases are known where this
happened, and not seldom the bug was in gcc
testing the same opts with 3.3.3 or 3.4 will
show if gcc is to blame or not.
best,
Herbert
> Will this workaround (use of -O1 instead of -O2 for tcp_input.c) enter
> the qemu CVS?
>
> --
> Alexander E. Patrakov
>
>
> _______________________________________________
> Qemu-devel mailing list
> address@hidden
> http://mail.nongnu.org/mailman/listinfo/qemu-devel
- [Qemu-devel] Overoptimization, Alexander E. Patrakov, 2004/05/27
- Re: [Qemu-devel] Overoptimization,
Herbert Poetzl <=
- Re: [Qemu-devel] Overoptimization, Alexander E. Patrakov, 2004/05/28
- Re: [Qemu-devel] Overoptimization, Herbert Poetzl, 2004/05/28
- [SOLVED] Re: [Qemu-devel] Overoptimization, Alexander E. Patrakov, 2004/05/28
- Re: [SOLVED] Re: [Qemu-devel] Overoptimization, Lionel Ulmer, 2004/05/29
- [Qemu-devel] VNC patch, the 3rd version, Johannes Schindelin, 2004/05/29
- Re: [Qemu-devel] VNC patch, the 3rd version, Jim C. Brown, 2004/05/30