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
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CA+Z25wWsv6z+GEQzvUhPL1NXnE-ha+pkobD7X-eUnds3jp2a8g@mail.gmail.com>
Date:	Wed, 9 Jan 2013 19:44:42 +0530
From:	Rajagopal Venkat <rajagopal.venkat@...aro.org>
To:	Abhilash Kesavan <a.kesavan@...sung.com>
Cc:	myungjoo.ham@...sung.com, linux-kernel@...r.kernel.org,
	linux-pm@...r.kernel.org, kgene.kim@...sung.com,
	kyungmin.park@...sung.com, rjw@...k.pl, jhbird.choi@...sung.com
Subject: Re: [PATCH 4/4] PM/Devfreq: Add Exynos5-bus devfreq driver for Exynos5250

On 9 January 2013 17:36, Abhilash Kesavan <a.kesavan@...sung.com> wrote:
> Exynos5-bus device devfreq driver monitors PPMU counters and
> adjusts operating frequencies and voltages with OPP. ASV should
> be used to provide appropriate voltages as per the speed group
> of the SoC rather than using a constant 1.025V.
>
> Signed-off-by: Abhilash Kesavan <a.kesavan@...sung.com>
> Cc: Jonghwan Choi <jhbird.choi@...sung.com>
> Cc: Kukjin Kim <kgene.kim@...sung.com>
> ---
>  drivers/devfreq/Kconfig               |   10 +
>  drivers/devfreq/Makefile              |    1 +
>  drivers/devfreq/exynos/Makefile       |    1 +
>  drivers/devfreq/exynos/exynos5_bus.c  |  469 +++++++++++++++++++++++++++++++++
>  drivers/devfreq/exynos/exynos5_ppmu.c |  412 +++++++++++++++++++++++++++++
>  drivers/devfreq/exynos/exynos_ppmu.c  |   56 ++++
>  include/linux/exynos5_ppmu.h          |   26 ++
>  include/linux/exynos_ppmu.h           |   79 ++++++
>  8 files changed, 1054 insertions(+), 0 deletions(-)
>  create mode 100644 drivers/devfreq/exynos/exynos5_bus.c
>  create mode 100644 drivers/devfreq/exynos/exynos5_ppmu.c
>  create mode 100644 drivers/devfreq/exynos/exynos_ppmu.c
>  create mode 100644 include/linux/exynos5_ppmu.h
>  create mode 100644 include/linux/exynos_ppmu.h
>
> diff --git a/drivers/devfreq/Kconfig b/drivers/devfreq/Kconfig
> index 0f079be..1560d0d 100644
> --- a/drivers/devfreq/Kconfig
> +++ b/drivers/devfreq/Kconfig
> @@ -78,4 +78,14 @@ config ARM_EXYNOS4_BUS_DEVFREQ
>           To operate with optimal voltages, ASV support is required
>           (CONFIG_EXYNOS_ASV).
>
> +config ARM_EXYNOS5_BUS_DEVFREQ
> +       bool "ARM Exynos5250 Bus DEVFREQ Driver"
> +       depends on SOC_EXYNOS5250
> +       select ARCH_HAS_OPP
> +       select DEVFREQ_GOV_SIMPLE_ONDEMAND
> +       help
> +         This adds the DEVFREQ driver for Exynos5250 bus interface (vdd_int).
> +         It reads PPMU counters of memory controllers and adjusts the
> +         operating frequencies and voltages with OPP support.
> +
>  endif # PM_DEVFREQ
> diff --git a/drivers/devfreq/Makefile b/drivers/devfreq/Makefile
> index 3bc1fef..16138c9 100644
> --- a/drivers/devfreq/Makefile
> +++ b/drivers/devfreq/Makefile
> @@ -6,3 +6,4 @@ obj-$(CONFIG_DEVFREQ_GOV_USERSPACE)     += governor_userspace.o
>
>  # DEVFREQ Drivers
>  obj-$(CONFIG_ARM_EXYNOS4_BUS_DEVFREQ)  += exynos/
> +obj-$(CONFIG_ARM_EXYNOS5_BUS_DEVFREQ)  += exynos/
> diff --git a/drivers/devfreq/exynos/Makefile b/drivers/devfreq/exynos/Makefile
> index 1498823..69713a8 100644
> --- a/drivers/devfreq/exynos/Makefile
> +++ b/drivers/devfreq/exynos/Makefile
> @@ -1,2 +1,3 @@
>  # Exynos DEVFREQ Drivers
>  obj-$(CONFIG_ARM_EXYNOS4_BUS_DEVFREQ)  += exynos4_bus.o
> +obj-$(CONFIG_ARM_EXYNOS5_BUS_DEVFREQ)  += exynos_ppmu.o exynos5_ppmu.o exynos5_bus.o
> diff --git a/drivers/devfreq/exynos/exynos5_bus.c b/drivers/devfreq/exynos/exynos5_bus.c
> new file mode 100644
> index 0000000..1d4a4b1
> --- /dev/null
> +++ b/drivers/devfreq/exynos/exynos5_bus.c
> @@ -0,0 +1,469 @@
> +/*
> + * Copyright (c) 2012 Samsung Electronics Co., Ltd.
> + *             http://www.samsung.com/
> + *
> + * EXYNOS5 INT clock frequency scaling support using DEVFREQ framework
> + * Based on work done by Jonghwan Choi <jhbird.choi@...sung.com>
> + * Support for only EXYNOS5250 is present.
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + *
> + */
> +
> +#include <linux/module.h>
> +#include <linux/slab.h>
> +#include <linux/suspend.h>
> +#include <linux/opp.h>
> +#include <linux/clk.h>
> +#include <linux/delay.h>
> +#include <linux/platform_device.h>
> +#include <linux/pm_qos.h>
> +#include <linux/regulator/consumer.h>
> +#include <linux/exynos_ppmu.h>
> +#include <linux/exynos5_ppmu.h>
> +
> +#include "../governor.h"

This header file is meant for governors use. What's the need of it here?

> +
> +#define MAX_SAFEVOLT                   1100000 /* 1.10V */
> +/* Assume that the bus is saturated if the utilization is 25% */
> +#define INT_BUS_SATURATION_RATIO       25
> +#define EXYNOS5_BUS_INT_POLL_TIME      msecs_to_jiffies(100)
> +
> +enum int_level_idx {
> +       LV_0,
> +       LV_1,
> +       LV_2,
> +       LV_3,
> +       LV_4,
> +       _LV_END
> +};
> +
> +struct busfreq_data_int {
> +       struct device *dev;
> +       struct devfreq *devfreq;
> +       bool disabled;
> +       struct regulator *vdd_int;
> +       unsigned long curr_freq;
> +       struct notifier_block pm_notifier;
> +       struct mutex lock;
> +       struct pm_qos_request int_req;
> +       struct clk *int_clk;
> +       struct exynos5_ppmu_handle *ppmu;
> +       struct delayed_work work;
> +       int busy;
> +};
> +
> +struct int_bus_opp_table {
> +       unsigned int idx;
> +       unsigned long clk;
> +       unsigned long volt;
> +};
> +
> +static struct int_bus_opp_table exynos5_int_opp_table[] = {
> +       {LV_0, 266000, 1025000},
> +       {LV_1, 200000, 1025000},
> +       {LV_2, 160000, 1025000},
> +       {LV_3, 133000, 1025000},
> +       {LV_4, 100000, 1025000},
> +       {0, 0, 0},
> +};
> +
> +static int exynos5_int_setvolt(struct busfreq_data_int *data,
> +               unsigned long volt)
> +{
> +       return regulator_set_voltage(data->vdd_int, volt, MAX_SAFEVOLT);
> +}
> +
> +static int exynos5_busfreq_int_target(struct device *dev, unsigned long *_freq,
> +                             u32 flags)
> +{
> +       int err = 0;
> +       struct platform_device *pdev = container_of(dev, struct platform_device,
> +                                                   dev);
> +       struct busfreq_data_int *data = platform_get_drvdata(pdev);
> +       struct opp *opp;
> +       unsigned long old_freq, freq;
> +       unsigned long volt;
> +
> +       rcu_read_lock();
> +       opp = devfreq_recommended_opp(dev, _freq, flags);
> +       if (IS_ERR(opp)) {
> +               rcu_read_unlock();
> +               dev_err(dev, "%s: Invalid OPP.\n", __func__);
> +               return PTR_ERR(opp);
> +       }
> +
> +       freq = opp_get_freq(opp);
> +       volt = opp_get_voltage(opp);
> +       rcu_read_unlock();
> +
> +       old_freq = data->curr_freq;
> +
> +       if (old_freq == freq)
> +               return 0;
> +
> +       dev_dbg(dev, "targetting %lukHz %luuV\n", freq, volt);
> +
> +       mutex_lock(&data->lock);
> +
> +       if (data->disabled)
> +               goto out;
> +
> +       if (freq > exynos5_int_opp_table[_LV_END - 1].clk)
> +               pm_qos_update_request(&data->int_req,
> +                               data->busy * old_freq * 16 / 100000);
> +       else
> +               pm_qos_update_request(&data->int_req, -1);
> +
> +       if (old_freq < freq)
> +               err = exynos5_int_setvolt(data, volt);
> +       if (err)
> +               goto out;
> +
> +       err = clk_set_rate(data->int_clk, freq * 1000);
> +
> +       if (err)
> +               goto out;
> +
> +       if (old_freq > freq)
> +               err = exynos5_int_setvolt(data, volt);
> +       if (err)
> +               goto out;
> +
> +       data->curr_freq = freq;
> +out:
> +       mutex_unlock(&data->lock);
> +       return err;
> +}
> +
> +static int exynos5_int_get_dev_status(struct device *dev,
> +                                     struct devfreq_dev_status *stat)
> +{
> +       struct platform_device *pdev = container_of(dev, struct platform_device,
> +                                                   dev);
> +       struct busfreq_data_int *data = platform_get_drvdata(pdev);
> +
> +       stat->current_frequency = data->curr_freq;
> +       stat->busy_time = data->busy;
> +       stat->total_time = 100;

How is busy_time is relative to total_time here? busy_time <=
total_time is guaranteed?

> +
> +       return 0;
> +}
> +
> +static void exynos5_int_poll_start(struct busfreq_data_int *data)
> +{
> +       schedule_delayed_work(&data->work, EXYNOS5_BUS_INT_POLL_TIME);
> +}
> +
> +static void exynos5_int_poll_stop(struct busfreq_data_int *data)
> +{
> +       cancel_delayed_work_sync(&data->work);
> +}
> +
> +static void exynos5_int_poll(struct work_struct *work)
> +{
> +       struct delayed_work *dwork;
> +       struct busfreq_data_int *data;
> +       int ret;
> +
> +       dwork = to_delayed_work(work);
> +       data = container_of(dwork, struct busfreq_data_int, work);
> +
> +       ret = exynos5_ppmu_get_busy(data->ppmu, PPMU_SET_RIGHT);
> +
> +       if (ret >= 0) {
> +               data->busy = ret;
> +               mutex_lock(&data->devfreq->lock);
> +               update_devfreq(data->devfreq);

Again, update_devfreq() is meant for devfreq governors use. Why is the devfreq
driver is doing devfreq governor job? any specific reason? The devfreq device
load monitoring is done by governors.

> +               mutex_unlock(&data->devfreq->lock);
> +       }
> +
> +       schedule_delayed_work(&data->work, EXYNOS5_BUS_INT_POLL_TIME);
> +}
> +
> +static void exynos5_int_exit(struct device *dev)
> +{
> +       struct platform_device *pdev = container_of(dev, struct platform_device,
> +                                                   dev);
> +       struct busfreq_data_int *data = platform_get_drvdata(pdev);
> +
> +       devfreq_unregister_opp_notifier(dev, data->devfreq);
> +}
> +
> +static struct devfreq_dev_profile exynos5_devfreq_int_profile = {
> +       .initial_freq           = 160000,
> +       .polling_ms             = 0,

why is polling_ms is set to zero? It defeats the purpose of devfreq driver.

> +       .target                 = exynos5_busfreq_int_target,
> +       .get_dev_status         = exynos5_int_get_dev_status,
> +       .exit                   = exynos5_int_exit,
> +};
> +
> +static int exynos5250_init_int_tables(struct busfreq_data_int *data)
> +{
> +       int i, err = 0;
> +
> +       for (i = LV_0; i < _LV_END; i++) {
> +               err = opp_add(data->dev, exynos5_int_opp_table[i].clk,
> +                               exynos5_int_opp_table[i].volt);
> +               if (err) {
> +                       dev_err(data->dev, "Cannot add opp entries.\n");
> +                       return err;
> +               }
> +       }
> +
> +       return 0;
> +}
> +static struct devfreq_simple_ondemand_data exynos5_int_ondemand_data = {
> +       .downdifferential = 2,
> +       .upthreshold = INT_BUS_SATURATION_RATIO,
> +};
> +
> +static int exynos5_busfreq_int_pm_notifier_event(struct notifier_block *this,
> +               unsigned long event, void *ptr)
> +{
> +       struct busfreq_data_int *data = container_of(this,
> +                                       struct busfreq_data_int, pm_notifier);
> +       struct opp *opp;
> +       unsigned long maxfreq = ULONG_MAX;
> +       unsigned long freq;
> +       unsigned long volt;
> +       int err = 0;
> +
> +       switch (event) {
> +       case PM_SUSPEND_PREPARE:
> +               /* Set Fastest and Deactivate DVFS */
> +               mutex_lock(&data->lock);
> +
> +               data->disabled = true;
> +
> +               rcu_read_lock();
> +               opp = opp_find_freq_floor(data->dev, &maxfreq);
> +               if (IS_ERR(opp)) {
> +                       rcu_read_unlock();
> +                       err = PTR_ERR(opp);
> +                       goto unlock;
> +               }
> +               freq = opp_get_freq(opp);
> +               volt = opp_get_voltage(opp);
> +               rcu_read_unlock();
> +
> +               err = exynos5_int_setvolt(data, volt);
> +               if (err)
> +                       goto unlock;
> +
> +               err = clk_set_rate(data->int_clk, freq * 1000);
> +
> +               if (err)
> +                       goto unlock;
> +
> +               data->curr_freq = freq;
> +unlock:
> +               mutex_unlock(&data->lock);
> +               if (err)
> +                       return NOTIFY_BAD;
> +               return NOTIFY_OK;
> +       case PM_POST_RESTORE:
> +       case PM_POST_SUSPEND:
> +               /* Reactivate */
> +               mutex_lock(&data->lock);
> +               data->disabled = false;
> +               mutex_unlock(&data->lock);
> +               return NOTIFY_OK;
> +       }
> +
> +       return NOTIFY_DONE;
> +}
> +
> +static __devinit int exynos5_busfreq_int_probe(struct platform_device *pdev)
> +{
> +       struct busfreq_data_int *data;
> +       struct opp *opp;
> +       struct device *dev = &pdev->dev;
> +       unsigned long initial_freq;
> +       unsigned long initial_volt;
> +       int err = 0;
> +
> +       data = devm_kzalloc(&pdev->dev, sizeof(struct busfreq_data_int),
> +                               GFP_KERNEL);
> +       if (data == NULL) {
> +               dev_err(dev, "Cannot allocate memory.\n");
> +               return -ENOMEM;
> +       }
> +
> +       data->pm_notifier.notifier_call = exynos5_busfreq_int_pm_notifier_event;
> +       data->dev = dev;
> +       mutex_init(&data->lock);
> +
> +       err = exynos5250_init_int_tables(data);
> +       if (err)
> +               goto err_regulator;
> +
> +       data->vdd_int = regulator_get(dev, "vdd_int");
> +       if (IS_ERR(data->vdd_int)) {
> +               dev_err(dev, "Cannot get the regulator \"vdd_int\"\n");
> +               err = PTR_ERR(data->vdd_int);
> +               goto err_regulator;
> +       }
> +
> +       data->int_clk = clk_get(dev, "int_clk");
> +       if (IS_ERR(data->int_clk)) {
> +               dev_err(dev, "Cannot get clock \"int_clk\"\n");
> +               err = PTR_ERR(data->int_clk);
> +               goto err_clock;
> +       }
> +
> +       rcu_read_lock();
> +       opp = opp_find_freq_floor(dev,
> +                       &exynos5_devfreq_int_profile.initial_freq);
> +       if (IS_ERR(opp)) {
> +               rcu_read_unlock();
> +               dev_err(dev, "Invalid initial frequency %lu kHz.\n",
> +                      exynos5_devfreq_int_profile.initial_freq);
> +               err = PTR_ERR(opp);
> +               goto err_opp_add;
> +       }
> +       initial_freq = opp_get_freq(opp);
> +       initial_volt = opp_get_voltage(opp);
> +       rcu_read_unlock();
> +       data->curr_freq = initial_freq;
> +
> +       err = clk_set_rate(data->int_clk, initial_freq * 1000);
> +       if (err) {
> +               dev_err(dev, "Failed to set initial frequency\n");
> +               goto err_opp_add;
> +       }
> +
> +       err = exynos5_int_setvolt(data, initial_volt);
> +       if (err)
> +               goto err_opp_add;
> +
> +       platform_set_drvdata(pdev, data);
> +
> +       data->ppmu = exynos5_ppmu_get();
> +       if (!data->ppmu)
> +               goto err_ppmu_get;
> +
> +       INIT_DELAYED_WORK(&data->work, exynos5_int_poll);
> +       exynos5_int_poll_start(data);
> +
> +       data->devfreq = devfreq_add_device(dev, &exynos5_devfreq_int_profile,
> +                                          "simple_ondemand",
> +                                          &exynos5_int_ondemand_data);
> +
> +       if (IS_ERR(data->devfreq)) {
> +               err = PTR_ERR(data->devfreq);
> +               goto err_devfreq_add;
> +       }
> +
> +       devfreq_register_opp_notifier(dev, data->devfreq);
> +
> +       err = register_pm_notifier(&data->pm_notifier);
> +       if (err) {
> +               dev_err(dev, "Failed to setup pm notifier\n");
> +               goto err_devfreq_add;
> +       }
> +
> +       /* TODO: Add a new QOS class for int/mif bus */
> +       pm_qos_add_request(&data->int_req, PM_QOS_NETWORK_THROUGHPUT, -1);
> +
> +       return 0;
> +
> +err_devfreq_add:
> +       devfreq_remove_device(data->devfreq);
> +       exynos5_int_poll_stop(data);
> +err_ppmu_get:
> +       platform_set_drvdata(pdev, NULL);
> +err_opp_add:
> +       clk_put(data->int_clk);
> +err_clock:
> +       regulator_put(data->vdd_int);
> +err_regulator:
> +       return err;
> +}
> +
> +static __devexit int exynos5_busfreq_int_remove(struct platform_device *pdev)
> +{
> +       struct busfreq_data_int *data = platform_get_drvdata(pdev);
> +
> +       pm_qos_remove_request(&data->int_req);
> +       unregister_pm_notifier(&data->pm_notifier);
> +       devfreq_remove_device(data->devfreq);
> +       regulator_put(data->vdd_int);
> +       clk_put(data->int_clk);
> +       platform_set_drvdata(pdev, NULL);
> +
> +       return 0;
> +}
> +
> +#ifdef CONFIG_PM_SLEEP
> +static int exynos5_busfreq_int_suspend(struct device *dev)
> +{
> +       struct platform_device *pdev = container_of(dev, struct platform_device,
> +                                                   dev);
> +       struct busfreq_data_int *data = platform_get_drvdata(pdev);
> +
> +       exynos5_int_poll_stop(data);
> +       return 0;
> +}
> +
> +static int exynos5_busfreq_int_resume(struct device *dev)
> +{
> +       struct platform_device *pdev = container_of(dev, struct platform_device,
> +                                                   dev);
> +       struct busfreq_data_int *data = platform_get_drvdata(pdev);
> +
> +       exynos5_int_poll_start(data);
> +       return 0;
> +}
> +#endif
> +
> +static SIMPLE_DEV_PM_OPS(exynos5_busfreq_int_pm, exynos5_busfreq_int_suspend,
> +               exynos5_busfreq_int_resume);
> +
> +/* platform device pointer for exynos5 devfreq device. */
> +static struct platform_device *exynos5_devfreq_pdev;
> +
> +static struct platform_driver exynos5_busfreq_int_driver = {
> +       .probe          = exynos5_busfreq_int_probe,
> +       .remove         = __devexit_p(exynos5_busfreq_int_remove),
> +       .driver         = {
> +               .name           = "exynos5-bus-int",
> +               .owner          = THIS_MODULE,
> +               .pm             = &exynos5_busfreq_int_pm,
> +       },
> +};
> +
> +static int __init exynos5_busfreq_int_init(void)
> +{
> +       int ret;
> +
> +       ret = platform_driver_register(&exynos5_busfreq_int_driver);
> +       if (ret < 0)
> +               goto out;
> +
> +       exynos5_devfreq_pdev =
> +               platform_device_register_simple("exynos5-bus-int", -1, NULL, 0);
> +       if (IS_ERR_OR_NULL(exynos5_devfreq_pdev)) {
> +               ret = PTR_ERR(exynos5_devfreq_pdev);
> +               goto out1;
> +       }
> +
> +       return 0;
> +out1:
> +       platform_driver_unregister(&exynos5_busfreq_int_driver);
> +out:
> +       return ret;
> +}
> +late_initcall(exynos5_busfreq_int_init);
> +
> +static void __exit exynos5_busfreq_int_exit(void)
> +{
> +       platform_device_unregister(exynos5_devfreq_pdev);
> +       platform_driver_unregister(&exynos5_busfreq_int_driver);
> +}
> +module_exit(exynos5_busfreq_int_exit);
> +
> +MODULE_LICENSE("GPL");
> +MODULE_DESCRIPTION("EXYNOS5 busfreq driver with devfreq framework");
> diff --git a/drivers/devfreq/exynos/exynos5_ppmu.c b/drivers/devfreq/exynos/exynos5_ppmu.c
> new file mode 100644
> index 0000000..0620f24
> --- /dev/null
> +++ b/drivers/devfreq/exynos/exynos5_ppmu.c
> @@ -0,0 +1,412 @@
> +/*
> + * Copyright (c) 2012 Samsung Electronics Co., Ltd.
> + *             http://www.samsung.com/
> + *
> + * EXYNOS5 PPMU support
> + * Support for only EXYNOS5250 is present.
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + *
> + */
> +
> +#include <linux/module.h>
> +#include <linux/platform_device.h>
> +#include <linux/kernel.h>
> +#include <linux/debugfs.h>
> +#include <linux/delay.h>
> +#include <linux/hrtimer.h>
> +#include <linux/io.h>
> +#include <linux/of.h>
> +#include <linux/slab.h>
> +#include <linux/exynos_ppmu.h>
> +#include <linux/exynos5_ppmu.h>
> +
> +#include <mach/map.h>
> +
> +#define FIXED_POINT_OFFSET     (0x8)
> +#define FIXED_POINT_MASK       (0xff)
> +
> +enum exynos5_ppmu_list {
> +       PPMU_DDR_C,
> +       PPMU_DDR_R1,
> +       PPMU_DDR_L,
> +       PPMU_RIGHT,
> +       PPMU_CPU,
> +       PPMU_END,
> +};
> +
> +struct exynos5_ppmu_handle {
> +       struct list_head node;
> +       struct exynos_ppmu ppmu[PPMU_END];
> +};
> +
> +static DEFINE_SPINLOCK(exynos5_ppmu_lock);
> +static LIST_HEAD(exynos5_ppmu_handle_list);
> +static struct exynos_ppmu ppmu[PPMU_END];
> +
> +static const char *exynos5_ppmu_name[PPMU_END] = {
> +       [PPMU_DDR_C]    = "DDR_C",
> +       [PPMU_DDR_R1]   = "DDR_R1",
> +       [PPMU_DDR_L]    = "DDR_L",
> +       [PPMU_RIGHT]    = "RIGHT",
> +       [PPMU_CPU]      = "CPU",
> +};
> +
> +static void exynos5_ppmu_reset(struct exynos_ppmu *ppmu)
> +{
> +       unsigned long flags;
> +       void __iomem *ppmu_base = ppmu->hw_base;
> +
> +       /* Reset the performance and cycle counters */
> +       exynos_ppmu_reset(ppmu_base);
> +
> +       /* Setup count registers for monitoring read/write transactions */
> +       ppmu->event[PPMU_PMNCNT0] = RD_DATA_COUNT;
> +       exynos_ppmu_setevent(ppmu_base, PPMU_PMNCNT0,
> +                       ppmu->event[PPMU_PMNCNT0]);
> +       ppmu->event[PPMU_PMCCNT1] = WR_DATA_COUNT;
> +       exynos_ppmu_setevent(ppmu_base, PPMU_PMCCNT1,
> +                       ppmu->event[PPMU_PMCCNT1]);
> +       ppmu->event[PPMU_PMNCNT3] = RDWR_DATA_COUNT;
> +       exynos_ppmu_setevent(ppmu_base, PPMU_PMNCNT3,
> +                       ppmu->event[PPMU_PMNCNT3]);
> +
> +       local_irq_save(flags);
> +       ppmu->reset_time = ktime_get();
> +       exynos_ppmu_start(ppmu_base);
> +       local_irq_restore(flags);
> +}
> +
> +static void exynos5_ppmu_read(struct exynos_ppmu *ppmu)
> +{
> +       unsigned long flags;
> +       ktime_t read_time;
> +       ktime_t t;
> +       int reg, j;
> +       void __iomem *ppmu_base = ppmu->hw_base;
> +
> +       local_irq_save(flags);
> +       read_time = ktime_get();
> +       exynos_ppmu_stop(ppmu_base);
> +       local_irq_restore(flags);
> +
> +       /* Update local data from PPMU */
> +       ppmu->ccnt = __raw_readl(ppmu_base + PPMU_CCNT);
> +       reg = __raw_readl(ppmu_base + PPMU_FLAG);
> +       ppmu->ccnt_overflow = reg & PPMU_CCNT_OVERFLOW;
> +
> +       for (j = PPMU_PMNCNT0; j < PPMU_PMNCNT_MAX; j++) {
> +               if (ppmu->event[j] == 0)
> +                       ppmu->count[j] = 0;
> +               else
> +                       ppmu->count[j] = exynos_ppmu_read(ppmu_base, j);
> +       }
> +       t = ktime_sub(read_time, ppmu->reset_time);
> +       ppmu->ns = ktime_to_ns(t);
> +}
> +
> +static void exynos5_ppmu_add(struct exynos_ppmu *to, struct exynos_ppmu *from)
> +{
> +       int i, j;
> +
> +       for (i = 0; i < PPMU_END; i++) {
> +               for (j = PPMU_PMNCNT0; j < PPMU_PMNCNT_MAX; j++)
> +                       to[i].count[j] += from[i].count[j];
> +
> +               to[i].ccnt += from[i].ccnt;
> +               if (to[i].ccnt < from[i].ccnt)
> +                       to[i].ccnt_overflow = true;
> +
> +               to[i].ns += from[i].ns;
> +
> +               if (from[i].ccnt_overflow)
> +                       to[i].ccnt_overflow = true;
> +       }
> +}
> +
> +static void exynos5_ppmu_handle_clear(struct exynos5_ppmu_handle *handle)
> +{
> +       memset(&handle->ppmu, 0, sizeof(struct exynos_ppmu) * PPMU_END);
> +}
> +
> +static void exynos5_ppmu_update(void)
> +{
> +       int i;
> +       struct exynos5_ppmu_handle *handle;
> +
> +       for (i = 0; i < PPMU_END; i++) {
> +               exynos5_ppmu_read(&ppmu[i]);
> +               exynos5_ppmu_reset(&ppmu[i]);
> +       }
> +
> +       list_for_each_entry(handle, &exynos5_ppmu_handle_list, node)
> +               exynos5_ppmu_add(handle->ppmu, ppmu);
> +}
> +
> +static int exynos5_ppmu_get_filter(enum exynos5_ppmu_sets filter,
> +       enum exynos5_ppmu_list *start, enum exynos5_ppmu_list *end)
> +{
> +       switch (filter) {
> +       case PPMU_SET_DDR:
> +               *start = PPMU_DDR_C;
> +               *end = PPMU_DDR_L;
> +               break;
> +       case PPMU_SET_RIGHT:
> +               *start = PPMU_RIGHT;
> +               *end = PPMU_RIGHT;
> +               break;
> +       case PPMU_SET_CPU:
> +               *start = PPMU_CPU;
> +               *end = PPMU_CPU;
> +               break;
> +       default:
> +               return -EINVAL;
> +       }
> +
> +       return 0;
> +}
> +
> +int exynos5_ppmu_get_busy(struct exynos5_ppmu_handle *handle,
> +       enum exynos5_ppmu_sets filter)
> +{
> +       unsigned long flags;
> +       int i, temp, ret, busy = 0;
> +       enum exynos5_ppmu_list start;
> +       enum exynos5_ppmu_list end;
> +
> +       ret = exynos5_ppmu_get_filter(filter, &start, &end);
> +       if (ret < 0)
> +               return ret;
> +
> +       spin_lock_irqsave(&exynos5_ppmu_lock, flags);
> +
> +       exynos5_ppmu_update();
> +
> +       for (i = start; i <= end; i++) {
> +               if (handle->ppmu[i].ccnt_overflow) {
> +                       busy = -EOVERFLOW;
> +                       break;
> +               }
> +               temp = handle->ppmu[i].count[PPMU_PMNCNT3] * 100;
> +               if (handle->ppmu[i].ccnt > 0)
> +                       temp /= handle->ppmu[i].ccnt;
> +               if (temp > busy)
> +                       busy = temp;
> +       }
> +
> +       exynos5_ppmu_handle_clear(handle);
> +
> +       spin_unlock_irqrestore(&exynos5_ppmu_lock, flags);
> +
> +       return busy;
> +}
> +
> +static void exynos5_ppmu_put(struct exynos5_ppmu_handle *handle)
> +{
> +       unsigned long flags;
> +
> +       spin_lock_irqsave(&exynos5_ppmu_lock, flags);
> +
> +       list_del(&handle->node);
> +
> +       spin_unlock_irqrestore(&exynos5_ppmu_lock, flags);
> +
> +       kfree(handle);
> +}
> +
> +struct exynos5_ppmu_handle *exynos5_ppmu_get(void)
> +{
> +       struct exynos5_ppmu_handle *handle;
> +       unsigned long flags;
> +
> +       handle = kzalloc(sizeof(struct exynos5_ppmu_handle), GFP_KERNEL);
> +       if (!handle)
> +               return NULL;
> +
> +       spin_lock_irqsave(&exynos5_ppmu_lock, flags);
> +
> +       exynos5_ppmu_update();
> +       list_add_tail(&handle->node, &exynos5_ppmu_handle_list);
> +
> +       spin_unlock_irqrestore(&exynos5_ppmu_lock, flags);
> +
> +       return handle;
> +}
> +
> +static void exynos5_ppmu_debug_compute(struct exynos_ppmu *ppmu,
> +       enum ppmu_counter i, u32 *sat, u32 *freq, u32 *bw)
> +{
> +       u64 ns = ppmu->ns;
> +       u32 busy = ppmu->count[i];
> +       u32 total = ppmu->ccnt;
> +
> +       u64 s;
> +       u64 f;
> +       u64 b;
> +
> +       s = (u64)busy * 100 * (1 << FIXED_POINT_OFFSET);
> +       s += total / 2;
> +       do_div(s, total);
> +
> +       f = (u64)total * 1000 * (1 << FIXED_POINT_OFFSET);
> +       f += ns / 2;
> +       f = div64_u64(f, ns);
> +
> +       b = (u64)busy * (128 / 8) * 1000 * (1 << FIXED_POINT_OFFSET);
> +       b += ns / 2;
> +       b = div64_u64(b, ns);
> +
> +       *sat = s;
> +       *freq = f;
> +       *bw = b;
> +}
> +
> +static void exynos5_ppmu_debug_show_one_counter(struct seq_file *s,
> +       const char *name, const char *type, struct exynos_ppmu *ppmu,
> +       enum ppmu_counter i, u32 *bw_total)
> +{
> +       u32 sat;
> +       u32 freq;
> +       u32 bw;
> +
> +       exynos5_ppmu_debug_compute(ppmu, i, &sat, &freq, &bw);
> +
> +       seq_printf(s, "%-10s %-10s %4u.%02u MBps %3u.%02u MHz %2u.%02u%%\n",
> +               name, type,
> +               bw >> FIXED_POINT_OFFSET,
> +               (bw & FIXED_POINT_MASK) * 100 / (1 << FIXED_POINT_OFFSET),
> +               freq >> FIXED_POINT_OFFSET,
> +               (freq & FIXED_POINT_MASK) * 100 / (1 << FIXED_POINT_OFFSET),
> +               sat >> FIXED_POINT_OFFSET,
> +               (sat & FIXED_POINT_MASK) * 100 / (1 << FIXED_POINT_OFFSET));
> +
> +       *bw_total += bw;
> +}
> +
> +static void exynos5_ppmu_debug_show_one(struct seq_file *s,
> +       const char *name, struct exynos_ppmu *ppmu,
> +       u32 *bw_total)
> +{
> +       exynos5_ppmu_debug_show_one_counter(s, name, "read+write",
> +               ppmu, PPMU_PMNCNT3, &bw_total[PPMU_PMNCNT3]);
> +       exynos5_ppmu_debug_show_one_counter(s, "", "read",
> +               ppmu, PPMU_PMNCNT0, &bw_total[PPMU_PMNCNT0]);
> +       exynos5_ppmu_debug_show_one_counter(s, "", "write",
> +               ppmu, PPMU_PMCCNT1, &bw_total[PPMU_PMCCNT1]);
> +
> +}
> +
> +static int exynos5_ppmu_debug_show(struct seq_file *s, void *d)
> +{
> +       int i;
> +       u32 bw_total[PPMU_PMNCNT_MAX];
> +       struct exynos5_ppmu_handle *handle;
> +       unsigned long flags;
> +
> +       memset(bw_total, 0, sizeof(bw_total));
> +
> +       handle = exynos5_ppmu_get();
> +       msleep(100);
> +
> +       spin_lock_irqsave(&exynos5_ppmu_lock, flags);
> +
> +       exynos5_ppmu_update();
> +
> +       for (i = 0; i < PPMU_CPU; i++)
> +               exynos5_ppmu_debug_show_one(s, exynos5_ppmu_name[i],
> +                               &handle->ppmu[i], bw_total);
> +
> +       seq_printf(s, "%-10s %-10s %4u.%02u MBps\n", "total", "read+write",
> +               bw_total[PPMU_PMNCNT3] >> FIXED_POINT_OFFSET,
> +               (bw_total[PPMU_PMNCNT3] & FIXED_POINT_MASK) *
> +                               100 / (1 << FIXED_POINT_OFFSET));
> +       seq_printf(s, "%-10s %-10s %4u.%02u MBps\n", "", "read",
> +               bw_total[PPMU_PMNCNT0] >> FIXED_POINT_OFFSET,
> +               (bw_total[PPMU_PMNCNT0] & FIXED_POINT_MASK) *
> +                               100 / (1 << FIXED_POINT_OFFSET));
> +       seq_printf(s, "%-10s %-10s %4u.%02u MBps\n", "", "write",
> +               bw_total[PPMU_PMCCNT1] >> FIXED_POINT_OFFSET,
> +               (bw_total[PPMU_PMCCNT1] & FIXED_POINT_MASK) *
> +                               100 / (1 << FIXED_POINT_OFFSET));
> +
> +       seq_printf(s, "\n");
> +
> +       exynos5_ppmu_debug_show_one(s, exynos5_ppmu_name[PPMU_CPU],
> +                       &ppmu[PPMU_CPU], bw_total);
> +
> +       spin_unlock_irqrestore(&exynos5_ppmu_lock, flags);
> +
> +       exynos5_ppmu_put(handle);
> +
> +       return 0;
> +}
> +
> +static int exynos5_ppmu_debug_open(struct inode *inode, struct file *file)
> +{
> +       return single_open(file, exynos5_ppmu_debug_show, inode->i_private);
> +}
> +
> +static const struct file_operations exynos5_ppmu_debug_fops = {
> +       .open           = exynos5_ppmu_debug_open,
> +       .read           = seq_read,
> +       .llseek         = seq_lseek,
> +       .release        = single_release,
> +};
> +
> +static int __init exynos5_ppmu_debug_init(void)
> +{
> +       debugfs_create_file("exynos5_bus", S_IRUGO, NULL, NULL,
> +               &exynos5_ppmu_debug_fops);
> +       return 0;
> +}
> +late_initcall(exynos5_ppmu_debug_init);
> +
> +static int exynos5_ppmu_probe(struct platform_device *pdev)
> +{
> +       struct resource *res;
> +       int i;
> +
> +       for (i = 0; i < PPMU_END; i++) {
> +               /* get the memory region */
> +               res = platform_get_resource(pdev, IORESOURCE_MEM, i);
> +               if (res == NULL) {
> +                       dev_err(&pdev->dev, "failed to get memory region resource\n");
> +                       return -ENOENT;
> +               }
> +
> +               ppmu[i].hw_base = devm_request_and_ioremap(&pdev->dev, res);
> +               if (ppmu->hw_base == NULL) {
> +                       dev_err(&pdev->dev, "failed to ioremap memory region\n");
> +                       return -EINVAL;
> +               }
> +       }
> +
> +       return 0;
> +}
> +
> +static int __devexit exynos5_ppmu_remove(struct platform_device *pdev)
> +{
> +       dev_dbg(&pdev->dev, "%s driver unloaded\n", pdev->name);
> +
> +       return 0;
> +}
> +
> +static const struct of_device_id exynos5_ppmu_match[] = {
> +       {
> +               .compatible = "samsung,exynos5250-ppmu",
> +       },
> +       {},
> +};
> +
> +static struct platform_driver exynos5_ppmu_driver = {
> +       .probe          = exynos5_ppmu_probe,
> +       .remove         = __devexit_p(exynos5_ppmu_remove),
> +       .driver         = {
> +               .name   = "exynos5-ppmu",
> +               .owner  = THIS_MODULE,
> +               .of_match_table = of_match_ptr(exynos5_ppmu_match),
> +       },
> +};
> +module_platform_driver(exynos5_ppmu_driver);
> diff --git a/drivers/devfreq/exynos/exynos_ppmu.c b/drivers/devfreq/exynos/exynos_ppmu.c
> new file mode 100644
> index 0000000..da373be
> --- /dev/null
> +++ b/drivers/devfreq/exynos/exynos_ppmu.c
> @@ -0,0 +1,56 @@
> +/*
> + * Copyright (c) 2012 Samsung Electronics Co., Ltd.
> + *             http://www.samsung.com/
> + *
> + * EXYNOS - PPMU support
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + */
> +
> +#include <linux/kernel.h>
> +#include <linux/types.h>
> +#include <linux/io.h>
> +
> +#include <linux/exynos_ppmu.h>
> +
> +void exynos_ppmu_reset(void __iomem *ppmu_base)
> +{
> +       __raw_writel(PPMU_CYCLE_RESET | PPMU_COUNTER_RESET, ppmu_base);
> +       __raw_writel(PPMU_ENABLE_CYCLE  |
> +                    PPMU_ENABLE_COUNT0 |
> +                    PPMU_ENABLE_COUNT1 |
> +                    PPMU_ENABLE_COUNT2 |
> +                    PPMU_ENABLE_COUNT3,
> +                    ppmu_base + PPMU_CNTENS);
> +}
> +
> +void exynos_ppmu_setevent(void __iomem *ppmu_base, unsigned int ch,
> +                       unsigned int evt)
> +{
> +       __raw_writel(evt, ppmu_base + PPMU_BEVTSEL(ch));
> +}
> +
> +void exynos_ppmu_start(void __iomem *ppmu_base)
> +{
> +       __raw_writel(PPMU_ENABLE, ppmu_base);
> +}
> +
> +void exynos_ppmu_stop(void __iomem *ppmu_base)
> +{
> +       __raw_writel(PPMU_DISABLE, ppmu_base);
> +}
> +
> +unsigned int exynos_ppmu_read(void __iomem *ppmu_base, unsigned int ch)
> +{
> +       unsigned int total;
> +
> +       if (ch == PPMU_PMNCNT3)
> +               total = ((__raw_readl(ppmu_base + PMCNT_OFFSET(ch)) << 8) |
> +                         __raw_readl(ppmu_base + PMCNT_OFFSET(ch + 1)));
> +       else
> +               total = __raw_readl(ppmu_base + PMCNT_OFFSET(ch));
> +
> +       return total;
> +}
> diff --git a/include/linux/exynos5_ppmu.h b/include/linux/exynos5_ppmu.h
> new file mode 100644
> index 0000000..9f492c1
> --- /dev/null
> +++ b/include/linux/exynos5_ppmu.h
> @@ -0,0 +1,26 @@
> +/*
> + * Copyright (c) 2012 Samsung Electronics Co., Ltd.
> + *             http://www.samsung.com/
> + *
> + * EXYNOS5 PPMU header
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> +*/
> +
> +#ifndef __DEVFREQ_EXYNOS5_PPMU_H
> +#define __DEVFREQ_EXYNOS5_PPMU_H __FILE__
> +
> +enum exynos5_ppmu_sets {
> +       PPMU_SET_DDR,
> +       PPMU_SET_RIGHT,
> +       PPMU_SET_CPU,
> +};
> +
> +struct exynos5_ppmu_handle *exynos5_ppmu_get(void);
> +extern int exynos5_ppmu_get_busy(struct exynos5_ppmu_handle *handle,
> +       enum exynos5_ppmu_sets filter);
> +
> +#endif /* __DEVFREQ_EXYNOS5_PPMU_H */
> +
> diff --git a/include/linux/exynos_ppmu.h b/include/linux/exynos_ppmu.h
> new file mode 100644
> index 0000000..b46d31b
> --- /dev/null
> +++ b/include/linux/exynos_ppmu.h
> @@ -0,0 +1,79 @@
> +/*
> + * Copyright (c) 2012 Samsung Electronics Co., Ltd.
> + *             http://www.samsung.com/
> + *
> + * EXYNOS PPMU header
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> +*/
> +
> +#ifndef __DEVFREQ_EXYNOS_PPMU_H
> +#define __DEVFREQ_EXYNOS_PPMU_H __FILE__
> +
> +#include <linux/ktime.h>
> +
> +/* For PPMU Control */
> +#define PPMU_ENABLE             BIT(0)
> +#define PPMU_DISABLE            0x0
> +#define PPMU_CYCLE_RESET        BIT(1)
> +#define PPMU_COUNTER_RESET      BIT(2)
> +
> +#define PPMU_ENABLE_COUNT0      BIT(0)
> +#define PPMU_ENABLE_COUNT1      BIT(1)
> +#define PPMU_ENABLE_COUNT2      BIT(2)
> +#define PPMU_ENABLE_COUNT3      BIT(3)
> +#define PPMU_ENABLE_CYCLE       BIT(31)
> +
> +#define PPMU_CNTENS            0x10
> +#define PPMU_FLAG              0x50
> +#define PPMU_CCNT_OVERFLOW     BIT(31)
> +#define PPMU_CCNT              0x100
> +
> +#define PPMU_PMCNT0            0x110
> +#define PPMU_PMCNT_OFFSET      0x10
> +#define PMCNT_OFFSET(x)                (PPMU_PMCNT0 + (PPMU_PMCNT_OFFSET * x))
> +
> +#define PPMU_BEVT0SEL          0x1000
> +#define PPMU_BEVTSEL_OFFSET    0x100
> +#define PPMU_BEVTSEL(x)                (PPMU_BEVT0SEL + (ch * PPMU_BEVTSEL_OFFSET))
> +
> +/* For Event Selection */
> +#define RD_DATA_COUNT          0x5
> +#define WR_DATA_COUNT          0x6
> +#define RDWR_DATA_COUNT                0x7
> +
> +enum ppmu_counter {
> +       PPMU_PMNCNT0,
> +       PPMU_PMCCNT1,
> +       PPMU_PMNCNT2,
> +       PPMU_PMNCNT3,
> +       PPMU_PMNCNT_MAX,
> +};
> +
> +struct bus_opp_table {
> +       unsigned int idx;
> +       unsigned long clk;
> +       unsigned long volt;
> +};
> +
> +struct exynos_ppmu {
> +       void __iomem *hw_base;
> +       unsigned int ccnt;
> +       unsigned int event[PPMU_PMNCNT_MAX];
> +       unsigned int count[PPMU_PMNCNT_MAX];
> +       unsigned long long ns;
> +       ktime_t reset_time;
> +       bool ccnt_overflow;
> +       bool count_overflow[PPMU_PMNCNT_MAX];
> +};
> +
> +void exynos_ppmu_reset(void __iomem *ppmu_base);
> +void exynos_ppmu_setevent(void __iomem *ppmu_base, unsigned int ch,
> +                       unsigned int evt);
> +void exynos_ppmu_start(void __iomem *ppmu_base);
> +void exynos_ppmu_stop(void __iomem *ppmu_base);
> +unsigned int exynos_ppmu_read(void __iomem *ppmu_base, unsigned int ch);
> +#endif /* __DEVFREQ_EXYNOS_PPMU_H */
> +
> --
> 1.7.8.6
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-pm" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html



-- 
Regards,
Rajagopal
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ