)]}' { "log": [ { "commit": "53c8c8fdfd2d2d515bdcb3d0f2a11d1f3f42ece1", "tree": "37c0af18e73feeeb8301473de1932163df1d96be", "parents": [ "ed86a59071a4ccd0e9bcefc61e013759a21eda78" ], "author": { "name": "Mathieu Desnoyers", "email": "mathieu.desnoyers@polymtl.ca", "time": "Fri Oct 03 11:52:54 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:38:40 2008 +0200" }, "message": "markers: turn marker_synchronize_unregister() into an inline\n\nTurn marker 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": "ed86a59071a4ccd0e9bcefc61e013759a21eda78", "tree": "013abe0cb9e3cd4d7e6c2a9655f86517423a2623", "parents": [ "5b9261d93e5fa3db4995d5b77b5ed365166e001c" ], "author": { "name": "Mathieu Desnoyers", "email": "mathieu.desnoyers@polymtl.ca", "time": "Wed Oct 01 12:03:25 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:38:38 2008 +0200" }, "message": "markers: re-enable fast batch registration\n\nLai Jiangshan discovered a reentrancy issue with markers and fixed it by\nadding synchronize_sched() calls at each registration/unregistraiton.\n\nIt works, but it removes the ability to do batch\nregistration/unregistration and can cause registration of ~100 markers\nto take about 30 seconds on a loaded machine (synchronize_sched() is\nmuch slower on such workloads).\n\nThis patch implements a version of the fix which won\u0027t slow down marker batch\nregistration/unregistration. It also go back to the original non-synchronized\nreg/unreg.\n\nSigned-off-by: Mathieu Desnoyers \u003cmathieu.desnoyers@polymtl.ca\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "5b9261d93e5fa3db4995d5b77b5ed365166e001c", "tree": "ac134765e5518af79da1845fccac7683488e0b22", "parents": [ "91a8d46c47e7eb1c53c181e4328a3cfa45ae4ad3" ], "author": { "name": "Mathieu Desnoyers", "email": "mathieu.desnoyers@polymtl.ca", "time": "Mon Sep 29 11:11:47 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:38:35 2008 +0200" }, "message": "sputrace: use marker_synchronize_unregister()\n\nWe need a marker_synchronize_unregister() before the end of exit() to make sure\nevery probe callers have exited the non preemptible section and thus are not\nexecuting the probe code anymore.\n\nSigned-off-by: Mathieu Desnoyers \u003cmathieu.desnoyers@polymtl.ca\u003e\nAcked-by: Jeremy Kerr \u003cjk@ozlabs.org\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "91a8d46c47e7eb1c53c181e4328a3cfa45ae4ad3", "tree": "61a4f531bd68a386610d6ecaa520bda1d7c80174", "parents": [ "531d297569014e8f889b167a2d15d72429faead1" ], "author": { "name": "Mathieu Desnoyers", "email": "mathieu.desnoyers@polymtl.ca", "time": "Mon Sep 29 11:10:34 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:38:33 2008 +0200" }, "message": "markers: documentation fix for teardown\n\nDocument the need for a marker_synchronize_unregister() before the end of\nexit() 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": "531d297569014e8f889b167a2d15d72429faead1", "tree": "41bd1b7d66cc454aaab3f699698c75ca0c88c92f", "parents": [ "e2d3b75dbc486253c910722486ac64087f96c59f" ], "author": { "name": "Mathieu Desnoyers", "email": "mathieu.desnoyers@polymtl.ca", "time": "Mon Sep 29 11:09:15 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:38:30 2008 +0200" }, "message": "markers: probe example, fix teardown\n\nNeed a marker_synchronize_unregister() before the end of exit() to make sure\nevery probe callers have exited the non preemptible section and thus are not\nexecuting 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": "e2d3b75dbc486253c910722486ac64087f96c59f", "tree": "c59bf9066f6f4bc290e5330823dbd87429b09e28", "parents": [ "e98d0eabef2748d88fa58760d104e8e68517406b" ], "author": { "name": "Mathieu Desnoyers", "email": "mathieu.desnoyers@polymtl.ca", "time": "Mon Sep 29 11:08:03 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:38:28 2008 +0200" }, "message": "markers: fix unregister bug and reenter bug, cleanup\n\nUse the new rcu_read_lock_sched/unlock_sched() in marker code around the call\nsite instead of preempt_disable/enable(). It helps reviewing the code more\neasily.\n\nSigned-off-by: Mathieu Desnoyers \u003cmathieu.desnoyers@polymtl.ca\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "e98d0eabef2748d88fa58760d104e8e68517406b", "tree": "6a7782971bdd598380eab43c0cb9aefbb2c6e2d6", "parents": [ "9a1e9693f534174945154197fec4ec92f168ce21" ], "author": { "name": "Mathieu Desnoyers", "email": "mathieu.desnoyers@polymtl.ca", "time": "Mon Sep 29 11:05:13 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:38:26 2008 +0200" }, "message": "markers: marker_synchronize_unregister()\n\nCreate marker_synchronize_unregister() which must be called before the end of\nexit() to make sure every probe callers have exited the non preemptible section\nand 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": "9a1e9693f534174945154197fec4ec92f168ce21", "tree": "d1477dc3492812c2f8a93afe37075857d4b1ce0f", "parents": [ "ca2db6cf30d6a45798b7a760d0c4f7735b16799d" ], "author": { "name": "Mathieu Desnoyers", "email": "mathieu.desnoyers@polymtl.ca", "time": "Tue Sep 30 01:51:12 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:38:23 2008 +0200" }, "message": "tracepoints: fix reentrancy\n\nThe tracepoints had the same problem markers did have wrt reentrancy. Apply a\nsimilar fix using a rcu_barrier after each tracepoint mutex lock.\n\nSigned-off-by: Mathieu Desnoyers \u003cmathieu.desnoyers@polymtl.ca\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "ca2db6cf30d6a45798b7a760d0c4f7735b16799d", "tree": "5c1658ec172fdaa04ee018972eb96f0f2c409676", "parents": [ "d74185ed27651ad8d920b37d7851306ad01f7d6f" ], "author": { "name": "Mathieu Desnoyers", "email": "mathieu.desnoyers@polymtl.ca", "time": "Tue Sep 30 01:49:39 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:38:21 2008 +0200" }, "message": "tracepoints: use rcu sched\n\nMake tracepoints use rcu sched. (cleanup)\n\nSigned-off-by: Mathieu Desnoyers \u003cmathieu.desnoyers@polymtl.ca\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "d74185ed27651ad8d920b37d7851306ad01f7d6f", "tree": "d3eab74f08808e9596641cd2dcc66f02df060705", "parents": [ "ac2b86fdef5b44f194eefaa6b7b6aea9423d1bc2" ], "author": { "name": "Lai Jiangshan", "email": "laijs@cn.fujitsu.com", "time": "Mon Sep 29 16:00:05 2008 +0800" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:38:19 2008 +0200" }, "message": "markers: fix unregister bug and reenter bug\n\nunregister bug:\n\ncodes using makers are typically calling marker_probe_unregister()\nand then destroying the data that marker_probe_func needs(or\nunloading this module). This is bug when the corresponding\nmarker_probe_func is still running(on other cpus),\nit is using the destroying/ed data.\n\nwe should call synchronize_sched() after marker_update_probes().\n\nreenter bug:\n\nmarker_probe_register(), marker_probe_unregister() and\nmarker_probe_unregister_private_data() are not reentrant safe\nfunctions. these 3 functions release markers_mutex and then\nrequire it again and do \"entry-\u003eoldptr \u003d old; ...\", but entry-\u003eoldptr\nmaybe is using now for these 3 functions may reenter when markers_mutex\nis released.\n\nwe use synchronize_sched() instead of call_rcu_sched() to fix\nthis bug. actually we can do:\n\"\nif (entry-\u003ercu_pending)\n\t\trcu_barrier_sched();\n\"\nafter require markers_mutex again. but synchronize_sched()\nis better and simpler. For these 3 functions are not critical path.\n\nSigned-off-by: Lai Jiangshan \u003claijs@cn.fujitsu.com\u003e\nCc: Mathieu Desnoyers \u003cmathieu.desnoyers@polymtl.ca\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "ac2b86fdef5b44f194eefaa6b7b6aea9423d1bc2", "tree": "7ca5225be1cd41e328a54158e1c4cfaaefc1b9e4", "parents": [ "37a52f5ef120b93734bb2461744512b55695f69c" ], "author": { "name": "Frédéric Weisbecker", "email": "fweisbec@gmail.com", "time": "Wed Sep 24 16:31:56 2008 +0100" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:38:16 2008 +0200" }, "message": "x86/ftrace: use uaccess in atomic context\n\nWith latest -tip I get this bug:\n\n[ 49.439988] in_atomic():0, irqs_disabled():1\n[ 49.440118] INFO: lockdep is turned off.\n[ 49.440118] Pid: 2814, comm: modprobe Tainted: G W 2.6.27-rc7 #4\n[ 49.440118] [\u003cc01215e1\u003e] __might_sleep+0xe1/0x120\n[ 49.440118] [\u003cc01148ea\u003e] ftrace_modify_code+0x2a/0xd0\n[ 49.440118] [\u003cc01148a2\u003e] ? ftrace_test_p6nop+0x0/0xa\n[ 49.440118] [\u003cc016e80e\u003e] __ftrace_update_code+0xfe/0x2f0\n[ 49.440118] [\u003cc01148a2\u003e] ? ftrace_test_p6nop+0x0/0xa\n[ 49.440118] [\u003cc016f190\u003e] ftrace_convert_nops+0x50/0x80\n[ 49.440118] [\u003cc016f1d6\u003e] ftrace_init_module+0x16/0x20\n[ 49.440118] [\u003cc015498b\u003e] load_module+0x185b/0x1d30\n[ 49.440118] [\u003cc01767a0\u003e] ? find_get_page+0x0/0xf0\n[ 49.440118] [\u003cc02463c0\u003e] ? sprintf+0x0/0x30\n[ 49.440118] [\u003cc034e012\u003e] ? mutex_lock_interruptible_nested+0x1f2/0x350\n[ 49.440118] [\u003cc0154eb3\u003e] sys_init_module+0x53/0x1b0\n[ 49.440118] [\u003cc0352340\u003e] ? do_page_fault+0x0/0x740\n[ 49.440118] [\u003cc0104012\u003e] syscall_call+0x7/0xb\n[ 49.440118] \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n\nIt is because ftrace_modify_code() calls copy_to_user and\ncopy_from_user.\nThese functions have been inserted after guessing that there\ncouldn\u0027t be any race condition but copy_[to/from]_user might\nsleep and __ftrace_update_code is called with local_irq_saved.\n\nThese function have been inserted since this commit:\nd5e92e8978fd2574e415dc2792c5eb592978243d:\n\"ftrace: x86 use copy from user function\"\n\nSigned-off-by: Frederic Weisbecker \u003cfweisbec@gmail.com\u003e\nAcked-by: Steven Rostedt \u003crostedt@goodmis.org\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "37a52f5ef120b93734bb2461744512b55695f69c", "tree": "c50d991252df8cdff2ae3c95b82c13631b7768be", "parents": [ "05736a427f7e16be948ccbf39782bd3a6ae16b14" ], "author": { "name": "Harvey Harrison", "email": "harvey.harrison@gmail.com", "time": "Tue Sep 23 15:05:46 2008 -0700" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:38:14 2008 +0200" }, "message": "x86: suppress trivial sparse signedness warnings\n\nCould just as easily change the three casts to cast to the correct\ntype...this patch changes the type of ftrace_nop instead.\n\nSupresses sparse warnings:\n\n arch/x86/kernel/ftrace.c:157:14: warning: incorrect type in assignment (different signedness)\n arch/x86/kernel/ftrace.c:157:14: expected long *static [toplevel] ftrace_nop\n arch/x86/kernel/ftrace.c:157:14: got unsigned long *\u003cnoident\u003e\n arch/x86/kernel/ftrace.c:161:14: warning: incorrect type in assignment (different signedness)\n arch/x86/kernel/ftrace.c:161:14: expected long *static [toplevel] ftrace_nop\n arch/x86/kernel/ftrace.c:161:14: got unsigned long *\u003cnoident\u003e\n arch/x86/kernel/ftrace.c:165:14: warning: incorrect type in assignment (different signedness)\n arch/x86/kernel/ftrace.c:165:14: expected long *static [toplevel] ftrace_nop\n arch/x86/kernel/ftrace.c:165:14: got unsigned long *\u003cnoident\u003e\n\nSigned-off-by: Harvey Harrison \u003charvey.harrison@gmail.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "05736a427f7e16be948ccbf39782bd3a6ae16b14", "tree": "6efd6991853918242947df308be60b14a6fa5e80", "parents": [ "43a15386c4faf913f7d70a47748c266d6210cd6e" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Mon Sep 22 14:55:47 2008 -0700" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:38:11 2008 +0200" }, "message": "ftrace: warn on failure to disable mcount callers\n\nWith the recent updates to ftrace, there should not be any failures when\nmodifying the code. If there is, then we need to warn about it.\n\nThis patch has a cleaned up version of the code that I used to discover\nthat the weak symbols were causing failures.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "43a15386c4faf913f7d70a47748c266d6210cd6e", "tree": "2af766a34661123bd520fb30b1f83fbff1e65dab", "parents": [ "2a3a4f669df2164288d11406d11d5e4933bf5e53" ], "author": { "name": "Frédéric Weisbecker", "email": "fweisbec@gmail.com", "time": "Sun Sep 21 20:16:30 2008 +0200" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:38:09 2008 +0200" }, "message": "tracing/ftrace: replace none tracer by nop tracer\n\nReplace \"none\" tracer by the recently created \"nop\" tracer.\nBoth are pretty similar except that nop accepts TRACE_PRINT\nor TRACE_SPECIAL entries.\n\nAnd as a consequence, changing the size of the ring buffer now\nrequires that tracing has already been disabled.\n\nSigned-off-by: Frederic Weisbecker \u003cfweisbec@gmail.com\u003e\nCc: Steven Rostedt \u003crostedt@goodmis.org\u003e\nCc: Steven Noonan \u003csteven@uplinklabs.net\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "2a3a4f669df2164288d11406d11d5e4933bf5e53", "tree": "cbcab61ae346f24731b52c706a3650978a95aba3", "parents": [ "35cb5ed01261f5669657d2f1720aca902299887d" ], "author": { "name": "Frédéric Weisbecker", "email": "fweisbec@gmail.com", "time": "Sun Sep 21 20:12:14 2008 +0200" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:38:06 2008 +0200" }, "message": "tracing/ftrace: tracing engine depends on Nop Tracer\n\nNow that the nop tracer is used as the default tracer by\nreplacing the \"none\" tracer, tracing engine depends on it.\n\nSigned-off-by: Frederic Weisbecker \u003cfweisbec@gmail.com\u003e\nCc: Steven Rostedt \u003crostedt@goodmis.org\u003e\nCc: Steven Noonan \u003csteven@uplinklabs.net\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "35cb5ed01261f5669657d2f1720aca902299887d", "tree": "b4c91f58d56f0744d076cfb69967efa747f1385d", "parents": [ "8925b394eca0bb0a444a6487d702d0f650e94a10" ], "author": { "name": "Frédéric Weisbecker", "email": "fweisbec@gmail.com", "time": "Sun Sep 21 20:10:14 2008 +0200" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:38:04 2008 +0200" }, "message": "tracing/ftrace: make nop tracer reset previous entries\n\nIf nop tracer is selected, some old entries from the previous tracer\ncould still be enqueued. Tracing have to be reset.\n\nSigned-off-by: Frederic Weisbecker \u003cfweisbec@gmail.com\u003e\nCc: Steven Rostedt \u003crostedt@goodmis.org\u003e\nCc: Steven Noonan \u003csteven@uplinklabs.net\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "8925b394eca0bb0a444a6487d702d0f650e94a10", "tree": "855a9f3ba8f6e74eaf0301898d79619077fa9b8d", "parents": [ "71c67d58b5660f8e42c7d4c3e77cbc03fac5ed31" ], "author": { "name": "Steven Noonan", "email": "steven@uplinklabs.net", "time": "Sat Sep 20 01:00:38 2008 -0700" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:38:01 2008 +0200" }, "message": "trace: remove pointless ifdefs\n\nThe functions are already \u0027extern\u0027 anyway, so there\u0027s no problem\nwith linkage. Removing these ifdefs also helps find any potential\ncompiler errors.\n\nSuggested by Andrew Morton.\n\nSigned-off-by: Steven Noonan \u003csteven@uplinklabs.net\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "71c67d58b5660f8e42c7d4c3e77cbc03fac5ed31", "tree": "c098b6b717c3bf1cb80bef83bde7c9a0ccbacaad", "parents": [ "fb1b6d8b5154c692172a424e45fbd0573295cb93" ], "author": { "name": "Steven Noonan", "email": "steven@uplinklabs.net", "time": "Sat Sep 20 01:00:37 2008 -0700" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:37:58 2008 +0200" }, "message": "ftrace: mcount_addr defined but not used\n\nWhen CONFIG_DYNAMIC_FTRACE isn\u0027t used, neither is mcount_addr. This\npatch eliminates that warning.\n\nSigned-off-by: Steven Noonan \u003csteven@uplinklabs.net\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "fb1b6d8b5154c692172a424e45fbd0573295cb93", "tree": "d9a7ad2c629a6133998402354e77cd721e4962b4", "parents": [ "5bf9a1ee350a10feb94107de32a203d81fbbe706" ], "author": { "name": "Steven Noonan", "email": "steven@uplinklabs.net", "time": "Fri Sep 19 03:06:43 2008 -0700" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:37:43 2008 +0200" }, "message": "ftrace: add nop tracer\n\nA no-op tracer which can serve two purposes:\n\n 1. A template for development of a new tracer.\n 2. A convenient way to see ftrace_printk() calls without\n an irrelevant trace making the output messy.\n\n[ mingo@elte.hu: resolved conflicts ]\nSigned-off-by: Steven Noonan \u003csteven@uplinklabs.net\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "5bf9a1ee350a10feb94107de32a203d81fbbe706", "tree": "ac6b11aaa35194aed9fe9f1403c9542d070d0c62", "parents": [ "4427414170a63331a9cc36b9598502c5cdfe453b" ], "author": { "name": "Pekka Paalanen", "email": "pq@iki.fi", "time": "Tue Sep 16 22:06:42 2008 +0300" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:37:20 2008 +0200" }, "message": "ftrace: inject markers via trace_marker file\n\nAllow a user to inject a marker (TRACE_PRINT entry) into the trace ring\nbuffer. The related file operations are derived from code by Frédéric\nWeisbecker \u003cfweisbec@gmail.com\u003e.\n\nSigned-off-by: Pekka Paalanen \u003cpq@iki.fi\u003e\nAcked-by: Steven Rostedt \u003crostedt@goodmis.org\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "4427414170a63331a9cc36b9598502c5cdfe453b", "tree": "684d77d4052d5ab59d2e0786fc627d113a403833", "parents": [ "fc5e27ae4b45a0619701a83f30d9b7fad7ed9400" ], "author": { "name": "Pekka Paalanen", "email": "pq@iki.fi", "time": "Tue Sep 16 22:03:56 2008 +0300" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:37:17 2008 +0200" }, "message": "mmiotrace: remove left-over marker cruft\n\nSigned-off-by: Pekka Paalanen \u003cpq@iki.fi\u003e\nAcked-by: Steven Rostedt \u003crostedt@goodmis.org\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "fc5e27ae4b45a0619701a83f30d9b7fad7ed9400", "tree": "236b53e76430aae9f6b1abc37de9259ba95c38c6", "parents": [ "9e57fb35d711331a9b1410c5c56ebeb3733428a0" ], "author": { "name": "Pekka Paalanen", "email": "pq@iki.fi", "time": "Tue Sep 16 22:02:27 2008 +0300" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:37:14 2008 +0200" }, "message": "mmiotrace: handle TRACE_PRINT entries\n\nAlso make trace_seq_print_cont() non-static, and add a newline if the\nseq buffer can\u0027t hold all data.\n\nSigned-off-by: Pekka Paalanen \u003cpq@iki.fi\u003e\nAcked-by: Steven Rostedt \u003crostedt@goodmis.org\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "9e57fb35d711331a9b1410c5c56ebeb3733428a0", "tree": "2e68b1b8bb233378cbcad0c8e9bc8f99abf7263a", "parents": [ "801fe40001dfc263848552fb28924b766ed44ea4" ], "author": { "name": "Pekka Paalanen", "email": "pq@iki.fi", "time": "Tue Sep 16 22:00:34 2008 +0300" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:37:11 2008 +0200" }, "message": "x86 mmiotrace: implement mmiotrace_printk()\n\nOffer mmiotrace users a function to inject markers from inside the kernel.\nThis depends on the trace_vprintk() patch.\n\nSigned-off-by: Pekka Paalanen \u003cpq@iki.fi\u003e\nAcked-by: Steven Rostedt \u003crostedt@goodmis.org\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "801fe40001dfc263848552fb28924b766ed44ea4", "tree": "fbb0a6ff09deb9f605d955ccdc6f5cb89793e1cf", "parents": [ "45dcd8b8a8ca855591e3ac882d3a7fc255d09d43" ], "author": { "name": "Pekka Paalanen", "email": "pq@iki.fi", "time": "Tue Sep 16 21:58:24 2008 +0300" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:37:07 2008 +0200" }, "message": "ftrace: add trace_vprintk()\n\ntrace_vprintk() for easier implementation of tracer specific *_printk\nfunctions. Add check check for no_tracer, and implement\n__ftrace_printk() as a wrapper.\n\nSigned-off-by: Pekka Paalanen \u003cpq@iki.fi\u003e\nAcked-by: Steven Rostedt \u003crostedt@goodmis.org\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "45dcd8b8a8ca855591e3ac882d3a7fc255d09d43", "tree": "55b1772dfc9a6381ab1ecad7f3f6fbd605427f20", "parents": [ "bbe5c7830c6dbde58726d44ec0337bc8b2d95d37" ], "author": { "name": "Pekka Paalanen", "email": "pq@iki.fi", "time": "Tue Sep 16 21:56:41 2008 +0300" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:37:04 2008 +0200" }, "message": "ftrace: move mmiotrace functions out of trace.c\n\nMoves the mmiotrace specific functions from trace.c to\ntrace_mmiotrace.c. Functions trace_wake_up(), tracing_get_trace_entry(),\nand tracing_generic_entry_update() are therefore made available outside\ntrace.c.\n\nSigned-off-by: Pekka Paalanen \u003cpq@iki.fi\u003e\nAcked-by: Steven Rostedt \u003crostedt@goodmis.org\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "bbe5c7830c6dbde58726d44ec0337bc8b2d95d37", "tree": "48ddada3c3d68db5b5476234fb04a8da768f4d9c", "parents": [ "644f991d4b920ab1f5043509651479420b293490" ], "author": { "name": "Pekka Paalanen", "email": "pq@iki.fi", "time": "Tue Sep 16 21:54:16 2008 +0300" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:37:01 2008 +0200" }, "message": "x86 mmiotrace: fix a rare memory leak\n\nSigned-off-by: Pekka Paalanen \u003cpq@iki.fi\u003e\nAcked-by: Steven Rostedt \u003crostedt@goodmis.org\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "644f991d4b920ab1f5043509651479420b293490", "tree": "075c57f5fd43206d54e5debed0e738e1878d73aa", "parents": [ "c0719e5a4b1ccc04180b7a7b71095c9fb7131919" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Sat Sep 06 01:06:04 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:36:58 2008 +0200" }, "message": "ftrace: fix unlocking of hash\n\nThis must be brown paper bag week for Steven Rostedt!\n\nWhile working on ftrace for PPC, I discovered that the hash locking done\nwhen CONFIG_FTRACE_MCOUNT_RECORD is not set, is totally incorrect.\n\nWith a cut and paste error, I had the hash lock macro to lock for both\nhash_lock _and_ hash_unlock!\n\nThis bug did not affect x86 since this bug was introduced when\nCONFIG_FTRACE_MCOUNT_RECORD was added to x86.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "c0719e5a4b1ccc04180b7a7b71095c9fb7131919", "tree": "a22ebc58b34f94caf63491ad35f034d0564f4cfd", "parents": [ "d3ee6d992821f471193a7ee7a00af9ebb4bf5d01" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Sat Sep 06 01:06:03 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:36:55 2008 +0200" }, "message": "ftrace: use ftrace_release for all dynamic ftrace functions\n\nftrace_release is necessary for all uses of dynamic ftrace and not just\nthe archs that have CONFIG_FTRACE_MCOUNT_RECORD defined.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "d3ee6d992821f471193a7ee7a00af9ebb4bf5d01", "tree": "4dbd9712266848c101f249c9df7840ba784b5a31", "parents": [ "80b5e940050c273ba277acbf3a9fbc1d4441e681" ], "author": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Thu Sep 04 14:04:51 2008 +0200" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:36:51 2008 +0200" }, "message": "ftrace: make it depend on DEBUG_KERNEL\n\nmake most of the tracers depend on DEBUG_KERNEL - that\u0027s their intended\npurpose. (most distributions have DEBUG_KERNEL enabled anyway so this is\nnot a practical limitation - but it simplifies the tracing menu in the\nnormal case)\n\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "80b5e940050c273ba277acbf3a9fbc1d4441e681", "tree": "dd40f96b91fe3dc5508395cf9a559e14070499e7", "parents": [ "f09ce573f57ddc35c67b39e51f34545877b30031" ], "author": { "name": "Peter Zijlstra", "email": "peterz@infradead.org", "time": "Thu Sep 04 10:24:16 2008 +0200" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:36:48 2008 +0200" }, "message": "ftrace: sched_switch: show the wakee\u0027s cpu\n\nWhile profiling the smp behaviour of the scheduler it was needed to know to\nwhich cpu a task got woken.\n\nSigned-off-by: Peter Zijlstra \u003ca.p.zijlstra@chello.nl\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "f09ce573f57ddc35c67b39e51f34545877b30031", "tree": "776620a03aa55eecf5497b2a2a8027de6d819624", "parents": [ "5a90f577e5369a84b720ead42e621fcb1b8a8b21" ], "author": { "name": "Peter Zijlstra", "email": "peterz@infradead.org", "time": "Thu Sep 04 10:24:14 2008 +0200" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:36:45 2008 +0200" }, "message": "ftrace: make ftrace_printk usable with the other tracers\n\nCurrently ftrace_printk only works with the ftrace tracer, switch it to an\niter_ctrl setting so we can make us of them with other tracers too.\n\n[rostedt@redhat.com: tweak to the disable condition]\n\nSigned-off-by: Peter Zijlstra \u003ca.p.zijlstra@chello.nl\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "5a90f577e5369a84b720ead42e621fcb1b8a8b21", "tree": "55263f6d25613ce1ba98062ca96635aaac5dd426", "parents": [ "652567aa2000f1d4a1fd434382a30d8dd4a7c980" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Wed Sep 03 17:42:51 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:36:42 2008 +0200" }, "message": "ftrace: print continue index fix\n\nAn item in the trace buffer that is bigger than one entry may be split\nup using the TRACE_CONT entry. This makes it a virtual single entry.\nThe current code increments the iterator index even while traversing\nTRACE_CONT entries, making it look like the iterator is further than\nit actually is.\n\nThis patch adds code to not increment the iterator index while skipping\nover TRACE_CONT entries.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nAcked-by: Peter Zijlstra \u003cpeterz@infradead.org\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "652567aa2000f1d4a1fd434382a30d8dd4a7c980", "tree": "0686a5ceeeb9f6ecedf5d063d05563ac32359554", "parents": [ "a6168353d1c0b24b7512e14d1c3e47ed69881a23" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Wed Sep 03 17:42:50 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:36:39 2008 +0200" }, "message": "ftrace: binary and not logical for continue test\n\nPeter Zijlstra provided me with a nice brown paper bag while letting me know\nthat I was doing a logical AND and not a binary one, making a condition\ntrue more often than it should be.\n\nLuckily, a false true is handled by the calling function and no harm is\ndone. But this needs to be fixed regardless.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nAcked-by: Peter Zijlstra \u003cpeterz@infradead.org\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "a6168353d1c0b24b7512e14d1c3e47ed69881a23", "tree": "f019f0304ed7465b5dbe25374a62fa704196749a", "parents": [ "2ff01c6a17391225a18256d510b6e5b4aba40aa1" ], "author": { "name": "Michael Ellerman", "email": "michael@ellerman.id.au", "time": "Wed Aug 20 16:36:11 2008 -0700" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:36:34 2008 +0200" }, "message": "ftrace: make output nicely spaced for up to 999 cpus\n\nCurrently some of the ftrace output goes skewiff if you have more\nthan 9 cpus, and some if you have more than 99.\n\nTwiddle with the headers and format strings to make up to 999 cpus\ndisplay without causing spacing problems.\n\nSigned-off-by: Michael Ellerman \u003cmichael@ellerman.id.au\u003e\nAcked-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "2ff01c6a17391225a18256d510b6e5b4aba40aa1", "tree": "9645990cbdcf5009c6b901de1004c3369e44c1aa", "parents": [ "1b6cced6ec9677fa65471e890dfdcb4bf5387643" ], "author": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Thu Sep 04 15:04:37 2008 +0200" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:36:31 2008 +0200" }, "message": "stack tracer: depends on DEBUG_KERNEL\n\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "1b6cced6ec9677fa65471e890dfdcb4bf5387643", "tree": "3728f61fff36a4c2f1f41ae15e638be58437c6bc", "parents": [ "d53475b5aa946752e3306b2ecb5a8c9c51cf8dd0" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Fri Aug 29 16:51:43 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:36:28 2008 +0200" }, "message": "ftrace: stack trace add indexes\n\nThis patch adds indexes into the stack that the functions in the\nstack dump were found at. As an added bonus, I also added a diff\nto show which function is the most notorious consumer of the stack.\n\nThe output now looks like this:\n\n# cat /debug/tracing/stack_trace\n Depth Size Location (48 entries)\n ----- ---- --------\n 0) 2476 212 blk_recount_segments+0x39/0x59\n 1) 2264 12 bio_phys_segments+0x16/0x1d\n 2) 2252 20 blk_rq_bio_prep+0x23/0xaf\n 3) 2232 12 init_request_from_bio+0x74/0x77\n 4) 2220 56 __make_request+0x294/0x331\n 5) 2164 136 generic_make_request+0x34f/0x37d\n 6) 2028 56 submit_bio+0xe7/0xef\n 7) 1972 28 submit_bh+0xd1/0xf0\n 8) 1944 112 block_read_full_page+0x299/0x2a9\n 9) 1832 8 blkdev_readpage+0x14/0x16\n 10) 1824 28 read_cache_page_async+0x7e/0x109\n 11) 1796 16 read_cache_page+0x11/0x49\n 12) 1780 32 read_dev_sector+0x3c/0x72\n 13) 1748 48 read_lba+0x4d/0xaa\n 14) 1700 168 efi_partition+0x85/0x61b\n 15) 1532 72 rescan_partitions+0x10e/0x266\n 16) 1460 40 do_open+0x1c7/0x24e\n 17) 1420 292 __blkdev_get+0x79/0x84\n 18) 1128 12 blkdev_get+0x12/0x14\n 19) 1116 20 register_disk+0xd1/0x11e\n 20) 1096 28 add_disk+0x34/0x90\n 21) 1068 52 sd_probe+0x2b1/0x366\n 22) 1016 20 driver_probe_device+0xa5/0x120\n 23) 996 8 __device_attach+0xd/0xf\n 24) 988 32 bus_for_each_drv+0x3e/0x68\n 25) 956 24 device_attach+0x56/0x6c\n 26) 932 16 bus_attach_device+0x26/0x4d\n 27) 916 64 device_add+0x380/0x4b4\n 28) 852 28 scsi_sysfs_add_sdev+0xa1/0x1c9\n 29) 824 160 scsi_probe_and_add_lun+0x919/0xa2a\n 30) 664 36 __scsi_add_device+0x88/0xae\n 31) 628 44 ata_scsi_scan_host+0x9e/0x21c\n 32) 584 28 ata_host_register+0x1cb/0x1db\n 33) 556 24 ata_host_activate+0x98/0xb5\n 34) 532 192 ahci_init_one+0x9bd/0x9e9\n 35) 340 20 pci_device_probe+0x3e/0x5e\n 36) 320 20 driver_probe_device+0xa5/0x120\n 37) 300 20 __driver_attach+0x3f/0x5e\n 38) 280 36 bus_for_each_dev+0x40/0x62\n 39) 244 12 driver_attach+0x19/0x1b\n 40) 232 28 bus_add_driver+0x9c/0x1af\n 41) 204 28 driver_register+0x76/0xd2\n 42) 176 20 __pci_register_driver+0x44/0x71\n 43) 156 8 ahci_init+0x14/0x16\n 44) 148 100 _stext+0x42/0x122\n 45) 48 20 kernel_init+0x175/0x1dc\n 46) 28 28 kernel_thread_helper+0x7/0x10\n\nThe first column is simply an index starting from the inner most function\nand counting down to the outer most.\n\nThe next column is the location that the function was found on the stack.\n\nThe next column is the size of the stack for that function.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "d53475b5aa946752e3306b2ecb5a8c9c51cf8dd0", "tree": "0be25db731cdd69f23ebc18f1579af043fcf9dc7", "parents": [ "3b47bfc1fca01cccad9cce2d18b79b18ef2e4131" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Wed Aug 27 13:02:01 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:36:25 2008 +0200" }, "message": "ftrace: remove warning of old objcopy and local functions\n\nThe warning messages about old objcopy and local functions spam the\nuser quite drastically. Remove the warning until we can find a nicer\nway of tell the user to upgrade their objcopy.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nCc: Stephen Rothwell \u003csfr@canb.auug.org.au\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "3b47bfc1fca01cccad9cce2d18b79b18ef2e4131", "tree": "5be5a241139b505dcb0edf6150183096c23b47ed", "parents": [ "e5a81b629ea8feb9e7530cfac35cfb41c45facf3" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Wed Aug 27 23:24:15 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:36:22 2008 +0200" }, "message": "ftrace: remove direct reference to mcount in trace code\n\nThe mcount record method of ftrace scans objdump for references to mcount.\nUsing mcount as the reference to test if the calls to mcount being replaced\nare indeed calls to mcount, this use of mcount was also caught as a\nlocation to change. Using a variable that points to the mcount address\nmoves this reference into the data section that is not scanned, and\nwe do not use a false location to try and modify.\n\nThe warn on code was what was used to detect this bug.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "e5a81b629ea8feb9e7530cfac35cfb41c45facf3", "tree": "71437955371e0913521d4ddb02db974df9fd8d34", "parents": [ "b3a320417484a6d6b9d28098944df58341353992" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Wed Aug 27 23:31:01 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:36:19 2008 +0200" }, "message": "ftrace: add stack tracer\n\nThis is another tracer using the ftrace infrastructure, that examines\nat each function call the size of the stack. If the stack use is greater\nthan the previous max it is recorded.\n\nYou can always see (and set) the max stack size seen. By setting it\nto zero will start the recording again. The backtrace is also available.\n\nFor example:\n\n# cat /debug/tracing/stack_max_size\n1856\n\n# cat /debug/tracing/stack_trace\n[\u003cc027764d\u003e] stack_trace_call+0x8f/0x101\n[\u003cc021b966\u003e] ftrace_call+0x5/0x8\n[\u003cc02553cc\u003e] clocksource_get_next+0x12/0x48\n[\u003cc02542a5\u003e] update_wall_time+0x538/0x6d1\n[\u003cc0245913\u003e] do_timer+0x23/0xb0\n[\u003cc0257657\u003e] tick_do_update_jiffies64+0xd9/0xf1\n[\u003cc02576b9\u003e] tick_sched_timer+0x4a/0xad\n[\u003cc0250fe6\u003e] __run_hrtimer+0x3e/0x75\n[\u003cc02518ed\u003e] hrtimer_interrupt+0xf1/0x154\n[\u003cc022c870\u003e] smp_apic_timer_interrupt+0x71/0x84\n[\u003cc021b7e9\u003e] apic_timer_interrupt+0x2d/0x34\n[\u003cc0238597\u003e] finish_task_switch+0x29/0xa0\n[\u003cc05abd13\u003e] schedule+0x765/0x7be\n[\u003cc05abfca\u003e] schedule_timeout+0x1b/0x90\n[\u003cc05ab4d4\u003e] wait_for_common+0xab/0x101\n[\u003cc05ab5ac\u003e] wait_for_completion+0x12/0x14\n[\u003cc033cfc3\u003e] blk_execute_rq+0x84/0x99\n[\u003cc0402470\u003e] scsi_execute+0xc2/0x105\n[\u003cc040250a\u003e] scsi_execute_req+0x57/0x7f\n[\u003cc043afe0\u003e] sr_test_unit_ready+0x3e/0x97\n[\u003cc043bbd6\u003e] sr_media_change+0x43/0x205\n[\u003cc046b59f\u003e] media_changed+0x48/0x77\n[\u003cc046b5ff\u003e] cdrom_media_changed+0x31/0x37\n[\u003cc043b091\u003e] sr_block_media_changed+0x16/0x18\n[\u003cc02b9e69\u003e] check_disk_change+0x1b/0x63\n[\u003cc046f4c3\u003e] cdrom_open+0x7a1/0x806\n[\u003cc043b148\u003e] sr_block_open+0x78/0x8d\n[\u003cc02ba4c0\u003e] do_open+0x90/0x257\n[\u003cc02ba869\u003e] blkdev_open+0x2d/0x56\n[\u003cc0296a1f\u003e] __dentry_open+0x14d/0x23c\n[\u003cc0296b32\u003e] nameidata_to_filp+0x24/0x38\n[\u003cc02a1c68\u003e] do_filp_open+0x347/0x626\n[\u003cc02967ef\u003e] do_sys_open+0x47/0xbc\n[\u003cc02968b0\u003e] sys_open+0x23/0x2b\n[\u003cc021aadd\u003e] sysenter_do_call+0x12/0x26\n\nI\u0027ve tested this on both x86_64 and i386.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nCc: Peter Zijlstra \u003cpeterz@infradead.org\u003e\nCc: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nCc: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "b3a320417484a6d6b9d28098944df58341353992", "tree": "c2b408434feb6ddf7547b19d85ad5f5c2649d6de", "parents": [ "f2f8458e751f9ae41dfec3c00a46d3e62dc38f60" ], "author": { "name": "Andrew Morton", "email": "akpm@linux-foundation.org", "time": "Wed Aug 27 09:08:30 2008 +0200" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:36:16 2008 +0200" }, "message": "kbuild: ftrace: don\u0027t assume that scripts/recordmcount.pl is executable\n\nCHK include/linux/version.h\n CHK include/linux/utsrelease.h\n CC scripts/mod/empty.o\n/bin/sh: /usr/src/25/scripts/recordmcount.pl: Permission denied\n\nWe shouldn\u0027t assume that files have their `x\u0027 bits set. There are various\nways in which file permissions get lost, including use of patch(1).\n\nIt might not be correct to assume that perl lives in $PATH?\n\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "f2f8458e751f9ae41dfec3c00a46d3e62dc38f60", "tree": "c16c97b3891790c9ffedc59579876510a8352ae4", "parents": [ "ac8825ec6d941b6899331b84c7d6bf027c3bb4f1" ], "author": { "name": "Steven Rostedt", "email": "srostedt@redhat.com", "time": "Mon Aug 25 14:52:11 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:36:12 2008 +0200" }, "message": "ftrace: objcopy version test for local symbols\n\nThe --globalize-symbols option came out in objcopy version 2.17.\nIf the kernel is being compiled on a system with a lower version of\nobjcopy, then we can not use the globalize / localize trick to\nlink to symbols pointing to local functions.\n\nThis patch tests the version of objcopy and will only use the trick\nif the version is greater than or equal to 2.17. Otherwise, if an\nobject has only local functions within a section, it will give a\nnice warning and recommend the user to upgrade their objcopy.\n\nLeaving the symbols unrecorded is not that big of a deal, since the\nmcount record method changes the actual mcount code to be a simple\n\"ret\" without recording registers or anything.\n\nReported-by: Stephen Rothwell \u003csfr@canb.auug.org.au\u003e\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "ac8825ec6d941b6899331b84c7d6bf027c3bb4f1", "tree": "3c8673f34f88e9438405216416ad22af8b9e4998", "parents": [ "2d7da80f7138c4276ef4fa0334be400b805d0fbf" ], "author": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Mon Aug 25 08:12:04 2008 +0200" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:36:09 2008 +0200" }, "message": "ftrace: clean up macro usage\n\nenclose the argument in parenthesis. (especially since we cast it,\nwhich is a high prio operation)\n\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "2d7da80f7138c4276ef4fa0334be400b805d0fbf", "tree": "2c2c21758498c989cf22414bf66bff1093edd263", "parents": [ "6f93fc076a464bfe24e8d4c5fea3f6ca5bdb264d" ], "author": { "name": "Stephen Rothwell", "email": "sfr@canb.auug.org.au", "time": "Mon Aug 25 13:08:44 2008 +1000" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:36:06 2008 +0200" }, "message": "ftrace: fix build failure\n\nAfter disabling FTRACE_MCOUNT_RECORD via a patch, a dormant build\nfailure surfaced:\n\n kernel/trace/ftrace.c: In function \u0027ftrace_record_ip\u0027:\n kernel/trace/ftrace.c:416: error: incompatible type for argument 1 of \u0027_spin_lock_irqsave\u0027\n kernel/trace/ftrace.c:433: error: incompatible type for argument 1 of \u0027_spin_lock_irqsave\u0027\n\nIntroduced by commit 6dad8e07f4c10b17b038e84d29f3ca41c2e55cd0 (\"ftrace:\nadd necessary locking for ftrace records\").\n\nSigned-off-by: Stephen Rothwell \u003csfr@canb.auug.org.au\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "6f93fc076a464bfe24e8d4c5fea3f6ca5bdb264d", "tree": "06dd8772efe5be7432e2191761744eb3d8d9c9bc", "parents": [ "8feff1cacc29e9cfdc6d1ce5f2108db87b91046e" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Wed Aug 20 12:55:07 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:36:03 2008 +0200" }, "message": "ftrace: x86 use copy to and from user functions\n\nThe modification of code is performed either by kstop_machine, before\nSMP starts, or on module code before the module is executed. There is\nno reason to do the modifications from assembly. The copy to and from\nuser functions are sufficient and produces cleaner and easier to read\ncode.\n\nThanks to Benjamin Herrenschmidt for suggesting the idea.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "8feff1cacc29e9cfdc6d1ce5f2108db87b91046e", "tree": "b8670f1cd683039263f6f1f98962c8a6ed4d6ad6", "parents": [ "d74fcd1e4e8842d5302cd303ef25cef7e67f68b4" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Wed Aug 20 10:07:35 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:36:00 2008 +0200" }, "message": "ftrace: handle weak symbol functions\n\nDuring tests and checks, I\u0027ve discovered that there were failures to\nconvert mcount callers into nops. Looking deeper into these failures,\ncode that was attempted to be changed was not an mcount caller.\nThe current code only updates if the code being changed is what it expects,\nbut I still investigate any time there is a failure.\n\nWhat was happening is that a weak symbol was being used as a reference\nfor other mcount callers. That weak symbol was also referenced elsewhere\nso the offsets were using the strong symbol and not the function symbol\nthat it was referenced from.\n\nThis patch changes the setting up of the mcount_loc section to search\nfor a global function that is not weak. It will pick a local over a weak\nbut if only a weak is found in a section, a warning is printed and the\nmcount location is not recorded (just to be safe).\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "d74fcd1e4e8842d5302cd303ef25cef7e67f68b4", "tree": "71b65e0b54f918ec47f8d9813a00c0d3d47e39fd", "parents": [ "6a4917e3ae5194a10e0723f96edc854c381e3063" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Fri Aug 15 11:40:24 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:35:56 2008 +0200" }, "message": "ftrace: update recordmount.pl arch changes\n\nI\u0027m trying to keep all the arch changes in recordmcount.pl in one place.\nI moved your code into that area, by adding the flags to the commands\nthat were passed in.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "6a4917e3ae5194a10e0723f96edc854c381e3063", "tree": "8aaefd805a3e4cc6df2050838d1ae44c5a8f9ca5", "parents": [ "3700273586ee6a58b95dd07d9f8a02db4a9b476f" ], "author": { "name": "Jeremy Fitzhardinge", "email": "jeremy@goop.org", "time": "Mon Aug 18 15:58:12 2008 -0700" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:35:53 2008 +0200" }, "message": "ftrace: fix build problem with CONFIG_FTRACE\n\nI\u0027m seeing when I use separate src/build dirs:\n\nmake[3]: *** [arch/x86/kernel/time_32.o] Error 1\n/bin/sh: scripts/recordmcount.pl: No such file or directory\nmake[3]: *** [arch/x86/kernel/irq_32.o] Error 1\n/bin/sh: scripts/recordmcount.pl: No such file or directory\nmake[3]: *** [arch/x86/kernel/ldt.o] Error 1\n/bin/sh: scripts/recordmcount.pl: No such file or directory\nmake[3]: *** [arch/x86/kernel/i8259.o] Error 1\n/bin/sh: scripts/recordmcount.pl: No such file or directory\n\nThis fixes it.\n\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "3700273586ee6a58b95dd07d9f8a02db4a9b476f", "tree": "f291a04bd0ab0d866c00c3f2d241a870b873ec25", "parents": [ "99ecdc43bc17faf5fa571db8569df171ecd0e5b8" ], "author": { "name": "Huang Ying", "email": "ying.huang@intel.com", "time": "Mon Aug 18 16:24:56 2008 +0800" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:35:50 2008 +0200" }, "message": "ftrace: fix incorrect comment style of __ftrace_enabled_save()\n\nThis patch fixes incorrect comment style of __ftrace_enabled_save().\n\nSigned-off-by: Huang Ying \u003cying.huang@intel.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "99ecdc43bc17faf5fa571db8569df171ecd0e5b8", "tree": "aedefb2859247aefe7538e479ab71bcd3a1717c8", "parents": [ "00fd61aee10533e003f2f00ab7163207660a4051" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Fri Aug 15 21:40:05 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:35:47 2008 +0200" }, "message": "ftrace: add necessary locking for ftrace records\n\nThe new design of pre-recorded mcounts and updating the code outside of\nkstop_machine has changed the way the records themselves are protected.\n\nThis patch uses the ftrace_lock to protect the records. Note, the lock\nstill does not need to be taken within calls that are only called via\nkstop_machine, since the that code can not run while the spin lock is held.\n\nAlso removed the hash_lock needed for the daemon when MCOUNT_RECORD is\nconfigured. Also did a slight cleanup of an unused variable.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "00fd61aee10533e003f2f00ab7163207660a4051", "tree": "72f42d3a4b238d84e3697c53b31b1ba9f8d022cb", "parents": [ "98a983aad2e5b3dc83a8a761675445cdd8f3e6bd" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Fri Aug 15 21:40:04 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:35:43 2008 +0200" }, "message": "ftrace: do not init module on ftrace disabled\n\nIf one of the self tests of ftrace has disabled the function tracer,\ndo not run the code to convert the mcount calls in modules.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "98a983aad2e5b3dc83a8a761675445cdd8f3e6bd", "tree": "f207a80453ff556fa1ddcb7eef4f17485a647161", "parents": [ "3989cce82b272bd6312555fcc47c11715c157102" ], "author": { "name": "Frédéric Weisbecker", "email": "fweisbec@gmail.com", "time": "Fri Aug 15 21:08:22 2008 +0200" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:35:40 2008 +0200" }, "message": "ftrace: fix some mistakes in error messages\n\nThis patch fixes some mistakes on the tracer in warning messages when\ndebugfs fails to create tracing files.\n\nSigned-off-by: Frederic Weisbecker \u003cfweisbec@gmail.com\u003e\nCc: srostedt@redhat.com\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "3989cce82b272bd6312555fcc47c11715c157102", "tree": "6c49985859e25991268317b23cc4260ce459134a", "parents": [ "c5131ad6c3cbe8f6674993e29a76cecf8deb4384" ], "author": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Mon Jun 09 20:54:22 2008 +0200" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:35:36 2008 +0200" }, "message": "ftrace: scripts/recordmcount.pl cross-build hack\n\nhack around:\n\n ld: Relocatable linking with relocations from format elf32-i386 (init/.tmp_gl_calibrate.o) to format elf64-x86-64 (init/.tmp_mx_calibrate.o) i CC arch/x86/mm/extable.o\n objcopy: \u0027init/.tmp_mx_calibrate.o\u0027: No such file\n rm: cannot remove `init/.tmp_mx_calibrate.o\u0027: No such file or directory\n ld: Relocatable linking with relocations from format elf32-i386 (arch/x86/mm/extable.o) to format elf64-x86-64 (arch/x86/mm/.tmp_mx_extable.o) is not supported\n mv: cannot stat `arch/x86/mm/.tmp_mx_extable.o\u0027: No such file or directory\n ld: Relocatable linking with relocations from format elf32-i386 (arch/x86/mm/fault.o) to format elf64-x86-64 (arch/x86/mm/.tmp_mx_fault.o) is not supported\n\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "c5131ad6c3cbe8f6674993e29a76cecf8deb4384", "tree": "ce9f9cd79cd987295d0cbcbb3cf99ada9b29fc1a", "parents": [ "7b928c23fa3e9fa37d1d4ba52ba963f41ee5aae0" ], "author": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Fri Aug 15 18:22:09 2008 +0200" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:35:33 2008 +0200" }, "message": "ftrace: ftrace_kill_atomic() build fix\n\nfix:\n\n kernel/built-in.o: In function `ftrace_dump\u0027:\n (.text+0x2e2ea): undefined reference to `ftrace_kill_atomic\u0027\n\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "7b928c23fa3e9fa37d1d4ba52ba963f41ee5aae0", "tree": "a8873d531424a53b7c5f3a7cd4e280c50ca2e2a8", "parents": [ "3f5a54e371ca20b119b73704f6c01b71295c1714" ], "author": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Fri Aug 15 17:48:02 2008 +0200" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:35:29 2008 +0200" }, "message": "ftrace: build fix\n\nfix:\n\n In file included from init/main.c:65:\n include/linux/ftrace.h:166: error: expected ‘,\u0027 or ‘;\u0027 before ‘{\u0027 token\n make[1]: *** [init/main.o] Error 1\n make: *** [init/main.o] Error 2\n\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "3f5a54e371ca20b119b73704f6c01b71295c1714", "tree": "1453315564a19e967f11e612e264d4d2989afbe5", "parents": [ "2f2c99dba2398ef7d9c21f7c793180a50e68b1f0" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Wed Jul 30 22:36:46 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:35:26 2008 +0200" }, "message": "ftrace: dump out ftrace buffers to console on panic\n\nAt OLS I had a lot of interest to be able to have the ftrace buffers\ndumped on panic. Usually one would expect to uses kexec and examine\nthe buffers after a new kernel is loaded. But sometimes the resources\ndo not permit kdump and kexec, so having an option to still see the\nsequence of events up to the crash is very advantageous.\n\nThis patch adds the option to have the ftrace buffers dumped to the\nconsole in the latency_trace format on a panic. When the option is set,\nthe default entries per CPU buffer are lowered to 16384, since the writing\nto the serial (if that is the console) may take an awful long time\notherwise.\n\n[\n Changes since -v1:\n Got alpine to send correctly (as well as spell check working).\n Removed config option.\n Moved the static variables into ftrace_dump itself.\n Gave printk a log level.\n]\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nCc: Peter Zijlstra \u003cpeterz@infradead.org\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "2f2c99dba2398ef7d9c21f7c793180a50e68b1f0", "tree": "4948b0cd7b9828aa19f6862b10173cbc9f59ea68", "parents": [ "dd0e545f061f90099a3dcc13aa77e29c6295cf23" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Fri Aug 01 16:45:49 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:35:22 2008 +0200" }, "message": "ftrace: ftrace_printk doc moved\n\nBased on Randy Dunlap\u0027s suggestion, the ftrace_printk kernel-doc belongs\nwith the ftrace_printk macro that should be used. Not with the\n__ftrace_printk internal function.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nAcked-by: Randy Dunlap \u003crandy.dunlap@oracle.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "dd0e545f061f90099a3dcc13aa77e29c6295cf23", "tree": "91842e81cacd2ae785bf14b96f3f65af96985658", "parents": [ "2e2ca155cd2213b4f398031180fb3d399d5b7db9" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Fri Aug 01 12:26:41 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:35:19 2008 +0200" }, "message": "ftrace: printk formatting infrastructure\n\nThis patch adds a feature that can help kernel developers debug their\ncode using ftrace.\n\n int ftrace_printk(const char *fmt, ...);\n\nThis records into the ftrace buffer using printf formatting. The entry\nsize in the buffers are still a fixed length. A new type has been added\nthat allows for more entries to be used for a single recording.\n\nThe start of the print is still the same as the other entries.\n\nIt returns the number of characters written to the ftrace buffer.\n\nFor example:\n\nHaving a module with the following code:\n\nstatic int __init ftrace_print_test(void)\n{\n ftrace_printk(\"jiffies are %ld\\n\", jiffies);\n return 0;\n}\n\nGives me:\n\n insmod-5441 3...1 7569us : ftrace_print_test: jiffies are 4296626666\n\nfor the latency_trace file and:\n\n insmod-5441 [03] 1959.370498: ftrace_print_test jiffies are 4296626666\n\nfor the trace file.\n\nNote: Only the infrastructure should go into the kernel. It is to help\nfacilitate debugging for other kernel developers. Calls to ftrace_printk\nis not intended to be left in the kernel, and should be frowned upon just\nlike scattering printks around in the code.\n\nBut having this easily at your fingertips helps the debugging go faster\nand bugs be solved quicker.\n\nMaybe later on, we can hook this with markers and have their printf format\nbe sucked into ftrace output.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "2e2ca155cd2213b4f398031180fb3d399d5b7db9", "tree": "027295832950e23ac4edc84e43550d02e9a1acc6", "parents": [ "fed1939c64d2288938fdc1c367d49082da65e195" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Fri Aug 01 12:26:40 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:35:15 2008 +0200" }, "message": "ftrace: new continue entry - separate out from trace_entry\n\nSome tracers will need to work with more than one entry. In order to do this\nthe trace_entry structure was split into two fields. One for the start of\nall entries, and one to continue an existing entry.\n\nThe trace_entry structure now has a \"field\" entry that consists of the previous\ncontent of the trace_entry, and a \"cont\" entry that is just a string buffer\nthe size of the \"field\" entry.\n\nThanks to Andrew Morton for suggesting this idea.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "fed1939c64d2288938fdc1c367d49082da65e195", "tree": "a4592352d28efcfe82379c71d061b9127e49a115", "parents": [ "28614889bcb2558a47d02d52394b7fd9795a9547" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Thu Aug 14 22:47:19 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:35:12 2008 +0200" }, "message": "ftrace: remove old pointers to mcount\n\nWhen a mcount pointer is recorded into a table, it is used to add or\nremove calls to mcount (replacing them with nops). If the code is removed\nvia removing a module, the pointers still exist. At modifying the code\na check is always made to make sure the code being replaced is the code\nexpected. In-other-words, the code being replaced is compared to what\nit is expected to be before being replaced.\n\nThere is a very small chance that the code being replaced just happens\nto look like code that calls mcount (very small since the call to mcount\nis relative). To remove this chance, this patch adds ftrace_release to\nallow module unloading to remove the pointers to mcount within the module.\n\nAnother change for init calls is made to not trace calls marked with\n__init. The tracing can not be started until after init is done anyway.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "28614889bcb2558a47d02d52394b7fd9795a9547", "tree": "1b9e3b91c6db5927a6e53f5f1175aa25b27c4a9f", "parents": [ "a9fdda33cd7c7519b082e37538fe790f9ff684bb" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Thu Aug 14 22:47:18 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:35:08 2008 +0200" }, "message": "ftrace: move notrace to compiler.h\n\nThe notrace define belongs in compiler.h so that it can be used in\ninit.h\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "a9fdda33cd7c7519b082e37538fe790f9ff684bb", "tree": "56d16d69e519e127a2090df6a16aa9029e8c0e05", "parents": [ "732f3ca7d4ba3c1be8d051d52302ef441ee7748b" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Thu Aug 14 22:47:17 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:35:05 2008 +0200" }, "message": "ftrace: do not show freed records in available_filter_functions\n\nSeems that freed records can appear in the available_filter_functions list.\nThis patch fixes that.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "732f3ca7d4ba3c1be8d051d52302ef441ee7748b", "tree": "503f2c44e3a1617cb4aa45454071cb52330574b9", "parents": [ "0a37605c2261a06d8cafc62dee11374ad676c8c4" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Thu Aug 14 18:05:05 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:35:01 2008 +0200" }, "message": "ftrace: use only 5 byte nops for x86\n\nMathieu Desnoyers revealed a bug in the original code. The nop that is\nused to relpace the mcount caller can be a two part nop. This runs the\nrisk where a process can be preempted after executing the first nop, but\nbefore the second part of the nop.\n\nThe ftrace code calls kstop_machine to keep multiple CPUs from executing\ncode that is being modified, but it does not protect against a task preempting\nin the middle of a two part nop.\n\nIf the above preemption happens and the tracer is enabled, after the\nkstop_machine runs, all those nops will be calls to the trace function.\nIf the preempted process that was preempted between the two nops is executed\nagain, it will execute half of the call to the trace function, and this\nmight crash the system.\n\nThis patch instead uses what both the latest Intel and AMD spec suggests.\nThat is the P6_NOP5 sequence of \"0x0f 0x1f 0x44 0x00 0x00\".\n\nNote, some older CPUs and QEMU might fault on this nop, so this nop\nis executed with fault handling first. If it detects a fault, it will then\nuse the code \"0x66 0x66 0x66 0x66 0x90\". If that faults, it will then\ndefault to a simple \"jmp 1f; .byte 0x00 0x00 0x00; 1:\". The jmp is\nnot optimal but will do if the first two can not be executed.\n\nTODO: Examine the cpuid to determine the nop to use.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "0a37605c2261a06d8cafc62dee11374ad676c8c4", "tree": "0867723688dbdb824a5a15f91551f88bd47c6936", "parents": [ "e4b2b8866121bd06d5f3d9de0f79a04339ffa252" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Thu Aug 14 15:45:12 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:34:58 2008 +0200" }, "message": "ftrace: x86 mcount stub\n\nx86 now sets up the mcount locations through the build and no longer\nneeds to record the ip when the function is executed. This patch changes\nthe initial mcount to simply return. There\u0027s no need to do any other work.\nIf the ftrace start up test fails, the original mcount will be what everything\nwill use, so having this as fast as possible is a good thing.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "e4b2b8866121bd06d5f3d9de0f79a04339ffa252", "tree": "4fc7408e20bbe742adaabcf827ace8c4266f1263", "parents": [ "29e71abf56cebc5c5a4e184a6eb4360cc58554ad" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Thu Aug 14 15:45:11 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:34:54 2008 +0200" }, "message": "ftrace: enable using mcount recording on x86\n\nEnable the use of the __mcount_loc infrastructure on x86_64 and i386.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "29e71abf56cebc5c5a4e184a6eb4360cc58554ad", "tree": "2e86f88ce888e15c3d5aa007795092194a0f24c8", "parents": [ "90d595fe5ca4b685465c068907e6e554760abea8" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Thu Aug 14 15:45:10 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:34:51 2008 +0200" }, "message": "ftrace: rebuild everything on change to FTRACE_MCOUNT_RECORD\n\nWhen enabling or disabling CONFIG_FTRACE_MCOUNT_RECORD, we want a full\nkernel compile to handle the adding of the __mcount_loc sections.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "90d595fe5ca4b685465c068907e6e554760abea8", "tree": "03f98454af8c6756177aa053ae7440373007f67d", "parents": [ "68bf21aa15c85d2e9b623dcda2b1ed8893275fa1" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Thu Aug 14 15:45:09 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:34:47 2008 +0200" }, "message": "ftrace: enable mcount recording for modules\n\nThis patch enables the loading of the __mcount_section of modules and\nchanging all the callers of mcount into nops.\n\nThe modification is done before the init_module function is called, so\nagain, we do not need to use kstop_machine to make these changes.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "68bf21aa15c85d2e9b623dcda2b1ed8893275fa1", "tree": "54d4d4f7e8311b77f3166a8807a635347d479081", "parents": [ "8da3821ba5634497da63d58a69e24a97697c4a2b" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Thu Aug 14 15:45:08 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:34:44 2008 +0200" }, "message": "ftrace: mcount call site on boot nops core\n\nThis is the infrastructure to the converting the mcount call sites\nrecorded by the __mcount_loc section into nops on boot. It also allows\nfor using these sites to enable tracing as normal. When the __mcount_loc\nsection is used, the \"ftraced\" kernel thread is disabled.\n\nThis uses the current infrastructure to record the mcount call sites\nas well as convert them to nops. The mcount function is kept as a stub\non boot up and not converted to the ftrace_record_ip function. We use the\nftrace_record_ip to only record from the table.\n\nThis patch does not handle modules. That comes with a later patch.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "8da3821ba5634497da63d58a69e24a97697c4a2b", "tree": "47d4d097b1cd2aafae2e3596a9ae0d1293228782", "parents": [ "8b1fa1d7b22f386747c7b78b918d4c680c16066f" ], "author": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Thu Aug 14 15:45:07 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:34:40 2008 +0200" }, "message": "ftrace: create __mcount_loc section\n\nThis patch creates a section in the kernel called \"__mcount_loc\".\nThis will hold a list of pointers to the mcount relocation for\neach call site of mcount.\n\nFor example:\n\nobjdump -dr init/main.o\n[...]\nDisassembly of section .text:\n\n0000000000000000 \u003cdo_one_initcall\u003e:\n 0: 55 push %rbp\n[...]\n000000000000017b \u003cinit_post\u003e:\n 17b: 55 push %rbp\n 17c: 48 89 e5 mov %rsp,%rbp\n 17f: 53 push %rbx\n 180: 48 83 ec 08 sub $0x8,%rsp\n 184: e8 00 00 00 00 callq 189 \u003cinit_post+0xe\u003e\n 185: R_X86_64_PC32 mcount+0xfffffffffffffffc\n[...]\n\nWe will add a section to point to each function call.\n\n .section __mcount_loc,\"a\",@progbits\n[...]\n .quad .text + 0x185\n[...]\n\nThe offset to of the mcount call site in init_post is an offset from\nthe start of the section, and not the start of the function init_post.\nThe mcount relocation is at the call site 0x185 from the start of the\n.text section.\n\n .text + 0x185 \u003d\u003d init_post + 0xa\n\nWe need a way to add this __mcount_loc section in a way that we do not\nlose the relocations after final link. The .text section here will\nbe attached to all other .text sections after final link and the\noffsets will be meaningless. We need to keep track of where these\n.text sections are.\n\nTo do this, we use the start of the first function in the section.\ndo_one_initcall. We can make a tmp.s file with this function as a reference\nto the start of the .text section.\n\n .section __mcount_loc,\"a\",@progbits\n[...]\n .quad do_one_initcall + 0x185\n[...]\n\nThen we can compile the tmp.s into a tmp.o\n\n gcc -c tmp.s -o tmp.o\n\nAnd link it into back into main.o.\n\n ld -r main.o tmp.o -o tmp_main.o\n mv tmp_main.o main.o\n\nBut we have a problem. What happens if the first function in a section\nis not exported, and is a static function. The linker will not let\nthe tmp.o use it. This case exists in main.o as well.\n\nDisassembly of section .init.text:\n\n0000000000000000 \u003cset_reset_devices\u003e:\n 0: 55 push %rbp\n 1: 48 89 e5 mov %rsp,%rbp\n 4: e8 00 00 00 00 callq 9 \u003cset_reset_devices+0x9\u003e\n 5: R_X86_64_PC32 mcount+0xfffffffffffffffc\n\nThe first function in .init.text is a static function.\n\n00000000000000a8 t __setup_set_reset_devices\n000000000000105f t __setup_str_set_reset_devices\n0000000000000000 t set_reset_devices\n\nThe lowercase \u0027t\u0027 means that set_reset_devices is local and is not exported.\nIf we simply try to link the tmp.o with the set_reset_devices we end\nup with two symbols: one local and one global.\n\n .section __mcount_loc,\"a\",@progbits\n .quad set_reset_devices + 0x10\n\n00000000000000a8 t __setup_set_reset_devices\n000000000000105f t __setup_str_set_reset_devices\n0000000000000000 t set_reset_devices\n U set_reset_devices\n\nWe still have an undefined reference to set_reset_devices, and if we try\nto compile the kernel, we will end up with an undefined reference to\nset_reset_devices, or even worst, it could be exported someplace else,\nand then we will have a reference to the wrong location.\n\nTo handle this case, we make an intermediate step using objcopy.\nWe convert set_reset_devices into a global exported symbol before linking\nit with tmp.o and set it back afterwards.\n\n00000000000000a8 t __setup_set_reset_devices\n000000000000105f t __setup_str_set_reset_devices\n0000000000000000 T set_reset_devices\n\n00000000000000a8 t __setup_set_reset_devices\n000000000000105f t __setup_str_set_reset_devices\n0000000000000000 T set_reset_devices\n\n00000000000000a8 t __setup_set_reset_devices\n000000000000105f t __setup_str_set_reset_devices\n0000000000000000 t set_reset_devices\n\nNow we have a section in main.o called __mcount_loc that we can place\nsomewhere in the kernel using vmlinux.ld.S and access it to convert\nall these locations that call mcount into nops before starting SMP\nand thus, eliminating the need to do this with kstop_machine.\n\nNote, A well documented perl script (scripts/recordmcount.pl) is used\nto do all this in one location.\n\nSigned-off-by: Steven Rostedt \u003csrostedt@redhat.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "8b1fa1d7b22f386747c7b78b918d4c680c16066f", "tree": "9cd3deebefd072adc9ee4b79d32076db2b53a033", "parents": [ "36dcd67ae994fece615b7c700958d215e884b9ae" ], "author": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Jul 29 12:36:02 2008 +0200" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:34:36 2008 +0200" }, "message": "ftrace: mark lapic_wd_event() notrace\n\nit can be called in the NMI path:\n\n[ 0.645999] calling ftrace_dynamic_init+0x0/0xd6\n[ 0.647521] ------------[ cut here ]------------\n[ 0.647521] WARNING: at kernel/trace/ftrace.c:348 ftrace_record_ip+0x4e/0x252()\n[ 0.647521] Modules linked in:\n[ 0.647521] Pid: 15, comm: kstop1 Not tainted 2.6.27-rc1-tip #22686\n[ 0.647521]\n[ 0.647521] Call Trace:\n[ 0.647521] \u003cNMI\u003e [\u003cffffffff8024593f\u003e] warn_on_slowpath+0x5d/0x84\n[ 0.647521] [\u003cffffffff80220b99\u003e] ? lapic_wd_event+0xb/0x5c\n[ 0.647521] [\u003cffffffff80287b3b\u003e] ftrace_record_ip+0x4e/0x252\n[ 0.647521] [\u003cffffffff80211274\u003e] mcount_call+0x5/0x31\n[ 0.647521] [\u003cffffffff80220b9e\u003e] ? lapic_wd_event+0x10/0x5c\n[ 0.647521] [\u003cffffffff8083f3ec\u003e] nmi_watchdog_tick+0x19d/0x1ad\n[ 0.647521] [\u003cffffffff8083e875\u003e] default_do_nmi+0x75/0x1e3\n[ 0.647521] [\u003cffffffff8083f0b3\u003e] do_nmi+0x5d/0x94\n[ 0.647521] [\u003cffffffff8083e2d2\u003e] nmi+0xa2/0xc2\n[ 0.647521] [\u003cffffffff802b48c3\u003e] ? check_bytes_and_report+0x11/0xcc\n[ 0.647521] \u003c\u003cEOE\u003e\u003e [\u003cffffffff80211274\u003e] ? mcount_call+0x5/0x31\n[ 0.647521] [\u003cffffffff802b49df\u003e] check_object+0x61/0x1b0\n[ 0.647521] [\u003cffffffff802b502a\u003e] __slab_free+0x169/0x2ae\n[ 0.647521] [\u003cffffffff80242dbf\u003e] ? __cleanup_sighand+0x25/0x27\n[ 0.647521] [\u003cffffffff80242dbf\u003e] ? __cleanup_sighand+0x25/0x27\n[ 0.647521] [\u003cffffffff802b60cd\u003e] kmem_cache_free+0x85/0xb9\n[ 0.647521] [\u003cffffffff80242dbf\u003e] __cleanup_sighand+0x25/0x27\n[ 0.647521] [\u003cffffffff80247b3d\u003e] release_task+0x256/0x339\n[ 0.647521] [\u003cffffffff802490b4\u003e] do_exit+0x764/0x7ef\n[ 0.647521] [\u003cffffffff8027624c\u003e] __xchg+0x0/0x38\n[ 0.647521] [\u003cffffffff8027619a\u003e] ? stop_cpu+0x0/0xb2\n[ 0.647521] [\u003cffffffff8027619a\u003e] ? stop_cpu+0x0/0xb2\n[ 0.647521] [\u003cffffffff8025922f\u003e] kthread+0x4e/0x7b\n[ 0.647521] [\u003cffffffff80212979\u003e] child_rip+0xa/0x11\n[ 0.647521] [\u003cffffffff80211c17\u003e] ? restore_args+0x0/0x30\n[ 0.647521] [\u003cffffffff802283a5\u003e] ? native_load_tls+0x14/0x2e\n[ 0.647521] [\u003cffffffff802591e1\u003e] ? kthread+0x0/0x7b\n[ 0.647521] [\u003cffffffff8021296f\u003e] ? child_rip+0x0/0x11\n[ 0.647521]\n[ 0.647521] ---[ end trace 4eaa2a86a8e2da22 ]---\n[ 0.672032] initcall ftrace_dynamic_init+0x0/0xd6 returned 0 after 19 msecs\n\nalso mark it no-kprobes while at it.\n\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "36dcd67ae994fece615b7c700958d215e884b9ae", "tree": "2a4bed0cb9de673b7760b81f9fe03fd5f262c77e", "parents": [ "9795302acf2817d0842e56d23df6008e43df0970" ], "author": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Jul 29 12:00:59 2008 +0200" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:34:22 2008 +0200" }, "message": "ftrace: ignore functions that cannot be kprobe-ed\n\nkprobes already has an extensive list of annotations for functions\nthat should not be instrumented. Add notrace annotations to these\nfunctions as well.\n\nThis is particularly useful for functions called by the NMI path.\n\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "9795302acf2817d0842e56d23df6008e43df0970", "tree": "de2219d52413770567e656b8e6e6b4da7fbf6fca", "parents": [ "611b1597680dd4a57896bcca1af0484be463c55e" ], "author": { "name": "Mathieu Desnoyers", "email": "mathieu.desnoyers@polymtl.ca", "time": "Thu Jul 24 16:37:23 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:34:07 2008 +0200" }, "message": "tracepoints: use TABLE_SIZE macro\n\nSteven Rostedt suggested:\n\n| Wouldn\u0027t it look nicer to have: (TRACEPOINT_TABLE_SIZE - 1) ?\n\nSigned-off-by: Mathieu Desnoyers \u003cmathieu.desnoyers@polymtl.ca\u003e\nCc: Steven Rostedt \u003crostedt@goodmis.org\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "611b1597680dd4a57896bcca1af0484be463c55e", "tree": "29356a233cad80a0bcf8c42c966b782726ac9892", "parents": [ "5f87f1121895dc09d2d1c1db5f14af6aa4ce3e94" ], "author": { "name": "Pekka Paalanen", "email": "pq@iki.fi", "time": "Mon Jul 21 18:49:56 2008 +0300" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:33:50 2008 +0200" }, "message": "x86: fix mmiotrace 8-bit register decoding\n\nWhen SIL, DIL, BPL or SPL registers were used in MMIO, the datum\nwas extracted from AH, BH, CH, or DH, which are incorrect.\n\nSigned-off-by: Pekka Paalanen \u003cpq@iki.fi\u003e\nCc: \"Vegard Nossum\" \u003cvegard.nossum@gmail.com\u003e\nCc: \"Steven Rostedt\" \u003csrostedt@redhat.com\u003e\nCc: proski@gnu.org\nCc: \"Pekka Enberg\"\n\t\u003cpenberg@cs.helsinki.fi\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "5f87f1121895dc09d2d1c1db5f14af6aa4ce3e94", "tree": "4b8354ab2dc0890d0d4c91d42ff9e13db50ebeab", "parents": [ "cf569a932217b97e2fc2c48aa597fe29519a0cff" ], "author": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Wed Jul 23 14:15:22 2008 +0200" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:33:32 2008 +0200" }, "message": "tracing: clean up tracepoints kconfig structure\n\ndo not expose users to CONFIG_TRACEPOINTS - tracers can select it\njust fine.\n\nupdate ftrace to select CONFIG_TRACEPOINTS.\n\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "cf569a932217b97e2fc2c48aa597fe29519a0cff", "tree": "85be252713acf89df2b63dd6f5169b7ae1b13ae2", "parents": [ "fa340d9c050e78fb21a142b617304214ae5e0c2d" ], "author": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Wed Jul 23 13:48:22 2008 +0200" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:33:14 2008 +0200" }, "message": "sched: clean up tracepoints\n\nmake it a bit more structured hence more readable.\n\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "fa340d9c050e78fb21a142b617304214ae5e0c2d", "tree": "ef7b912de0c922ce7c819d26aa36e113c9c81858", "parents": [ "b07c3f193a8074aa4afe43cfa8ae38ec4c7ccfa9" ], "author": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Wed Jul 23 13:38:00 2008 +0200" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:32:56 2008 +0200" }, "message": "tracing: disable tracepoints by default\n\nwhile it\u0027s arguably low overhead, we dont enable new features by default.\n\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "b07c3f193a8074aa4afe43cfa8ae38ec4c7ccfa9", "tree": "7d2b1d9efc5a8e24cb07c8d7f0b3e056fec8f150", "parents": [ "0a16b6075843325dc402edf80c1662838b929aff" ], "author": { "name": "Mathieu Desnoyers", "email": "mathieu.desnoyers@polymtl.ca", "time": "Fri Jul 18 12:16:17 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:32:26 2008 +0200" }, "message": "ftrace: port to tracepoints\n\nPorting the trace_mark() used by ftrace to tracepoints. (cleanup)\n\nChangelog :\n- Change error messages : marker -\u003e tracepoint\n\n[ mingo@elte.hu: conflict resolutions ]\nSigned-off-by: Mathieu Desnoyers \u003cmathieu.desnoyers@polymtl.ca\u003e\nAcked-by: \u0027Peter Zijlstra\u0027 \u003cpeterz@infradead.org\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "0a16b6075843325dc402edf80c1662838b929aff", "tree": "124e49d8c0196f3c6eb7aa09a7c4c3c3157fff7a", "parents": [ "4a0897526bbc5c6ac0df80b16b8c60339e717ae2" ], "author": { "name": "Mathieu Desnoyers", "email": "mathieu.desnoyers@polymtl.ca", "time": "Fri Jul 18 12:16:17 2008 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Tue Oct 14 10:30:52 2008 +0200" }, "message": "tracing, sched: LTTng instrumentation - scheduler\n\nInstrument the scheduler activity (sched_switch, migration, wakeups,\nwait for a task, signal delivery) and process/thread\ncreation/destruction (fork, exit, kthread stop). Actually, kthread\ncreation is not instrumented in this patch because it is architecture\ndependent. It allows to connect tracers such as ftrace which detects\nscheduling latencies, good/bad scheduler decisions. Tools like LTTng can\nexport this scheduler information along with instrumentation of the rest\nof the kernel activity to perform post-mortem analysis on the scheduler\nactivity.\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. See the \"Tracepoints\" patch header for\nperformance result detail.\n\nChangelog :\n\n- Change instrumentation location and parameter to match ftrace\n instrumentation, previously done with kernel markers.\n\n[ mingo@elte.hu: conflict resolutions ]\nSigned-off-by: Mathieu Desnoyers \u003cmathieu.desnoyers@polymtl.ca\u003e\nAcked-by: \u0027Peter Zijlstra\u0027 \u003cpeterz@infradead.org\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "4a0897526bbc5c6ac0df80b16b8c60339e717ae2", "tree": "aa8eb17c791cdff175f8cd6cb76f02d05f00029a", "parents": [ "24b8d831d56aac7907752d22d2aba5d8127db6f6" ], "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:29:05 2008 +0200" }, "message": "tracing: tracepoints, samples\n\nTracepoint example code under samples/.\n\nSigned-off-by: Mathieu Desnoyers \u003cmathieu.desnoyers@polymtl.ca\u003e\nAcked-by: \u0027Peter Zijlstra\u0027 \u003cpeterz@infradead.org\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "24b8d831d56aac7907752d22d2aba5d8127db6f6", "tree": "7a92d53f0c5e237099c0d39e7f11e4fef79fe951", "parents": [ "97e1c18e8d17bd87e1e383b2e9d9fc740332c8e2" ], "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:47 2008 +0200" }, "message": "tracing: tracepoints, documentation\n\nDocumentation of tracepoint usage.\n\nSigned-off-by: Mathieu Desnoyers \u003cmathieu.desnoyers@polymtl.ca\u003e\nAcked-by: \u0027Peter Zijlstra\u0027 \u003cpeterz@infradead.org\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" }, { "commit": "e7f2f9918c0e97aa98ba147ca387e2c7238f0711", "tree": "dd85d6d2907bffeda76b42ce55a445e3142fe738", "parents": [ "11a96d1820113fde0d55c3487b7da7a9031326b8", "c00193f9f09f9b852249a66391985f585d066084" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Oct 13 09:54:35 2008 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Oct 13 09:54:45 2008 -0700" }, "message": "Merge phase #5 (misc) of git://git.kernel.org/pub/scm/linux/kernel/git/tip/linux-2.6-tip\n\nMerges oprofile, timers/hpet, x86/traps, x86/time, and x86/core misc items.\n\n* \u0027x86-core-v4-for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/tip/linux-2.6-tip: (132 commits)\n x86: change early_ioremap to use slots instead of nesting\n x86: adjust dependencies for CONFIG_X86_CMOV\n dumpstack: x86: various small unification steps, fix\n x86: remove additional_cpus\n x86: remove additional_cpus configurability\n x86: improve UP kernel when CPU-hotplug and SMP is enabled\n dumpstack: x86: various small unification steps\n dumpstack: i386: make kstack\u003d an early boot-param and add oops\u003dpanic\n dumpstack: x86: use log_lvl and unify trace formatting\n dumptrace: x86: consistently include loglevel, print stack switch\n dumpstack: x86: add \"end\" parameter to valid_stack_ptr and print_context_stack\n dumpstack: x86: make printk_address equal\n dumpstack: x86: move die_nmi to dumpstack_32.c\n traps: x86: finalize unification of traps.c\n traps: x86: make traps_32.c and traps_64.c equal\n traps: x86: various noop-changes preparing for unification of traps_xx.c\n traps: x86_64: use task_pid_nr(tsk) instead of tsk-\u003epid in do_general_protection\n traps: i386: expand clear_mem_error and remove from mach_traps.h\n traps: x86_64: make io_check_error equal to the one on i386\n traps: i386: use preempt_conditional_sti/cli in do_int3\n ...\n" }, { "commit": "11a96d1820113fde0d55c3487b7da7a9031326b8", "tree": "dfbb1a6f7a8aa89ea9c5133aa661c5427b6c8b3a", "parents": [ "a18992d4652553f65469ac72b54b7fdd4f058277" ], "author": { "name": "Alan Cox", "email": "alan@redhat.com", "time": "Mon Oct 13 10:46:24 2008 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Oct 13 09:51:45 2008 -0700" }, "message": "tty: rename the remaining oddly named n_tty functions\n\nOriginal idea for this from a patch by Rodolfo Giometti which merges various\nbits of PPS support\n\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "a18992d4652553f65469ac72b54b7fdd4f058277", "tree": "76b9347d3e11f0ef125f3466560f565e88a52d38", "parents": [ "a90610e50b98e18385d4fbea2c9eaefb12c5a75c" ], "author": { "name": "Alan Cox", "email": "alan@redhat.com", "time": "Mon Oct 13 10:46:09 2008 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Oct 13 09:51:44 2008 -0700" }, "message": "fs3270: Correct error returns\n\nDrop the kernel lock further and also correct cases where we set rc to an\nerror code, and then return 0\n\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "a90610e50b98e18385d4fbea2c9eaefb12c5a75c", "tree": "03fd450b5d94710105a6c6f4f3f20519fd815461", "parents": [ "402fda9201ac619ea1f120996b1a142c57aeeb6b" ], "author": { "name": "Alan Cox", "email": "alan@redhat.com", "time": "Mon Oct 13 10:45:52 2008 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Oct 13 09:51:44 2008 -0700" }, "message": "fs3270: remove extra locks\n\nget_current_tty now does internal locking and returns a referenced object,\nthus our use of tty_mutex here can go away.\n\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "402fda9201ac619ea1f120996b1a142c57aeeb6b", "tree": "9a0e1411e55148aac0a8dd662335b29ab2e13063", "parents": [ "8440838bc5337243917f13bc14ea2445da5e0197" ], "author": { "name": "Jason Wessel", "email": "jason.wessel@windriver.com", "time": "Mon Oct 13 10:45:36 2008 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Oct 13 09:51:44 2008 -0700" }, "message": "tty: tty_io.c shadows sparse fix\n\ndrivers/char/tty_io.c:1413:17: warning: symbol \u0027buf\u0027 shadows an earlier one\ndrivers/char/tty_io.c:1379:20: originally declared here\n\nSigned-off-by: Jason Wessel \u003cjason.wessel@windriver.com\u003e\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "8440838bc5337243917f13bc14ea2445da5e0197", "tree": "175af8fd7d678073a9794248132b606fb536c887", "parents": [ "a7be18d436f0c7007794965e5af29fa1ffff1e05" ], "author": { "name": "David S. Miller", "email": "davem@davemloft.net", "time": "Mon Oct 13 10:45:26 2008 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Oct 13 09:51:44 2008 -0700" }, "message": "serial: fix device name reporting when minor space is shared between drivers\n\nThe multiple drivers share the minor space occupied by a particular major\nnumber, the actual index within the device name\u0027s space is indicated by\nthe tty_driver-\u003ename_base + uart_port-\u003eline\n\nAnother usable formula is (uart_driver-\u003eminor - MINOR_BASE) + port-\u003eline\n\nUse those to print the device names properly in such situations in\nserial_core.c and 8250.c\n\nSigned-off-by: David S. Miller \u003cdavem@davemloft.net\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "a7be18d436f0c7007794965e5af29fa1ffff1e05", "tree": "4675ce7e3a5255108bf1d9c7f4ec4507592d50b7", "parents": [ "17b820606e7bdd3be31b001265d0177423fea5eb" ], "author": { "name": "Alan Cox", "email": "alan@redhat.com", "time": "Mon Oct 13 10:45:17 2008 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Oct 13 09:51:44 2008 -0700" }, "message": "applicom: Fix an unchecked user ioctl range and an error return\n\nCloses bug #11408 by checking the card index range for command 0\nFixes the ioctl to return ENOTTY which is correct for unknown ioctls\n\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "17b820606e7bdd3be31b001265d0177423fea5eb", "tree": "df962dfca41e133983ee1a68deaad646a311d8b2", "parents": [ "51383f69ec56fe3f425d5255fb43e18c445e061c" ], "author": { "name": "Alan Cox", "email": "alan@redhat.com", "time": "Mon Oct 13 10:45:06 2008 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Oct 13 09:51:44 2008 -0700" }, "message": "tty: Minor tidyups and document fixes for n_tty\n\nRemove/fix some bogus NULL checks, comment some locking etc\n\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "51383f69ec56fe3f425d5255fb43e18c445e061c", "tree": "f9d9e280892702c3a47ffa807c0937ea9c80e315", "parents": [ "ea1afd256258f04a290aaa7fd5c2d4deb2e79e26" ], "author": { "name": "Alan Cox", "email": "alan@redhat.com", "time": "Mon Oct 13 10:44:57 2008 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Oct 13 09:51:44 2008 -0700" }, "message": "tty: Remove lots of NULL checks\n\nMany tty drivers contain \u0027can\u0027t happen\u0027 checks against NULL pointers passed\nin by the tty layer. These have never been possible to occur. Even more\nimportantly if they ever do occur we want to know as it would be a serious\nbug.\n\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "ea1afd256258f04a290aaa7fd5c2d4deb2e79e26", "tree": "6ef94459212ed68e64b9a86c3b4542e935f32e10", "parents": [ "53e86317e967b9a1b1b78d6a076547144454bce1" ], "author": { "name": "Alan Cox", "email": "alan@redhat.com", "time": "Mon Oct 13 10:44:43 2008 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Oct 13 09:51:44 2008 -0700" }, "message": "tty: fix up gigaset a bit\n\nStephen\u0027s fixes reminded me that gigaset is still rather broken so fix it up\na bit\n\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "53e86317e967b9a1b1b78d6a076547144454bce1", "tree": "87f38a002bc278769aa632d3a6eadddee1e44732", "parents": [ "47afa7a5a8a8fb9e60cdb6a3bd612e07c37e9d90" ], "author": { "name": "Stephen Rothwell", "email": "sfr@canb.auug.org.au", "time": "Mon Oct 13 10:44:33 2008 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Oct 13 09:51:44 2008 -0700" }, "message": "tty: Fallout from tty-move-canon-specials\n\nToday\u0027s linux-next build (x86_64 allmodconfig) failed like this:\n\n/drivers/char/tty_ioctl.c: In function \u0027change_termios\u0027:\ndrivers/isdn/capi/capi.c:1234: error: implicit declaration of function \u0027n_tty_ioctl\u0027\ndrivers/isdn/gigaset/ser-gigaset.c: In function \u0027gigaset_tty_ioctl\u0027:\ndrivers/isdn/gigaset/ser-gigaset.c:648: error: implicit declaration of function \u0027n_tty_ioctl\u0027\n\nIntroduced by commit 686b5e4aea05a80e370dc931b7f4a8d03c80da54\n(\"tty-move-canon-specials\"). I added the following patch (which may not\nbe correct).\n\nSigned-off-by: Stephen Rothwell \u003csfr@canb.auug.org.au\u003e\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "47afa7a5a8a8fb9e60cdb6a3bd612e07c37e9d90", "tree": "4f07d3aed0a08516162e529df75a014bd0798f8f", "parents": [ "fe6e29fdb1a7b94891bbdd3c67358fe4ed14639d" ], "author": { "name": "Alan Cox", "email": "alan@redhat.com", "time": "Mon Oct 13 10:44:17 2008 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Oct 13 09:51:44 2008 -0700" }, "message": "tty: some ICANON magic is in the wrong places\n\nMove the set up on ldisc change into the ldisc\nMove the INQ/OUTQ cases into the driver not in shared ioctl code where it\ngives bogus answers for other ldisc values\n\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "fe6e29fdb1a7b94891bbdd3c67358fe4ed14639d", "tree": "a1b9524ec42081e6573dd5a77bbb881117c6a73b", "parents": [ "8dff04ea316125639120c0a565ce0346b892fef7" ], "author": { "name": "Alan Cox", "email": "alan@redhat.com", "time": "Mon Oct 13 10:44:08 2008 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Oct 13 09:51:44 2008 -0700" }, "message": "tty: simplify ktermios allocation\n\nCopy the simplification from the pty unix98 special case to the generic one.\nThis allows us to kill off driver-\u003etermios_locked entirely which is nice. We\nhave to whack bits of the cris driver as it meddles in places it shouldn\u0027t\nproviding its own arrays that were never used anyway.\n\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "8dff04ea316125639120c0a565ce0346b892fef7", "tree": "f6c4d0455d951d550ab55cebe109ac526c957011", "parents": [ "335adde689150d2fcf4df3cb26a6fc6740ed1f3e" ], "author": { "name": "Alan Cox", "email": "alan@redhat.com", "time": "Mon Oct 13 10:43:58 2008 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Oct 13 09:51:44 2008 -0700" }, "message": "pty: simplify unix98 allocation\n\nWe need both termios and termios_locked so allocate them as one\n\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "335adde689150d2fcf4df3cb26a6fc6740ed1f3e", "tree": "93e11b89e96e8baf15a276e4008e08bb5873fb90", "parents": [ "fe9cd962a62cb5f666cf48b9941d3f3cde134254" ], "author": { "name": "Alan Cox", "email": "alan@redhat.com", "time": "Mon Oct 13 10:43:48 2008 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Oct 13 09:51:44 2008 -0700" }, "message": "pty: Fix allocation failure double free\n\nThe updating and moving around of the pty code added a bug where both the\nhelper and caller free the main tty struct (the pty driver must free the\no_tty pair itself however).\n\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "fe9cd962a62cb5f666cf48b9941d3f3cde134254", "tree": "c493585da2814bc8061c84fd2052a4407105d1e1", "parents": [ "a6f37daa8b892fd29dd71be0de61460a478cb122" ], "author": { "name": "Alan Cox", "email": "alan@redhat.com", "time": "Mon Oct 13 10:43:38 2008 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Oct 13 09:51:43 2008 -0700" }, "message": "pty: Coding style and polish\n\nWe\u0027ve done the heavy lifting now its time to mop up a bit\n\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "a6f37daa8b892fd29dd71be0de61460a478cb122", "tree": "158010784c65dc0183b62551058ae0ed0fa2e887", "parents": [ "89a52e109e2e2fe8bbd4e316cdb910774519c029" ], "author": { "name": "Sukadev Bhattiprolu", "email": "sukadev@us.ibm.com", "time": "Mon Oct 13 10:43:27 2008 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Oct 13 09:51:43 2008 -0700" }, "message": "Simplify devpts_pty_kill\n\nWhen creating a new pty, save the pty\u0027s inode in the tty-\u003edriver_data.\nUse this inode in pty_kill() to identify the devpts instance. Since\nwe now have the inode for the pty, we can skip get_node() lookup and\nremove the unused get_node().\n\nTODO:\n\t- check if the mutex_lock is needed in pty_kill().\n\nSigned-off-by: Sukadev Bhattiprolu \u003csukadev@us.ibm.com\u003e\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "89a52e109e2e2fe8bbd4e316cdb910774519c029", "tree": "5e469fcbfe109216ea1e046377c00b439b558bdd", "parents": [ "527b3e4773628b30d03323a2cb5fb0d84441990f" ], "author": { "name": "Sukadev Bhattiprolu", "email": "sukadev@us.ibm.com", "time": "Mon Oct 13 10:43:18 2008 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Oct 13 09:51:43 2008 -0700" }, "message": "Simplify devpts_pty_new()\n\ndevpts_pty_new() is called when setting up a new pty and would not\nwill not have an existing dentry or inode for the pty. So don\u0027t bother\nlooking for an existing dentry - just create a new one.\n\nSigned-off-by: Sukadev Bhattiprolu \u003csukadev@us.ibm.com\u003e\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "527b3e4773628b30d03323a2cb5fb0d84441990f", "tree": "d7dfcc1b716881162d93bf24ade602a77ca171ac", "parents": [ "15f1a6338ddd4e69fff965d4b3a0e1bfb7a13d9c" ], "author": { "name": "Sukadev Bhattiprolu", "email": "sukadev@us.ibm.com", "time": "Mon Oct 13 10:43:08 2008 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Oct 13 09:51:43 2008 -0700" }, "message": "Simplify devpts_get_tty()\n\nAs pointed out by H. Peter Anvin, since the inode for the pty is known,\nwe don\u0027t need to look it up.\n\nSigned-off-by: Sukadev Bhattiprolu \u003csukadev@us.ibm.com\u003e\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "15f1a6338ddd4e69fff965d4b3a0e1bfb7a13d9c", "tree": "301eea94cb514dc12b5070fc4b3ab33d3aab4322", "parents": [ "4a2b5fddd53b80efcb3266ee36e23b8de28e761a" ], "author": { "name": "Sukadev Bhattiprolu", "email": "sukadev@us.ibm.com", "time": "Mon Oct 13 10:42:59 2008 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Oct 13 09:51:43 2008 -0700" }, "message": "Add an instance parameter devpts interfaces\n\nPass-in \u0027inode\u0027 or \u0027tty\u0027 parameter to devpts interfaces. With multiple\ndevpts instances, these parameters will be used in subsequent patches\nto identify the instance of devpts mounted. The parameters also help\nsimplify devpts implementation.\n\nChangelog[v3]:\n\t- minor changes due to merge with ttydev updates\n\t- rename parameters to emphasize they are ptmx or pts inodes\n\t- pass-in tty_struct * to devpts_pty_kill() (this will help\n\t cleanup the get_node() call in a subsequent patch)\n\nSigned-off-by: Sukadev Bhattiprolu \u003csukadev@us.ibm.com\u003e\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" } ], "next": "4a2b5fddd53b80efcb3266ee36e23b8de28e761a" }