2016-11-29 17:00 GMT+08:00 Richard W.M. Jones <[email protected]>: > On Tue, Nov 29, 2016 at 04:56:55PM +0800, Li Qiang wrote: > > Hi > > > > 2016-11-29 16:39 GMT+08:00 Richard W.M. Jones <[email protected]>: > > > > > On Mon, Nov 28, 2016 at 05:49:04PM -0800, Li Qiang wrote: > > > > From: Li Qiang <[email protected]> > > > > > > > > When the Intel 6300ESB watchdog is hot unplug. The timer allocated > > > > in realize isn't freed thus leaking memory leak. This patch avoid > > > > this through adding the exit function. > > > > > > I will just note that the real hardware is not hot-pluggable. However > > > we don't need to stick to the real hardware capabilities, so that's OK. > > > > > > > > > > If the hardware is not hot-pluggable, we can set dc->hotpluggable = > false. > > No no no! The hardware hasn't been made for a decade or two, > we don't need to stick with what the real hardware did. > > OK, then I think this patch is OK.
> Rich. > > -- > Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~ > rjones > Read my programming and virtualization blog: http://rwmj.wordpress.com > libguestfs lets you edit virtual machines. Supports shell scripting, > bindings from many languages. http://libguestfs.org >
