qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Mateusz Loskot <mateusz@loskot.net>
To: Blue Swirl <blauwirbel@gmail.com>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [sparc] Floating point exception issue
Date: Sat, 22 Jan 2011 15:30:45 +0000	[thread overview]
Message-ID: <4D3AF825.1020104@loskot.net> (raw)
In-Reply-To: <AANLkTikdY=5TZojTdT-QBUsAD-B5WKATLZjAceJb8oCO@mail.gmail.com>

On 18/01/11 21:51, Blue Swirl wrote:
> On Tue, Jan 18, 2011 at 6:00 PM, Mateusz Loskot <mateusz@loskot.net> wrote:
>> On 18/01/11 17:36, Blue Swirl wrote:
>>>
>>> On Tue, Jan 18, 2011 at 3:27 PM, Mateusz Loskot<mateusz@loskot.net>
>>>  wrote:
>>>>
>>>> Hi,
>>>>
>>>> Recently, I have reported mysterious issues on NetBSD 5.1
>>>> emulated on SPARC. The whole first thread is here:
>>>>
>>>> http://lists.gnu.org/archive/html/qemu-devel/2011-01/msg01509.html
>>>>
>>>> I decided to investigate the problem deeper and with great help
>>>> from NetBSD folks, I managed to find reproducible test case.
>>>> Initially, it was AWK command:
>>>>
>>>> # echo NaN | awk '{print "test"}'
>>>> awk: floating point exception 8
>>>>  source line number 1
>>>>
>>>> and next it boiled down to simple C program (see below).
>>>> Details of the investigation are archived in the NetBSD Problem
>>>> Report #44389 here:
>>>>
>>>> http://gnats.netbsd.org/cgi-bin/query-pr-single.pl?number=44389
>>>>
>>>>
>>>> Here is final version of the test program which reproduces the problem:
>>>>
>>>> #include<stdio.h>
>>>> #include<stdlib.h>
>>>> #include<math.h>
>>>> #include<errno.h>
>>>>
>>>> int is_number(const char *s)
>>>> {
>>>>        double r;
>>>>        char *ep;
>>>>        errno = 0;
>>>>        r = strtod(s,&ep);
>>>>        if (r == HUGE_VAL)
>>>>                printf("X:%g\n", r);
>>>>
>>>>        if (ep == s || r == HUGE_VAL || errno == ERANGE)
>>>>                return 0;
>>>>        while (*ep == ' ' || *ep == '\t' || *ep == '\n')
>>>>                ep++;
>>>>        if (*ep == '\0')
>>>>                return 1;
>>>>        else
>>>>                return 0;
>>>> }
>>>>
>>>> int main(int argc, char **argv)
>>>> {
>>>>        double v;
>>>>
>>>>        if (is_number("NaN")) {
>>>>                printf("is a number\n");
>>>>                v = atof("NaN");
>>>>        } else {
>>>>                printf("not a number\n");
>>>>                v = 0.0;
>>>>        }
>>>>        printf("%.4f\n", v);
>>>>
>>>>        return 0;
>>>> }
>>>>
>>>>
>>>> On NetBSD/SPARC, the program receives SIGFPE:
>>>>
>>>> $ gcc ./nan_test_2.c
>>>>  $ ./a.out
>>>>  [1]   Floating point exception (core dumped) ./a.out
>>>>
>>>> Specifically, it's caused by r == HUGE_VAL condition in
>>>>  if (ep == s || r == HUGE_VAL || errno == ERANGE)
>>>> where r is NaN.
>>>>
>>>> All the signs indicate there is a bug in QEMU.
>>>
>>> I'll install 5.1, but on 4.0 which I had installed, the program works
>>> fine:
>>> $ ./sigfpe
>>> is a number
>>> nan
>>
>> I just tested on NetBSD 5.0/SPARC under QEMU 0.13 (same version I use with
>> NetBSD 5.1/SPARC) and it works well indeed:
>>
>> mloskot@qemu-netbsd-50-sparc:~/tmp# ./a.out
>> is a number
>> nan
>> mloskot@qemu-netbsd-50-sparc:~/tmp#
>>
>> Hmm, this is becoming interesting.
>>
>> I run QEMU 0.13 on Windows Vista (64-bit).
>> Perhaps host system and QEMU binaries are relevant here.
>> I will try on Linux host system later tonight.
>>
>> BTW, here are my images:
>>
>> http://mateusz.loskot.net/tmp/qemu/
> 
> The problem was with NaN handling in fcmped instruction. I've
> committed a patch that fixes the problem, please test. Thanks for
> reporting and the test case.

FYI, this problem seems to be occurring in qemu on Windows only.
I tested git version dated before you applied the fix
and built on Linux and the problem does not happening there.

Best regards,
-- 
Mateusz Loskot, http://mateusz.loskot.net
Charter Member of OSGeo, http://osgeo.org
Member of ACCU, http://accu.org

  parent reply	other threads:[~2011-01-22 15:30 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-18 15:27 [Qemu-devel] [sparc] Floating point exception issue Mateusz Loskot
2011-01-18 17:36 ` Blue Swirl
2011-01-18 18:00   ` Mateusz Loskot
2011-01-18 21:51     ` Blue Swirl
2011-01-19 12:03       ` Mateusz Loskot
2011-01-22 15:30       ` Mateusz Loskot [this message]
2011-01-22 15:38         ` Blue Swirl
2011-01-22 16:45           ` Robert Reif
2011-01-22 23:03             ` Artyom Tarasenko

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4D3AF825.1020104@loskot.net \
    --to=mateusz@loskot.net \
    --cc=blauwirbel@gmail.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).