[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation |
Date: |
Tue, 8 Sep 2015 21:25:16 +0100 |
On 8 September 2015 at 20:28, Richard Henderson <address@hidden> wrote:
> On 09/08/2015 12:06 PM, Peter Maydell wrote:
>> On 8 September 2015 at 20:00, Richard Henderson <address@hidden> wrote:
>>> On 09/08/2015 11:56 AM, Peter Maydell wrote:
>>>> My sparc test image (which is just the 32-bit debian from
>>>> Aurelien's website) boots fine even with this patchset...
>>>
>>> Odd, it shouldn't. ;-)
>>>
>>> Anyway, I've just fixed the sparc problem and re-pushed the tree to
>>>
>>> git://github.com/rth7680/qemu.git tcg-search-2
>>>
>>> for anyone who wants to do any more testing.
>>
>> ...so what was the bug? (Push doesn't seem to have made it
>> to github yet.)
>
> Err.. it has. Tip should be 98cb3e2ecffd126177f43634b643be81bdc764e7.
> So I guess you pulled it post fix?
Yep, that's what I've been reviewing...would explain why I
couldn't get it to fall over in testing :-)
> The problem was in 12/20, "target-sparc: Remove gen_opc_jump_pc".
>
> The original was slightly off in how it was computing the npc in a delay slot.
> The replacement keeps the dc->jump_pc array, but verifies that the value of
> dc->jump_pc[1] is as expected: jump false to next insn. It's a smaller change
> to the translator, and easier to verify correctness.
Yeah. My r-b applies to the fixed version, not the mail on the list.
thanks
-- PMM
- Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation, (continued)
- Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation, Peter Maydell, 2015/09/04
- Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation, Richard Henderson, 2015/09/04
- Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation, Max Filippov, 2015/09/04
- Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation, Mark Cave-Ayland, 2015/09/05
- Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation, Richard Henderson, 2015/09/06
- Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation, Artyom Tarasenko, 2015/09/09
Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation, Peter Maydell, 2015/09/08
Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation, Artyom Tarasenko, 2015/09/09
Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation, Paolo Bonzini, 2015/09/09
Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation, Artyom Tarasenko, 2015/09/09
Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation, Dennis Luehring, 2015/09/10
Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation, Artyom Tarasenko, 2015/09/10
Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation, Dennis Luehring, 2015/09/10
Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation, Artyom Tarasenko, 2015/09/10
Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation, Dennis Luehring, 2015/09/10
Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation, Paolo Bonzini, 2015/09/10
Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation, Dennis Luehring, 2015/09/10