[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [Qemu-devel] [PATCH 10/15] target/arm: Don't use cpsr_wri
From: |
Richard Henderson |
Subject: |
Re: [Qemu-arm] [Qemu-devel] [PATCH 10/15] target/arm: Don't use cpsr_write/cpsr_read to transfer M profile XPSR |
Date: |
Thu, 3 Aug 2017 15:13:27 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 |
On 08/02/2017 09:43 AM, Peter Maydell wrote:
> + if (val & XPSR_EXCP) {
> + /* This is a CPSR format value from an older QEMU. (We can tell
> + * because values transferred in XPSR format always have zero
> + * for the EXCP field, and CPSR format will always have bit 4
> + * set in CPSR_M.) Rearrange it into XPSR format. The significant
> + * differences are that the T bit is not in the same place, the
> + * primask/faultmask info may be in the CPSR I and F bits, and
> + * we do not want the mode bits.
> + */
> + uint32_t newval = val;
> +
> + newval &= (CPSR_NZCV | CPSR_Q | CPSR_IT | CPSR_GE);
> + if (val & CPSR_T) {
> + newval |= XPSR_T;
> + }
> + /* If the I or F bits are set then this is a migration from
> + * an old QEMU which still stored the M profile FAULTMASK
> + * and PRIMASK in env->daif. For a new QEMU, the data is
> + * transferred using the vmstate_m_faultmask_primask subsection.
> + */
The second comment seems sort of redundant with the first now.
r~
- Re: [Qemu-arm] [Qemu-devel] [PATCH 11/15] target/arm: Make arm_cpu_dump_state() handle the M-profile XPSR, (continued)
- [Qemu-arm] [PATCH 06/15] target/arm: Remove incorrect comment about MPU_CTRL, Peter Maydell, 2017/08/02
- [Qemu-arm] [PATCH 10/15] target/arm: Don't use cpsr_write/cpsr_read to transfer M profile XPSR, Peter Maydell, 2017/08/02
- Re: [Qemu-arm] [Qemu-devel] [PATCH 10/15] target/arm: Don't use cpsr_write/cpsr_read to transfer M profile XPSR,
Richard Henderson <=
- [Qemu-arm] [PATCH 07/15] target/arm: Fix outdated comment about exception exit, Peter Maydell, 2017/08/02
- [Qemu-arm] [PATCH 08/15] target/arm: Define and use XPSR bit masks, Peter Maydell, 2017/08/02
- [Qemu-arm] [PATCH 05/15] hw/intc/armv7m_nvic.c: Remove out of date comment, Peter Maydell, 2017/08/02