From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wu Fengguang Subject: [PATCH 10/10] trace task_io Date: Fri, 26 Aug 2011 19:38:23 +0800 Message-ID: <20110826114619.923651827@intel.com> References: <20110826113813.895522398@intel.com> Cc: Peter Zijlstra , Wu Fengguang To: Return-path: cc: Andrew Morton CC: Jan Kara CC: Christoph Hellwig CC: Dave Chinner CC: Greg Thelen CC: Minchan Kim CC: Vivek Goyal CC: Andrea Righi Cc: linux-mm Cc: LKML Content-Disposition: inline; filename=writeback-trace-task-total-dirtied.patch Sender: owner-linux-mm@kvack.org List-Id: linux-fsdevel.vger.kernel.org The "write_bytes" curve over time is very handy for viewing the smoothness and fairness of page dirties for each task. XXX: It looks not a good fit for writeback traces, shall we create a include/trace/events/vfs.h for this? XXX: the other fields of struct task_io_accounting are not dumped. Signed-off-by: Wu Fengguang --- include/trace/events/writeback.h | 32 +++++++++++++++++++++++++++++ mm/page-writeback.c | 1 2 files changed, 33 insertions(+) --- linux-next.orig/include/trace/events/writeback.h 2011-08-22 11:58:21.000000000 +0800 +++ linux-next/include/trace/events/writeback.h 2011-08-22 12:22:38.000000000 +0800 @@ -180,6 +180,38 @@ TRACE_EVENT(writeback_queue_io, __entry->moved) ); +TRACE_EVENT(task_io, + TP_PROTO(struct task_struct *task), + TP_ARGS(task), + + TP_STRUCT__entry( + __field(unsigned long long, read_bytes) + __field(unsigned long long, write_bytes) + __field(unsigned long long, cancelled_write_bytes) + ), + + TP_fast_assign( + struct task_io_accounting *ioac = &task->ioac; + +#ifdef CONFIG_TASK_IO_ACCOUNTING + __entry->read_bytes = ioac->read_bytes; + __entry->write_bytes = ioac->write_bytes; + __entry->cancelled_write_bytes = ioac->cancelled_write_bytes; +#else + __entry->read_bytes = 0; + __entry->write_bytes = 0; + __entry->cancelled_write_bytes = 0; +#endif + ), + + TP_printk("read=%llu write=%llu cancelled_write=%llu", + __entry->read_bytes, + __entry->write_bytes, + __entry->cancelled_write_bytes + ) +); + + TRACE_EVENT(global_dirty_state, TP_PROTO(unsigned long background_thresh, --- linux-next.orig/mm/page-writeback.c 2011-08-22 12:10:33.000000000 +0800 +++ linux-next/mm/page-writeback.c 2011-08-22 12:20:33.000000000 +0800 @@ -407,6 +407,7 @@ void global_dirty_limits(unsigned long * *pbackground = background; *pdirty = dirty; trace_global_dirty_state(background, dirty); + trace_task_io(current); } /** -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: email@kvack.org