[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH 2/2] PPC: Fix TLB invalidation bug within the PPC
From: |
Scott Wood |
Subject: |
Re: [Qemu-ppc] [PATCH 2/2] PPC: Fix TLB invalidation bug within the PPC interrupt handler. |
Date: |
Fri, 23 Mar 2012 12:56:05 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:8.0) Gecko/20111115 Thunderbird/8.0 |
On 03/23/2012 08:53 AM, Mark Cave-Ayland wrote:
> On 23/03/12 02:20, David Gibson wrote:
>
>>> I wonder why this is done again at the end of the function for booke
>>> (without regard to MSR bits). It seems like the above flush should
>>> handle booke as well as classic -- though the comment should be
>>> "changed/deactivated" rather than "disactivated", since on booke those
>>> bits just switch from one translation to another.
>>
>> Right, which means I don't think this test will work as is for BookE.
>> There, we'd need to check for any change in the IS/DS bits instead of
>> just testing presence of IR/DR bits.
>
> Are these still MSR bits? I don't see any MSR_IS/MSR_DS defines in
> target-ppc/cpu.h?
IS/DS are the booke version of IR/DR. They occupy the same bits in MSR.
The only difference is that on booke, you can never totally turn off
translation, but instead are choosing between two different address spaces.
-Scott
Re: [Qemu-ppc] [PATCH 0/2] PPC: interrupt handler bugfixes, Andreas Färber, 2012/03/23