From mboxrd@z Thu Jan 1 00:00:00 1970 From: Leo Yan Subject: Re: [PATCH 2/2] ARM: cpuidle: replace cpuidle_get_driver with cpuidle_get_cpu_driver Date: Sat, 7 Oct 2017 20:12:47 +0800 Message-ID: <20171007121247.GB23080@leoy-linaro> References: <1504507934-14218-1-git-send-email-leo.yan@linaro.org> <1504507934-14218-2-git-send-email-leo.yan@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail-pf0-f182.google.com ([209.85.192.182]:50163 "EHLO mail-pf0-f182.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751309AbdJGMMz (ORCPT ); Sat, 7 Oct 2017 08:12:55 -0400 Received: by mail-pf0-f182.google.com with SMTP id l188so10733770pfc.6 for ; Sat, 07 Oct 2017 05:12:54 -0700 (PDT) Content-Disposition: inline In-Reply-To: <1504507934-14218-2-git-send-email-leo.yan@linaro.org> Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org To: "Rafael J. Wysocki" , Daniel Lezcano , linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Stefan Wahren On Mon, Sep 04, 2017 at 02:52:14PM +0800, Leo Yan wrote: > commit d50a7d8acd78 ("ARM: cpuidle: Support asymmetric idle definition") > supports multiple CPU idle driver so every CPU has its own driver. When > the initialization fails, the failure handling releases the resources > for every previous CPU; so it needs to retrieve every CPU device and > driver handler and unregister them. But the function > cpuidle_get_driver() can only return current CPU driver handler but not > the iterated CPU driver handler, so it cannot release resource properly. > > This patch is to replace cpuidle_get_driver() with > cpuidle_get_cpu_driver(), every CPU has its own device handler so this > function can get back correct driver handler for the CPU according to > the CPU device handler. By using this CPU driver handler we can release > resource properly. > > Cc: Daniel Lezcano > Cc: Stefan Wahren > Signed-off-by: Leo Yan > Fixes: d50a7d8acd78 ("ARM: cpuidle: Support asymmetric idle definition") Gentle ping. > --- > drivers/cpuidle/cpuidle-arm.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/cpuidle/cpuidle-arm.c b/drivers/cpuidle/cpuidle-arm.c > index f419f6a..ef34780 100644 > --- a/drivers/cpuidle/cpuidle-arm.c > +++ b/drivers/cpuidle/cpuidle-arm.c > @@ -161,9 +161,9 @@ static int __init arm_idle_init(void) > > while (--cpu >= 0) { > dev = per_cpu(cpuidle_devices, cpu); > + drv = cpuidle_get_cpu_driver(dev); > cpuidle_unregister_device(dev); > kfree(dev); > - drv = cpuidle_get_driver(); > cpuidle_unregister_driver(drv); > kfree(drv); > } > -- > 2.7.4 >