Message-ID: <4C7A806D.4010106@linux.intel.com>
Date: Sun, 29 Aug 2010 08:44:45 -0700
From: Arjan van de Ven <arjan@linux.intel.com>
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.1.11) Gecko/20100711 Thunderbird/3.0.6
MIME-Version: 1.0
To: Yong Zhang <yong.zhang0@gmail.com>
CC: linux-kernel@vger.kernel.org, tglx@linutronix.de,
        akpm@linux-foundation.org, mingo@elte.hu, a.p.zijlstra@chello.nl,
        tj@kernel.org, oleg@redhat.com
Subject: Re: [RFC PATCHSET 0/6] timer/hrtimer/timer_stats: Improvement on
 timer_stats
References: <1283094309-6635-1-git-send-email-yong.zhang0@gmail.com>
In-Reply-To: <1283094309-6635-1-git-send-email-yong.zhang0@gmail.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 945
Lines: 26

On 8/29/2010 8:05 AM, Yong Zhang wrote:
> Hi,
>
> This patchset focus on two problem:
>
> 1)timer_stats_timer_*_start_info() should belong to timer core only.
>    It should not be seen by outsider, such as workqueue. And should
>    not resident in timer.h.
>    Patch-0001~0003 first remove the only explicit out call, then move
>    timer_stats_timer_*_start_info() from timer.h to timer.c.
>
> 2)call site recorded by timer_stats sometimes show the timer core functions
>    instead of the real call site.
>    Now /proc/timer_stats is like below on my machine:
> ---now---
>    


one question; have you tested this with powertop? (powertop being one of 
the prime consumers of this interface)

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/