Messages in this thread | | | Date | Tue, 27 Jan 2015 00:45:19 +0900 | From | Minchan Kim <> | Subject | Re: [PATCH] zram: free meta table in zram_meta_free |
| |
Hello,
On Sat, Jan 24, 2015 at 09:45:53PM +0800, Ganesh Mahendran wrote: > zram_meta_alloc() and zram_meta_free() are a pair. > In zram_meta_alloc(), meta table is allocated. So it it better to free > it in zram_meta_free(). > > Signed-off-by: Ganesh Mahendran <opensource.ganesh@gmail.com> > Cc: Nitin Gupta <ngupta@vflare.org> > Cc: Minchan Kim <minchan@kernel.org>
Looks good to me but it seems the patch is based on my recent work "zram: free meta out of init_lock". Please resend it on recent mmotm because I will respin my patch and your patch is orthogonal with mine.
Thanks.
| |