From patchwork Wed Sep 12 08:28:45 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Viresh Kumar X-Patchwork-Id: 146517 Delivered-To: patch@linaro.org Received: by 2002:a2e:1648:0:0:0:0:0 with SMTP id 8-v6csp4546929ljw; Wed, 12 Sep 2018 01:30:08 -0700 (PDT) X-Google-Smtp-Source: ANB0VdavfroNTv8h9oz2CXuYfO7N0H0556VwO5m5Grf4XOn9Q5a2KnVuwnlurV6idlm73So7e55q X-Received: by 2002:a62:225d:: with SMTP id i90-v6mr930879pfi.246.1536741008317; Wed, 12 Sep 2018 01:30:08 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1536741008; cv=none; d=google.com; s=arc-20160816; b=lAxDLbEsHXm5h5CS+mZlhIJi0oLYAeNFYg2asGN+zXHIQZFMtUKZWPgsbe2LTWhJbQ fNC9k4hTkamPM9VQv7k5lrtujPitd3l6U7XnnIEWuCwUTXSWpVYXtUMWr/PCeuy3vNwJ VEhpYD1CUHcw9b6VZnDkm4rhHzxqo0Gh173ioWLXfQ2JQgIdfTscTbhrVf2J8pHxPMdq J6ERE1uNt9/1db6EoB527iIGPHMcZ/4GKim8JbosWUdIQ98Vt9dFr087HqeRNK4JTJK5 uoQRHL2xLKpSJ3HUlTUvmi5mzZiabkWwt3jJI/fayOFxSbLMTwTK8eFeMB8B7O0xMNCV rnLQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:references:in-reply-to:message-id:date :subject:cc:to:from:dkim-signature; bh=bOqb8Y/Ki92J5bVwFaxRmefkd408CdxSloxrbWIZXp8=; b=neXssT21Jgn/UOIpKOZoMkAgtuXSYEHdFhzwggMy13QEYLp0x5/EbmYOlBNKodgVlY AgQvV6EkBJMuUy7rpn/QvRnMTyMSxgZXqgKEeCbCSJey2Y7KGlMxPbHuDN01CnsIYxHs 3Q99D7SeQvyBUKHEsQsyvKuan24Ys36lTRANrnKJ4sZZYXWZ45vnci4Y2A2yV8mO5+HP 0Ga/olhEiQqD0DbP0xvaBrjWbjC54OxDfQVY29zx6PViOTPNKPV/K3qwVuj2ong5UKJp ZapRexCucRvqfOgmjRQXJpDEis7Eg7Ggmsg8SH3B6heu3k6oORcYM08BAPncFfJoCstE jQVA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=E4a4JHHF; spf=pass (google.com: best guess record for domain of linux-pm-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-pm-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id s71-v6si363611pgs.499.2018.09.12.01.30.08; Wed, 12 Sep 2018 01:30:08 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-pm-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=@linaro.org header.s=google header.b=E4a4JHHF; spf=pass (google.com: best guess record for domain of linux-pm-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-pm-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728094AbeILNde (ORCPT + 10 others); Wed, 12 Sep 2018 09:33:34 -0400 Received: from mail-pl1-f196.google.com ([209.85.214.196]:41818 "EHLO mail-pl1-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728090AbeILNdd (ORCPT ); Wed, 12 Sep 2018 09:33:33 -0400 Received: by mail-pl1-f196.google.com with SMTP id b12-v6so620657plr.8 for ; Wed, 12 Sep 2018 01:30:06 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=bOqb8Y/Ki92J5bVwFaxRmefkd408CdxSloxrbWIZXp8=; b=E4a4JHHFS0WobIR2+BcmYpszw/c/1g7azPbMpoGJwlWdpQrEJ3YzrvcA+bk1N8WTje is2DwvZR8+vGHkRPuqk46vO+GYzNQMmQUIk8C33EbDRHduliE/ZyDWdrjWRo/bOo5mfH YhJc5mj4WdiPqiiLk1GW3qr4+sZovlmGtEm4E= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=bOqb8Y/Ki92J5bVwFaxRmefkd408CdxSloxrbWIZXp8=; b=UVgGbkyayURVdevwTq7BmEMj4pwGNdZND/UpJKA5u4O5jGv/SCb9pI7HbozgvqFow+ GkxWVbXZfmcTfxZ3U8QEeeFRoE03JBTHdZLdpPrWwsVuU94E5eV7SuFXgxDz88pH1bVz 30HQ5k5K4JnxdQk7OLhN0t96OgoY4coa246+VTXQFT2f8dZCDxMjKlWcXhLqF9bMqg72 6deDcUyBGQ94qIGUUKveQT8zP9ZWulg7ZOZRR7zWWqXo8xFMCSWi3+LlRrbVy9Fb8HJV Xilq9/D8N3kpqBglR/tB0Ap0oGHaUyTQOYTwUNwP9Ojcsabg4ERitObklXKEq/ftQl98 MeVQ== X-Gm-Message-State: APzg51B51gyHO6lzro5r84rEryda8bk+kNaEjWkBNkMp1rMNde/aaEHV LFaSZTa2fEmNRiSrpfpnIt4sGA== X-Received: by 2002:a17:902:5a87:: with SMTP id r7-v6mr874291pli.247.1536741005892; Wed, 12 Sep 2018 01:30:05 -0700 (PDT) Received: from localhost ([122.171.64.45]) by smtp.gmail.com with ESMTPSA id f87-v6sm1147545pfh.168.2018.09.12.01.30.04 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 12 Sep 2018 01:30:05 -0700 (PDT) From: Viresh Kumar To: niklas.cassel@linaro.org, Viresh Kumar , Nishanth Menon , Stephen Boyd Cc: Viresh Kumar , Rafael Wysocki , linux-pm@vger.kernel.org, Vincent Guittot , linux-kernel@vger.kernel.org Subject: [PATCH 06/11] OPP: Create separate kref for static OPPs list Date: Wed, 12 Sep 2018 13:58:45 +0530 Message-Id: <42981293f33d4baf752f7629f41625f009b3cebb.1536736872.git.viresh.kumar@linaro.org> X-Mailer: git-send-email 2.18.0.rc1.242.g61856ae69a2c In-Reply-To: References: Sender: linux-pm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pm@vger.kernel.org The static OPPs don't always get freed with the OPP table, it can happen before that as well. For example, if the OPP table is first created using helpers like dev_pm_opp_set_supported_hw() and the OPPs are created at a later point. Now when the OPPs are removed, the OPP table stays until the time dev_pm_opp_put_supported_hw() is called. Later patches will streamline the freeing of OPP table and that requires the static OPPs to get freed with help of a separate kernel reference. This patch prepares for that by creating a separate kref for static OPPs list. Signed-off-by: Viresh Kumar --- drivers/opp/core.c | 33 ++++++++++++++++++++++++++++++++- drivers/opp/of.c | 7 +++++++ drivers/opp/opp.h | 3 +++ 3 files changed, 42 insertions(+), 1 deletion(-) -- 2.18.0.rc1.242.g61856ae69a2c diff --git a/drivers/opp/core.c b/drivers/opp/core.c index 2a6976265580..b555121b878b 100644 --- a/drivers/opp/core.c +++ b/drivers/opp/core.c @@ -892,6 +892,33 @@ static void _opp_table_kref_release(struct kref *kref) mutex_unlock(&opp_table_lock); } +void _opp_remove_all_static(struct opp_table *opp_table) +{ + struct dev_pm_opp *opp, *tmp; + + list_for_each_entry_safe(opp, tmp, &opp_table->opp_list, node) { + if (!opp->dynamic) + dev_pm_opp_put(opp); + } + + opp_table->parsed_static_opps = false; +} + +static void _opp_table_list_kref_release(struct kref *kref) +{ + struct opp_table *opp_table = container_of(kref, struct opp_table, + list_kref); + + _opp_remove_all_static(opp_table); + mutex_unlock(&opp_table_lock); +} + +void _put_opp_list_kref(struct opp_table *opp_table) +{ + kref_put_mutex(&opp_table->list_kref, _opp_table_list_kref_release, + &opp_table_lock); +} + void dev_pm_opp_put_opp_table(struct opp_table *opp_table) { kref_put_mutex(&opp_table->kref, _opp_table_kref_release, @@ -1746,8 +1773,11 @@ void _dev_pm_opp_remove_table(struct opp_table *opp_table, struct device *dev, /* Find if opp_table manages a single device */ if (list_is_singular(&opp_table->dev_list)) { /* Free static OPPs */ + _put_opp_list_kref(opp_table); + + /* Free dynamic OPPs */ list_for_each_entry_safe(opp, tmp, &opp_table->opp_list, node) { - if (remove_all || !opp->dynamic) + if (remove_all) dev_pm_opp_put(opp); } @@ -1758,6 +1788,7 @@ void _dev_pm_opp_remove_table(struct opp_table *opp_table, struct device *dev, if (opp_table->genpd_performance_state) dev_pm_genpd_set_performance_state(dev, 0); } else { + _put_opp_list_kref(opp_table); _remove_opp_dev(_find_opp_dev(dev, opp_table), opp_table); } diff --git a/drivers/opp/of.c b/drivers/opp/of.c index ebf467b4d99e..892d17069f05 100644 --- a/drivers/opp/of.c +++ b/drivers/opp/of.c @@ -411,6 +411,8 @@ static int _of_add_opp_table_v2(struct device *dev, struct device_node *opp_np, ret = -ENOMEM; else if (!opp_table->parsed_static_opps) goto initialize_static_opps; + else + kref_get(&opp_table->list_kref); goto put_opp_table; } @@ -420,6 +422,8 @@ static int _of_add_opp_table_v2(struct device *dev, struct device_node *opp_np, return -ENOMEM; initialize_static_opps: + kref_init(&opp_table->list_kref); + /* We have opp-table node now, iterate over it and add OPPs */ for_each_available_child_of_node(opp_np, np) { count++; @@ -437,6 +441,7 @@ static int _of_add_opp_table_v2(struct device *dev, struct device_node *opp_np, /* There should be one of more OPP defined */ if (WARN_ON(!count)) { ret = -ENOENT; + _put_opp_list_kref(opp_table); goto put_opp_table; } @@ -491,6 +496,8 @@ static int _of_add_opp_table_v1(struct device *dev) if (!opp_table) return -ENOMEM; + kref_init(&opp_table->list_kref); + val = prop->value; while (nr) { unsigned long freq = be32_to_cpup(val++) * 1000; diff --git a/drivers/opp/opp.h b/drivers/opp/opp.h index d218fc0a498d..783428fa9c33 100644 --- a/drivers/opp/opp.h +++ b/drivers/opp/opp.h @@ -126,6 +126,7 @@ enum opp_table_access { * @dev_list: list of devices that share these OPPs * @opp_list: table of opps * @kref: for reference count of the table. + * @list_kref: for reference count of the OPP list. * @lock: mutex protecting the opp_list and dev_list. * @np: struct device_node pointer for opp's DT node. * @clock_latency_ns_max: Max clock latency in nanoseconds. @@ -157,6 +158,7 @@ struct opp_table { struct list_head dev_list; struct list_head opp_list; struct kref kref; + struct kref list_kref; struct mutex lock; struct device_node *np; @@ -200,6 +202,7 @@ int _opp_add(struct device *dev, struct dev_pm_opp *new_opp, struct opp_table *o int _opp_add_v1(struct opp_table *opp_table, struct device *dev, unsigned long freq, long u_volt, bool dynamic); void _dev_pm_opp_cpumask_remove_table(const struct cpumask *cpumask, bool of); struct opp_table *_add_opp_table(struct device *dev); +void _put_opp_list_kref(struct opp_table *opp_table); #ifdef CONFIG_OF void _of_init_opp_table(struct opp_table *opp_table, struct device *dev, int index);