[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH v4 2/9] target-ppc: Implement mtvsrdd instruction
From: |
Nikunj A Dadhania |
Subject: |
Re: [Qemu-ppc] [PATCH v4 2/9] target-ppc: Implement mtvsrdd instruction |
Date: |
Thu, 29 Sep 2016 08:50:34 +0530 |
User-agent: |
Notmuch/0.21 (https://notmuchmail.org) Emacs/25.0.94.1 (x86_64-redhat-linux-gnu) |
David Gibson <address@hidden> writes:
> [ Unknown signature status ]
> On Wed, Sep 28, 2016 at 11:01:20AM +0530, Nikunj A Dadhania wrote:
>> From: Ravi Bangoria <address@hidden>
>>
>> mtvsrdd: Move To VSR Double Doubleword
>>
>> Signed-off-by: Ravi Bangoria <address@hidden>
>> Signed-off-by: Nikunj A Dadhania <address@hidden>
>> ---
>> target-ppc/translate/vsx-impl.inc.c | 23 +++++++++++++++++++++++
>> target-ppc/translate/vsx-ops.inc.c | 1 +
>> 2 files changed, 24 insertions(+)
>>
>> diff --git a/target-ppc/translate/vsx-impl.inc.c
>> b/target-ppc/translate/vsx-impl.inc.c
>> index b669e8c..f9db1d4 100644
>> --- a/target-ppc/translate/vsx-impl.inc.c
>> +++ b/target-ppc/translate/vsx-impl.inc.c
>> @@ -234,6 +234,29 @@ static void gen_mfvsrld(DisasContext *ctx)
>> tcg_gen_mov_i64(cpu_gpr[rA(ctx->opcode)], cpu_vsrl(xS(ctx->opcode)));
>> }
>>
>> +static void gen_mtvsrdd(DisasContext *ctx)
>> +{
>> + if (xT(ctx->opcode) < 32) {
>> + if (unlikely(!ctx->vsx_enabled)) {
>> + gen_exception(ctx, POWERPC_EXCP_VSXU);
>> + return;
>> + }
>> + } else {
>> + if (unlikely(!ctx->altivec_enabled)) {
>> + gen_exception(ctx, POWERPC_EXCP_VPU);
>> + return;
>> + }
>> + }
>
> Huh.. so in the ISA doc version I have at least (p114), the
> pseudo-code for the instruction states either vector or VSX
> exceptions. The text however says either FP or vector exceptions.
>
> The pseudo-code version seems more sensible which is what you've
> implemented, so I'm guessing this is just an error in the descriptive
> text.
Right.
> It'd be nice to confirm that against real hardware behaviour if
> possible though.
Sure, will check it.
Regards,
Nikunj
- [Qemu-ppc] [PATCH v4 0/9] POWER9 TCG enablements - part4, Nikunj A Dadhania, 2016/09/28
- [Qemu-ppc] [PATCH v4 1/9] target-ppc: Implement mfvsrld instruction, Nikunj A Dadhania, 2016/09/28
- [Qemu-ppc] [PATCH v4 2/9] target-ppc: Implement mtvsrdd instruction, Nikunj A Dadhania, 2016/09/28
- [Qemu-ppc] [PATCH v4 3/9] target-ppc: Implement mtvsrws instruction, Nikunj A Dadhania, 2016/09/28
- [Qemu-ppc] [PATCH v4 4/9] target-ppc: improve lxvw4x implementation, Nikunj A Dadhania, 2016/09/28
- Re: [Qemu-ppc] [PATCH v4 4/9] target-ppc: improve lxvw4x implementation, Richard Henderson, 2016/09/28
- Re: [Qemu-ppc] [PATCH v4 4/9] target-ppc: improve lxvw4x implementation, David Gibson, 2016/09/28
- Re: [Qemu-ppc] [PATCH v4 4/9] target-ppc: improve lxvw4x implementation, Nikunj A Dadhania, 2016/09/28
- Re: [Qemu-ppc] [PATCH v4 4/9] target-ppc: improve lxvw4x implementation, Nikunj A Dadhania, 2016/09/28
- Re: [Qemu-ppc] [PATCH v4 4/9] target-ppc: improve lxvw4x implementation, Richard Henderson, 2016/09/28
- Re: [Qemu-ppc] [PATCH v4 4/9] target-ppc: improve lxvw4x implementation, David Gibson, 2016/09/29
- Re: [Qemu-ppc] [PATCH v4 4/9] target-ppc: improve lxvw4x implementation, David Gibson, 2016/09/29