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.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=no 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 1A3D4ECE58E for ; Thu, 17 Oct 2019 10:00:08 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id DEF712089C for ; Thu, 17 Oct 2019 10:00:07 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2404039AbfJQKAG (ORCPT ); Thu, 17 Oct 2019 06:00:06 -0400 Received: from [217.140.110.172] ([217.140.110.172]:37756 "EHLO foss.arm.com" rhost-flags-FAIL-FAIL-OK-OK) by vger.kernel.org with ESMTP id S1726638AbfJQKAG (ORCPT ); Thu, 17 Oct 2019 06:00:06 -0400 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id C0BAC1AED; Thu, 17 Oct 2019 02:59:45 -0700 (PDT) Received: from bogus (e107155-lin.cambridge.arm.com [10.1.196.42]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id D9AFF3F718; Thu, 17 Oct 2019 02:59:44 -0700 (PDT) Date: Thu, 17 Oct 2019 10:59:42 +0100 From: Sudeep Holla To: Viresh Kumar Cc: "Rafael J. Wysocki" , Linux PM , Linux ACPI , LKML , Dmitry Osipenko Subject: Re: [RFT][PATCH 0/3] cpufreq / PM: QoS: Introduce frequency QoS and use it in cpufreq Message-ID: <20191017095942.GF8978@bogus> References: <2811202.iOFZ6YHztY@kreacher> <20191016142343.GB5330@bogus> <20191017095725.izchzl7enfylvpf3@vireshk-i7> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20191017095725.izchzl7enfylvpf3@vireshk-i7> User-Agent: Mutt/1.9.4 (2018-02-28) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Oct 17, 2019 at 03:27:25PM +0530, Viresh Kumar wrote: > On 16-10-19, 15:23, Sudeep Holla wrote: > > Thanks for the spinning these patches so quickly. > > > > I did give it a spin, but unfortunately it doesn't fix the bug I reported. > > So I looked at my bug report in detail and looks like the cpufreq_driver > > variable is set to NULL at that point and it fails to dereference it > > while trying to execute: > > ret = cpufreq_driver->verify(new_policy); > > (Hint verify is at offset 0x1c/28) > > > > So I suspect some race as this platform with bL switcher tries to > > unregister and re-register the cpufreq driver during the boot. > > > > I need to spend more time on this as reverting the initial PM QoS patch > > to cpufreq.c makes the issue disappear. > > Is this easily reproducible ? cpufreq_driver == NULL shouldn't be the case, it > get updated only once while registering/unregistering cpufreq drivers. That is > the last thing which can go wrong from my point of view :) > Yes, if I boot my TC2 with bL switcher enabled, it always crashes on boot. -- Regards, Sudeep