| Message ID | 20220322190745.37727-1-philippe.mathieu.daude@gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [PATCH-for-7.0] target/i386/hvf: Free ressources when vCPU is destroyed | expand |
Typo s/ressources/resources/ in subject. On 22/3/22 20:07, Philippe Mathieu-Daudé wrote: > From: Philippe Mathieu-Daudé <f4bug@amsat.org> > > Both xsave_buf and hvf_caps are allocated in hvf_arch_init_vcpu(), > free them in hvf_arch_vcpu_destroy(). > > Reported-by: Mark Kanda <mark.kanda@oracle.com> > Suggested-by: Igor Mammedov <imammedo@redhat.com> > Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org> > --- > target/i386/hvf/hvf.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/target/i386/hvf/hvf.c b/target/i386/hvf/hvf.c > index fc12c02fb2..39fa4641b9 100644 > --- a/target/i386/hvf/hvf.c > +++ b/target/i386/hvf/hvf.c > @@ -163,7 +163,9 @@ void hvf_arch_vcpu_destroy(CPUState *cpu) > X86CPU *x86_cpu = X86_CPU(cpu); > CPUX86State *env = &x86_cpu->env; > > + g_free(env->xsave_buf); > g_free(env->hvf_mmio_buf); > + g_free(hvf_state->hvf_caps); > } > > static void init_tsc_freq(CPUX86State *env)
On Tue, 22 Mar 2022 20:07:45 +0100 Philippe Mathieu-Daudé <philippe.mathieu.daude@gmail.com> wrote: > From: Philippe Mathieu-Daudé <f4bug@amsat.org> > > Both xsave_buf and hvf_caps are allocated in hvf_arch_init_vcpu(), > free them in hvf_arch_vcpu_destroy(). > > Reported-by: Mark Kanda <mark.kanda@oracle.com> > Suggested-by: Igor Mammedov <imammedo@redhat.com> > Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org> Reviewed-by: Igor Mammedov <imammedo@redhat.com> > --- > target/i386/hvf/hvf.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/target/i386/hvf/hvf.c b/target/i386/hvf/hvf.c > index fc12c02fb2..39fa4641b9 100644 > --- a/target/i386/hvf/hvf.c > +++ b/target/i386/hvf/hvf.c > @@ -163,7 +163,9 @@ void hvf_arch_vcpu_destroy(CPUState *cpu) > X86CPU *x86_cpu = X86_CPU(cpu); > CPUX86State *env = &x86_cpu->env; > > + g_free(env->xsave_buf); > g_free(env->hvf_mmio_buf); > + g_free(hvf_state->hvf_caps); > } > > static void init_tsc_freq(CPUX86State *env)
diff --git a/target/i386/hvf/hvf.c b/target/i386/hvf/hvf.c index fc12c02fb2..39fa4641b9 100644 --- a/target/i386/hvf/hvf.c +++ b/target/i386/hvf/hvf.c @@ -163,7 +163,9 @@ void hvf_arch_vcpu_destroy(CPUState *cpu) X86CPU *x86_cpu = X86_CPU(cpu); CPUX86State *env = &x86_cpu->env; + g_free(env->xsave_buf); g_free(env->hvf_mmio_buf); + g_free(hvf_state->hvf_caps); } static void init_tsc_freq(CPUX86State *env)