)]}' { "log": [ { "commit": "55dff4954ebdeba2be59e19398a607d799c5fa9f", "tree": "b442f972d503de39bf0d4a3eed007294a45b4dc9", "parents": [ "0b4b2ad5307c76c7105d6e7c724b1c14b8daf482" ], "author": { "name": "Randy Dunlap", "email": "randy.dunlap@oracle.com", "time": "Wed Sep 23 15:56:17 2009 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Sep 24 07:20:57 2009 -0700" }, "message": "docs: fix various Documentation/ paths in header files\n\nFix various Documentation/ paths in include/linux/.\n\nSigned-off-by: Randy Dunlap \u003crandy.dunlap@oracle.com\u003e\nReviewed-by: Jesper Juhl \u003cjj@chaosbits.net\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "35dce1a99d010f3d738af4ce1b9b77302fdfe69c", "tree": "e34a37de965a79a2ae301de4d0557f500111dde6", "parents": [ "7cb2e3ee2aeec5b83ecadba929a2dc575dd4008f", "1c569f0264ea629c10bbab471dd0626ce4d3f19f" ], "author": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Wed Aug 26 08:29:02 2009 +0200" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Wed Aug 26 08:29:02 2009 +0200" }, "message": "Merge branch \u0027tracing/core\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/frederic/random-tracing into tracing/core\n\nConflicts:\n\tinclude/linux/tracepoint.h\n\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "7cb2e3ee2aeec5b83ecadba929a2dc575dd4008f", "tree": "8918c12cac3cd816bf3dfaafc3fac46e36f4a73d", "parents": [ "5ac35daa9343936038a3c9c4f4d6d3fe6a2a7bd8" ], "author": { "name": "Steven Rostedt", "email": "srostedt@redhat.com", "time": "Wed Aug 26 00:32:37 2009 -0400" }, "committer": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Wed Aug 26 00:32:37 2009 -0400" }, "message": "tracing: add comments to explain TRACE_EVENT out of protection\n\nThe commit:\n commit 5ac35daa9343936038a3c9c4f4d6d3fe6a2a7bd8\n Author: Xiao Guangrong \u003cxiaoguangrong@cn.fujitsu.com\u003e\n tracing/events: fix the include file dependencies\n\nMoved the TRACE_EVENT out of the ifdef protection of tracepoints.h\nbut uses the define of TRACE_EVENT itself as protection. This patch\nadds comments to explain why.\n\nSigned-off-by: Steven Rostedt \u003crostedt@goodmis.org\u003e\n" }, { "commit": "5ac35daa9343936038a3c9c4f4d6d3fe6a2a7bd8", "tree": "0810db04bd8ad675aee30681576903e393e9d4c6", "parents": [ "5079f3261ffd7fe4a537679af695f2328943a245" ], "author": { "name": "Xiao Guangrong", "email": "xiaoguangrong@cn.fujitsu.com", "time": "Tue Aug 25 14:06:22 2009 +0800" }, "committer": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Wed Aug 26 00:32:09 2009 -0400" }, "message": "tracing/events: fix the include file dependencies\n\nThe TRACE_EVENT depends on the include/linux/tracepoint.h first\nand include/trace/ftrace.h later, if we include the ftrace.h early,\na building error will occur.\n\nBoth define TRACE_EVENT in trace_a.h and trace_b.h, if we include\nthose in .c file, like this:\n\n#define CREATE_TRACE_POINTS\ninclude \u003ctrace/events/trace_a.h\u003e\ninclude \u003ctrace/events/trace_b.h\u003e\n\nThe above will not work, because the TRACE_EVENT was re-defined by\nthe previous .h file.\n\nReported-by: Wei Yongjun \u003cyjwei@cn.fujitsu.com\u003e\nSigned-off-by: Xiao Guangrong \u003cxiaoguangrong@cn.fujitsu.com\u003e\nLKML-Reference: \u003c4A937F5E.3020802@cn.fujitsu.com\u003e\nSigned-off-by: Steven Rostedt \u003crostedt@goodmis.org\u003e\n" }, { "commit": "97419875865859fd2403e66266c02ce028e2f5ab", "tree": "7df6e6df767e9c8ff538a50bcae17638a1c8da99", "parents": [ "3d27d8cb34fc156beb86de2338ca4029873a5cc6" ], "author": { "name": "Josh Stone", "email": "jistone@redhat.com", "time": "Mon Aug 24 14:43:13 2009 -0700" }, "committer": { "name": "Frederic Weisbecker", "email": "fweisbec@gmail.com", "time": "Wed Aug 26 00:36:41 2009 +0200" }, "message": "tracing: Move tracepoint callbacks from declaration to definition\n\nIt\u0027s not strictly correct for the tracepoint reg/unreg callbacks to\noccur when a client is hooking up, because the actual tracepoint may not\nbe present yet. This happens to be fine for syscall, since that\u0027s in\nthe core kernel, but it would cause problems for tracepoints defined in\na module that hasn\u0027t been loaded yet. It also means the reg/unreg has\nto be EXPORTed for any modules to use the tracepoint (as in SystemTap).\n\nThis patch removes DECLARE_TRACE_WITH_CALLBACK, and instead introduces\nDEFINE_TRACE_FN which stores the callbacks in struct tracepoint. The\ncallbacks are used now when the active state of the tracepoint changes\nin set_tracepoint \u0026 disable_tracepoint.\n\nThis also introduces TRACE_EVENT_FN, so ftrace events can also provide\nregistration callbacks if needed.\n\nSigned-off-by: Josh Stone \u003cjistone@redhat.com\u003e\nCc: Jason Baron \u003cjbaron@redhat.com\u003e\nCc: Frederic Weisbecker \u003cfweisbec@gmail.com\u003e\nCc: Ingo Molnar \u003cmingo@elte.hu\u003e\nCc: Li Zefan \u003clizf@cn.fujitsu.com\u003e\nCc: Steven Rostedt \u003crostedt@goodmis.org\u003e\nCc: Peter Zijlstra \u003cpeterz@infradead.org\u003e\nCc: Mathieu Desnoyers \u003cmathieu.desnoyers@polymtl.ca\u003e\nCc: Jiaying Zhang \u003cjiayingz@google.com\u003e\nCc: Martin Bligh \u003cmbligh@google.com\u003e\nCc: Lai Jiangshan \u003claijs@cn.fujitsu.com\u003e\nCc: Paul Mundt \u003clethal@linux-sh.org\u003e\nCc: Martin Schwidefsky \u003cschwidefsky@de.ibm.com\u003e\nCc: Heiko Carstens \u003cheiko.carstens@de.ibm.com\u003e\nLKML-Reference: \u003c1251150194-1713-4-git-send-email-jistone@redhat.com\u003e\nSigned-off-by: Frederic Weisbecker \u003cfweisbec@gmail.com\u003e\n" }, { "commit": "63fbdab3157b72467013fe4dcf88c85e45280ef7", "tree": "c0e8647e18528dca9682e98a441729adfa41696c", "parents": [ "066e0378c23f0a3db730893f6a041e4a3922a385" ], "author": { "name": "Jason Baron", "email": "jbaron@redhat.com", "time": "Mon Aug 10 16:52:27 2009 -0400" }, "committer": { "name": "Frederic Weisbecker", "email": "fweisbec@gmail.com", "time": "Tue Aug 11 20:35:26 2009 +0200" }, "message": "tracing: Add DECLARE_TRACE_WITH_CALLBACK() macro\n\nIntroduce a new \u0027DECLARE_TRACE_WITH_CALLBACK()\u0027 macro, so that\ntracepoints can associate an external register/unregister function.\n\nThis prepares for the syscalls tracer conversion to trace events. We\nwill need to perform arch level operations once a syscall event is\nturned on/off, such as TIF flags setting, hence the need of such\nspecific callbacks.\n\nSigned-off-by: Jason Baron \u003cjbaron@redhat.com\u003e\nCc: Lai Jiangshan \u003claijs@cn.fujitsu.com\u003e\nCc: Steven Rostedt \u003crostedt@goodmis.org\u003e\nCc: Peter Zijlstra \u003cpeterz@infradead.org\u003e\nCc: Mathieu Desnoyers \u003cmathieu.desnoyers@polymtl.ca\u003e\nCc: Jiaying Zhang \u003cjiayingz@google.com\u003e\nCc: Martin Bligh \u003cmbligh@google.com\u003e\nCc: Li Zefan \u003clizf@cn.fujitsu.com\u003e\nCc: Masami Hiramatsu \u003cmhiramat@redhat.com\u003e\nSigned-off-by: Frederic Weisbecker \u003cfweisbec@gmail.com\u003e\n" }, { "commit": "156f5a7801195fa2ce44aeeb62d6cf8468f3332a", "tree": "dd506816ca6f14bb650189aa364eb0a2f51ad5cc", "parents": [ "1b713e00500c6f03317742981674e89a21629399" ], "author": { "name": "GeunSik Lim", "email": "leemgs1@gmail.com", "time": "Tue Jun 02 15:01:37 2009 +0900" }, "committer": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Mon Jun 15 21:30:28 2009 -0700" }, "message": "debugfs: Fix terminology inconsistency of dir name to mount debugfs filesystem.\n\nMany developers use \"/debug/\" or \"/debugfs/\" or \"/sys/kernel/debug/\"\ndirectory name to mount debugfs filesystem for ftrace according to\n./Documentation/tracers/ftrace.txt file.\n\nAnd, three directory names(ex:/debug/, /debugfs/, /sys/kernel/debug/) is\nexisted in kernel source like ftrace, DRM, Wireless, Documentation,\nNetwork[sky2]files to mount debugfs filesystem.\n\ndebugfs means debug filesystem for debugging easy to use by greg kroah\nhartman. \"/sys/kernel/debug/\" name is suitable as directory name\nof debugfs filesystem.\n- debugfs related reference: http://lwn.net/Articles/334546/\n\nFix inconsistency of directory name to mount debugfs filesystem.\n\n* From Steven Rostedt\n - find_debugfs() and tracing_files() in this patch.\n\nSigned-off-by: GeunSik Lim \u003cgeunsik.lim@samsung.com\u003e\nAcked-by : Inaky Perez-Gonzalez \u003cinaky@linux.intel.com\u003e\nReviewed-by : Steven Rostedt \u003crostedt@goodmis.org\u003e\nReviewed-by : James Smart \u003cjames.smart@emulex.com\u003e\nCC: Jiri Kosina \u003ctrivial@kernel.org\u003e\nCC: David Airlie \u003cairlied@linux.ie\u003e\nCC: Peter Osterlund \u003cpetero2@telia.com\u003e\nCC: Ananth N Mavinakayanahalli \u003cananth@in.ibm.com\u003e\nCC: Anil S Keshavamurthy \u003canil.s.keshavamurthy@intel.com\u003e\nCC: Masami Hiramatsu \u003cmhiramat@redhat.com\u003e\nSigned-off-by: Greg Kroah-Hartman \u003cgregkh@suse.de\u003e\n\n" }, { "commit": "b8e65554d80b4c560d201362d0e8fa02109d89fd", "tree": "21ebf291cf1ac97533cac99619652cd21bbd04f2", "parents": [ "160031b556e93590fa8635210d73d93c3d3853a9" ], "author": { "name": "Steven Rostedt", "email": "srostedt@redhat.com", "time": "Fri Apr 24 11:50:39 2009 -0400" }, "committer": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Fri Apr 24 11:50:39 2009 -0400" }, "message": "tracing: remove deprecated TRACE_FORMAT\n\nThe TRACE_FORMAT macro has been deprecated by the TRACE_EVENT macro.\nThere are no more users. All new users must use the TRACE_EVENT macro.\n\n[ Impact: remove old functionality ]\n\nCc: Peter Zijlstra \u003cpeterz@infradead.org\u003e\nSigned-off-by: Steven Rostedt \u003crostedt@goodmis.org\u003e\n" }, { "commit": "ea20d9293ce423a39717ed4375393129a2e701f9", "tree": "30cbfd532e6541c17eb69a63044cfe7bce6cf974", "parents": [ "0a19e53c1514ad8e9c3cbab40c6c3f52c86f403d" ], "author": { "name": "Steven Rostedt", "email": "srostedt@redhat.com", "time": "Fri Apr 10 08:54:16 2009 -0400" }, "committer": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Tue Apr 14 09:43:40 2009 -0400" }, "message": "tracing: consolidate trace and trace_event headers\n\nImpact: clean up\n\nNeil Horman (et. al.) criticized the way the trace events were broken up\ninto two files. The reason for that was that ftrace needed to separate out\nthe declarations from where the #include \u003clinux/tracepoint.h\u003e was used.\nIt then dawned on me that the tracepoint.h header only needs to define the\nTRACE_EVENT macro if it is not already defined.\n\nThe solution is simply to test if TRACE_EVENT is defined, and if it is not\nthen the linux/tracepoint.h header can define it. This change consolidates\nall the \u003ctraces\u003e.h and \u003ctraces\u003e_event_types.h into the \u003ctraces\u003e.h file.\n\nReported-by: Neil Horman \u003cnhorman@tuxdriver.com\u003e\nReported-by: Theodore Tso \u003ctytso@mit.edu\u003e\nReported-by: Jiaying Zhang \u003cjiayingz@google.com\u003e\nCc: Zhaolei \u003czhaolei@cn.fujitsu.com\u003e\nCc: Frederic Weisbecker \u003cfweisbec@gmail.com\u003e\nCc: Peter Zijlstra \u003cpeterz@infradead.org\u003e\nCc: Jason Baron \u003cjbaron@redhat.com\u003e\nCc: Mathieu Desnoyers \u003cmathieu.desnoyers@polymtl.ca\u003e\nSigned-off-by: Steven Rostedt \u003crostedt@goodmis.org\u003e\n" }, { "commit": "ef18012b248b47ec9a12c3a83ca5e99782d39c5d", "tree": "8103b4d14c6b81a6a5ec39cd43e4ca7ffb51625d", "parents": [ "0e3d0f0566f3fcf664782f597070bbc669d78454" ], "author": { "name": "Steven Rostedt", "email": "srostedt@redhat.com", "time": "Tue Mar 10 14:10:56 2009 -0400" }, "committer": { "name": "Steven Rostedt", "email": "srostedt@redhat.com", "time": "Tue Mar 10 14:13:14 2009 -0400" }, "message": "tracing: remove funky whitespace in the trace code\n\nImpact: clean up\n\nThere existed a lot of \u003cspace\u003e\u003ctab\u003e\u0027s in the tracing code. This\npatch removes them.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\n" }, { "commit": "823f9124fb2e33eeb624d139978a52089f8a02ae", "tree": "e1094e94a958e2098c3946de4ac648a7d87a1bcb", "parents": [ "30a8fecc2d34f086df34fe2f2b926f080e002600" ], "author": { "name": "Steven Rostedt", "email": "srostedt@redhat.com", "time": "Tue Mar 10 12:58:51 2009 -0400" }, "committer": { "name": "Steven Rostedt", "email": "srostedt@redhat.com", "time": "Tue Mar 10 12:58:51 2009 -0400" }, "message": "tracing: document TRACE_EVENT macro in tracepoint.h\n\nImpact: clean up / comments\n\nKosaki Motohiro asked about an explanation to the TRACE_EVENT macro.\nIngo Molnar replied with a nice description.\n\nThis patch takes the description that Ingo wrote (with some slight\nmodifications) and adds it to the tracepoint.h file.\n\nReported-by: KOSAKI Motohiro \u003ckosaki.motohiro@jp.fujitsu.com\u003e\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\n" }, { "commit": "30a8fecc2d34f086df34fe2f2b926f080e002600", "tree": "a6959a40a6a3065b61ddcf4954861dfd8892a843", "parents": [ "2314c4ae1461c9e8b26cf8b9a851f280bc5769e1" ], "author": { "name": "Steven Rostedt", "email": "srostedt@redhat.com", "time": "Tue Mar 10 12:41:38 2009 -0400" }, "committer": { "name": "Steven Rostedt", "email": "srostedt@redhat.com", "time": "Tue Mar 10 12:41:38 2009 -0400" }, "message": "tracing: flip the TP_printk and TP_fast_assign in the TRACE_EVENT macro\n\nImpact: clean up\n\nIn trying to stay consistant with the C style format in the TRACE_EVENT\nmacro, it makes more sense to do the printk after the assigning of\nthe variables.\n\nReported-by: Ingo Molnar \u003cmingo@elte.hu\u003e\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\n" }, { "commit": "157587d7ac555458da9f682e3250135e468470a6", "tree": "81033336672eb821f3833a1667bedfc646a3a6f5", "parents": [ "d6e2ca4c05be6a5ab16030a9f227301bd6acc9f0" ], "author": { "name": "Steven Rostedt", "email": "srostedt@redhat.com", "time": "Tue Mar 10 00:15:34 2009 -0400" }, "committer": { "name": "Steven Rostedt", "email": "srostedt@redhat.com", "time": "Tue Mar 10 00:35:12 2009 -0400" }, "message": "tracing: remove obsolete TRACE_EVENT_FORMAT macro\n\nImpact: clean up\n\nThe TRACE_EVENT_FORMAT macro is no longer used by trace points\nand only the DECLARE_TRACE, TRACE_FORMAT or TRACE_EVENT macros should\nbe used by them. Although the TRACE_EVENT_FORMAT macro is still used\nby the internal tracing utility, it should not be used in core\nkernel code.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\n" }, { "commit": "da4d03020c2af32f73e8bfbab0a66620d85bb9bb", "tree": "3ee6d7d69754df7910454315a6011c14d8664d01", "parents": [ "9cc26a261d43e5898287a1f5808132f8f05ceb1c" ], "author": { "name": "Steven Rostedt", "email": "srostedt@redhat.com", "time": "Mon Mar 09 17:14:30 2009 -0400" }, "committer": { "name": "Steven Rostedt", "email": "srostedt@redhat.com", "time": "Tue Mar 10 00:35:07 2009 -0400" }, "message": "tracing: new format for specialized trace points\n\nImpact: clean up and enhancement\n\nThe TRACE_EVENT_FORMAT macro looks quite ugly and is limited in its\nability to save data as well as to print the record out. Working with\nIngo Molnar, we came up with a new format that is much more pleasing to\nthe eye of C developers. This new macro is more C style than the old\nmacro, and is more obvious to what it does.\n\nHere\u0027s the example. The only updated macro in this patch is the\nsched_switch trace point.\n\nThe old method looked like this:\n\n TRACE_EVENT_FORMAT(sched_switch,\n TP_PROTO(struct rq *rq, struct task_struct *prev,\n struct task_struct *next),\n TP_ARGS(rq, prev, next),\n TP_FMT(\"task %s:%d \u003d\u003d\u003e %s:%d\",\n prev-\u003ecomm, prev-\u003epid, next-\u003ecomm, next-\u003epid),\n TRACE_STRUCT(\n TRACE_FIELD(pid_t, prev_pid, prev-\u003epid)\n TRACE_FIELD(int, prev_prio, prev-\u003eprio)\n TRACE_FIELD_SPECIAL(char next_comm[TASK_COMM_LEN],\n next_comm,\n TP_CMD(memcpy(TRACE_ENTRY-\u003enext_comm,\n next-\u003ecomm,\n TASK_COMM_LEN)))\n TRACE_FIELD(pid_t, next_pid, next-\u003epid)\n TRACE_FIELD(int, next_prio, next-\u003eprio)\n ),\n TP_RAW_FMT(\"prev %d:%d \u003d\u003d\u003e next %s:%d:%d\")\n );\n\nThe above method is hard to read and requires two format fields.\n\nThe new method:\n\n /*\n * Tracepoint for task switches, performed by the scheduler:\n *\n * (NOTE: the \u0027rq\u0027 argument is not used by generic trace events,\n * but used by the latency tracer plugin. )\n */\n TRACE_EVENT(sched_switch,\n\n\tTP_PROTO(struct rq *rq, struct task_struct *prev,\n\t\t struct task_struct *next),\n\n\tTP_ARGS(rq, prev, next),\n\n\tTP_STRUCT__entry(\n\t\t__array(\tchar,\tprev_comm,\tTASK_COMM_LEN\t)\n\t\t__field(\tpid_t,\tprev_pid\t\t\t)\n\t\t__field(\tint,\tprev_prio\t\t\t)\n\t\t__array(\tchar,\tnext_comm,\tTASK_COMM_LEN\t)\n\t\t__field(\tpid_t,\tnext_pid\t\t\t)\n\t\t__field(\tint,\tnext_prio\t\t\t)\n\t),\n\n\tTP_printk(\"task %s:%d [%d] \u003d\u003d\u003e %s:%d [%d]\",\n\t\t__entry-\u003eprev_comm, __entry-\u003eprev_pid, __entry-\u003eprev_prio,\n\t\t__entry-\u003enext_comm, __entry-\u003enext_pid, __entry-\u003enext_prio),\n\n\tTP_fast_assign(\n\t\tmemcpy(__entry-\u003enext_comm, next-\u003ecomm, TASK_COMM_LEN);\n\t\t__entry-\u003eprev_pid\t\u003d prev-\u003epid;\n\t\t__entry-\u003eprev_prio\t\u003d prev-\u003eprio;\n\t\tmemcpy(__entry-\u003eprev_comm, prev-\u003ecomm, TASK_COMM_LEN);\n\t\t__entry-\u003enext_pid\t\u003d next-\u003epid;\n\t\t__entry-\u003enext_prio\t\u003d next-\u003eprio;\n\t)\n );\n\nThis macro is called TRACE_EVENT, it is broken up into 5 parts:\n\n TP_PROTO: the proto type of the trace point\n TP_ARGS: the arguments of the trace point\n TP_STRUCT_entry: the structure layout of the entry in the ring buffer\n TP_printk: the printk format\n TP_fast_assign: the method used to write the entry into the ring buffer\n\nThe structure is the definition of how the event will be saved in the\nring buffer. The printk is used by the internal tracing in case of\nan oops, and the kernel needs to print out the format of the record\nto the console. This the TP_printk gives a means to show the records\nin a human readable format. It is also used to print out the data\nfrom the trace file.\n\nThe TP_fast_assign is executed directly. It is basically like a C function,\nwhere the __entry is the handle to the record.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\n" }, { "commit": "2939b0469d04ba9ac791aca9a81625d7eb50662b", "tree": "573f10c39f34c670fdc6832415642738c5afb3f9", "parents": [ "156b5f172a64103bcb13b6d26288388b9019caa3" ], "author": { "name": "Steven Rostedt", "email": "srostedt@redhat.com", "time": "Mon Mar 09 15:47:18 2009 -0400" }, "committer": { "name": "Steven Rostedt", "email": "srostedt@redhat.com", "time": "Tue Mar 10 00:35:04 2009 -0400" }, "message": "tracing: replace TP\u003cvar\u003e with TP_\u003cvar\u003e\n\nImpact: clean up\n\nThe macros TPPROTO, TPARGS, TPFMT, TPRAWFMT, and TPCMD all look a bit\nugly. This patch adds an underscore to their names.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\n" }, { "commit": "629928041c53771f9902753d50fef6b35f36d33d", "tree": "e960bf1f0311a2d00eadebfc1304662a149e32fe", "parents": [ "fd99498989f3b3feeab89dcadf537138ba136d24" ], "author": { "name": "Steven Rostedt", "email": "srostedt@redhat.com", "time": "Sat Feb 28 02:47:59 2009 -0500" }, "committer": { "name": "Steven Rostedt", "email": "srostedt@redhat.com", "time": "Sat Feb 28 04:04:13 2009 -0500" }, "message": "tracing: create the C style tracing for the sched subsystem\n\nThis patch utilizes the TRACE_EVENT_FORMAT macro to enable the C style\nfaster tracing for the sched subsystem trace points.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\n" }, { "commit": "3cdfdf91fcc77cfc82592e2b5c2ab35abe819c41", "tree": "176e2dbbf4e8c6ab59d0243f501338d104d7a31f", "parents": [ "eef62a6826b8ab530cefff5aa55c1661a209c803" ], "author": { "name": "Steven Rostedt", "email": "srostedt@redhat.com", "time": "Wed Feb 25 15:54:30 2009 -0500" }, "committer": { "name": "Steven Rostedt", "email": "srostedt@redhat.com", "time": "Wed Feb 25 21:44:26 2009 -0500" }, "message": "tracing: wrap arguments with PARAMS\n\nPeter Zijlstra warned that TPPROTO and TPARGS might become something\nother than a simple copy of itself. To prevent this from having\nside effects in the TRACE_FORMAT macro in tracepoint.h, we add a\nPARAMS() macro to be defined as just a wrapper.\n\nReported-by: Peter Zijlstra \u003cpeterz@infradead.org\u003e\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\n" }, { "commit": "eef62a6826b8ab530cefff5aa55c1661a209c803", "tree": "f24c7f08b9a6ed443c09f26509cf2cf26fb8a19c", "parents": [ "d7350c3f45694104e820041969c8185c5f99e57c" ], "author": { "name": "Steven Rostedt", "email": "srostedt@redhat.com", "time": "Wed Feb 25 15:49:52 2009 -0500" }, "committer": { "name": "Steven Rostedt", "email": "srostedt@redhat.com", "time": "Wed Feb 25 21:44:22 2009 -0500" }, "message": "tracing: rename DEFINE_TRACE_FMT to just TRACE_FORMAT\n\nThere\u0027s been a bit confusion to whether DEFINE/DECLARE_TRACE_FMT should\nbe a DEFINE or a DECLARE. Ingo Molnar suggested simply calling it\nTRACE_FORMAT.\n\nReported-by: Ingo Molnar \u003cmingo@elte.hu\u003e\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\n" }, { "commit": "7c37730cd31ddb2d3a1da142af9b18c29b8c433b", "tree": "60e2d4a1b53dba29824b967345bc43814853b8a4", "parents": [ "c478f8786973d6d7552c652ddad3f6fd86b5af28" ], "author": { "name": "Steven Rostedt", "email": "srostedt@redhat.com", "time": "Tue Feb 24 12:07:53 2009 -0500" }, "committer": { "name": "Steven Rostedt", "email": "srostedt@redhat.com", "time": "Tue Feb 24 21:53:32 2009 -0500" }, "message": "tracing: add DEFINE_TRACE_FMT to tracepoint.h\n\nThis patch creates a DEFINE_TRACE_FMT to map to DECLARE_TRACE.\nThis allows for the developers to place format strings and\nargs in with their tracepoint declaration. A tracer may now\noverride the DEFINE_TRACE_FMT macro and use it to record\na default format.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\n" }, { "commit": "7e066fb870fcd1025ec3ba7bbde5d541094f4ce1", "tree": "52acda06de25c029b9834110d7bf6b4abc50353b", "parents": [ "32f85742778dfc2c74975cf0b9f5bdb13470cb32" ], "author": { "name": "Mathieu Desnoyers", "email": "mathieu.desnoyers@polymtl.ca", "time": "Fri Nov 14 17:47:47 2008 -0500" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Sun Nov 16 09:01:36 2008 +0100" }, "message": "tracepoints: add DECLARE_TRACE() and DEFINE_TRACE()\n\nImpact: API *CHANGE*. Must update all tracepoint users.\n\nAdd DEFINE_TRACE() to tracepoints to let them declare the tracepoint\nstructure in a single spot for all the kernel. It helps reducing memory\nconsumption, especially when declaring a lot of tracepoints, e.g. for\nkmalloc tracing.\n\n*API CHANGE WARNING*: now, DECLARE_TRACE() must be used in headers for\ntracepoint declarations rather than DEFINE_TRACE(). This is the sane way\nto do it. The name previously used was misleading.\n\nUpdates scheduler instrumentation to follow this API change.\n\nSigned-off-by: Mathieu Desnoyers \u003cmathieu.desnoyers@polymtl.ca\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "5f382671def7cb9c0f4b75d586dc5f60dca5e1c3", "tree": "ee90d469b24a893c8ec777775a43d41cdfb6f76f", "parents": [ "c420970ef476d7d68df119711700666224001f43" ], "author": { "name": "Mathieu Desnoyers", "email": "mathieu.desnoyers@polymtl.ca", "time": "Fri Nov 14 17:47:45 2008 -0500" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Sun Nov 16 09:01:34 2008 +0100" }, "message": "tracepoints: do not put arguments in name\n\nImpact: cleanup\n\nThat\u0027s overkill, takes space. We have a global tracepoint registery in\nheader files anyway.\n\nSigned-off-by: Mathieu Desnoyers \u003cmathieu.desnoyers@polymtl.ca\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "c420970ef476d7d68df119711700666224001f43", "tree": "c10ebb11a03199f99d779472037bd49129b43abb", "parents": [ "da7b3eab167091693ad215ad7692f7d0d24d1356" ], "author": { "name": "Mathieu Desnoyers", "email": "mathieu.desnoyers@polymtl.ca", "time": "Fri Nov 14 17:47:44 2008 -0500" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Sun Nov 16 09:01:33 2008 +0100" }, "message": "tracepoints: use unregister return value\n\nImpact: bugfix.\n\nUnregistering a tracepoint can fail. Return the error value.\n\nSigned-off-by: Mathieu Desnoyers \u003cmathieu.desnoyers@polymtl.ca\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "da7b3eab167091693ad215ad7692f7d0d24d1356", "tree": "a879fc523166fdaf196fe128e11246996eda0480", "parents": [ "de0baf9ad661ac630a45a50ea1717cc4f4b33ace" ], "author": { "name": "Mathieu Desnoyers", "email": "mathieu.desnoyers@polymtl.ca", "time": "Fri Nov 14 17:47:43 2008 -0500" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Sun Nov 16 09:01:32 2008 +0100" }, "message": "tracepoints: use rcu_*_sched_notrace\n\nMake sure tracepoints can be called within ftrace callbacks.\n\nSigned-off-by: Mathieu Desnoyers \u003cmathieu.desnoyers@polymtl.ca\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "127cafbb276266b1b8da967bfe25a062ab1d42ab", "tree": "8f7183fa6f9a68b2bd3b3734da3575e4eff46a1d", "parents": [ "19dba33c43a2f0f2aa727ae075ec3b11330775ef" ], "author": { "name": "Lai Jiangshan", "email": "laijs@cn.fujitsu.com", "time": "Tue Oct 28 10:51:53 2008 +0800" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Mon Nov 03 10:28:52 2008 +0100" }, "message": "tracepoint: introduce *_noupdate APIs.\n\nImpact: add new tracepoint APIs to allow the batched registration of probes\n\nnew APIs separate tracepoint_probe_register(),\ntracepoint_probe_unregister() into 2 steps. The first step of them\nis just update tracepoint_entry, not connect or disconnect.\n\nthis patch introduces tracepoint_probe_update_all() for update all.\n\nthese APIs are very useful for registering lots of probes\nbut just updating once. Another very important thing is that\n*_noupdate APIs do not require module_mutex.\n\nSigned-off-by: Lai Jiangshan \u003claijs@cn.fujitsu.com\u003e\nAcked-by: Mathieu Desnoyers \u003cmathieu.desnoyers@polymtl.ca\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "231375cc5cc3549bb413f94a164bdcbd5f9ce943", "tree": "03ff935268f5c551da14704d14667e6951a5b25f", "parents": [ "f2461fc82a083dd60062e05e704c5fcc1c658ba1" ], "author": { "name": "Mathieu Desnoyers", "email": "mathieu.desnoyers@polymtl.ca", "time": "Fri Oct 03 15:01:33 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:39:31 2008 +0200" }, "message": "tracepoints: synchronize unregister static inline\n\nTurn tracepoint synchronize unregister into a static inline. There is no\nreason to keep it as a macro over a static inline.\n\nSigned-off-by: Mathieu Desnoyers \u003cmathieu.desnoyers@polymtl.ca\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "f2461fc82a083dd60062e05e704c5fcc1c658ba1", "tree": "1b5e13fd4895351d26e9bd4a404c40c8d8381a09", "parents": [ "8b27386a9ce9c7f0f8702cff7565a46802ad57d1" ], "author": { "name": "Mathieu Desnoyers", "email": "mathieu.desnoyers@polymtl.ca", "time": "Mon Oct 06 10:33:00 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:39:30 2008 +0200" }, "message": "tracepoints: tracepoint_synchronize_unregister()\n\nCreate tracepoint_synchronize_unregister() which must be called before the end\nof exit() to make sure every probe callers have exited the non preemptible\nsection and thus are not executing the probe code anymore.\n\nSigned-off-by: Mathieu Desnoyers \u003cmathieu.desnoyers@polymtl.ca\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "97e1c18e8d17bd87e1e383b2e9d9fc740332c8e2", "tree": "5c6bfce8bacf04c2993a0029788a1370a483afa6", "parents": [ "e7f2f9918c0e97aa98ba147ca387e2c7238f0711" ], "author": { "name": "Mathieu Desnoyers", "email": "mathieu.desnoyers@polymtl.ca", "time": "Fri Jul 18 12:16:16 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:28:28 2008 +0200" }, "message": "tracing: Kernel Tracepoints\n\nImplementation of kernel tracepoints. Inspired from the Linux Kernel\nMarkers. Allows complete typing verification by declaring both tracing\nstatement inline functions and probe registration/unregistration static\ninline functions within the same macro \"DEFINE_TRACE\". No format string\nis required. See the tracepoint Documentation and Samples patches for\nusage examples.\n\nTaken from the documentation patch :\n\n\"A tracepoint placed in code provides a hook to call a function (probe)\nthat you can provide at runtime. A tracepoint can be \"on\" (a probe is\nconnected to it) or \"off\" (no probe is attached). When a tracepoint is\n\"off\" it has no effect, except for adding a tiny time penalty (checking\na condition for a branch) and space penalty (adding a few bytes for the\nfunction call at the end of the instrumented function and adds a data\nstructure in a separate section). When a tracepoint is \"on\", the\nfunction you provide is called each time the tracepoint is executed, in\nthe execution context of the caller. When the function provided ends its\nexecution, it returns to the caller (continuing from the tracepoint\nsite).\n\nYou can put tracepoints at important locations in the code. They are\nlightweight hooks that can pass an arbitrary number of parameters, which\nprototypes are described in a tracepoint declaration placed in a header\nfile.\"\n\nAddition and removal of tracepoints is synchronized by RCU using the\nscheduler (and preempt_disable) as guarantees to find a quiescent state\n(this is really RCU \"classic\"). The update side uses rcu_barrier_sched()\nwith call_rcu_sched() and the read/execute side uses\n\"preempt_disable()/preempt_enable()\".\n\nWe make sure the previous array containing probes, which has been\nscheduled for deletion by the rcu callback, is indeed freed before we\nproceed to the next update. It therefore limits the rate of modification\nof a single tracepoint to one update per RCU period. The objective here\nis to permit fast batch add/removal of probes on _different_\ntracepoints.\n\nChangelog :\n- Use #name \":\" #proto as string to identify the tracepoint in the\n tracepoint table. This will make sure not type mismatch happens due to\n connexion of a probe with the wrong type to a tracepoint declared with\n the same name in a different header.\n- Add tracepoint_entry_free_old.\n- Change __TO_TRACE to get rid of the \u0027i\u0027 iterator.\n\nMasami Hiramatsu \u003cmhiramat@redhat.com\u003e :\nTested on x86-64.\n\nPerformance impact of a tracepoint : same as markers, except that it\nadds about 70 bytes of instructions in an unlikely branch of each\ninstrumented function (the for loop, the stack setup and the function\ncall). It currently adds a memory read, a test and a conditional branch\nat the instrumentation site (in the hot path). Immediate values will\neventually change this into a load immediate, test and branch, which\nremoves the memory read which will make the i-cache impact smaller\n(changing the memory read for a load immediate removes 3-4 bytes per\nsite on x86_32 (depending on mov prefixes), or 7-8 bytes on x86_64, it\nalso saves the d-cache hit).\n\nAbout the performance impact of tracepoints (which is comparable to\nmarkers), even without immediate values optimizations, tests done by\nHideo Aoki on ia64 show no regression. His test case was using hackbench\non a kernel where scheduler instrumentation (about 5 events in code\nscheduler code) was added.\n\nQuoting Hideo Aoki about Markers :\n\nI evaluated overhead of kernel marker using linux-2.6-sched-fixes git\ntree, which includes several markers for LTTng, using an ia64 server.\n\nWhile the immediate trace mark feature isn\u0027t implemented on ia64, there\nis no major performance regression. So, I think that we don\u0027t have any\nissues to propose merging marker point patches into Linus\u0027s tree from\nthe viewpoint of performance impact.\n\nI prepared two kernels to evaluate. The first one was compiled without\nCONFIG_MARKERS. The second one was enabled CONFIG_MARKERS.\n\nI downloaded the original hackbench from the following URL:\nhttp://devresources.linux-foundation.org/craiger/hackbench/src/hackbench.c\n\nI ran hackbench 5 times in each condition and calculated the average and\ndifference between the kernels.\n\n The parameter of hackbench: every 50 from 50 to 800\n The number of CPUs of the server: 2, 4, and 8\n\nBelow is the results. As you can see, major performance regression\nwasn\u0027t found in any case. Even if number of processes increases,\ndifferences between marker-enabled kernel and marker- disabled kernel\ndoesn\u0027t increase. Moreover, if number of CPUs increases, the differences\ndoesn\u0027t increase either.\n\nCuriously, marker-enabled kernel is better than marker-disabled kernel\nin more than half cases, although I guess it comes from the difference\nof memory access pattern.\n\n* 2 CPUs\n\nNumber of | without | with | diff | diff |\nprocesses | Marker [Sec] | Marker [Sec] | [Sec] | [%] |\n--------------------------------------------------------------\n 50 | 4.811 | 4.872 | +0.061 | +1.27 |\n 100 | 9.854 | 10.309 | +0.454 | +4.61 |\n 150 | 15.602 | 15.040 | -0.562 | -3.6 |\n 200 | 20.489 | 20.380 | -0.109 | -0.53 |\n 250 | 25.798 | 25.652 | -0.146 | -0.56 |\n 300 | 31.260 | 30.797 | -0.463 | -1.48 |\n 350 | 36.121 | 35.770 | -0.351 | -0.97 |\n 400 | 42.288 | 42.102 | -0.186 | -0.44 |\n 450 | 47.778 | 47.253 | -0.526 | -1.1 |\n 500 | 51.953 | 52.278 | +0.325 | +0.63 |\n 550 | 58.401 | 57.700 | -0.701 | -1.2 |\n 600 | 63.334 | 63.222 | -0.112 | -0.18 |\n 650 | 68.816 | 68.511 | -0.306 | -0.44 |\n 700 | 74.667 | 74.088 | -0.579 | -0.78 |\n 750 | 78.612 | 79.582 | +0.970 | +1.23 |\n 800 | 85.431 | 85.263 | -0.168 | -0.2 |\n--------------------------------------------------------------\n\n* 4 CPUs\n\nNumber of | without | with | diff | diff |\nprocesses | Marker [Sec] | Marker [Sec] | [Sec] | [%] |\n--------------------------------------------------------------\n 50 | 2.586 | 2.584 | -0.003 | -0.1 |\n 100 | 5.254 | 5.283 | +0.030 | +0.56 |\n 150 | 8.012 | 8.074 | +0.061 | +0.76 |\n 200 | 11.172 | 11.000 | -0.172 | -1.54 |\n 250 | 13.917 | 14.036 | +0.119 | +0.86 |\n 300 | 16.905 | 16.543 | -0.362 | -2.14 |\n 350 | 19.901 | 20.036 | +0.135 | +0.68 |\n 400 | 22.908 | 23.094 | +0.186 | +0.81 |\n 450 | 26.273 | 26.101 | -0.172 | -0.66 |\n 500 | 29.554 | 29.092 | -0.461 | -1.56 |\n 550 | 32.377 | 32.274 | -0.103 | -0.32 |\n 600 | 35.855 | 35.322 | -0.533 | -1.49 |\n 650 | 39.192 | 38.388 | -0.804 | -2.05 |\n 700 | 41.744 | 41.719 | -0.025 | -0.06 |\n 750 | 45.016 | 44.496 | -0.520 | -1.16 |\n 800 | 48.212 | 47.603 | -0.609 | -1.26 |\n--------------------------------------------------------------\n\n* 8 CPUs\n\nNumber of | without | with | diff | diff |\nprocesses | Marker [Sec] | Marker [Sec] | [Sec] | [%] |\n--------------------------------------------------------------\n 50 | 2.094 | 2.072 | -0.022 | -1.07 |\n 100 | 4.162 | 4.273 | +0.111 | +2.66 |\n 150 | 6.485 | 6.540 | +0.055 | +0.84 |\n 200 | 8.556 | 8.478 | -0.078 | -0.91 |\n 250 | 10.458 | 10.258 | -0.200 | -1.91 |\n 300 | 12.425 | 12.750 | +0.325 | +2.62 |\n 350 | 14.807 | 14.839 | +0.032 | +0.22 |\n 400 | 16.801 | 16.959 | +0.158 | +0.94 |\n 450 | 19.478 | 19.009 | -0.470 | -2.41 |\n 500 | 21.296 | 21.504 | +0.208 | +0.98 |\n 550 | 23.842 | 23.979 | +0.137 | +0.57 |\n 600 | 26.309 | 26.111 | -0.198 | -0.75 |\n 650 | 28.705 | 28.446 | -0.259 | -0.9 |\n 700 | 31.233 | 31.394 | +0.161 | +0.52 |\n 750 | 34.064 | 33.720 | -0.344 | -1.01 |\n 800 | 36.320 | 36.114 | -0.206 | -0.57 |\n--------------------------------------------------------------\n\nSigned-off-by: Mathieu Desnoyers \u003cmathieu.desnoyers@polymtl.ca\u003e\nAcked-by: Masami Hiramatsu \u003cmhiramat@redhat.com\u003e\nAcked-by: \u0027Peter Zijlstra\u0027 \u003cpeterz@infradead.org\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" } ] }