Date: Wed, 25 Jun 2008 21:04:59 +0000 (UTC) From: Marius Strobl <marius@FreeBSD.org> To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sys/sparc64/include in_cksum.h Message-ID: <200806252105.m5PL5AUp064418@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
marius 2008-06-25 21:04:59 UTC
FreeBSD src repository
Modified files:
sys/sparc64/include in_cksum.h
Log:
SVN rev 180011 on 2008-06-25 21:04:59Z by marius
Use "__asm __volatile" rather than "__asm" for instruction sequences
that modify condition codes (the carry bit, in this case). Without
"__volatile", the compiler might add the inline assembler instructions
between unrelated code which also uses condition codes, modifying the
latter.
This prevents the TCP pseudo header checksum calculation done in
tcp_output() from having effects on other conditions when compiled
with GCC 4.2.1 at "-O2" and "options INET6" left out. [1]
Reported & tested by: Boris Kochergin [1]
MFC after: 3 days
Revision Changes Path
1.4 +7 -7 src/sys/sparc64/include/in_cksum.h
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200806252105.m5PL5AUp064418>