All of lore.kernel.org
 help / color / mirror / Atom feed
From: Matthieu Bucchianeri <1611394@bugs.launchpad.net>
To: qemu-devel@nongnu.org
Subject: [Bug 1611394] Re: qemu-ppc: Scalar Single-Precision Floating-Point instructions should not test MSR[SPV]
Date: Sat, 25 Jul 2020 19:41:37 -0000	[thread overview]
Message-ID: <159570609787.30951.5981878549696934726.launchpad@gac.canonical.com> (raw)
In-Reply-To: 20160809142016.20365.3095.malonedeb@wampee.canonical.com

** Changed in: qemu
     Assignee: (unassigned) => Matthieu Bucchianeri (matthieu-bucchianeri)

** Changed in: qemu
       Status: New => In Progress

-- 
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1611394

Title:
  qemu-ppc: Scalar Single-Precision Floating-Point instructions should
  not test  MSR[SPV]

Status in QEMU:
  In Progress

Bug description:
  According to "Signal Processing Engine (SPE) Programming Environments Manual" at
  http://cache.nxp.com/files/32bit/doc/ref_manual/SPEPEM.pdf?fsrch=1&sr=1&pageNum=1

  c.f section 4.2.3  and also Figure 2-2.

  When MSR[SPV] is _NOT_ set, then the embedded scalar single-precision floating-point instructions
  should _NOT_ generate an Embedded Floating-Point Unavailable Interrupt.

  
  Hence, some tests for MSR[SPV] in file target-ppc/translate.c need to be removed.
  Namely, in the definitions of 
  1. GEN_SPEFPUOP_ARITH2_32_32
  2. gen_efsabs
  3. gen_efsnabs
  4. gen_efsneg
  5. GEN_SPEFPUOP_COMP_32

  Note, the macro GEN_SPEFPUOP_CONV_32_32 is already correct.

  One more thing, afaict the macro GEN_SPEFPUOP_CONV_32_64 is used by both
  efs- and efd- instructions, and will need to be split in two versions.
  The efs-use (i.e for efscfd) should be as it is today, but the use by efd-instructions 
  (e.g efdctui) will need to add a test for MSR[SPV].


  (I've looked at today's HEAD-revision of target-ppc/translate.c).

To manage notifications about this bug go to:
https://bugs.launchpad.net/qemu/+bug/1611394/+subscriptions


  parent reply	other threads:[~2020-07-25 19:55 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-09 14:20 [Qemu-devel] [Bug 1611394] [NEW] qemu-ppc: Scalar Single-Precision Floating-Point instructions should not test MSR[SPV] Andreas Rasmusson
2020-07-25  2:56 ` [Bug 1611394] " Matthieu Bucchianeri
2020-07-25 19:41 ` Matthieu Bucchianeri [this message]
2020-11-04 22:53 ` John Snow
2020-12-10  8:48 ` Thomas Huth

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=159570609787.30951.5981878549696934726.launchpad@gac.canonical.com \
    --to=1611394@bugs.launchpad.net \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.