Re: [Qemu-devel] [PATCH] tcg: Use bitmaps for free temporaries

2013-12-10 Thread Aurelien Jarno
On Mon, Dec 09, 2013 at 11:23:14AM -0800, Richard Henderson wrote: > On 12/08/2013 05:48 AM, Aurelien Jarno wrote: > > It looks fine to me, though I don't know if it has any impact on the > > performances. > > I couldn't measure any performance difference at all. Ok great. Still in the long term

Re: [Qemu-devel] [PATCH] tcg: Use bitmaps for free temporaries

2013-12-09 Thread Richard Henderson
On 12/08/2013 05:48 AM, Aurelien Jarno wrote: > It looks fine to me, though I don't know if it has any impact on the > performances. I couldn't measure any performance difference at all. But for me, the improvement in debugging for the translators is important. r~

Re: [Qemu-devel] [PATCH] tcg: Use bitmaps for free temporaries

2013-12-08 Thread Aurelien Jarno
On Thu, Sep 19, 2013 at 12:58:37PM -0700, Richard Henderson wrote: > We previously allocated 32-bits per temp for the next_free_temp entry. > We now allocate 4 bits per temp across the 4 bitmaps. > > Using a linked list meant that if a translator is tweeked, resulting in > temps being freed in a d

Re: [Qemu-devel] [PATCH] tcg: Use bitmaps for free temporaries

2013-12-07 Thread Peter Maydell
On 19 September 2013 20:58, Richard Henderson wrote: > We previously allocated 32-bits per temp for the next_free_temp entry. > We now allocate 4 bits per temp across the 4 bitmaps. > > Using a linked list meant that if a translator is tweeked, resulting in > temps being freed in a different order

Re: [Qemu-devel] [PATCH] tcg: Use bitmaps for free temporaries

2013-12-06 Thread Richard Henderson
Ping. r~ On 11/19/2013 07:57 PM, Richard Henderson wrote: > Ping? > > r~ > > On 09/20/2013 05:58 AM, Richard Henderson wrote: >> We previously allocated 32-bits per temp for the next_free_temp entry. >> We now allocate 4 bits per temp across the 4 bitmaps. >> >> Using a linked list meant that i

Re: [Qemu-devel] [PATCH] tcg: Use bitmaps for free temporaries

2013-11-18 Thread Richard Henderson
Ping? r~ On 09/20/2013 05:58 AM, Richard Henderson wrote: > We previously allocated 32-bits per temp for the next_free_temp entry. > We now allocate 4 bits per temp across the 4 bitmaps. > > Using a linked list meant that if a translator is tweeked, resulting in > temps being freed in a differen

[Qemu-devel] [PATCH] tcg: Use bitmaps for free temporaries

2013-09-19 Thread Richard Henderson
We previously allocated 32-bits per temp for the next_free_temp entry. We now allocate 4 bits per temp across the 4 bitmaps. Using a linked list meant that if a translator is tweeked, resulting in temps being freed in a different order, that would have follow-on effects throughout the TB. Always