Re: [PATCH v2 1/1] hw/ppc: free env->tb_env in spapr_unrealize_vcpu()

2022-04-03 Thread Cédric Le Goater
On 3/29/22 14:45, Daniel Henrique Barboza wrote: The timebase is allocated during spapr_realize_vcpu() and it's not freed. This results in memory leaks when doing vcpu unplugs: ==636935== ==636935== 144 (96 direct, 48 indirect) bytes in 1 blocks are definitely lost in loss record 6 ,461 of 8,13

Re: [PATCH v2 1/1] hw/ppc: free env->tb_env in spapr_unrealize_vcpu()

2022-03-29 Thread David Gibson
On Tue, Mar 29, 2022 at 09:45:45AM -0300, Daniel Henrique Barboza wrote: > The timebase is allocated during spapr_realize_vcpu() and it's not > freed. This results in memory leaks when doing vcpu unplugs: > > ==636935== > ==636935== 144 (96 direct, 48 indirect) bytes in 1 blocks are definitely los

Re: [PATCH v2 1/1] hw/ppc: free env->tb_env in spapr_unrealize_vcpu()

2022-03-29 Thread Cédric Le Goater
On 3/29/22 14:45, Daniel Henrique Barboza wrote: The timebase is allocated during spapr_realize_vcpu() and it's not freed. This results in memory leaks when doing vcpu unplugs: ==636935== ==636935== 144 (96 direct, 48 indirect) bytes in 1 blocks are definitely lost in loss record 6 ,461 of 8,13

[PATCH v2 1/1] hw/ppc: free env->tb_env in spapr_unrealize_vcpu()

2022-03-29 Thread Daniel Henrique Barboza
The timebase is allocated during spapr_realize_vcpu() and it's not freed. This results in memory leaks when doing vcpu unplugs: ==636935== ==636935== 144 (96 direct, 48 indirect) bytes in 1 blocks are definitely lost in loss record 6 ,461 of 8,135 ==636935==at 0x4897468: calloc (vg_replace_ma