From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40174) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1X1qpZ-0005GF-Tu for qemu-devel@nongnu.org; Tue, 01 Jul 2014 01:42:20 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1X1qpT-0001Ur-LS for qemu-devel@nongnu.org; Tue, 01 Jul 2014 01:42:13 -0400 Received: from mail-pd0-f169.google.com ([209.85.192.169]:53492) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1X1qpT-0001Uj-G1 for qemu-devel@nongnu.org; Tue, 01 Jul 2014 01:42:07 -0400 Received: by mail-pd0-f169.google.com with SMTP id g10so9467363pdj.14 for ; Mon, 30 Jun 2014 22:42:06 -0700 (PDT) Message-ID: <53B24A29.1030607@ozlabs.ru> Date: Tue, 01 Jul 2014 15:42:01 +1000 From: Alexey Kardashevskiy MIME-Version: 1.0 References: <1404179460-29303-1-git-send-email-aik@ozlabs.ru> In-Reply-To: <1404179460-29303-1-git-send-email-aik@ozlabs.ru> Content-Type: text/plain; charset=KOI8-R Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH fix for 2.1] makefile: Fix tools compile List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Peter Maydell , Alex Graf , Paolo Bonzini On 07/01/2014 11:51 AM, Alexey Kardashevskiy wrote: > The existing test whether "-lm" needs to be included or not is > insufficient as it reports false negative on Fedora20/ppc64. > As the result, qemu-nbd/qemu-io/qemu-img tools cannot compile. > > This replaces sin() with log() in the test. > > Signed-off-by: Alexey Kardashevskiy > > --- > > The bug was triggered by efc6de0d0eb21bbd8fbc90da1faf7dd8ed9f5321 > "block/iscsi: handle BUSY condition" which does not anything wrong :) > > It compiled well on Fedora19 though what is weird. > > Is log() good enough or we need to test for both? > > Thanks. > > > ps. this is my test on the system upgraded from fc19 yesterday: > > [aik@vpl2 ~]$ cat b.c > #include > int main(void) { return isnan(log(0.0)); } > > [aik@vpl2 ~]$ gcc b.c -o b > /tmp/ccqp1EI4.o: In function `main': > b.c:(.text+0x20): undefined reference to `log' > collect2: error: ld returned 1 exit status > [aik@vpl2 ~]$ gcc b.c -o b -lm > [aik@vpl2 ~]$ cat /etc/issue > Fedora release 20 (Heisenbug) > Kernel \r on an \m (\l) > > > --- > configure | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/configure b/configure > index 23ecb37..c7a2922 100755 > --- a/configure > +++ b/configure > @@ -3453,7 +3453,7 @@ fi > # Do we need libm > cat > $TMPC << EOF > #include > -int main(void) { return isnan(sin(0.0)); } > +int main(void) { return isnan(log(0.0)); } This is wrong actually. The problem here that compiler knows how to optimize constants. sin(0.0) is the one while log(0.0) is not (it is supposed to throw error or something as it the result is infinity). So the correct test here could be: int main(void) { volatile double x = 1; return isnan(sin(x)); } But I am afraid pretty soon compilers will learn how to optimize this as well :) So - what would be the right fix there? Always add "-lm" (on ppc64?)? > EOF > if compile_prog "" "" ; then > : > -- Alexey