From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752771AbbKQBgF (ORCPT ); Mon, 16 Nov 2015 20:36:05 -0500 Received: from www1246.sakura.ne.jp ([219.94.162.56]:47565 "EHLO www1246.sakura.ne.jp" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751611AbbKQBgE (ORCPT ); Mon, 16 Nov 2015 20:36:04 -0500 Message-ID: <564A847C.1090406@jinso.co.jp> Date: Tue, 17 Nov 2015 10:35:56 +0900 From: Cao Minh Hiep User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 MIME-Version: 1.0 To: Ulf Hansson CC: Nguyen Viet Dung , Kuninori Morimoto , Linux-SH , Linux-Kernel , "linux-pm@vger.kernel.org" , "Rafael J. Wysocki" , Sakato Ryusuke Subject: Re: PM: after suspend/resume the system,can not suspends again References: <87h9kulkfg.wl%kuninori.morimoto.gx@renesas.com> <87fv0elibh.wl%kuninori.morimoto.gx@renesas.com> <56417D61.80407@jinso.co.jp> <56499E53.2050905@jinso.co.jp> In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Dear Ulf Hansson Thanks for your feedback! On 2015年11月17日 06:12, Ulf Hansson wrote: > On 16 November 2015 at 10:13, Cao Minh Hiep wrote: >> Dear Ulf Hansson, >> >> I am Hiep from Jinzai Solution (a Japanese Company) >> We have tested PM driver on Rcar-Gen2 (Lager, Koelsch Board) at Linux v4.3. >> >> I would like to contact you because we found a patch that relate to a PM bug >> : >> >> After Suspend/resume system one time, we can not suspend the system again!. >> (I attach the test log file) >> >> The patch is show as below: >> a98f1b7 PM / Domains: Fix validation of latency constraints in genpd >> governor >> >> Tested-by: Harunaga >> Reported-by: Cao Minh Hiep >> >> This problem disapeared if revert this patch. >> Please review it, when you have time! > Thanks for reporting and bisecting! > > I had a look and I believe have found the cause of the problem, though > I will need a few days to post a fix for it. > > The problem is actually in genpd's ->runtime_suspend() callback, as it > validates PM QoS latencies even when it's being invoked from the > system PM path. > I will post a fixup to genpd in the next couple of days. I would > appreciate if you could help out in testing it. Please let me know when you post the fixup. I will test it. Thank you! Jinso/Cao Minh Hiep. > Kind regards > Uffe >