On Fri, Jul 15, 2011 at 4:44 PM, Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote: > On Fri, Jul 15, 2011 at 04:41:38PM -0700, Curt Wohlgemuth wrote: >> $ cat /proc/writeback/stats >> call: balance_dirty_pages 2253 >> call: background_writeout 45 >> call: try_to_free_pages 0 >> call: sync 14 >> call: kupdate 228 >> call: shrink_page_list 0 >> call: fdatawrite 6755 >> call: laptop_periodic 0 >> call: free_more_memory 0 >> page: balance_dirty_pages 455830 >> page: background_writeout 391568 >> page: try_to_free_pages 0 >> page: sync 133 >> page: kupdate 1158779 >> page: shrink_page_list 0 >> page: fdatawrite 777320 >> page: laptop_periodic 0 >> page: free_more_memory 0 >> periodic writeback 241 >> single inode wait 0 >> writeback_wb wait 7 >> metadata pages cleaned 19473 > > A fair amount of the page stats should be provided by Wu and Jan's > recent patches, shouldn't they? I don't think the calls stats > are such a good idea. We can triviall get these stats by using > perf - just place a probe at the function and then count it using > perf record. Well, sure, you can get all these stats via perf/ftrace/whatever, if you want to place the probes in the right spot, and count them up. I personally think that having total counts (that are monotonically increasing) always available is pretty convenient, including the call counts. But I agree, the page counts and the "metadata pages cleaned" are the most valuable. I looked through Jan's and Fengguang's recent patches for such stats, but didn't see anything -- though I could easily have missed them. > But if would be good to get the remaining page stats in, as they > seems pretty useful. I'll resend another patch without the call counts, see what people think. Thanks, Curt -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
