Re: [Qemu-devel] [PATCH v2 1/4] add and link a statistic struct to TBs

2019-06-26 Thread Alex Bennée
Alex Bennée writes: > Vanderson Martins do Rosario writes: > >> When the tb_flush removes a block and it's recreated, this shouldn't >> be creating a new block but using the one that is found by: >> >> lookup_result = g_list_find_custom(tb_ctx.tb_statistics, new_stats, >> statistics_cmp); >> >

Re: [Qemu-devel] [PATCH v2 1/4] add and link a statistic struct to TBs

2019-06-25 Thread Alex Bennée
Vanderson Martins do Rosario writes: > When the tb_flush removes a block and it's recreated, this shouldn't > be creating a new block but using the one that is found by: > > lookup_result = g_list_find_custom(tb_ctx.tb_statistics, new_stats, > statistics_cmp); > > So the tb_statisticics will be

Re: [Qemu-devel] [PATCH v2 1/4] add and link a statistic struct to TBs

2019-06-25 Thread Vanderson Martins do Rosario
When the tb_flush removes a block and it's recreated, this shouldn't be creating a new block but using the one that is found by: lookup_result = g_list_find_custom(tb_ctx.tb_statistics, new_stats, statistics_cmp); So the tb_statisticics will be reused and we could just add this regen counter in t

Re: [Qemu-devel] [PATCH v2 1/4] add and link a statistic struct to TBs

2019-06-25 Thread Alex Bennée
Alex Bennée writes: > vandersonmr writes: > >> We want to store statistics for each TB even after flushes. >> We do not want to modify or grow the TB struct. >> So we create a new struct to contain this statistics and >> link it to each TB while they are created. >> >> Signed-off-by: Vanderson

Re: [Qemu-devel] [PATCH v2 1/4] add and link a statistic struct to TBs

2019-06-24 Thread Alex Bennée
vandersonmr writes: > We want to store statistics for each TB even after flushes. > We do not want to modify or grow the TB struct. > So we create a new struct to contain this statistics and > link it to each TB while they are created. > > Signed-off-by: Vanderson M. do Rosario > --- > accel/

[Qemu-devel] [PATCH v2 1/4] add and link a statistic struct to TBs

2019-06-23 Thread vandersonmr
We want to store statistics for each TB even after flushes. We do not want to modify or grow the TB struct. So we create a new struct to contain this statistics and link it to each TB while they are created. Signed-off-by: Vanderson M. do Rosario --- accel/tcg/translate-all.c | 40 ++