From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: Regression in v5.0-rc1 with autosuspend hrtimers Date: Wed, 9 Jan 2019 08:28:46 -0800 Message-ID: <20190109162846.GP5544@atomide.com> References: <20190107233833.GI5544@atomide.com> <20190108155354.GL5544@atomide.com> <20190108213743.GN5544@atomide.com> <20190109014218.GA8363@linaro.org> <20190109015124.GO5544@atomide.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: "Rafael J. Wysocki" Cc: Ulf Hansson , Ladislav Michl , Vincent Guittot , "open list:THERMAL" , "Rafael J. Wysocki" , linux-kernel , Linux OMAP Mailing List , LAK List-Id: linux-omap@vger.kernel.org KiBSYWZhZWwgSi4gV3lzb2NraSA8cmFmYWVsQGtlcm5lbC5vcmc+IFsxOTAxMDkgMDk6NDRdOgo+ IE9uIFdlZCwgSmFuIDksIDIwMTkgYXQgMjo1MSBBTSBUb255IExpbmRncmVuIDx0b255QGF0b21p ZGUuY29tPiB3cm90ZToKPiA+Cj4gPiAqIFZpbmNlbnQgR3VpdHRvdCA8dmluY2VudC5ndWl0dG90 QGxpbmFyby5vcmc+IFsxOTAxMDkgMDE6NDJdOgo+ID4gPiBMZSBUdWVzZGF5IDA4IEphbiAyMDE5 IMOgIDEzOjM3OjQzICgtMDgwMCksIFRvbnkgTGluZGdyZW4gYSDDqWNyaXQgOgo+ID4gPiA+IExv d2VyaW5nIHRoZSBhdXRvc3VzcGVuZF9kZWxheV9tcyB0byAyMTAwIG1zIG1ha2VzIHRoaW5ncyB3 b3JrIGFnYWluLgo+ID4gPiA+IEFueXRoaW5nIGhpZ2hlciB0aGFuIDIyMDAgbXMgc2VlbXMgdG8g c29tZWhvdyB0aW1lIG91dCBpbW1lZGlhdGVseQo+ID4gPiA+IG5vdyA6KQo+ID4gPgo+ID4gPiBU aGlzIGlzIHF1aXRlIGNsb3NlIHRvIHRoZSBtYXggbnMgb2YgYW4gaW50IG9uIGFybSAzMmJpdHMK PiA+ID4KPiA+ID4gQ291bGQgeW91IHRyeSB0aGUgcGF0Y2ggYmVsb3cgPwo+ID4KPiA+IFl1cCBn cmVhdCB0aGFua3MsIHRoYXQncyBpdDoKPiA+Cj4gPiBUZXN0ZWQtYnk6IFRvbnkgTGluZGdyZW4g PHRvbnlAYXRvbWlkZS5jb20+Cj4gCj4gQ29vbC4gIFRoYW5rcyBmb3IgZ2V0dGluZyB0byB0aGUg Ym90dG9tIG9mIHRoaXMhCgpObyBwcm9ibGVtLgoKT25lIG1vcmUgdGhpbmcgSSBub3RpY2VkOiBU aGUgMjUlIHNsYWNrIGNhbiBnZXQgbm90aWNlYWJsZQpmb3IgbGFyZ2VyIHZhbHVlcy4gRm9yIHRo aW5ncyBsaWtlIGEgMyBzZWNvbmQgdWFydCBjb25zb2xlCnRpbWVvdXQgc2xhY2sgb2YgNzUwIG1z IGlzIHF1aXRlIGxhcmdlIHZhcmlhdGlvbi4KClNob3VsZCB3ZSBoYXZlIGEgbGltaXQgb2YgbWF4 IDEwMCBtcyBmb3IgdGhlIHNsYWNrPwoKUmVnYXJkcywKClRvbnkKCl9fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmxpbnV4LWFybS1rZXJuZWwgbWFpbGluZyBs aXN0CmxpbnV4LWFybS1rZXJuZWxAbGlzdHMuaW5mcmFkZWFkLm9yZwpodHRwOi8vbGlzdHMuaW5m cmFkZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4LWFybS1rZXJuZWwK From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.5 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_PASS,URIBL_BLOCKED,USER_AGENT_MUTT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3EC99C43614 for ; Wed, 9 Jan 2019 16:28:56 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 10D61206BA for ; Wed, 9 Jan 2019 16:28:56 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726154AbfAIQ2v (ORCPT ); Wed, 9 Jan 2019 11:28:51 -0500 Received: from muru.com ([72.249.23.125]:60994 "EHLO muru.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725468AbfAIQ2u (ORCPT ); Wed, 9 Jan 2019 11:28:50 -0500 Received: from atomide.com (localhost [127.0.0.1]) by muru.com (Postfix) with ESMTPS id CB48080C5; Wed, 9 Jan 2019 16:28:55 +0000 (UTC) Date: Wed, 9 Jan 2019 08:28:46 -0800 From: Tony Lindgren To: "Rafael J. Wysocki" Cc: Vincent Guittot , "Rafael J. Wysocki" , Ulf Hansson , "open list:THERMAL" , linux-kernel , LAK , Linux OMAP Mailing List , Ladislav Michl Subject: Re: Regression in v5.0-rc1 with autosuspend hrtimers Message-ID: <20190109162846.GP5544@atomide.com> References: <20190107233833.GI5544@atomide.com> <20190108155354.GL5544@atomide.com> <20190108213743.GN5544@atomide.com> <20190109014218.GA8363@linaro.org> <20190109015124.GO5544@atomide.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.11.1 (2018-12-01) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org * Rafael J. Wysocki [190109 09:44]: > On Wed, Jan 9, 2019 at 2:51 AM Tony Lindgren wrote: > > > > * Vincent Guittot [190109 01:42]: > > > Le Tuesday 08 Jan 2019 à 13:37:43 (-0800), Tony Lindgren a écrit : > > > > Lowering the autosuspend_delay_ms to 2100 ms makes things work again. > > > > Anything higher than 2200 ms seems to somehow time out immediately > > > > now :) > > > > > > This is quite close to the max ns of an int on arm 32bits > > > > > > Could you try the patch below ? > > > > Yup great thanks, that's it: > > > > Tested-by: Tony Lindgren > > Cool. Thanks for getting to the bottom of this! No problem. One more thing I noticed: The 25% slack can get noticeable for larger values. For things like a 3 second uart console timeout slack of 750 ms is quite large variation. Should we have a limit of max 100 ms for the slack? Regards, Tony