[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 29/37] ipmi: free extern timer
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-devel] [PATCH 29/37] ipmi: free extern timer |
Date: |
Thu, 21 Jul 2016 09:10:15 -0400 (EDT) |
Hi
----- Original Message -----
> I believe this is correct:
>
> Reviewed-by: Corey Minyard <address@hidden>
>
> I looked around at other devices with timers, I found several (two
> watchdogs, and ipmi_bmc_sim.c, for instance) that allocate the timer in
> the realize function but don't deallocate it in the unrealize function.
> Do those need to be fixed, too?
Yes, I haven't found them myself, but I can include fixes in the series too.
thanks
>
> -corey
>
>
> On 07/19/2016 03:54 AM, address@hidden wrote:
> > From: Marc-André Lureau <address@hidden>
> >
> > Free the timer allocated during instance init.
> >
> > Signed-off-by: Marc-André Lureau <address@hidden>
> > ---
> > hw/ipmi/ipmi_bmc_extern.c | 9 +++++++++
> > 1 file changed, 9 insertions(+)
> >
> > diff --git a/hw/ipmi/ipmi_bmc_extern.c b/hw/ipmi/ipmi_bmc_extern.c
> > index 157879e..5b73983 100644
> > --- a/hw/ipmi/ipmi_bmc_extern.c
> > +++ b/hw/ipmi/ipmi_bmc_extern.c
> > @@ -487,6 +487,14 @@ static void ipmi_bmc_extern_init(Object *obj)
> > vmstate_register(NULL, 0, &vmstate_ipmi_bmc_extern, ibe);
> > }
> >
> > +static void ipmi_bmc_extern_finalize(Object *obj)
> > +{
> > + IPMIBmcExtern *ibe = IPMI_BMC_EXTERN(obj);
> > +
> > + timer_del(ibe->extern_timer);
> > + timer_free(ibe->extern_timer);
> > +}
> > +
> > static Property ipmi_bmc_extern_properties[] = {
> > DEFINE_PROP_CHR("chardev", IPMIBmcExtern, chr),
> > DEFINE_PROP_END_OF_LIST(),
> > @@ -508,6 +516,7 @@ static const TypeInfo ipmi_bmc_extern_type = {
> > .parent = TYPE_IPMI_BMC,
> > .instance_size = sizeof(IPMIBmcExtern),
> > .instance_init = ipmi_bmc_extern_init,
> > + .instance_finalize = ipmi_bmc_extern_finalize,
> > .class_init = ipmi_bmc_extern_class_init,
> > };
> >
>
>
- Re: [Qemu-devel] [PATCH 25/37] ahci: free irqs array, (continued)
- [Qemu-devel] [PATCH 27/37] qjson: free str, marcandre . lureau, 2016/07/19
- [Qemu-devel] [PATCH 28/37] virtio-input: free config list, marcandre . lureau, 2016/07/19
- [Qemu-devel] [PATCH 29/37] ipmi: free extern timer, marcandre . lureau, 2016/07/19
- [Qemu-devel] [PATCH 30/37] usb: free USBDevice.strings, marcandre . lureau, 2016/07/19
- [Qemu-devel] [PATCH 31/37] tests: free a bunch of qmp responses, marcandre . lureau, 2016/07/19
- [Qemu-devel] [PATCH 32/37] usb: free leaking path, marcandre . lureau, 2016/07/19
- [Qemu-devel] [PATCH 33/37] bus: simplify name handling, marcandre . lureau, 2016/07/19
- [Qemu-devel] [PATCH 34/37] tests: pc-cpu-test, marcandre . lureau, 2016/07/19
- [Qemu-devel] [PATCH 36/37] ahci: fix sglist leak on retry, marcandre . lureau, 2016/07/19