| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Mon, 17 Jun 2019 11:23:19 +0200
From: Ulf Hansson <ulf.hansson@...aro.org>
To: Viresh Kumar <viresh.kumar@...aro.org>
Cc: Rafael Wysocki <rjw@...ysocki.net>,
Len Brown <len.brown@...el.com>, Pavel Machek <pavel@....cz>,
Kevin Hilman <khilman@...nel.org>,
Linux PM <linux-pm@...r.kernel.org>,
Vincent Guittot <vincent.guittot@...aro.org>,
Qais.Yousef@....com, Matthias Kaehlcke <mka@...omium.org>,
juri.lelli@...il.com,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH V3 1/5] PM / QOS: Pass request type to dev_pm_qos_{add|remove}_notifier()
On Mon, 10 Jun 2019 at 12:51, Viresh Kumar <viresh.kumar@...aro.org> wrote:
>
> In order to use the same set of routines to register notifiers for
> different request types, update the existing
> dev_pm_qos_{add|remove}_notifier() routines with an additional
> parameter: request-type.
>
> For now, it only supports resume-latency request type.
>
> Signed-off-by: Viresh Kumar <viresh.kumar@...aro.org>
Reviewed-by: Ulf Hansson <ulf.hansson@...aro.org>
Kind regards
Uffe
> ---
> Documentation/power/pm_qos_interface.txt | 10 ++++++----
> drivers/base/power/domain.c | 8 +++++---
> drivers/base/power/qos.c | 14 ++++++++++++--
> include/linux/pm_qos.h | 12 ++++++++----
> 4 files changed, 31 insertions(+), 13 deletions(-)
>
> diff --git a/Documentation/power/pm_qos_interface.txt b/Documentation/power/pm_qos_interface.txt
> index 19c5f7b1a7ba..ec7d662d1707 100644
> --- a/Documentation/power/pm_qos_interface.txt
> +++ b/Documentation/power/pm_qos_interface.txt
> @@ -164,12 +164,14 @@ directory.
> Notification mechanisms:
> The per-device PM QoS framework has a per-device notification tree.
>
> -int dev_pm_qos_add_notifier(device, notifier):
> -Adds a notification callback function for the device.
> +int dev_pm_qos_add_notifier(device, notifier, type):
> +Adds a notification callback function for the device for a particular request
> +type.
> +
> The callback is called when the aggregated value of the device constraints list
> -is changed (for resume latency device PM QoS only).
> +is changed.
>
> -int dev_pm_qos_remove_notifier(device, notifier):
> +int dev_pm_qos_remove_notifier(device, notifier, type):
> Removes the notification callback function for the device.
>
>
> diff --git a/drivers/base/power/domain.c b/drivers/base/power/domain.c
> index 33c30c1e6a30..b063bc41b0a9 100644
> --- a/drivers/base/power/domain.c
> +++ b/drivers/base/power/domain.c
> @@ -1536,7 +1536,8 @@ static int genpd_add_device(struct generic_pm_domain *genpd, struct device *dev,
> if (ret)
> genpd_free_dev_data(dev, gpd_data);
> else
> - dev_pm_qos_add_notifier(dev, &gpd_data->nb);
> + dev_pm_qos_add_notifier(dev, &gpd_data->nb,
> + DEV_PM_QOS_RESUME_LATENCY);
>
> return ret;
> }
> @@ -1569,7 +1570,8 @@ static int genpd_remove_device(struct generic_pm_domain *genpd,
>
> pdd = dev->power.subsys_data->domain_data;
> gpd_data = to_gpd_data(pdd);
> - dev_pm_qos_remove_notifier(dev, &gpd_data->nb);
> + dev_pm_qos_remove_notifier(dev, &gpd_data->nb,
> + DEV_PM_QOS_RESUME_LATENCY);
>
> genpd_lock(genpd);
>
> @@ -1597,7 +1599,7 @@ static int genpd_remove_device(struct generic_pm_domain *genpd,
>
> out:
> genpd_unlock(genpd);
> - dev_pm_qos_add_notifier(dev, &gpd_data->nb);
> + dev_pm_qos_add_notifier(dev, &gpd_data->nb, DEV_PM_QOS_RESUME_LATENCY);
>
> return ret;
> }
> diff --git a/drivers/base/power/qos.c b/drivers/base/power/qos.c
> index 6c91f8df1d59..cfd463212513 100644
> --- a/drivers/base/power/qos.c
> +++ b/drivers/base/power/qos.c
> @@ -467,6 +467,7 @@ EXPORT_SYMBOL_GPL(dev_pm_qos_remove_request);
> *
> * @dev: target device for the constraint
> * @notifier: notifier block managed by caller.
> + * @type: request type.
> *
> * Will register the notifier into a notification chain that gets called
> * upon changes to the target value for the device.
> @@ -474,10 +475,14 @@ EXPORT_SYMBOL_GPL(dev_pm_qos_remove_request);
> * If the device's constraints object doesn't exist when this routine is called,
> * it will be created (or error code will be returned if that fails).
> */
> -int dev_pm_qos_add_notifier(struct device *dev, struct notifier_block *notifier)
> +int dev_pm_qos_add_notifier(struct device *dev, struct notifier_block *notifier,
> + enum dev_pm_qos_req_type type)
> {
> int ret = 0;
>
> + if (WARN_ON(type != DEV_PM_QOS_RESUME_LATENCY))
> + return -EINVAL;
> +
> mutex_lock(&dev_pm_qos_mtx);
>
> if (IS_ERR(dev->power.qos))
> @@ -500,15 +505,20 @@ EXPORT_SYMBOL_GPL(dev_pm_qos_add_notifier);
> *
> * @dev: target device for the constraint
> * @notifier: notifier block to be removed.
> + * @type: request type.
> *
> * Will remove the notifier from the notification chain that gets called
> * upon changes to the target value.
> */
> int dev_pm_qos_remove_notifier(struct device *dev,
> - struct notifier_block *notifier)
> + struct notifier_block *notifier,
> + enum dev_pm_qos_req_type type)
> {
> int retval = 0;
>
> + if (WARN_ON(type != DEV_PM_QOS_RESUME_LATENCY))
> + return -EINVAL;
> +
> mutex_lock(&dev_pm_qos_mtx);
>
> /* Silently return if the constraints object is not present. */
> diff --git a/include/linux/pm_qos.h b/include/linux/pm_qos.h
> index 6ea1ae373d77..1f4d456e8fff 100644
> --- a/include/linux/pm_qos.h
> +++ b/include/linux/pm_qos.h
> @@ -146,9 +146,11 @@ int dev_pm_qos_add_request(struct device *dev, struct dev_pm_qos_request *req,
> int dev_pm_qos_update_request(struct dev_pm_qos_request *req, s32 new_value);
> int dev_pm_qos_remove_request(struct dev_pm_qos_request *req);
> int dev_pm_qos_add_notifier(struct device *dev,
> - struct notifier_block *notifier);
> + struct notifier_block *notifier,
> + enum dev_pm_qos_req_type type);
> int dev_pm_qos_remove_notifier(struct device *dev,
> - struct notifier_block *notifier);
> + struct notifier_block *notifier,
> + enum dev_pm_qos_req_type type);
> void dev_pm_qos_constraints_init(struct device *dev);
> void dev_pm_qos_constraints_destroy(struct device *dev);
> int dev_pm_qos_add_ancestor_request(struct device *dev,
> @@ -202,10 +204,12 @@ static inline int dev_pm_qos_update_request(struct dev_pm_qos_request *req,
> static inline int dev_pm_qos_remove_request(struct dev_pm_qos_request *req)
> { return 0; }
> static inline int dev_pm_qos_add_notifier(struct device *dev,
> - struct notifier_block *notifier)
> + struct notifier_block *notifier,
> + enum dev_pm_qos_req_type type);
> { return 0; }
> static inline int dev_pm_qos_remove_notifier(struct device *dev,
> - struct notifier_block *notifier)
> + struct notifier_block *notifier,
> + enum dev_pm_qos_req_type type)
> { return 0; }
> static inline void dev_pm_qos_constraints_init(struct device *dev)
> {
> --
> 2.21.0.rc0.269.g1a574e7a288b
>
Powered by blists - more mailing lists