Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754807Ab0DOQdY (ORCPT ); Thu, 15 Apr 2010 12:33:24 -0400 Received: from mail-bw0-f225.google.com ([209.85.218.225]:35649 "EHLO mail-bw0-f225.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754236Ab0DOQdH (ORCPT ); Thu, 15 Apr 2010 12:33:07 -0400 Message-ID: <4BC73FB9.9060303@petalogix.com> Date: Thu, 15 Apr 2010 18:32:57 +0200 From: Michal Simek Reply-To: michal.simek@petalogix.com User-Agent: Thunderbird 2.0.0.22 (X11/20090625) MIME-Version: 1.0 To: Steven Rostedt CC: Thomas Gleixner , LKML , Wu Zhangjin Subject: Re: sched_clock - microblaze References: <4BC728DD.5020404@petalogix.com> <1271343888.1880.1.camel@localhost> In-Reply-To: <1271343888.1880.1.camel@localhost> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3075 Lines: 83 Steven Rostedt wrote: > On Thu, 2010-04-15 at 16:55 +0200, Michal Simek wrote: >> Hi Thomas and Steven, >> >> I would like to improve time measuring for ftrace (Log below) >> >> I looked at http://lkml.org/lkml/2009/11/16/181 >> where Thomas suggested to create sched_clock() function. >> I used the same solution as Wu proposed but it is not nice. >> >> Is unimplemented sched_clock the reason why ftrace not show fine grain time? > > Yeah, sched_clock is used by ftrace for timings, so if it only returns > jiffies, then that will, unfortunately, be the resolution of the tracer. > > I've been told that if you make a higher resolution timer for sched > clock, it will improve the scheduling in CFS. We have two timers (in one IP). timer0 - in IRQ mode - clock event timer1 - free running up counter without IRQ - clock source I hope that I can use timer1 in sched_clock too. Anyway. I played with it a little bit and I also implemented microblaze specific sched_clock function (inspired by arch/arm/mach-ixp4xx/common.c) unsigned long long notrace sched_clock(void) { printk("%s\n", __func__); cycle_t cyc = microblaze_read(NULL); struct clocksource *cs = &clocksource_microblaze; return clocksource_cyc2ns(cyc, cs->mult, cs->shift); } (And comment sched_clock in kernel/sched_clock.c because our toolchain has the problem with weak symbols) Recompile the kernel and then show the log_buf <4>Ramdisk addr 0x00000003, Compiled-in FDT at 0xc0242a28 <5>Linux version 2.6.34-rc4-00053-gbc6ce8a-dirty (monstr@monstr.eu) (gcc version 4.1.2) #27 Thu Apr 15 16:38:16 CEST 2010 <6>setup_cpuinfo: initialising <6>setup_cpuinfo: Using full CPU PVR support <6>cache: wb_msr <6>setup_memory: Main mem: 0x48000000-0x50000000, size 0x08000000, klimit 0xc09c7000 <6>setup_memory: max_mapnr: 0x8000 <6>setup_memory: min_low_pfn: 0x48000 <6>setup_memory: max_low_pfn: 0x50000 <4>reserved 0 - 0x48000000-0x009c8000 <4>reserved 1 - 0x48fe0000-0x00020000 <4>reserved 2 - 0x4fffd8c8-0x00002738 <7>On node 0 totalpages: 32768 <7>free_area_init_node: node 0, pgdat c02bc70c, node_mem_map c09c8000 <7> Normal zone: 256 pages used for memmap <7> Normal zone: 32512 pages, LIFO batch:7 <4>Built 1 zonelists in Zone order, mobility grouping on. Total pages: 32512 <5>Kernel command line: console=ttyS0,115200 <6>PID hash table entries: 512 (order: -1, 2048 bytes) <6>Dentry cache hash table entries: 16384 (order: 4, 65536 bytes) <6>Inode-cache hash table entries: 8192 (order: 3, 32768 bytes) <6>Memory: 119796k/131072k available <4>sched_clock The problem I see is that scheck_clock is used before our timer subsystem is initialized. What am I missing? Thanks, Michal -- Michal Simek, Ing. (M.Eng) PetaLogix - Linux Solutions for a Reconfigurable World w: www.petalogix.com p: +61-7-30090663,+42-0-721842854 f: +61-7-30090663 -- 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/