From: Laurent Desnogues <laurent.desnogues@gmail.com>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: Roy Tam <roytam@gmail.com>,
seabios@seabios.org, qemu-devel@nongnu.org,
Kevin O'Connor <kevin@koconnor.net>,
Gerhard Wiesinger <lists@wiesinger.com>,
Aurelien Jarno <aurelien@aurel32.net>
Subject: Re: [Qemu-devel] Re: QEMU regression problems - Update FPU
Date: Wed, 23 Feb 2011 10:45:25 +0100 [thread overview]
Message-ID: <AANLkTi=msmzzaSYF0NQD3enbxbppnbdrdPThhUjK6nve@mail.gmail.com> (raw)
In-Reply-To: <AANLkTimsBF5nGEbxHQhx9G_SscWBJPX_52gtiqVN3CKa@mail.gmail.com>
On Wed, Feb 23, 2011 at 9:16 AM, Peter Maydell <peter.maydell@linaro.org> wrote:
> On 18 February 2011 07:12, Gerhard Wiesinger <lists@wiesinger.com> wrote:
>> Issue 1.) with FPU still present
>> I tracked down the problematic code and it is a rounding error from double
>> precision to 64bit floats: Any ideas how to fix such an issue in general?
>>
>> QEMU result in ST0: 0.42925860786976457 (wrong emulated)
>> KVM result in ST0: 0.42925860786975449 (correct)
>
> This is an error when running QEMU in TCG mode, right?
> At the moment x86 is the odd-one-out in that it doesn't
> use CONFIG_SOFTFLOAT for its FPU emulation, so somebody
> has made an explicit choice of preferring speed over
> accuracy, and I am unsurprised that there are rounding
> errors as a result.
Even if you were using SoftFloat, you'd probably still get wrong
results given that this test seems to test trigonometric instructions
which aren't implemented in SoftFloat, but are relying on libm.
Laurent
next prev parent reply other threads:[~2011-02-23 9:45 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-04-12 5:31 [Qemu-devel] QEMU regression problems Gerhard Wiesinger
2010-04-13 6:26 ` Roy Tam
2010-04-14 1:16 ` [SeaBIOS] " Kevin O'Connor
2010-04-19 6:19 ` Gerhard Wiesinger
2010-04-20 1:26 ` Kevin O'Connor
2010-04-21 19:16 ` Gerhard Wiesinger
2010-04-21 23:02 ` Kevin O'Connor
2010-04-22 6:16 ` Gerhard Wiesinger
2011-02-18 7:12 ` [Qemu-devel] Re: QEMU regression problems - Update FPU Gerhard Wiesinger
2011-02-23 7:09 ` Gerhard Wiesinger
2011-02-23 8:16 ` Peter Maydell
2011-02-23 9:45 ` Laurent Desnogues [this message]
2011-02-23 19:04 ` Aurelien Jarno
2011-02-24 11:10 ` Paolo Bonzini
2011-02-24 11:21 ` Laurent Desnogues
2011-02-24 7:03 ` Gerhard Wiesinger
2011-02-24 7:31 ` Aurelien Jarno
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='AANLkTi=msmzzaSYF0NQD3enbxbppnbdrdPThhUjK6nve@mail.gmail.com' \
--to=laurent.desnogues@gmail.com \
--cc=aurelien@aurel32.net \
--cc=kevin@koconnor.net \
--cc=lists@wiesinger.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=roytam@gmail.com \
--cc=seabios@seabios.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).