[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/3] target/arm: Flush only the TLBs affected by
From: |
Richard Henderson |
Subject: |
Re: [Qemu-devel] [PATCH 3/3] target/arm: Flush only the TLBs affected by TTBR*_EL1 |
Date: |
Thu, 18 Oct 2018 13:52:23 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.2.1 |
On 10/18/18 1:27 PM, Aaron Lindsay wrote:
> On Oct 18 11:27, Richard Henderson wrote:
>> @@ -2761,12 +2763,12 @@ static const ARMCPRegInfo vmsa_cp_reginfo[] = {
>> .fieldoffset = offsetof(CPUARMState, cp15.esr_el[1]), .resetvalue =
>> 0, },
>> { .name = "TTBR0_EL1", .state = ARM_CP_STATE_BOTH,
>> .opc0 = 3, .opc1 = 0, .crn = 2, .crm = 0, .opc2 = 0,
>> - .access = PL1_RW, .writefn = vmsa_ttbr_write, .resetvalue = 0,
>> + .access = PL1_RW, .writefn = vmsa_ttbr1_write, .resetvalue = 0,
>
> It's a little confusing that vmsa_ttbr1_write is used for TTBR0_EL1. Is
> the '1' indicating the EL instead of which TTBR is being used?
Yes. Perhaps I should have included "_el" in the symbol for clarity.
I expect to add a different function (vmsr_ttbr_el2_write?), for TTBR{0,1}_EL2,
which will also check HCR_EL2.E2H, when I get around to implementing
ARMv8.1-VHE.
r~