diff mbox

[v2] target-sparc64: implement Short Floating-Point Store Instructions

Message ID b6d3df1d58146c11a4cda9a002f99d224813b2f6.1407873114.git.atar4qemu@gmail.com
State New
Headers show

Commit Message

Artyom Tarasenko Aug. 12, 2014, 8:02 p.m. UTC 
Implement Short Floating-Point Store Instructions as described
in the chapter 13.5.2 of UltraSPARC-IIi User's Manual.

Particularly this instructions are used by NetBSD 4.0.1+ /sparc64

Signed-off-by: Artyom Tarasenko <atar4qemu@gmail.com>
Tested-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
---
With this patch applied on top of cmd646 patches it's possible to install
and boot NetBSD 6.1.4 /sparc64.

Changes from v0: corrected the comment messages, it's store, not load.

 target-sparc/ldst_helper.c | 15 ++++++++++++++-
 1 file changed, 14 insertions(+), 1 deletion(-)

Comments

Richard Henderson Aug. 13, 2014, 4:38 p.m. UTC | #1 
On 08/12/2014 10:02 AM, Artyom Tarasenko wrote:
> Implement Short Floating-Point Store Instructions as described
> in the chapter 13.5.2 of UltraSPARC-IIi User's Manual.
> 
> Particularly this instructions are used by NetBSD 4.0.1+ /sparc64
> 
> Signed-off-by: Artyom Tarasenko <atar4qemu@gmail.com>
> Tested-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
> ---
> With this patch applied on top of cmd646 patches it's possible to install
> and boot NetBSD 6.1.4 /sparc64.
> 
> Changes from v0: corrected the comment messages, it's store, not load.

Reviewed-by: Richard Henderson <rth@twiddle.net>


r~
Mark Cave-Ayland Aug. 17, 2014, 12:50 p.m. UTC | #2 
On 13/08/14 17:38, Richard Henderson wrote:

> On 08/12/2014 10:02 AM, Artyom Tarasenko wrote:
>> Implement Short Floating-Point Store Instructions as described
>> in the chapter 13.5.2 of UltraSPARC-IIi User's Manual.
>>
>> Particularly this instructions are used by NetBSD 4.0.1+ /sparc64
>>
>> Signed-off-by: Artyom Tarasenko <atar4qemu@gmail.com>
>> Tested-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
>> ---
>> With this patch applied on top of cmd646 patches it's possible to install
>> and boot NetBSD 6.1.4 /sparc64.
>>
>> Changes from v0: corrected the comment messages, it's store, not load.
>
> Reviewed-by: Richard Henderson <rth@twiddle.net>

Thanks! I've applied this to my qemu-sparc with a minor spacing tweak.


ATB,

Mark.
diff mbox

Patch

diff --git a/target-sparc/ldst_helper.c b/target-sparc/ldst_helper.c
index 03bd9f9..736d953 100644
--- a/target-sparc/ldst_helper.c
+++ b/target-sparc/ldst_helper.c
@@ -2154,7 +2154,6 @@  void helper_stf_asi(CPUSPARCState *env, target_ulong addr, int asi, int size,
     unsigned int i;
     target_ulong val;
 
-    helper_check_align(env, addr, 3);
     addr = asi_address_mask(env, asi, addr);
 
     switch (asi) {
@@ -2192,7 +2191,21 @@  void helper_stf_asi(CPUSPARCState *env, target_ulong addr, int asi, int size,
         }
 
         return;
+    case 0xd2: /* 16-bit floating point store primary */
+    case 0xd3: /* 16-bit floating point store secondary */
+    case 0xda: /* 16-bit floating point store primary, LE */
+    case 0xdb: /* 16-bit floating point store secondary, LE */
+        helper_check_align(env, addr, 1);
+        /* Fall through */
+    case 0xd0: /* 8-bit floating point store primary */
+    case 0xd1: /* 8-bit floating point store secondary */
+    case 0xd8: /* 8-bit floating point store primary, LE */
+    case 0xd9: /* 8-bit floating point store secondary, LE */
+        val = env->fpr[rd/2].l.lower;
+        helper_st_asi(env, addr, val, asi & 0x8d, ((asi & 2) >> 1) + 1);
+        return;
     default:
+        helper_check_align(env, addr, 3);
         break;
     }