Re: [Qemu-devel] [PATCH v2] ide: Fix memory leak in ide_register_restart_cb()

2016-09-27 Thread Ashijeet Acharya
On Tue, Sep 27, 2016 at 10:08 PM, John Snow wrote: > > > On 09/22/2016 01:47 AM, Ashijeet Acharya wrote: >> >> On Thu, Sep 22, 2016 at 1:08 AM, John Snow wrote: >>> >>> >>> >>> On 09/21/2016 01:53 PM, Ashijeet Acharya wrote: Fix a memory leak in

Re: [Qemu-devel] [PATCH v2] ide: Fix memory leak in ide_register_restart_cb()

2016-09-27 Thread John Snow
On 09/22/2016 01:47 AM, Ashijeet Acharya wrote: On Thu, Sep 22, 2016 at 1:08 AM, John Snow wrote: On 09/21/2016 01:53 PM, Ashijeet Acharya wrote: Fix a memory leak in ide_register_restart_cb() in hw/ide/core.c and add idebus_unrealize() in hw/ide/qdev.c to have calls to

Re: [Qemu-devel] [PATCH v2] ide: Fix memory leak in ide_register_restart_cb()

2016-09-21 Thread Ashijeet Acharya
On Thu, Sep 22, 2016 at 1:08 AM, John Snow wrote: > > > On 09/21/2016 01:53 PM, Ashijeet Acharya wrote: >> >> Fix a memory leak in ide_register_restart_cb() in hw/ide/core.c and add >> idebus_unrealize() in hw/ide/qdev.c to have calls to >> qemu_del_vm_change_state_handler() to

Re: [Qemu-devel] [PATCH v2] ide: Fix memory leak in ide_register_restart_cb()

2016-09-21 Thread John Snow
On 09/21/2016 01:53 PM, Ashijeet Acharya wrote: Fix a memory leak in ide_register_restart_cb() in hw/ide/core.c and add idebus_unrealize() in hw/ide/qdev.c to have calls to qemu_del_vm_change_state_handler() to deal with the dangling change state handler during hot-unplugging ide devices which