Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-16 Thread Richard Henderson
On 09/16/2015 01:59 AM, Alex Bennée wrote: > > Richard Henderson writes: > >> On 09/10/2015 11:55 AM, Alex Bennée wrote: >>> I've only had a quick glance so far but I'm fairly familiar with the >>> concept from a previous life. I'll aim to do a full review later once >>> I've gotten through my M

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-16 Thread Alex Bennée
Richard Henderson writes: > On 09/10/2015 11:55 AM, Alex Bennée wrote: >> I've only had a quick glance so far but I'm fairly familiar with the >> concept from a previous life. I'll aim to do a full review later once >> I've gotten through my MTTCG review backlog. >> >> Anyway some quick points:

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-15 Thread Dennis Luehring
Am 15.09.2015 um 22:19 schrieb Richard Henderson: Indeed. It would indeed be good to add a bunch of bare-metal tests. Pre-compiled and checked in so that one doesn't have to have a suite of cross-compilers in order to use them. OTOH, I don't see myself (or anyone else) really having the time to

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-15 Thread Richard Henderson
On 09/10/2015 11:55 AM, Alex Bennée wrote: > I've only had a quick glance so far but I'm fairly familiar with the > concept from a previous life. I'll aim to do a full review later once > I've gotten through my MTTCG review backlog. > > Anyway some quick points: > > * You can save data by only m

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-13 Thread Aurelien Jarno
On 2015-09-10 19:48, Aurelien Jarno wrote: > On 2015-09-01 22:51, Richard Henderson wrote: > > I've been looking at this problem off and on for the last week or so, > > prompted by the sparc performance work. Although I havn't been able > > to get a proper sparc64 guest install working, I see the

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-10 Thread Alex Bennée
Richard Henderson writes: > I've been looking at this problem off and on for the last week or so, > prompted by the sparc performance work. Although I havn't been able > to get a proper sparc64 guest install working, I see the exact same > problem with a mips guest. > > In the past we've talke

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-10 Thread Aurelien Jarno
On 2015-09-01 22:51, Richard Henderson wrote: > I've been looking at this problem off and on for the last week or so, > prompted by the sparc performance work. Although I havn't been able > to get a proper sparc64 guest install working, I see the exact same > problem with a mips guest. > > On alp

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-10 Thread Peter Maydell
On 8 September 2015 at 19:56, Peter Maydell wrote: > Looks sensible to me. For patches 1 2 4..16 20 > Reviewed-by: Peter Maydell > > Patches 3, 17, 19 I've sent "minor nit, otherwise r-by" followups to. > > Patch 18 is of course the meat of this series. It doesn't look > obviously wrong but I wan

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-10 Thread Artyom Tarasenko
On Thu, Sep 10, 2015 at 1:02 PM, Dennis Luehring wrote: > Am 10.09.2015 um 12:37 schrieb Dennis Luehring: >> >> Am 10.09.2015 um 11:54 schrieb Artyom Tarasenko: >> > On Thu, Sep 10, 2015 at 11:32 AM, Dennis Luehring >> > wrote: >> >> >Am 10.09.2015 um 09:00 schrieb Artyom Tarasenko: >> >>> >>

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-10 Thread Dennis Luehring
Am 10.09.2015 um 12:37 schrieb Dennis Luehring: Am 10.09.2015 um 11:54 schrieb Artyom Tarasenko: > On Thu, Sep 10, 2015 at 11:32 AM, Dennis Luehring wrote: >> >Am 10.09.2015 um 09:00 schrieb Artyom Tarasenko: >>> > >>> >strangly your branch doesn't changed anything for pure SPARC64 in m

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-10 Thread Paolo Bonzini
On 10/09/2015 12:37, Dennis Luehring wrote: >> >> Can you also show the perf top of the tcg-search-2 branch run? > > perf top from tcg-search-2 branch > > http://pastebin.com/AtASpQvk Still has gen_intermediate_code_pc in it. Paolo

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-10 Thread Dennis Luehring
Am 10.09.2015 um 11:54 schrieb Artyom Tarasenko: On Thu, Sep 10, 2015 at 11:32 AM, Dennis Luehring wrote: >Am 10.09.2015 um 09:00 schrieb Artyom Tarasenko: >>> >>> >strangly your branch doesn't changed anything for pure SPARC64 in my >>> >tests - >>> >i've always completely removed the qemu f

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-10 Thread Artyom Tarasenko
On Thu, Sep 10, 2015 at 11:32 AM, Dennis Luehring wrote: > Am 10.09.2015 um 09:00 schrieb Artyom Tarasenko: >>> >>> >strangly your branch doesn't changed anything for pure SPARC64 in my >>> > tests - >>> >i've always completely removed the qemu folder and cleanly rebuild >>> >(all based on stable

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-10 Thread Dennis Luehring
Am 10.09.2015 um 09:00 schrieb Artyom Tarasenko: >strangly your branch doesn't changed anything for pure SPARC64 in my tests - >i've always completely removed the qemu folder and cleanly rebuild >(all based on stable shell-scripts) Can you please show "perf top" of the qemu-system-sparc64 proces

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-10 Thread Artyom Tarasenko
On Thu, Sep 10, 2015 at 8:07 AM, Dennis Luehring wrote: > Am 08.09.2015 um 21:00 schrieb Richard Henderson: >> >> 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. > > > st

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-09 Thread Dennis Luehring
Am 08.09.2015 um 21:00 schrieb Richard Henderson: 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. strangly your branch doesn't changed anything for pure SPARC64 in my tests - i

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-09 Thread Artyom Tarasenko
On Wed, Sep 9, 2015 at 6:18 PM, Paolo Bonzini wrote: > > > On 09/09/2015 17:05, Artyom Tarasenko wrote: >> Hi Richard, >> >> On Tue, Sep 8, 2015 at 9:00 PM, Richard Henderson wrote: >>> On 09/08/2015 11:56 AM, Peter Maydell wrote: My sparc test image (which is just the 32-bit debian from >>>

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-09 Thread Paolo Bonzini
On 09/09/2015 17:05, Artyom Tarasenko wrote: > Hi Richard, > > On Tue, Sep 8, 2015 at 9:00 PM, Richard Henderson 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... >>

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-09 Thread Artyom Tarasenko
On Sun, Sep 6, 2015 at 10:19 PM, Richard Henderson wrote: > On Sep 5, 2015 07:11, Mark Cave-Ayland wrote: >> While I probably can't help too much on the TCG side, if it helps I can >> rustle up some SPARC images in order to help with testing? > > That would be helpful. Something more than the tr

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-09 Thread Artyom Tarasenko
Hi Richard, On Tue, Sep 8, 2015 at 9:00 PM, Richard Henderson 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

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-08 Thread Peter Maydell
On 8 September 2015 at 20:28, Richard Henderson wrote: > On 09/08/2015 12:06 PM, Peter Maydell wrote: >> On 8 September 2015 at 20:00, Richard Henderson 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)

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-08 Thread Richard Henderson
On 09/08/2015 12:06 PM, Peter Maydell wrote: > On 8 September 2015 at 20:00, Richard Henderson 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.

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-08 Thread Peter Maydell
On 8 September 2015 at 20:00, Richard Henderson 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

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-08 Thread Richard Henderson
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

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-08 Thread Peter Maydell
On 2 September 2015 at 06:51, Richard Henderson wrote: > I've been looking at this problem off and on for the last week or so, > prompted by the sparc performance work. Although I havn't been able > to get a proper sparc64 guest install working, I see the exact same > problem with a mips guest. >

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-06 Thread Richard Henderson
On Sep 5, 2015 07:11, Mark Cave-Ayland wrote: > While I probably can't help too much on the TCG side, if it helps I can > rustle up some SPARC images in order to help with testing? That would be helpful. Something more than the trivial sparc-test image on the wiki. Something with a sparc64 us

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-05 Thread Mark Cave-Ayland
On 04/09/15 17:46, Richard Henderson wrote: > On 09/04/2015 08:18 AM, Max Filippov wrote: >> On Wed, Sep 2, 2015 at 5:21 PM, Richard Henderson wrote: >>> git://github.com/rth7680/qemu.git tcg-search-2 >> >> That makes an impressive speedup for native kernel build on xtensa softmmu: >> down from 1

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-04 Thread Max Filippov
On Fri, Sep 4, 2015 at 7:46 PM, Richard Henderson wrote: > On 09/04/2015 08:18 AM, Max Filippov wrote: >> On Wed, Sep 2, 2015 at 5:21 PM, Richard Henderson wrote: >>> git://github.com/rth7680/qemu.git tcg-search-2 >> >> That makes an impressive speedup for native kernel build on xtensa softmmu: >

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-04 Thread Richard Henderson
On 09/04/2015 08:18 AM, Max Filippov wrote: > On Wed, Sep 2, 2015 at 5:21 PM, Richard Henderson wrote: >> git://github.com/rth7680/qemu.git tcg-search-2 > > That makes an impressive speedup for native kernel build on xtensa softmmu: > down from 1240 minutes to 690. > Yowza. Is xtensa a hardwar

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-04 Thread Peter Maydell
On 4 September 2015 at 16:18, Max Filippov wrote: > On Wed, Sep 2, 2015 at 5:21 PM, Richard Henderson wrote: >> git://github.com/rth7680/qemu.git tcg-search-2 > > That makes an impressive speedup for native kernel build on xtensa softmmu: > down from 1240 minutes to 690. Blimey. -- PMM

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-04 Thread Max Filippov
On Wed, Sep 2, 2015 at 5:21 PM, Richard Henderson wrote: > git://github.com/rth7680/qemu.git tcg-search-2 That makes an impressive speedup for native kernel build on xtensa softmmu: down from 1240 minutes to 690. -- Thanks. -- Max

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-02 Thread Richard Henderson
On 09/02/2015 05:21 AM, Max Filippov wrote: Richard, patch 01/20 haven't got to the list. Do you have that series somewhere in a public git? git://github.com/rth7680/qemu.git tcg-search-2 r~

Re: [Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-02 Thread Max Filippov
Richard, patch 01/20 haven't got to the list. Do you have that series somewhere in a public git? -- Thanks. -- Max

[Qemu-devel] [RFC 00/20] Do away with TB retranslation

2015-09-01 Thread Richard Henderson
I've been looking at this problem off and on for the last week or so, prompted by the sparc performance work. Although I havn't been able to get a proper sparc64 guest install working, I see the exact same problem with a mips guest. On alpha or x86, which seem to perform well, perf numbers for th