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@xxxxxxxxx> > Cc: Nitin Gupta <ngupta@xxxxxxxxxx> > Cc: Minchan Kim <minchan@xxxxxxxxxx> 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. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>