From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754160Ab2AIMyY (ORCPT ); Mon, 9 Jan 2012 07:54:24 -0500 Received: from hqemgate03.nvidia.com ([216.228.121.140]:17094 "EHLO hqemgate03.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752424Ab2AIMyW (ORCPT ); Mon, 9 Jan 2012 07:54:22 -0500 X-PGP-Universal: processed; by hqnvupgp08.nvidia.com on Mon, 09 Jan 2012 04:54:02 -0800 Date: Mon, 9 Jan 2012 14:53:55 +0200 From: Peter De Schrijver To: Stephen Warren CC: Colin Cross , Olof Johansson , "Russell King" , "linux-tegra@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH v1 0/3] Tegra30 clockframework Message-ID: <20120109125355.GU32619@tbergstrom-lnx.Nvidia.com> References: <1325517486-22458-1-git-send-email-pdeschrijver@nvidia.com> <74CDBE0F657A3D45AFBB94109FB122FF177EE39F6B@HQMAIL01.nvidia.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <74CDBE0F657A3D45AFBB94109FB122FF177EE39F6B@HQMAIL01.nvidia.com> X-NVConfidentiality: public User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Jan 06, 2012 at 10:13:08PM +0100, Stephen Warren wrote: > Peter De Schrijver wrote at Monday, January 02, 2012 8:18 AM: > > This patchset introduces the tegra30 clockframework. Clocks which require > > voltage scaling are not included in this version. The implementation doesn't > > use the generic clock code yet. It's the intention to move to it, once the > > semantics are fully clarified. > > I tried testing this, and the kernel crashes/hangs as shown below: > > [ 0.088340] CPU: Testing write buffer coherency: ok > [ 0.093638] CPU0: thread -1, cpu 0, socket 0, mpidr 80000000 > [ 0.099573] Division by zero in kernel. > [ 0.103643] [] (unwind_backtrace+0x0/0x11c) from [] (Ldiv0_64+0x8/0x18) > [ 0.112332] [] (Ldiv0_64+0x8/0x18) from [] (clockevents_config.part.0+0x1c/0x64) > [ 0.121810] [] (clockevents_config.part.0+0x1c/0x64) from [] (clockevents_config_and_register+0x20/0x2c) > [ 0.133433] [] (clockevents_config_and_register+0x20/0x2c) from [] (twd_timer_setup+0x144/0x190) > [ 0.144335] [] (twd_timer_setup+0x144/0x190) from [] (local_timer_setup+0x10/0x18) > [ 0.153986] [] (local_timer_setup+0x10/0x18) from [] (percpu_timer_setup+0x54/0xa8) > [ 0.163727] [] (percpu_timer_setup+0x54/0xa8) from [] (smp_prepare_cpus+0x6c/0x90) > [ 0.173391] [] (smp_prepare_cpus+0x6c/0x90) from [] (kernel_init+0x48/0x12c) > [ 0.182520] [] (kernel_init+0x48/0x12c) from [] (kernel_thread_exit+0x0/0x8) > [ 0.191628] ------------[ cut here ]------------ > [ 0.196454] WARNING: at kernel/time/clockevents.c:46 clockevent_delta2ns+0x3c/0x84() > [ 0.204490] Modules linked in: > [ 0.207735] [] (unwind_backtrace+0x0/0x11c) from [] (warn_slowpath_common+0x4c/0x64) > [ 0.217568] [] (warn_slowpath_common+0x4c/0x64) from [] (warn_slowpath_null+0x18/0x1c) > [ 0.227577] [] (warn_slowpath_null+0x18/0x1c) from [] (clockevent_delta2ns+0x3c/0x84) > [ 0.237497] [] (clockevent_delta2ns+0x3c/0x84) from [] (clockevents_config.part.0+0x48/0x64) > [ 0.248040] [] (clockevents_config.part.0+0x48/0x64) from [] (clockevents_config_and_register+0x20/0x2c) > [ 0.259654] [] (clockevents_config_and_register+0x20/0x2c) from [] (twd_timer_setup+0x144/0x190) > [ 0.270553] [] (twd_timer_setup+0x144/0x190) from [] (local_timer_setup+0x10/0x18) > [ 0.280202] [] (local_timer_setup+0x10/0x18) from [] (percpu_timer_setup+0x54/0xa8) > [ 0.289934] [] (percpu_timer_setup+0x54/0xa8) from [] (smp_prepare_cpus+0x6c/0x90) > [ 0.299582] [] (smp_prepare_cpus+0x6c/0x90) from [] (kernel_init+0x48/0x12c) > [ 0.308699] [] (kernel_init+0x48/0x12c) from [] (kernel_thread_exit+0x0/0x8) > [ 0.317830] ---[ end trace 1b75b31a2719ed1c ]--- > [ 0.322697] Setting up static identity map for 0x8028cf98 - 0x8028cff0 > <> > > Without these patches, the same tree gets all the way to waiting for the > root filesystem: > > [ 0.088329] CPU: Testing write buffer coherency: ok > [ 0.093631] CPU0: thread -1, cpu 0, socket 0, mpidr 80000000 > [ 0.099545] smp_twd: clock not found: -2 > [ 0.103667] Calibrating local timer... 500.06MHz. > [ 0.161249] Setting up static identity map for 0x8028a918 - 0x8028a970 > [ 2.261492] CPU1: failed to come online > [ 2.265684] Brought up 1 CPUs > [ 2.268813] SMP: Total of 1 processors activated (1987.37 BogoMIPS). > ... > [ 2.704845] Registering SWP/SWPB emulation handler > [ 2.710558] Waiting for root device /dev/mmcblk1p1... > > I tested using commit e4421e71fa89bee67d2b67079041ca346ae49ea3 in my > per-user repo on nv-tegra.nvidia.com. Ok... Figured out why it doesn't work. Cheers, Peter.