2016-07-06 10:32 GMT+08:00 Minchan Kim <minc...@kernel.org>: > Hi Ganesh, > > On Mon, Jul 04, 2016 at 02:49:52PM +0800, Ganesh Mahendran wrote: >> This patch changes trace_zsmalloc_compact_start[end] to >> trace_zs_compact_start[end] to keep function naming consistent >> with others in zsmalloc >> >> Also this patch remove pages_total_compacted information which >> may not really needed. >> >> Signed-off-by: Ganesh Mahendran <opensource.gan...@gmail.com> > > Once we decide to add event trace, I prefer getting more detailed > information which is hard to get it via /sys/block/zram/. > So, we can add trace __zs_compact as well as zs_compact with > some changes. > > IOW, > > zs_compact > trace_zs_compact_start(pool->name) > __zs_compact > trace_zs_compact(class, scanned_obj, freed_pages) > trace_zs_compact_end(pool->name)
Thanks, I will do it.