From patchwork Sun Dec 29 17:19:20 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Greg Kroah-Hartman X-Patchwork-Id: 182545 Delivered-To: patch@linaro.org Received: by 2002:a92:815a:0:0:0:0:0 with SMTP id e87csp5450372ild; Sun, 29 Dec 2019 09:34:00 -0800 (PST) X-Google-Smtp-Source: APXvYqwW1vIzNOujLyI51IHAcv1qvQ8lEcxbuWfHEHZ/xuAUdkT487bT0+ZirjV731uPMI0McxGt X-Received: by 2002:a9d:2028:: with SMTP id n37mr72696374ota.127.1577640840345; Sun, 29 Dec 2019 09:34:00 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1577640840; cv=none; d=google.com; s=arc-20160816; b=wSTrJbSeFbV+9mfN5F/ls6rjM7YmXVo2YD+0dKJr5UBpnNshTWwVz75CDxvpU7iAu3 CLD9hbOLgdZwpf8PJ8Z3GCbdiqIwgUvbItJr0l9gaFYv4oml3lXkfmC+mggr0WEzlAZ+ oIFbgl85qGfv1pHrZxHoqVzOmOje6+75UEQM7qRAjs2SUyDGepdA5Z2lnJOttz4SuE/7 XAKIdsX1m9+3Ncs3ys0NXYBM+9piMSPSjiNmITUtD4P7ipX5hSqjDr37yB7NkcahCuZ5 LDFRwXYSo0AFLL176AtNaN3omgg8lXTbmyqsgwlTAeplifXXLuwEW3YgGQkwvPYwHkKZ qK9A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :user-agent:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=gHUeP7V+20e8CXvAcGlm3kqOdyllhfCOqD3P1h1Ol50=; b=DlbhRLNZ9GXWGV27zG/GiUpkUv7d/Iu3SlebKSSzEwfcf6YzOT2s9RYhmOcUbbBAxz gCdptEGjbqJmr+9xiLbfpuRSMkqp0ySIHRMHoOeKeXHm07+VTq7miRXemo7amgk+86s0 q1Ap25VbgbZnKEue1IcBJzmB1f9WLZSRGav6wtsLZHrNpZJs7AJNNpYCX4COAEUxnoyH APJr0Y2EQq/C1ivYFjjtJQR2LQP9MiL8tCF2otnqmFFSa8tthjL1WwC7L7ZOAcbzM7lg goQVCkLcGc8+fz2Hif34X729xNyqq2d7dtFnCCIe5UYN0rwnXfSxIMneq5GlJkBW0kma lddA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=ZeuWt3MX; spf=pass (google.com: best guess record for domain of stable-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=stable-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id e13si15011938otp.164.2019.12.29.09.34.00; Sun, 29 Dec 2019 09:34:00 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of stable-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=ZeuWt3MX; spf=pass (google.com: best guess record for domain of stable-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=stable-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729552AbfL2Rd7 (ORCPT + 14 others); Sun, 29 Dec 2019 12:33:59 -0500 Received: from mail.kernel.org ([198.145.29.99]:36416 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729535AbfL2Rd6 (ORCPT ); Sun, 29 Dec 2019 12:33:58 -0500 Received: from localhost (83-86-89-107.cable.dynamic.v4.ziggo.nl [83.86.89.107]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 2F92520722; Sun, 29 Dec 2019 17:33:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1577640837; bh=lpKrn/z464txxM/CiCqCNfrmIYaLTF+arTlZnHGXzXM=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=ZeuWt3MXlpKOFg5O2eeQVYsTrk4Per7vayrFJscRDlZepxTqE/hYtp/4JN3F32nlN PRPtfzXUr7GyRJYXThqw0m+/uGU9Cy/erYYhZTA8gq9pUbl9m1CjMhnbniwsG+ZTIh j6fdtgtFrf5+r+Qr3FHPI4Ey2sUNfI8/CMApr3PI= From: Greg Kroah-Hartman To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman , stable@vger.kernel.org, Bjorn Andersson , Viresh Kumar , Amit Kucheria , "Rafael J. Wysocki" , Sasha Levin Subject: [PATCH 4.19 158/219] cpufreq: Register drivers only after CPU devices have been registered Date: Sun, 29 Dec 2019 18:19:20 +0100 Message-Id: <20191229162532.386006575@linuxfoundation.org> X-Mailer: git-send-email 2.24.1 In-Reply-To: <20191229162508.458551679@linuxfoundation.org> References: <20191229162508.458551679@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Sender: stable-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: stable@vger.kernel.org From: Viresh Kumar [ Upstream commit 46770be0cf94149ca48be87719bda1d951066644 ] The cpufreq core heavily depends on the availability of the struct device for CPUs and if they aren't available at the time cpufreq driver is registered, we will never succeed in making cpufreq work. This happens due to following sequence of events: - cpufreq_register_driver() - subsys_interface_register() - return 0; //successful registration of driver ... at a later point of time - register_cpu(); - device_register(); - bus_probe_device(); - sif->add_dev(); - cpufreq_add_dev(); - get_cpu_device(); //FAILS - per_cpu(cpu_sys_devices, num) = &cpu->dev; //used by get_cpu_device() - return 0; //CPU registered successfully Because the per-cpu variable cpu_sys_devices is set only after the CPU device is regsitered, cpufreq will never be able to get it when cpufreq_add_dev() is called. This patch avoids this failure by making sure device structure of at least CPU0 is available when the cpufreq driver is registered, else return -EPROBE_DEFER. Reported-by: Bjorn Andersson Co-developed-by: Amit Kucheria Signed-off-by: Viresh Kumar Tested-by: Amit Kucheria Signed-off-by: Rafael J. Wysocki Signed-off-by: Sasha Levin --- drivers/cpufreq/cpufreq.c | 7 +++++++ 1 file changed, 7 insertions(+) -- 2.20.1 diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c index 9d8d64f706e0..e35c397b1259 100644 --- a/drivers/cpufreq/cpufreq.c +++ b/drivers/cpufreq/cpufreq.c @@ -2480,6 +2480,13 @@ int cpufreq_register_driver(struct cpufreq_driver *driver_data) if (cpufreq_disabled()) return -ENODEV; + /* + * The cpufreq core depends heavily on the availability of device + * structure, make sure they are available before proceeding further. + */ + if (!get_cpu_device(0)) + return -EPROBE_DEFER; + if (!driver_data || !driver_data->verify || !driver_data->init || !(driver_data->setpolicy || driver_data->target_index || driver_data->target) ||