From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:45618) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Shz3s-000401-Ai for qemu-devel@nongnu.org; Fri, 22 Jun 2012 04:17:54 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Shz3l-0001u5-Ot for qemu-devel@nongnu.org; Fri, 22 Jun 2012 04:17:47 -0400 Received: from mailout3.w1.samsung.com ([210.118.77.13]:36908) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Shz3l-0001tn-Ij for qemu-devel@nongnu.org; Fri, 22 Jun 2012 04:17:41 -0400 Received: from eusync1.samsung.com (mailout3.w1.samsung.com [210.118.77.13]) by mailout3.w1.samsung.com (Oracle Communications Messaging Server 7u4-24.01(7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTP id <0M6000LVWF2EQ520@mailout3.w1.samsung.com> for qemu-devel@nongnu.org; Fri, 22 Jun 2012 09:18:14 +0100 (BST) Received: from [106.109.8.38] by eusync1.samsung.com (Oracle Communications Messaging Server 7u4-23.01(7.0.4.23.0) 64bit (built Aug 10 2011)) with ESMTPA id <0M60003X8F1CM530@eusync1.samsung.com> for qemu-devel@nongnu.org; Fri, 22 Jun 2012 09:17:38 +0100 (BST) Message-id: <4FE42A20.4020307@samsung.com> Date: Fri, 22 Jun 2012 12:17:36 +0400 From: Evgeny Voevodin MIME-version: 1.0 References: <1340349744-23920-1-git-send-email-e.voevodin@samsung.com> In-reply-to: Content-type: text/plain; charset=ISO-8859-1; format=flowed Content-transfer-encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] ARM: hw/exynos4210_mct.c: Fix a bug which hangs Linux kernel. List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Crosthwaite Cc: m.shcherbina@samsung.com, m.kozlov@samsung.com, i.mitsyanko@samsung.com, peter.maydell@linaro.org, qemu-devel@nongnu.org, kyungmin.park@samsung.com, Peter Chubb , Paul Brook , s.vorobiov@samsung.com, "Edgar E. Iglesias" , o.ogurtsov@samsung.com, d.solodkiy@samsung.com On 22.06.2012 11:56, Peter Crosthwaite wrote: > Hi Evgeny, > > Im just speculating here, but I recently ran into Linux hangs on > Microblaze due to ptimer issues and I think you may be suffering the > same base issue. > > The Microblaze timer (hw/xilinx_timer.c) has a similar implementation > to the exynos (chained one-shot ptimer). Recently Peter Chubb put > patch cf36b31db209a261ee3bc2737e788e1ced0a1bec through, which modified > ptimer to not set excessively short periods. Problem is, that only > works for periodic ptimers. No, that's another problem. MCT was developed earlier then this commit landed and it contains similar code to avoid such situation. But our patch fixes bug in logic. The thing is that since ptimer uses BHs, it can be the situation when ptimer is stopped but BH is not called yet. And exactly in this moment target reads counter value that was incorrect calculated. > Anyways, you may find that changing your set_count() calls to > set_limit (i.e. the function designed for periodic timers) calls works > for you, without changing the logic of your device. Heres the change > pattern: > > - ptimer_set_count(ptimer, count); > + ptimer_set_limit(ptimer, count, 1); > > More permanently (and a question for the cheif maintainers) can we > look into ways of fixing ptimer properly? > > Regards, > Peter > > On Fri, Jun 22, 2012 at 5:22 PM, Evgeny Voevodin wrote: >> From: Stanislav Vorobiov >> >> After some long period of time Linux kernel hanged due to >> ptimer_get_count may return 0 before timer interrupt occurs, >> thus, causing FRC to jump back in time >> >> Signed-off-by: Evgeny Voevodin > Reviewed-by Peter A. G. Crosthwaite Thanks. >> --- >> hw/exynos4210_mct.c | 4 ---- >> 1 file changed, 4 deletions(-) >> >> diff --git a/hw/exynos4210_mct.c b/hw/exynos4210_mct.c >> index 7474fcf..7a22b1f 100644 >> --- a/hw/exynos4210_mct.c >> +++ b/hw/exynos4210_mct.c >> @@ -376,10 +376,6 @@ static uint64_t exynos4210_gfrc_get_count(Exynos4210MCTGT *s) >> { >> uint64_t count = 0; >> count = ptimer_get_count(s->ptimer_frc); >> - if (!count) { >> - /* Timer event was generated and s->reg.cnt holds adequate value */ >> - return s->reg.cnt; >> - } >> count = s->count - count; >> return s->reg.cnt + count; >> } >> -- >> 1.7.9.5 >> >> -- Kind regards, Evgeny Voevodin, Leading Software Engineer, ASWG, Moscow R&D center, Samsung Electronics e-mail: e.voevodin@samsung.com