Re: [Qemu-devel] [PATCH v2 1/3] ipmi_bmc_sim: Add a proper unrealize function

2016-07-22 Thread Corey Minyard
On 07/22/2016 04:22 AM, Marc-André Lureau wrote: Hi - Original Message - From: Corey Minyard Add an unrealize function to free the timer allocated in the realize function, unregsiter the vmstate, and free any pending messages. I don't know how to test this

Re: [Qemu-devel] [PATCH v2 1/3] ipmi_bmc_sim: Add a proper unrealize function

2016-07-22 Thread Marc-André Lureau
Hi - Original Message - > From: Corey Minyard > > Add an unrealize function to free the timer allocated in the > realize function, unregsiter the vmstate, and free any > pending messages. I don't know how to test this either, the device seems to be hotpluggable,

[Qemu-devel] [PATCH v2 1/3] ipmi_bmc_sim: Add a proper unrealize function

2016-07-21 Thread minyard
From: Corey Minyard Add an unrealize function to free the timer allocated in the realize function, unregsiter the vmstate, and free any pending messages. Also, get rid of the unnecessary mutex, it was a vestige of something else that was not done. That way we don't have to