diff options
author | Li Zefan <lizf@cn.fujitsu.com> | 2009-07-07 01:54:28 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-07-10 05:59:43 -0400 |
commit | 0d109c8f70eab8b9f693bd5caea23012394e4876 (patch) | |
tree | 464df7244ab8fb311e054ba19539b80c4fdcbcf5 /kernel | |
parent | 011ed56853e07e30653d6f1bfddc56b396218664 (diff) |
ksym_tracer: Report error when failed to re-register hbp
When access type is changed, the hw break point will be
unregistered and then be registered again with new access
type. But the registration may fail, in this case, -errno
should be returned.
Signed-off-by: Li Zefan <lizf@cn.fujitsu.com>
Acked-by: Frederic Weisbecker <fweisbec@gmail.com>
Cc: "K.Prasad" <prasad@linux.vnet.ibm.com>
Cc: Alan Stern <stern@rowland.harvard.edu>
Cc: Steven Rostedt <rostedt@goodmis.org>
LKML-Reference: <4A52E314.7070004@cn.fujitsu.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/trace/trace_ksym.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/trace/trace_ksym.c b/kernel/trace/trace_ksym.c index 8cbed5a6286f..891e3b86b3f6 100644 --- a/kernel/trace/trace_ksym.c +++ b/kernel/trace/trace_ksym.c | |||
@@ -302,13 +302,13 @@ static ssize_t ksym_trace_filter_write(struct file *file, | |||
302 | ret = count; | 302 | ret = count; |
303 | goto unlock_ret_path; | 303 | goto unlock_ret_path; |
304 | } | 304 | } |
305 | } | 305 | } else |
306 | ret = count; | ||
306 | ksym_filter_entry_count--; | 307 | ksym_filter_entry_count--; |
307 | hlist_del_rcu(&(entry->ksym_hlist)); | 308 | hlist_del_rcu(&(entry->ksym_hlist)); |
308 | synchronize_rcu(); | 309 | synchronize_rcu(); |
309 | kfree(entry->ksym_hbp); | 310 | kfree(entry->ksym_hbp); |
310 | kfree(entry); | 311 | kfree(entry); |
311 | ret = count; | ||
312 | goto err_ret; | 312 | goto err_ret; |
313 | } else { | 313 | } else { |
314 | /* Check for malformed request: (4) */ | 314 | /* Check for malformed request: (4) */ |