From: Ulf Hansson <ulf.hansson@linaro.org> To: Jon Hunter <jonathanh@nvidia.com> Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net>, Kevin Hilman <khilman@kernel.org>, Thierry Reding <thierry.reding@gmail.com>, Kukjin Kim <kgene@kernel.org>, Krzysztof Kozlowski <k.kozlowski@samsung.com>, Alexander Aring <alex.aring@gmail.com>, Eric Anholt <eric@anholt.net>, "linux-pm@vger.kernel.org" <linux-pm@vger.kernel.org>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>, "linux-tegra@vger.kernel.org" <linux-tegra@vger.kernel.org> Subject: Re: [PATCH 08/10] PM / Domains: Add support for removing PM domains Date: Thu, 8 Sep 2016 13:49:14 +0200 [thread overview] Message-ID: <CAPDyKFrVO9wb3ffmxA7HUUDt1_aYxe=aqa1v+xNpdrYiLt_m0A@mail.gmail.com> (raw) In-Reply-To: <1471340976-5379-9-git-send-email-jonathanh@nvidia.com> On 16 August 2016 at 11:49, Jon Hunter <jonathanh@nvidia.com> wrote: > The genpd framework allows users to add PM domains via the pm_genpd_init() > function, however, there is no corresponding function to remove a PM > domain. For most devices this may be fine as the PM domains are never > removed, however, for devices that wish to populate the PM domains from > within a driver, having the ability to remove a PM domain if the probing > of the device fails or the driver is unloaded is necessary. > > Add the function pm_genpd_remove() to remove a PM domain by referencing > it's generic_pm_domain structure. > > PM domains can only be removed if they are not a parent domain to > another PM domain and have no devices associated with them. I think we should also check if the there's is a provider registered for the genpd, as it should also prevent the genpd from being removed. Right? I noticed that you are adding the ->provider pointer to the genpd struct in patch 9/10. Perhaps re-structure $subject patch and 9/10 a bit to deal with this, so we can add the pm_genpd_remove() API in a more safe manner. Kind regards Uffe > > Signed-off-by: Jon Hunter <jonathanh@nvidia.com> > --- > drivers/base/power/domain.c | 46 +++++++++++++++++++++++++++++++++++++++++++++ > include/linux/pm_domain.h | 5 +++++ > 2 files changed, 51 insertions(+) > > diff --git a/drivers/base/power/domain.c b/drivers/base/power/domain.c > index 7ce4608bbbe0..72b973539205 100644 > --- a/drivers/base/power/domain.c > +++ b/drivers/base/power/domain.c > @@ -1356,6 +1356,52 @@ int pm_genpd_init(struct generic_pm_domain *genpd, > } > EXPORT_SYMBOL_GPL(pm_genpd_init); > > +/** > + * pm_genpd_remove - Remove a generic I/O PM domain > + * @genpd: Pointer to PM domain that is to be removed. > + * > + * To remove the PM domain, this function: > + * - Removes the PM domain as a subdomain to any parent domains, > + * if it was added. > + * - Removes the PM domain from the list of registered PM domains. > + * > + * The PM domain will only be removed, if it is not a parent to any > + * other PM domain and has no devices associated with it. > + */ > +int pm_genpd_remove(struct generic_pm_domain *genpd) > +{ > + struct gpd_link *l, *link; > + int ret = 0; > + > + if (IS_ERR_OR_NULL(genpd)) > + return -EINVAL; > + > + mutex_lock(&gpd_list_lock); > + mutex_lock(&genpd->lock); > + > + if (!list_empty(&genpd->master_links) || genpd->device_count) { > + mutex_unlock(&genpd->lock); > + mutex_unlock(&gpd_list_lock); > + pr_err("%s: unable to remove %s\n", __func__, genpd->name); > + return -EBUSY; > + } > + > + list_for_each_entry_safe(link, l, &genpd->slave_links, slave_node) { > + list_del(&link->master_node); > + list_del(&link->slave_node); > + kfree(link); > + } > + > + list_del(&genpd->gpd_list_node); > + mutex_unlock(&genpd->lock); > + cancel_work_sync(&genpd->power_off_work); > + mutex_unlock(&gpd_list_lock); > + pr_debug("%s: removed %s\n", __func__, genpd->name); > + > + return ret; > +} > +EXPORT_SYMBOL_GPL(pm_genpd_remove); > + > #ifdef CONFIG_PM_GENERIC_DOMAINS_OF > > typedef struct generic_pm_domain *(*genpd_xlate_t)(struct of_phandle_args *args, > diff --git a/include/linux/pm_domain.h b/include/linux/pm_domain.h > index f103869db443..fbdc5c4588ef 100644 > --- a/include/linux/pm_domain.h > +++ b/include/linux/pm_domain.h > @@ -128,6 +128,7 @@ extern int pm_genpd_remove_subdomain(struct generic_pm_domain *genpd, > struct generic_pm_domain *target); > extern int pm_genpd_init(struct generic_pm_domain *genpd, > struct dev_power_governor *gov, bool is_off); > +extern int pm_genpd_remove(struct generic_pm_domain *genpd); > > extern struct dev_power_governor simple_qos_governor; > extern struct dev_power_governor pm_domain_always_on_gov; > @@ -163,6 +164,10 @@ static inline int pm_genpd_init(struct generic_pm_domain *genpd, > { > return -ENOSYS; > } > +static inline int pm_genpd_remove(struct generic_pm_domain *genpd) > +{ > + return -ENOTSUPP; > +} > #endif > > static inline int pm_genpd_add_device(struct generic_pm_domain *genpd, > -- > 2.1.4 >
WARNING: multiple messages have this Message-ID (diff)
From: ulf.hansson@linaro.org (Ulf Hansson) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 08/10] PM / Domains: Add support for removing PM domains Date: Thu, 8 Sep 2016 13:49:14 +0200 [thread overview] Message-ID: <CAPDyKFrVO9wb3ffmxA7HUUDt1_aYxe=aqa1v+xNpdrYiLt_m0A@mail.gmail.com> (raw) In-Reply-To: <1471340976-5379-9-git-send-email-jonathanh@nvidia.com> On 16 August 2016 at 11:49, Jon Hunter <jonathanh@nvidia.com> wrote: > The genpd framework allows users to add PM domains via the pm_genpd_init() > function, however, there is no corresponding function to remove a PM > domain. For most devices this may be fine as the PM domains are never > removed, however, for devices that wish to populate the PM domains from > within a driver, having the ability to remove a PM domain if the probing > of the device fails or the driver is unloaded is necessary. > > Add the function pm_genpd_remove() to remove a PM domain by referencing > it's generic_pm_domain structure. > > PM domains can only be removed if they are not a parent domain to > another PM domain and have no devices associated with them. I think we should also check if the there's is a provider registered for the genpd, as it should also prevent the genpd from being removed. Right? I noticed that you are adding the ->provider pointer to the genpd struct in patch 9/10. Perhaps re-structure $subject patch and 9/10 a bit to deal with this, so we can add the pm_genpd_remove() API in a more safe manner. Kind regards Uffe > > Signed-off-by: Jon Hunter <jonathanh@nvidia.com> > --- > drivers/base/power/domain.c | 46 +++++++++++++++++++++++++++++++++++++++++++++ > include/linux/pm_domain.h | 5 +++++ > 2 files changed, 51 insertions(+) > > diff --git a/drivers/base/power/domain.c b/drivers/base/power/domain.c > index 7ce4608bbbe0..72b973539205 100644 > --- a/drivers/base/power/domain.c > +++ b/drivers/base/power/domain.c > @@ -1356,6 +1356,52 @@ int pm_genpd_init(struct generic_pm_domain *genpd, > } > EXPORT_SYMBOL_GPL(pm_genpd_init); > > +/** > + * pm_genpd_remove - Remove a generic I/O PM domain > + * @genpd: Pointer to PM domain that is to be removed. > + * > + * To remove the PM domain, this function: > + * - Removes the PM domain as a subdomain to any parent domains, > + * if it was added. > + * - Removes the PM domain from the list of registered PM domains. > + * > + * The PM domain will only be removed, if it is not a parent to any > + * other PM domain and has no devices associated with it. > + */ > +int pm_genpd_remove(struct generic_pm_domain *genpd) > +{ > + struct gpd_link *l, *link; > + int ret = 0; > + > + if (IS_ERR_OR_NULL(genpd)) > + return -EINVAL; > + > + mutex_lock(&gpd_list_lock); > + mutex_lock(&genpd->lock); > + > + if (!list_empty(&genpd->master_links) || genpd->device_count) { > + mutex_unlock(&genpd->lock); > + mutex_unlock(&gpd_list_lock); > + pr_err("%s: unable to remove %s\n", __func__, genpd->name); > + return -EBUSY; > + } > + > + list_for_each_entry_safe(link, l, &genpd->slave_links, slave_node) { > + list_del(&link->master_node); > + list_del(&link->slave_node); > + kfree(link); > + } > + > + list_del(&genpd->gpd_list_node); > + mutex_unlock(&genpd->lock); > + cancel_work_sync(&genpd->power_off_work); > + mutex_unlock(&gpd_list_lock); > + pr_debug("%s: removed %s\n", __func__, genpd->name); > + > + return ret; > +} > +EXPORT_SYMBOL_GPL(pm_genpd_remove); > + > #ifdef CONFIG_PM_GENERIC_DOMAINS_OF > > typedef struct generic_pm_domain *(*genpd_xlate_t)(struct of_phandle_args *args, > diff --git a/include/linux/pm_domain.h b/include/linux/pm_domain.h > index f103869db443..fbdc5c4588ef 100644 > --- a/include/linux/pm_domain.h > +++ b/include/linux/pm_domain.h > @@ -128,6 +128,7 @@ extern int pm_genpd_remove_subdomain(struct generic_pm_domain *genpd, > struct generic_pm_domain *target); > extern int pm_genpd_init(struct generic_pm_domain *genpd, > struct dev_power_governor *gov, bool is_off); > +extern int pm_genpd_remove(struct generic_pm_domain *genpd); > > extern struct dev_power_governor simple_qos_governor; > extern struct dev_power_governor pm_domain_always_on_gov; > @@ -163,6 +164,10 @@ static inline int pm_genpd_init(struct generic_pm_domain *genpd, > { > return -ENOSYS; > } > +static inline int pm_genpd_remove(struct generic_pm_domain *genpd) > +{ > + return -ENOTSUPP; > +} > #endif > > static inline int pm_genpd_add_device(struct generic_pm_domain *genpd, > -- > 2.1.4 >
next prev parent reply other threads:[~2016-09-08 11:49 UTC|newest] Thread overview: 64+ messages / expand[flat|nested] mbox.gz Atom feed top 2016-08-16 9:49 [PATCH 00/10] PM / Domains: Add support for removing PM domains Jon Hunter 2016-08-16 9:49 ` Jon Hunter 2016-08-16 9:49 ` [PATCH 01/10] PM / Domains: Add new helper functions for device-tree Jon Hunter 2016-08-16 9:49 ` Jon Hunter [not found] ` <1471340976-5379-2-git-send-email-jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2016-09-08 11:34 ` Ulf Hansson 2016-09-08 11:34 ` Ulf Hansson 2016-08-16 9:49 ` [PATCH 02/10] ARM: EXYNOS: Remove calls to of_genpd_get_from_provider() Jon Hunter 2016-08-16 9:49 ` Jon Hunter [not found] ` <1471340976-5379-3-git-send-email-jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2016-08-16 19:26 ` Krzysztof Kozlowski 2016-08-16 19:26 ` Krzysztof Kozlowski 2016-09-08 11:35 ` Ulf Hansson 2016-09-08 11:35 ` Ulf Hansson 2016-08-16 9:49 ` [PATCH 03/10] staging: board: " Jon Hunter 2016-08-16 9:49 ` Jon Hunter [not found] ` <1471340976-5379-4-git-send-email-jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2016-09-08 11:35 ` Ulf Hansson 2016-09-08 11:35 ` Ulf Hansson 2016-08-16 9:49 ` [PATCH 04/10] PM / Domains: Don't expose generic_pm_domain structure to clients Jon Hunter 2016-08-16 9:49 ` Jon Hunter 2016-09-08 11:35 ` Ulf Hansson 2016-09-08 11:35 ` Ulf Hansson 2016-08-16 9:49 ` [PATCH 05/10] PM / Domains: Don't expose xlate and provider helper functions Jon Hunter 2016-08-16 9:49 ` Jon Hunter [not found] ` <1471340976-5379-6-git-send-email-jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2016-09-08 11:36 ` Ulf Hansson 2016-09-08 11:36 ` Ulf Hansson 2016-08-16 9:49 ` [PATCH 07/10] PM / Domains: Prepare for adding support to remove PM domains Jon Hunter 2016-08-16 9:49 ` Jon Hunter 2016-09-08 11:41 ` Ulf Hansson 2016-09-08 11:41 ` Ulf Hansson 2016-08-16 9:49 ` [PATCH 08/10] PM / Domains: Add support for removing " Jon Hunter 2016-08-16 9:49 ` Jon Hunter 2016-09-08 11:49 ` Ulf Hansson [this message] 2016-09-08 11:49 ` Ulf Hansson [not found] ` <CAPDyKFrVO9wb3ffmxA7HUUDt1_aYxe=aqa1v+xNpdrYiLt_m0A-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2016-09-09 13:54 ` Jon Hunter 2016-09-09 13:54 ` Jon Hunter [not found] ` <c8498eca-6934-1cab-31d9-3d8ce4c74897-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2016-09-09 15:17 ` Jon Hunter 2016-09-09 15:17 ` Jon Hunter [not found] ` <63871fe6-cda6-2a95-9c09-e4b3ebfa3419-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2016-09-12 7:21 ` Ulf Hansson 2016-09-12 7:21 ` Ulf Hansson 2016-09-12 7:26 ` Jon Hunter 2016-09-12 7:26 ` Jon Hunter 2016-08-16 9:49 ` [PATCH 09/10] PM / Domains: Store the provider in the PM domain structure Jon Hunter 2016-08-16 9:49 ` Jon Hunter 2016-09-08 11:56 ` Ulf Hansson 2016-09-08 11:56 ` Ulf Hansson [not found] ` <CAPDyKFp+93=CrmgLNx1ritm1Dm=eWFJn=vbuzAUb9VmWc4j1fg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2016-09-09 13:57 ` Jon Hunter 2016-09-09 13:57 ` Jon Hunter [not found] ` <73870e28-3600-8a84-7659-55d4a8320a1d-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2016-09-09 14:25 ` Ulf Hansson 2016-09-09 14:25 ` Ulf Hansson 2016-08-16 9:49 ` [PATCH 10/10] PM / Domains: Add support for removing nested PM domains by provider Jon Hunter 2016-08-16 9:49 ` Jon Hunter 2016-09-08 12:30 ` Ulf Hansson 2016-09-08 12:30 ` Ulf Hansson 2016-09-09 14:04 ` Jon Hunter 2016-09-09 14:04 ` Jon Hunter [not found] ` <f31e57aa-52aa-d340-ca2b-73dbdf769035-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2016-09-09 14:21 ` Ulf Hansson 2016-09-09 14:21 ` Ulf Hansson [not found] ` <1471340976-5379-1-git-send-email-jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2016-08-16 9:49 ` [PATCH 06/10] PM / Domains: Verify the PM domain is present when adding a provider Jon Hunter 2016-08-16 9:49 ` Jon Hunter 2016-09-08 11:39 ` Ulf Hansson 2016-09-08 11:39 ` Ulf Hansson 2016-09-09 9:41 ` Jon Hunter 2016-09-09 9:41 ` Jon Hunter 2016-08-26 13:12 ` [PATCH 00/10] PM / Domains: Add support for removing PM domains Jon Hunter 2016-08-26 13:12 ` Jon Hunter
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to='CAPDyKFrVO9wb3ffmxA7HUUDt1_aYxe=aqa1v+xNpdrYiLt_m0A@mail.gmail.com' \ --to=ulf.hansson@linaro.org \ --cc=alex.aring@gmail.com \ --cc=eric@anholt.net \ --cc=jonathanh@nvidia.com \ --cc=k.kozlowski@samsung.com \ --cc=kgene@kernel.org \ --cc=khilman@kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-pm@vger.kernel.org \ --cc=linux-tegra@vger.kernel.org \ --cc=rjw@rjwysocki.net \ --cc=thierry.reding@gmail.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.