linaro-dev October 2012

linaro-dev@lists.linaro.org

76 participants
108 discussions

by Zach Pfeffer

Hey everyone, About to put this in summit: https://docs.google.com/a/linaro.org/document/d/1gdmjguZSKyoVUlqteX1-ldIFEp… Are we missing anything big? -- Zach Pfeffer Android Platform Team Lead, Linaro Platform Teams Linaro.org | Open source software for ARM SoCs Follow Linaro: http://www.facebook.com/pages/Linaro http://twitter.com/#!/linaroorg - http://www.linaro.org/linaro-blog

11 years, 7 months

[PATCH 0/4] cpuidle - support multiple drivers at a time

by Daniel Lezcano

The discussion about having different cpus on the system with different latencies bring us to a first attemp by adding a pointer in the cpuidle_device to the states array. But as Rafael suggested, it would make more sense to create a driver per cpu [1]. This patch adds support for multiple cpuidle drivers. It creates a per cpu cpuidle driver pointer. In order to not break the different drivers, the function cpuidle_register_driver assign for each cpu, the driver. The multiple driver support is optional and if it is not set, the cpuide driver core code remains the same (except some code reorganisation). I did the following tests compiled, booted, tested without/with CONFIG_CPU_IDLE, with/without CONFIG_CPU_IDLE_MULTIPLE_DRIVERS. Tested on Core2 Duo T9500 with acpi_idle [and intel_idle] Tested on ARM Dual Cortex-A9 U8500 (aka Snowball) [1] http://www.spinics.net/lists/linux-acpi/msg37921.html Daniel Lezcano (4): cpuidle : move driver's refcount to cpuidle cpuidle : move driver checking within the lock section cpuidle - prepare the driver core to be multi drivers aware cpuidle - support multiple drivers drivers/cpuidle/Kconfig | 9 ++ drivers/cpuidle/cpuidle.c | 24 ++++-- drivers/cpuidle/driver.c | 194 ++++++++++++++++++++++++++++++++++++++------- drivers/cpuidle/sysfs.c | 50 +++++++++--- include/linux/cpuidle.h | 9 ++- 5 files changed, 238 insertions(+), 48 deletions(-) -- 1.7.5.4

11 years, 7 months

alsa UCM

by Sebastien LEDUC

Hi, We are potentially interested in contributing to the alsa UCM for Android initiative. Is there still some activities around that? Thanks Regards, Sebastien [cid:image001.png@01CDA7CF.6B9F6710] Sebastien Le Duc | Tel: +33 476 58 54 14 | Mobile: +33 684 43 07 00 APE | Audio Technical Leader ST online: www.st.com<http://www.st.com/> | Follow us on twitter: @st_world<http://twitter.com/#!/ST_World>

11 years, 7 months

[PATCH] cpufreq: vexpress: cpufreq driver name can't be longer than 15 chars

by Viresh Kumar

Following is the declaration of name field in struct cpufreq_driver: char name[CPUFREQ_NAME_LEN]; where CPUFREQ_NAME_LEN is 16. So, length of drivers name must be <=15 (one position for '\0'). Current name is crossing this limit and so name doesn't get printed properly when we do: $ cat /sys/devices/system/cpu/cpu0/cpufreq/scaling_driver This patch renames driver-name to fix this issue. Signed-off-by: Viresh Kumar <viresh.kumar(a)linaro.org> --- drivers/cpufreq/vexpress_bL_cpufreq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/cpufreq/vexpress_bL_cpufreq.c b/drivers/cpufreq/vexpress_bL_cpufreq.c index 8b7ec18..541fc40 100644 --- a/drivers/cpufreq/vexpress_bL_cpufreq.c +++ b/drivers/cpufreq/vexpress_bL_cpufreq.c @@ -244,7 +244,7 @@ static struct cpufreq_driver vexpress_cpufreq_driver = { .target = vexpress_cpufreq_set_target, .get = vexpress_cpufreq_get, .init = vexpress_cpufreq_init, - .name = "cpufreq_vexpress", + .name = "vexpress-bL", .attr = vexpress_cpufreq_attr, }; -- 1.7.12.rc2.18.g61b472e

11 years, 7 months

Interesting(?) health failure

by Dave Pigott

Hi all, I found an interesting health failure today on origen07 http://validation.linaro.org/lava-server/scheduler/job/35016/log_file When you look at the log, you see that the board starts off at the u-boot prompt. It then tries to do a "reboot", which (obviously) fails. So naturally, it then does a hard reset, and this is where it does something very odd: It interrupts the boot and tries to boot the previously installed test image. I haven't yet looked at the dispatcher code to figure out why (that's my next job). What then started alarm bells ringing was that I saw this: 1261680 bytes read reading uInitrd 1532597 bytes read reading board.dtb ** Unable to read "board.dtb" from mmc 0:5 ** So whatever the test image was, it was expecting a device tree blob, which I would have assumed would have to have been installed during deploy_linaro_image() being that if there is one it should just be part of the test boot deployment. So I looked at the log from the previous job: http://validation.linaro.org/lava-server/scheduler/job/34938/log_file#entry… and sure enough, you'll see at that mark the same issue. So there are two things: 1) There's some twisted logic in the dispatcher that's making it do odd things if it starts off in u-boot 2) Do we have an issue with dtbs not being handled properly by lava, or is it just that the hwpack was incomplete? Thanks Dave

11 years, 7 months

Re: [PATCH v4 1/3] devfreq: Core updates to support devices which can idle

by MyungJoo Ham

> Prepare devfreq core framework to support devices which > can idle. When device idleness is detected perhaps through > runtime-pm, need some mechanism to suspend devfreq load > monitoring and resume back when device is online. Present > code continues monitoring unless device is removed from > devfreq core. > > This patch introduces following design changes, > > - use per device work instead of global work to monitor device > load. This enables suspend/resume of device devfreq and > reduces monitoring code complexity. > - decouple delayed work based load monitoring logic from core > by introducing helpers functions to be used by governors. This > provides flexibility for governors either to use delayed work > based monitoring functions or to implement their own mechanism. > - devfreq core interacts with governors via events to perform > specific actions. These events include start/stop devfreq. > This sets ground for adding suspend/resume events. > > The devfreq apis are not modified and are kept intact. > > Signed-off-by: Rajagopal Venkat <rajagopal.venkat(a)linaro.org> Thank you! Reviewed and Tested (at Exynos4210-Nuri). Acked-by: MyungJoo Ham <myungjoo.ham(a)samsung.com> > --- > Documentation/ABI/testing/sysfs-class-devfreq | 8 - > drivers/devfreq/devfreq.c | 443 +++++++++++--------------- > drivers/devfreq/governor.h | 11 + > drivers/devfreq/governor_performance.c | 16 +- > drivers/devfreq/governor_powersave.c | 16 +- > drivers/devfreq/governor_simpleondemand.c | 24 ++ > drivers/devfreq/governor_userspace.c | 23 +- > include/linux/devfreq.h | 34 +- > 8 files changed, 279 insertions(+), 296 deletions(-) > > diff --git a/Documentation/ABI/testing/sysfs-class-devfreq b/Documentation/ABI/testing/sysfs-class-devfreq > index 23d78b5..89283b1 100644 > --- a/Documentation/ABI/testing/sysfs-class-devfreq > +++ b/Documentation/ABI/testing/sysfs-class-devfreq > @@ -21,14 +21,6 @@ Description: > The /sys/class/devfreq/.../cur_freq shows the current > frequency of the corresponding devfreq object. > > -What: /sys/class/devfreq/.../central_polling > -Date: September 2011 > -Contact: MyungJoo Ham <myungjoo.ham(a)samsung.com> > -Description: > - The /sys/class/devfreq/.../central_polling shows whether > - the devfreq ojbect is using devfreq-provided central > - polling mechanism or not. > - > What: /sys/class/devfreq/.../polling_interval > Date: September 2011 > Contact: MyungJoo Ham <myungjoo.ham(a)samsung.com> > diff --git a/drivers/devfreq/devfreq.c b/drivers/devfreq/devfreq.c > index b146d76..41a4099 100644 > --- a/drivers/devfreq/devfreq.c > +++ b/drivers/devfreq/devfreq.c > @@ -30,17 +30,11 @@ > struct class *devfreq_class; > > /* > - * devfreq_work periodically monitors every registered device. > - * The minimum polling interval is one jiffy. The polling interval is > - * determined by the minimum polling period among all polling devfreq > - * devices. The resolution of polling interval is one jiffy. > + * devfreq core provides delayed work based load monitoring helper > + * functions. Governors can use these or can implement their own > + * monitoring mechanism. > */ > -static bool polling; > static struct workqueue_struct *devfreq_wq; > -static struct delayed_work devfreq_work; > - > -/* wait removing if this is to be removed */ > -static struct devfreq *wait_remove_device; > > /* The list of all device-devfreq */ > static LIST_HEAD(devfreq_list); > @@ -72,6 +66,8 @@ static struct devfreq *find_device_devfreq(struct device *dev) > return ERR_PTR(-ENODEV); > } > > +/* Load monitoring helper functions for governors use */ > + > /** > * update_devfreq() - Reevaluate the device and configure frequency. > * @devfreq: the devfreq instance. > @@ -121,6 +117,152 @@ int update_devfreq(struct devfreq *devfreq) > } > > /** > + * devfreq_monitor() - Periodically poll devfreq objects. > + * @work: the work struct used to run devfreq_monitor periodically. > + * > + */ > +static void devfreq_monitor(struct work_struct *work) > +{ > + int err; > + struct devfreq *devfreq = container_of(work, > + struct devfreq, work.work); > + > + mutex_lock(&devfreq->lock); > + err = update_devfreq(devfreq); > + if (err) > + dev_err(&devfreq->dev, "dvfs failed with (%d) error\n", err); > + > + queue_delayed_work(devfreq_wq, &devfreq->work, > + msecs_to_jiffies(devfreq->profile->polling_ms)); > + mutex_unlock(&devfreq->lock); > +} > + > +/** > + * devfreq_monitor_start() - Start load monitoring of devfreq instance > + * @devfreq: the devfreq instance. > + * > + * Helper function for starting devfreq device load monitoing. By > + * default delayed work based monitoring is supported. Function > + * to be called from governor in response to DEVFREQ_GOV_START > + * event when device is added to devfreq framework. > + */ > +void devfreq_monitor_start(struct devfreq *devfreq) > +{ > + INIT_DEFERRABLE_WORK(&devfreq->work, devfreq_monitor); > + if (devfreq->profile->polling_ms) > + queue_delayed_work(devfreq_wq, &devfreq->work, > + msecs_to_jiffies(devfreq->profile->polling_ms)); > +} > + > +/** > + * devfreq_monitor_stop() - Stop load monitoring of a devfreq instance > + * @devfreq: the devfreq instance. > + * > + * Helper function to stop devfreq device load monitoing. Function > + * to be called from governor in response to DEVFREQ_GOV_STOP > + * event when device is removed from devfreq framework. > + */ > +void devfreq_monitor_stop(struct devfreq *devfreq) > +{ > + cancel_delayed_work_sync(&devfreq->work); > +} > + > +/** > + * devfreq_monitor_suspend() - Suspend load monitoring of a devfreq instance > + * @devfreq: the devfreq instance. > + * > + * Helper function to suspend devfreq device load monitoing. Function > + * to be called from governor in response to DEVFREQ_GOV_SUSPEND > + * event or when polling interval is set to zero. > + * > + * Note: Though this function is same as devfreq_monitor_stop(), > + * intentionally kept separate to provide hooks for collecting > + * transition statistics. > + */ > +void devfreq_monitor_suspend(struct devfreq *devfreq) > +{ > + mutex_lock(&devfreq->lock); > + if (devfreq->stop_polling) { > + mutex_unlock(&devfreq->lock); > + return; > + } > + > + devfreq->stop_polling = true; > + mutex_unlock(&devfreq->lock); > + cancel_delayed_work_sync(&devfreq->work); > +} > + > +/** > + * devfreq_monitor_resume() - Resume load monitoring of a devfreq instance > + * @devfreq: the devfreq instance. > + * > + * Helper function to resume devfreq device load monitoing. Function > + * to be called from governor in response to DEVFREQ_GOV_RESUME > + * event or when polling interval is set to non-zero. > + */ > +void devfreq_monitor_resume(struct devfreq *devfreq) > +{ > + mutex_lock(&devfreq->lock); > + if (!devfreq->stop_polling) > + goto out; > + > + if (!delayed_work_pending(&devfreq->work) && > + devfreq->profile->polling_ms) > + queue_delayed_work(devfreq_wq, &devfreq->work, > + msecs_to_jiffies(devfreq->profile->polling_ms)); > + devfreq->stop_polling = false; > + > +out: > + mutex_unlock(&devfreq->lock); > +} > + > +/** > + * devfreq_interval_update() - Update device devfreq monitoring interval > + * @devfreq: the devfreq instance. > + * @delay: new polling interval to be set. > + * > + * Helper function to set new load monitoring polling interval. Function > + * to be called from governor in response to DEVFREQ_GOV_INTERVAL event. > + */ > +void devfreq_interval_update(struct devfreq *devfreq, unsigned int *delay) > +{ > + unsigned int cur_delay = devfreq->profile->polling_ms; > + unsigned int new_delay = *delay; > + > + mutex_lock(&devfreq->lock); > + devfreq->profile->polling_ms = new_delay; > + > + if (devfreq->stop_polling) > + goto out; > + > + /* if new delay is zero, stop polling */ > + if (!new_delay) { > + mutex_unlock(&devfreq->lock); > + cancel_delayed_work_sync(&devfreq->work); > + return; > + } > + > + /* if current delay is zero, start polling with new delay */ > + if (!cur_delay) { > + queue_delayed_work(devfreq_wq, &devfreq->work, > + msecs_to_jiffies(devfreq->profile->polling_ms)); > + goto out; > + } > + > + /* if current delay is greater than new delay, restart polling */ > + if (cur_delay > new_delay) { > + mutex_unlock(&devfreq->lock); > + cancel_delayed_work_sync(&devfreq->work); > + mutex_lock(&devfreq->lock); > + if (!devfreq->stop_polling) > + queue_delayed_work(devfreq_wq, &devfreq->work, > + msecs_to_jiffies(devfreq->profile->polling_ms)); > + } > +out: > + mutex_unlock(&devfreq->lock); > +} > + > +/** > * devfreq_notifier_call() - Notify that the device frequency requirements > * has been changed out of devfreq framework. > * @nb the notifier_block (supposed to be devfreq->nb) > @@ -143,59 +285,33 @@ static int devfreq_notifier_call(struct notifier_block *nb, unsigned long type, > } > > /** > - * _remove_devfreq() - Remove devfreq from the device. > + * _remove_devfreq() - Remove devfreq from the devfreq list and > + release its resources. > * @devfreq: the devfreq struct > * @skip: skip calling device_unregister(). > - * > - * Note that the caller should lock devfreq->lock before calling > - * this. _remove_devfreq() will unlock it and free devfreq > - * internally. devfreq_list_lock should be locked by the caller > - * as well (not relased at return) > - * > - * Lock usage: > - * devfreq->lock: locked before call. > - * unlocked at return (and freed) > - * devfreq_list_lock: locked before call. > - * kept locked at return. > - * if devfreq is centrally polled. > - * > - * Freed memory: > - * devfreq > */ > static void _remove_devfreq(struct devfreq *devfreq, bool skip) > { > - if (!mutex_is_locked(&devfreq->lock)) { > - WARN(true, "devfreq->lock must be locked by the caller.\n"); > - return; > - } > - if (!devfreq->governor->no_central_polling && > - !mutex_is_locked(&devfreq_list_lock)) { > - WARN(true, "devfreq_list_lock must be locked by the caller.\n"); > + mutex_lock(&devfreq_list_lock); > + if (IS_ERR(find_device_devfreq(devfreq->dev.parent))) { > + mutex_unlock(&devfreq_list_lock); > + dev_warn(&devfreq->dev, "releasing devfreq which doesn't exist\n"); > return; > } > + list_del(&devfreq->node); > + mutex_unlock(&devfreq_list_lock); > > - if (devfreq->being_removed) > - return; > - > - devfreq->being_removed = true; > + devfreq->governor->event_handler(devfreq, DEVFREQ_GOV_STOP, NULL); > > if (devfreq->profile->exit) > devfreq->profile->exit(devfreq->dev.parent); > > - if (devfreq->governor->exit) > - devfreq->governor->exit(devfreq); > - > if (!skip && get_device(&devfreq->dev)) { > device_unregister(&devfreq->dev); > put_device(&devfreq->dev); > } > > - if (!devfreq->governor->no_central_polling) > - list_del(&devfreq->node); > - > - mutex_unlock(&devfreq->lock); > mutex_destroy(&devfreq->lock); > - > kfree(devfreq); > } > > @@ -210,130 +326,8 @@ static void _remove_devfreq(struct devfreq *devfreq, bool skip) > static void devfreq_dev_release(struct device *dev) > { > struct devfreq *devfreq = to_devfreq(dev); > - bool central_polling = !devfreq->governor->no_central_polling; > - > - /* > - * If devfreq_dev_release() was called by device_unregister() of > - * _remove_devfreq(), we cannot mutex_lock(&devfreq->lock) and > - * being_removed is already set. This also partially checks the case > - * where devfreq_dev_release() is called from a thread other than > - * the one called _remove_devfreq(); however, this case is > - * dealt completely with another following being_removed check. > - * > - * Because being_removed is never being > - * unset, we do not need to worry about race conditions on > - * being_removed. > - */ > - if (devfreq->being_removed) > - return; > > - if (central_polling) > - mutex_lock(&devfreq_list_lock); > - > - mutex_lock(&devfreq->lock); > - > - /* > - * Check being_removed flag again for the case where > - * devfreq_dev_release() was called in a thread other than the one > - * possibly called _remove_devfreq(). > - */ > - if (devfreq->being_removed) { > - mutex_unlock(&devfreq->lock); > - goto out; > - } > - > - /* devfreq->lock is unlocked and removed in _removed_devfreq() */ > _remove_devfreq(devfreq, true); > - > -out: > - if (central_polling) > - mutex_unlock(&devfreq_list_lock); > -} > - > -/** > - * devfreq_monitor() - Periodically poll devfreq objects. > - * @work: the work struct used to run devfreq_monitor periodically. > - * > - */ > -static void devfreq_monitor(struct work_struct *work) > -{ > - static unsigned long last_polled_at; > - struct devfreq *devfreq, *tmp; > - int error; > - unsigned long jiffies_passed; > - unsigned long next_jiffies = ULONG_MAX, now = jiffies; > - struct device *dev; > - > - /* Initially last_polled_at = 0, polling every device at bootup */ > - jiffies_passed = now - last_polled_at; > - last_polled_at = now; > - if (jiffies_passed == 0) > - jiffies_passed = 1; > - > - mutex_lock(&devfreq_list_lock); > - list_for_each_entry_safe(devfreq, tmp, &devfreq_list, node) { > - mutex_lock(&devfreq->lock); > - dev = devfreq->dev.parent; > - > - /* Do not remove tmp for a while */ > - wait_remove_device = tmp; > - > - if (devfreq->governor->no_central_polling || > - devfreq->next_polling == 0) { > - mutex_unlock(&devfreq->lock); > - continue; > - } > - mutex_unlock(&devfreq_list_lock); > - > - /* > - * Reduce more next_polling if devfreq_wq took an extra > - * delay. (i.e., CPU has been idled.) > - */ > - if (devfreq->next_polling <= jiffies_passed) { > - error = update_devfreq(devfreq); > - > - /* Remove a devfreq with an error. */ > - if (error && error != -EAGAIN) { > - > - dev_err(dev, "Due to update_devfreq error(%d), devfreq(%s) is removed from the device\n", > - error, devfreq->governor->name); > - > - /* > - * Unlock devfreq before locking the list > - * in order to avoid deadlock with > - * find_device_devfreq or others > - */ > - mutex_unlock(&devfreq->lock); > - mutex_lock(&devfreq_list_lock); > - /* Check if devfreq is already removed */ > - if (IS_ERR(find_device_devfreq(dev))) > - continue; > - mutex_lock(&devfreq->lock); > - /* This unlocks devfreq->lock and free it */ > - _remove_devfreq(devfreq, false); > - continue; > - } > - devfreq->next_polling = devfreq->polling_jiffies; > - } else { > - devfreq->next_polling -= jiffies_passed; > - } > - > - if (devfreq->next_polling) > - next_jiffies = (next_jiffies > devfreq->next_polling) ? > - devfreq->next_polling : next_jiffies; > - > - mutex_unlock(&devfreq->lock); > - mutex_lock(&devfreq_list_lock); > - } > - wait_remove_device = NULL; > - mutex_unlock(&devfreq_list_lock); > - > - if (next_jiffies > 0 && next_jiffies < ULONG_MAX) { > - polling = true; > - queue_delayed_work(devfreq_wq, &devfreq_work, next_jiffies); > - } else { > - polling = false; > - } > } > > /** > @@ -357,16 +351,13 @@ struct devfreq *devfreq_add_device(struct device *dev, > return ERR_PTR(-EINVAL); > } > > - > - if (!governor->no_central_polling) { > - mutex_lock(&devfreq_list_lock); > - devfreq = find_device_devfreq(dev); > - mutex_unlock(&devfreq_list_lock); > - if (!IS_ERR(devfreq)) { > - dev_err(dev, "%s: Unable to create devfreq for the device. It already has one.\n", __func__); > - err = -EINVAL; > - goto err_out; > - } > + mutex_lock(&devfreq_list_lock); > + devfreq = find_device_devfreq(dev); > + mutex_unlock(&devfreq_list_lock); > + if (!IS_ERR(devfreq)) { > + dev_err(dev, "%s: Unable to create devfreq for the device. It already has one.\n", __func__); > + err = -EINVAL; > + goto err_out; > } > > devfreq = kzalloc(sizeof(struct devfreq), GFP_KERNEL); > @@ -386,48 +377,41 @@ struct devfreq *devfreq_add_device(struct device *dev, > devfreq->governor = governor; > devfreq->previous_freq = profile->initial_freq; > devfreq->data = data; > - devfreq->next_polling = devfreq->polling_jiffies > - = msecs_to_jiffies(devfreq->profile->polling_ms); > devfreq->nb.notifier_call = devfreq_notifier_call; > > dev_set_name(&devfreq->dev, dev_name(dev)); > err = device_register(&devfreq->dev); > if (err) { > put_device(&devfreq->dev); > + mutex_unlock(&devfreq->lock); > goto err_dev; > } > > - if (governor->init) > - err = governor->init(devfreq); > - if (err) > - goto err_init; > - > mutex_unlock(&devfreq->lock); > > - if (governor->no_central_polling) > - goto out; > - > mutex_lock(&devfreq_list_lock); > - > list_add(&devfreq->node, &devfreq_list); > + mutex_unlock(&devfreq_list_lock); > > - if (devfreq_wq && devfreq->next_polling && !polling) { > - polling = true; > - queue_delayed_work(devfreq_wq, &devfreq_work, > - devfreq->next_polling); > + err = devfreq->governor->event_handler(devfreq, > + DEVFREQ_GOV_START, NULL); > + if (err) { > + dev_err(dev, "%s: Unable to start governor for the device\n", > + __func__); > + goto err_init; > } > - mutex_unlock(&devfreq_list_lock); > -out: > + > return devfreq; > > err_init: > + list_del(&devfreq->node); > device_unregister(&devfreq->dev); > err_dev: > - mutex_unlock(&devfreq->lock); > kfree(devfreq); > err_out: > return ERR_PTR(err); > } > +EXPORT_SYMBOL(devfreq_add_device); > > /** > * devfreq_remove_device() - Remove devfreq feature from a device. > @@ -435,30 +419,14 @@ err_out: > */ > int devfreq_remove_device(struct devfreq *devfreq) > { > - bool central_polling; > - > if (!devfreq) > return -EINVAL; > > - central_polling = !devfreq->governor->no_central_polling; > - > - if (central_polling) { > - mutex_lock(&devfreq_list_lock); > - while (wait_remove_device == devfreq) { > - mutex_unlock(&devfreq_list_lock); > - schedule(); > - mutex_lock(&devfreq_list_lock); > - } > - } > - > - mutex_lock(&devfreq->lock); > - _remove_devfreq(devfreq, false); /* it unlocks devfreq->lock */ > - > - if (central_polling) > - mutex_unlock(&devfreq_list_lock); > + _remove_devfreq(devfreq, false); > > return 0; > } > +EXPORT_SYMBOL(devfreq_remove_device); > > static ssize_t show_governor(struct device *dev, > struct device_attribute *attr, char *buf) > @@ -490,35 +458,13 @@ static ssize_t store_polling_interval(struct device *dev, > if (ret != 1) > goto out; > > - mutex_lock(&df->lock); > - df->profile->polling_ms = value; > - df->next_polling = df->polling_jiffies > - = msecs_to_jiffies(value); > - mutex_unlock(&df->lock); > - > + df->governor->event_handler(df, DEVFREQ_GOV_INTERVAL, &value); > ret = count; > > - if (df->governor->no_central_polling) > - goto out; > - > - mutex_lock(&devfreq_list_lock); > - if (df->next_polling > 0 && !polling) { > - polling = true; > - queue_delayed_work(devfreq_wq, &devfreq_work, > - df->next_polling); > - } > - mutex_unlock(&devfreq_list_lock); > out: > return ret; > } > > -static ssize_t show_central_polling(struct device *dev, > - struct device_attribute *attr, char *buf) > -{ > - return sprintf(buf, "%d\n", > - !to_devfreq(dev)->governor->no_central_polling); > -} > - > static ssize_t store_min_freq(struct device *dev, struct device_attribute *attr, > const char *buf, size_t count) > { > @@ -590,7 +536,6 @@ static ssize_t show_max_freq(struct device *dev, struct device_attribute *attr, > static struct device_attribute devfreq_attrs[] = { > __ATTR(governor, S_IRUGO, show_governor, NULL), > __ATTR(cur_freq, S_IRUGO, show_freq, NULL), > - __ATTR(central_polling, S_IRUGO, show_central_polling, NULL), > __ATTR(polling_interval, S_IRUGO | S_IWUSR, show_polling_interval, > store_polling_interval), > __ATTR(min_freq, S_IRUGO | S_IWUSR, show_min_freq, store_min_freq), > @@ -598,23 +543,6 @@ static struct device_attribute devfreq_attrs[] = { > { }, > }; > > -/** > - * devfreq_start_polling() - Initialize data structure for devfreq framework and > - * start polling registered devfreq devices. > - */ > -static int __init devfreq_start_polling(void) > -{ > - mutex_lock(&devfreq_list_lock); > - polling = false; > - devfreq_wq = create_freezable_workqueue("devfreq_wq"); > - INIT_DEFERRABLE_WORK(&devfreq_work, devfreq_monitor); > - mutex_unlock(&devfreq_list_lock); > - > - devfreq_monitor(&devfreq_work.work); > - return 0; > -} > -late_initcall(devfreq_start_polling); > - > static int __init devfreq_init(void) > { > devfreq_class = class_create(THIS_MODULE, "devfreq"); > @@ -622,7 +550,15 @@ static int __init devfreq_init(void) > pr_err("%s: couldn't create class\n", __FILE__); > return PTR_ERR(devfreq_class); > } > + > + devfreq_wq = create_freezable_workqueue("devfreq_wq"); > + if (IS_ERR(devfreq_wq)) { > + class_destroy(devfreq_class); > + pr_err("%s: couldn't create workqueue\n", __FILE__); > + return PTR_ERR(devfreq_wq); > + } > devfreq_class->dev_attrs = devfreq_attrs; > + > return 0; > } > subsys_initcall(devfreq_init); > @@ -630,6 +566,7 @@ subsys_initcall(devfreq_init); > static void __exit devfreq_exit(void) > { > class_destroy(devfreq_class); > + destroy_workqueue(devfreq_wq); > } > module_exit(devfreq_exit); > > diff --git a/drivers/devfreq/governor.h b/drivers/devfreq/governor.h > index ea7f13c..bb3aff3 100644 > --- a/drivers/devfreq/governor.h > +++ b/drivers/devfreq/governor.h > @@ -18,7 +18,18 @@ > > #define to_devfreq(DEV) container_of((DEV), struct devfreq, dev) > > +/* Devfreq events */ > +#define DEVFREQ_GOV_START 0x1 > +#define DEVFREQ_GOV_STOP 0x2 > +#define DEVFREQ_GOV_INTERVAL 0x3 > + > /* Caution: devfreq->lock must be locked before calling update_devfreq */ > extern int update_devfreq(struct devfreq *devfreq); > > +extern void devfreq_monitor_start(struct devfreq *devfreq); > +extern void devfreq_monitor_stop(struct devfreq *devfreq); > +extern void devfreq_monitor_suspend(struct devfreq *devfreq); > +extern void devfreq_monitor_resume(struct devfreq *devfreq); > +extern void devfreq_interval_update(struct devfreq *devfreq, > + unsigned int *delay); > #endif /* _GOVERNOR_H */ > diff --git a/drivers/devfreq/governor_performance.c b/drivers/devfreq/governor_performance.c > index af75ddd..eea3f9b 100644 > --- a/drivers/devfreq/governor_performance.c > +++ b/drivers/devfreq/governor_performance.c > @@ -26,14 +26,22 @@ static int devfreq_performance_func(struct devfreq *df, > return 0; > } > > -static int performance_init(struct devfreq *devfreq) > +static int devfreq_performance_handler(struct devfreq *devfreq, > + unsigned int event, void *data) > { > - return update_devfreq(devfreq); > + int ret = 0; > + > + if (event == DEVFREQ_GOV_START) { > + mutex_lock(&devfreq->lock); > + ret = update_devfreq(devfreq); > + mutex_unlock(&devfreq->lock); > + } > + > + return ret; > } > > const struct devfreq_governor devfreq_performance = { > .name = "performance", > - .init = performance_init, > .get_target_freq = devfreq_performance_func, > - .no_central_polling = true, > + .event_handler = devfreq_performance_handler, > }; > diff --git a/drivers/devfreq/governor_powersave.c b/drivers/devfreq/governor_powersave.c > index fec0cdb..2868d98 100644 > --- a/drivers/devfreq/governor_powersave.c > +++ b/drivers/devfreq/governor_powersave.c > @@ -23,14 +23,22 @@ static int devfreq_powersave_func(struct devfreq *df, > return 0; > } > > -static int powersave_init(struct devfreq *devfreq) > +static int devfreq_powersave_handler(struct devfreq *devfreq, > + unsigned int event, void *data) > { > - return update_devfreq(devfreq); > + int ret = 0; > + > + if (event == DEVFREQ_GOV_START) { > + mutex_lock(&devfreq->lock); > + ret = update_devfreq(devfreq); > + mutex_unlock(&devfreq->lock); > + } > + > + return ret; > } > > const struct devfreq_governor devfreq_powersave = { > .name = "powersave", > - .init = powersave_init, > .get_target_freq = devfreq_powersave_func, > - .no_central_polling = true, > + .event_handler = devfreq_powersave_handler, > }; > diff --git a/drivers/devfreq/governor_simpleondemand.c b/drivers/devfreq/governor_simpleondemand.c > index a2e3eae..cf94218 100644 > --- a/drivers/devfreq/governor_simpleondemand.c > +++ b/drivers/devfreq/governor_simpleondemand.c > @@ -12,6 +12,7 @@ > #include <linux/errno.h> > #include <linux/devfreq.h> > #include <linux/math64.h> > +#include "governor.h" > > /* Default constants for DevFreq-Simple-Ondemand (DFSO) */ > #define DFSO_UPTHRESHOLD (90) > @@ -88,7 +89,30 @@ static int devfreq_simple_ondemand_func(struct devfreq *df, > return 0; > } > > +int devfreq_simple_ondemand_handler(struct devfreq *devfreq, > + unsigned int event, void *data) > +{ > + switch (event) { > + case DEVFREQ_GOV_START: > + devfreq_monitor_start(devfreq); > + break; > + > + case DEVFREQ_GOV_STOP: > + devfreq_monitor_stop(devfreq); > + break; > + > + case DEVFREQ_GOV_INTERVAL: > + devfreq_interval_update(devfreq, (unsigned int *)data); > + break; > + default: > + break; > + } > + > + return 0; > +} > + > const struct devfreq_governor devfreq_simple_ondemand = { > .name = "simple_ondemand", > .get_target_freq = devfreq_simple_ondemand_func, > + .event_handler = devfreq_simple_ondemand_handler, > }; > diff --git a/drivers/devfreq/governor_userspace.c b/drivers/devfreq/governor_userspace.c > index 0681246..1fddee5 100644 > --- a/drivers/devfreq/governor_userspace.c > +++ b/drivers/devfreq/governor_userspace.c > @@ -116,10 +116,27 @@ static void userspace_exit(struct devfreq *devfreq) > devfreq->data = NULL; > } > > +int devfreq_userspace_handler(struct devfreq *devfreq, > + unsigned int event, void *data) > +{ > + int ret = 0; > + > + switch (event) { > + case DEVFREQ_GOV_START: > + ret = userspace_init(devfreq); > + break; > + case DEVFREQ_GOV_STOP: > + userspace_exit(devfreq); > + break; > + default: > + break; > + } > + > + return ret; > +} > + > const struct devfreq_governor devfreq_userspace = { > .name = "userspace", > .get_target_freq = devfreq_userspace_func, > - .init = userspace_init, > - .exit = userspace_exit, > - .no_central_polling = true, > + .event_handler = devfreq_userspace_handler, > }; > diff --git a/include/linux/devfreq.h b/include/linux/devfreq.h > index 281c72a..9cdffde 100644 > --- a/include/linux/devfreq.h > +++ b/include/linux/devfreq.h > @@ -91,25 +91,18 @@ struct devfreq_dev_profile { > * status of the device (load = busy_time / total_time). > * If no_central_polling is set, this callback is called > * only with update_devfreq() notified by OPP. > - * @init Called when the devfreq is being attached to a device > - * @exit Called when the devfreq is being removed from a > - * device. Governor should stop any internal routines > - * before return because related data may be > - * freed after exit(). > - * @no_central_polling Do not use devfreq's central polling mechanism. > - * When this is set, devfreq will not call > - * get_target_freq with devfreq_monitor(). However, > - * devfreq will call get_target_freq with > - * devfreq_update() notified by OPP framework. > + * @event_handler Callback for devfreq core framework to notify events > + * to governors. Events include per device governor > + * init and exit, opp changes out of devfreq, suspend > + * and resume of per device devfreq during device idle. > * > * Note that the callbacks are called with devfreq->lock locked by devfreq. > */ > struct devfreq_governor { > const char name[DEVFREQ_NAME_LEN]; > int (*get_target_freq)(struct devfreq *this, unsigned long *freq); > - int (*init)(struct devfreq *this); > - void (*exit)(struct devfreq *this); > - const bool no_central_polling; > + int (*event_handler)(struct devfreq *devfreq, > + unsigned int event, void *data); > }; > > /** > @@ -124,18 +117,13 @@ struct devfreq_governor { > * @nb notifier block used to notify devfreq object that it should > * reevaluate operable frequencies. Devfreq users may use > * devfreq.nb to the corresponding register notifier call chain. > - * @polling_jiffies interval in jiffies. > + * @work delayed work for load monitoring. > * @previous_freq previously configured frequency value. > - * @next_polling the number of remaining jiffies to poll with > - * "devfreq_monitor" executions to reevaluate > - * frequency/voltage of the device. Set by > - * profile's polling_ms interval. > * @data Private data of the governor. The devfreq framework does not > * touch this. > - * @being_removed a flag to mark that this object is being removed in > - * order to prevent trying to remove the object multiple times. > * @min_freq Limit minimum frequency requested by user (0: none) > * @max_freq Limit maximum frequency requested by user (0: none) > + * @stop_polling devfreq polling status of a device. > * > * This structure stores the devfreq information for a give device. > * > @@ -153,17 +141,15 @@ struct devfreq { > struct devfreq_dev_profile *profile; > const struct devfreq_governor *governor; > struct notifier_block nb; > + struct delayed_work work; > > - unsigned long polling_jiffies; > unsigned long previous_freq; > - unsigned int next_polling; > > void *data; /* private data for governors */ > > - bool being_removed; > - > unsigned long min_freq; > unsigned long max_freq; > + bool stop_polling; > }; > > #if defined(CONFIG_PM_DEVFREQ) > -- > 1.7.11.3 > > > > > > > >

11 years, 7 months

ANN: people.linaro.org going down for 30 min, Thursday at 19:00UTC

by Fathi Boudra

Hi All, There's a scheduled downtime for 30 min, tomorrow (Thursday) at 19:00UTC. We plan to increase the disk space available on people.linaro.org. Thanks. Cheers, -- Fathi Boudra Linaro Release Manager | LAVA Project Manager Linaro.org | Open source software for ARM SoCs

11 years, 7 months

big.LITTLE MP status Oct 9, 2012

by David Zinman

https://wiki.linaro.org/projects/big.LITTLE.MP Blueprint updates ================ https://blueprints.launchpad.net/linaro-power-kernel/+spec/sched-cooperatio… - slow progress https://blueprints.launchpad.net/linaro-qa/+spec/task-placement-testing https://blueprints.launchpad.net/linaro-qa/+spec/sched-test-porting-to-andr… - Both still blocked due to the TC2 hardware not available to the QA engineer. https://blueprints.launchpad.net/linaro-qa/+spec/bl-agitator-simultaneous-s… - in progress https://blueprints.launchpad.net/linaro-qa/+spec/bl-hotplug-tests - Blocked on bL MP development work Roadmap Cards ============== No updates this week General Items ============== The agenda for the big.LITTLE mini summit at LCE has been posted: http://summit.linaro.org/lce12/2012-11-01/ Ryan has a fix for UEFI so that LAVA boards can be booted with cpufreq/cpuidle enabled. Work is being done to get this deployed on LAVA. Issues ====== The TC2 hardware is still not in the hands of the Linaro QA engineer, it is currently hung up in customs. Linaro is attempting to expedite delivery. Little progress since last week. Testing infrastructure deployment has been progressing and we anticipate testing to begin on both TC2 and RTSM platforms shortly. The issues holding up execution are bootwrapper support for RTSM and hardware delivery for TC2. The project plan is at https://docs.google.com/a/linaro.org/document/d/1XI9FaEd6dYfCEDezboF7gWtrTO… -- David Zinman Linaro Release Manager | Project Manager Linaro.org | Open source software for ARM SoCs

11 years, 7 months

[Query]: CPUFREQ: Affected and related cpus in cpufreq

by viresh kumar

Hi All, Sorry for asking one of the most basic question of cpufreq :( I couldn't get the difference between affected (policy->cpus) and related cpus (policy->related_cpus) in cpufreq... As per Documentation/code: affected_cpus(policy->cpus): - List of CPUs that require software coordination of frequency. - Processors part of affected_cpus share policy struct - Policy limits the frequencies that the processor can work with. related_cpus(policy->related_cpus): - List of CPUs that need some sort of frequency coordination, whether software or hardware. - Processors part of related_cpus share governer. - Governer sets the rules, about when to change limits specified by policy. Correct? So, now comes the real question: - In which scenario's should we populate affected and related cpus? - Should related cpus will always be a superset of affected cpus? -- Viresh

11 years, 7 months

[PULL REQUEST]: ARM: perf: multi-PMU support v2

by Sudeep KarkadaNagesha

Hi Viresh, Please pull the following changes(v2) for the multiple CPU PMU support (re-based to v3.6). Changes v1->v2: 1. Incorporated review comments from Will for few patches(which he will be queuing for v3.8) 2. Dropped "ARM: perf: register the init functions with the bindings", still looking for alternate to list in implementation. 3. Added per-process event tracking support on multi-PMUs. Re-factored the original patch "ARM: perf: add support for per-cluster/multiple PMUs" 4. Fixed the bug reported by Tixy for non-FDT case ---------------------------------------------------------------- The following changes since commit a0d271cbfed1dd50278c6b06bead3d00ba0a88f9: Linux 3.6 (2012-09-30 16:47:46 -0700) are available in the git repository at: ssh://username@pdsw-ci.cambridge.arm.com:29418/kernel.git multi_pmu_v2 for you to fetch changes up to bd7fc4c65bd8ed994ded5aa0feeef91190fbd7be: ARM: perf: save/restore pmu registers in pm notifier (2012-10-09 18:20:45 +0100) ---------------------------------------------------------------- Axel Lin (1): ARM: ux500: Fix build error due to missing include of asm/pmu.h in cpu-db8500.c Jon Hunter (1): ARM: PMU: Add runtime PM Support Lorenzo Pieralisi (1): ARM: kernel: provide cluster to logical cpu mask mapping API Marc Zyngier (1): ARM: perf: add guest vs host discrimination Mark Rutland (1): ARM: perf: register cpu_notifier at driver init Sudeep KarkadaNagesha (11): ARM: pmu: remove arm_pmu_type enumeration ARM: perf: move irq registration into pmu implementation ARM: perf: allocate CPU PMU dynamically at probe time ARM: perf: consistently use struct perf_event in arm_pmu functions ARM: perf: check ARMv7 counter validity on a per-pmu basis ARM: perf: replace global CPU PMU pointer with per-cpu pointers ARM: perf: register CPU PMUs with idr types ARM: perf: set cpu affinity to support multiple PMUs ARM: perf: set cpu affinity for the irqs correctly ARM: perf: remove spaces in CPU PMU names ARM: perf: save/restore pmu registers in pm notifier Will Deacon (8): ARM: perf: add devicetree bindings for 11MPcore, A5, A7 and A15 PMUs ARM: pmu: remove unused reservation mechanism ARM: perf: remove mysterious compiler barrier ARM: perf: probe devicetree in preference to current CPU ARM: perf: prepare for moving CPU PMU code into separate file ARM: perf: move CPU-specific PMU handling code into separate file ARM: perf: return NOTIFY_DONE from cpu notifier when no available PMU ARM: perf: consistently use arm_pmu->name for PMU name Documentation/devicetree/bindings/arm/pmu.txt | 7 + MAINTAINERS | 1 - arch/arm/Kconfig | 8 +- arch/arm/include/asm/perf_event.h | 14 +- arch/arm/include/asm/pmu.h | 111 +++---- arch/arm/include/asm/topology.h | 3 + arch/arm/kernel/Makefile | 3 +- arch/arm/kernel/perf_event.c | 444 +++++++------------------ arch/arm/kernel/perf_event_cpu.c | 380 +++++++++++++++++++++ arch/arm/kernel/perf_event_v6.c | 134 ++++---- arch/arm/kernel/perf_event_v7.c | 307 +++++++++-------- arch/arm/kernel/perf_event_xscale.c | 163 ++++----- arch/arm/kernel/pmu.c | 36 -- arch/arm/kernel/topology.c | 27 ++ arch/arm/mach-bcmring/arch.c | 3 +- arch/arm/mach-omap2/devices.c | 3 +- arch/arm/mach-pxa/devices.c | 3 +- arch/arm/mach-realview/realview_eb.c | 3 +- arch/arm/mach-realview/realview_pb1176.c | 3 +- arch/arm/mach-realview/realview_pb11mp.c | 3 +- arch/arm/mach-realview/realview_pba8.c | 3 +- arch/arm/mach-realview/realview_pbx.c | 3 +- arch/arm/mach-tegra/devices.c | 3 +- arch/arm/mach-ux500/cpu-db8500.c | 4 +- arch/arm/mach-vexpress/ct-ca9x4.c | 3 +- arch/arm/plat-iop/pmu.c | 3 +- arch/arm/plat-samsung/devs.c | 3 +- 27 files changed, 942 insertions(+), 736 deletions(-) create mode 100644 arch/arm/kernel/perf_event_cpu.c delete mode 100644 arch/arm/kernel/pmu.c -- IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.

11 years, 7 months

← Newer
1
...
5
6
7
8
9
10
11
Older →

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

linaro-dev October 2012