[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] target-ppc: clear MSR_POW on interrupt
From: |
Alexander Graf |
Subject: |
Re: [Qemu-devel] [PATCH] target-ppc: clear MSR_POW on interrupt |
Date: |
Fri, 10 Sep 2010 17:26:50 +0200 |
User-agent: |
Thunderbird 2.0.0.23 (X11/20090817) |
Thomas Monjalon wrote:
> Alexander Graf wrote:
>
>> Thomas Monjalon wrote:
>>
>>> From: till <address@hidden>
>>>
>>> According to FreeScale's 'Programming Environments Manual for 32-bit
>>> Implementations of the PowerPC Architecture' [MPCFPE32B, Rev.3, 9/2005],
>>> section 6.5, table 6-7, an interrupt resets MSR_POW to zero but
>>> qemu-0.12.4 fails to do so.
>>> Resetting the bit is necessary in order to bring the processor out of
>>> power management since otherwise it goes to sleep right away in the
>>> exception handler, i.e., it is impossible to leave PM-mode.
>>>
>> This doesn't look right. POW shouldn't even get stored in SRR1. Could
>> you please redo the patch and make sure that mtmsr masks out MSR_POW?
>>
>
> Could you point sections of the specification for these requirements ?
>
> I think SRR1 can save any bits. Non significant bits can be overriden and are
> masked out when RFI occurs.
>
I'm not saying I'm 100% sure on this, but take a look at the e300
reference manual
(http://cache.freescale.com/files/32bit/doc/ref_manual/e300coreRM.pdf)
section 5.2.3. POW is bit 13 in this notion. I don't see it getting
saved to SRR1.
Alex
- [Qemu-devel] [PATCH] target-ppc: clear MSR_POW on interrupt, Thomas Monjalon, 2010/09/10
- Re: [Qemu-devel] [PATCH] target-ppc: clear MSR_POW on interrupt, Alexander Graf, 2010/09/10
- Re: [Qemu-devel] [PATCH] target-ppc: clear MSR_POW on interrupt, Thomas Monjalon, 2010/09/10
- Re: [Qemu-devel] [PATCH] target-ppc: clear MSR_POW on interrupt,
Alexander Graf <=
- Re: [Qemu-devel] [PATCH] target-ppc: clear MSR_POW on interrupt, Thomas Monjalon, 2010/09/10
- Re: [Qemu-devel] [PATCH] target-ppc: clear MSR_POW on interrupt, Alexander Graf, 2010/09/10
- Re: [Qemu-devel] [PATCH] target-ppc: clear MSR_POW on interrupt, Edgar E. Iglesias, 2010/09/10
- Re: [Qemu-devel] [PATCH] target-ppc: clear MSR_POW on interrupt, Alexander Graf, 2010/09/10
- Re: [Qemu-devel] [PATCH] target-ppc: clear MSR_POW on interrupt, Thomas Monjalon, 2010/09/10
- Re: [Qemu-devel] [PATCH] target-ppc: clear MSR_POW on interrupt, Edgar E. Iglesias, 2010/09/10
- Re: [Qemu-devel] [PATCH] target-ppc: clear MSR_POW on interrupt, Alexander Graf, 2010/09/10