From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753903AbZHFDYz (ORCPT ); Wed, 5 Aug 2009 23:24:55 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752750AbZHFDYy (ORCPT ); Wed, 5 Aug 2009 23:24:54 -0400 Received: from [222.73.24.84] ([222.73.24.84]:50433 "EHLO song.cn.fujitsu.com" rhost-flags-FAIL-FAIL-OK-OK) by vger.kernel.org with ESMTP id S1752045AbZHFDYy (ORCPT ); Wed, 5 Aug 2009 23:24:54 -0400 Message-ID: <4A7A4DBC.6020403@cn.fujitsu.com> Date: Thu, 06 Aug 2009 11:27:56 +0800 From: Zhaolei User-Agent: Thunderbird 2.0.0.6 (Windows/20070728) MIME-Version: 1.0 To: Steven Rostedt , Ingo Molnar , Frederic Weisbecker CC: LKML Subject: [PATCH 2/2] ftrace: Unify effect of writing to trace_options and option/* References: <4A7A4D7B.7010706@cn.fujitsu.com> In-Reply-To: <4A7A4D7B.7010706@cn.fujitsu.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org "echo noglobal-clock > trace_options" can be used to change trace clock but "echo 0 > options/global-clock" can't. We can fix it by using set_tracer_flags() in trace_options_core_write(). Signed-off-by: Zhao Lei --- kernel/trace/trace.c | 5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index 464b7bb..2fa2bac 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -3893,12 +3893,11 @@ trace_options_core_write(struct file *filp, const char __user *ubuf, size_t cnt, switch (val) { case 0: - trace_flags &= ~(1 << index); + set_tracer_flags(1 << index, 0); break; case 1: - trace_flags |= 1 << index; + set_tracer_flags(1 << index, 1); break; - default: return -EINVAL; } -- 1.5.5.3