On Fri, Sep 25, 2015 at 03:14:59PM +0200, Gerd Hoffmann wrote: > On Do, 2015-09-24 at 13:16 +0300, Michael S. Tsirkin wrote: > > On Thu, Sep 24, 2015 at 11:04:55AM +0200, Gerd Hoffmann wrote: > > > Signed-off-by: Gerd Hoffmann <kra...@redhat.com> > > > > It's easy to see this is what the patch does. But why? Some > > explanation in the commit log about why it's done, as opposed to what is > > done, would be better. > > It's for symmetry reasons: virtio_gpu_create_mapping_iov() allocates it > so virtio_gpu_cleanup_mapping_iov() should free it, otherwise it's easy > to miss a free() needed and leak memory. > > cheers, > Gerd >
Makes sense. Pls include this info in the commit log.