[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/2] target-mips: add missing MSA and correct FP
From: |
Leon Alrae |
Subject: |
Re: [Qemu-devel] [PATCH 2/2] target-mips: add missing MSA and correct FP in VMState |
Date: |
Thu, 19 Feb 2015 16:43:41 +0000 |
User-agent: |
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:24.0) Gecko/20100101 Thunderbird/24.5.0 |
On 19/02/2015 15:43, Richard Henderson wrote:
> On 02/18/2015 06:51 AM, Leon Alrae wrote:
>> static VMStateField vmstate_fpu_fields[] = {
>> VMSTATE_FPR_ARRAY(fpr, CPUMIPSFPUContext, 32),
>> - VMSTATE_INT8(fp_status.float_detect_tininess, CPUMIPSFPUContext),
>> + VMSTATE_UINT8(fp_status.flush_to_zero, CPUMIPSFPUContext),
>> VMSTATE_INT8(fp_status.float_rounding_mode, CPUMIPSFPUContext),
>> VMSTATE_INT8(fp_status.float_exception_flags, CPUMIPSFPUContext),
>> VMSTATE_UINT32(fcr0, CPUMIPSFPUContext),
>> @@ -70,6 +78,11 @@ static VMStateField vmstate_tc_fields[] = {
>> VMSTATE_UINTTL(CP0_TCScheFBack, TCState),
>> VMSTATE_INT32(CP0_Debug_tcstatus, TCState),
>> VMSTATE_UINTTL(CP0_UserLocal, TCState),
>> + VMSTATE_INT32(msacsr, TCState),
>> + VMSTATE_INT8(msa_fp_status.float_rounding_mode, TCState),
>> + VMSTATE_INT8(msa_fp_status.float_exception_flags, TCState),
>> + VMSTATE_UINT8(msa_fp_status.flush_to_zero, TCState),
>> + VMSTATE_UINT8(msa_fp_status.flush_inputs_to_zero, TCState),
>> VMSTATE_END_OF_LIST()
>> };
>
> Surely these fp_status fields are simply implementation of the architectural
> CSR registers?
>
> IMO you shouldn't store things related to TCG state, but always how the
> architecture represents it. That way you're free to change the TCG
> implementation without breaking save/restore.
Good point. Saving fp_status and msa_fp_status doesn't seem to be needed
at all as they can be restored from FCSR and MSACSR respectively.
Presumably I can use vmstate post_load() for that.
Thanks,
Leon