From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754247AbbI3OI1 (ORCPT ); Wed, 30 Sep 2015 10:08:27 -0400 Received: from mailout4.w1.samsung.com ([210.118.77.14]:39226 "EHLO mailout4.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751793AbbI3OIT (ORCPT ); Wed, 30 Sep 2015 10:08:19 -0400 X-AuditID: cbfec7f5-f794b6d000001495-b4-560becd0674d Message-id: <560BECCE.6080304@samsung.com> Date: Wed, 30 Sep 2015 16:08:14 +0200 From: Jacek Anaszewski User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130804 Thunderbird/17.0.8 MIME-version: 1.0 To: Maciek Borzecki Cc: linux-kernel@vger.kernel.org, Richard Purdie , linux-leds@vger.kernel.org, linux-doc@vger.kernel.org, Jonathan Corbet Subject: Re: [PATCH 2/3] leds: add debugfs to device trigger References: In-reply-to: Content-type: text/plain; charset=ISO-8859-1; format=flowed Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFrrMLMWRmVeSWpSXmKPExsVy+t/xq7oX3nCHGaybwmnx5EA7o8XCtiUs Fpd3zWGz2PpmHaPFlDWtzBa7dz1ldWDz2DnrLrvH4r7JrB575v9g9fi8SS6AJYrLJiU1J7Ms tUjfLoErY/5GqYJv6hVfV6xgbGA8Lt/FyMkhIWAi0XzpCBuELSZx4d56IJuLQ0hgKaPEtGnH mSGcZ4wSPe+bmUGqeAW0JObO72YEsVkEVCU2L+4Gi7MJGEr8fPGaCcQWFYiQ+HN6HytEvaDE j8n3WEBsEQF9ib8Ne8GGMgssZJRYdHYTWJGwgI3EgrX7WCC2TWKUuLZqNdhUToEkiTl3PrKD 2MwC1hIrJ21jhLDlJTavecs8gVFgFpIls5CUzUJStoCReRWjaGppckFxUnqukV5xYm5xaV66 XnJ+7iZGSFB/3cG49JjVIUYBDkYlHt4XAtxhQqyJZcWVuYcYJTiYlUR4H90GCvGmJFZWpRbl xxeV5qQWH2KU5mBREueduet9iJBAemJJanZqakFqEUyWiYNTqoGxdQFXT9MC/keS575c2/s/ //6LbybGEhMCCpTE968xWs2zaJYOA2vk8s1uqgtv8KkEH1w7537AuvDJnyM/HsuQZK+q3nVW 0u7Hw4oLlyM2vYj+cGihMWN8/jue61flZdgVZgTuiVxWe3vq6iUBwb+nTPKYt6/w1Cop/d1T PcQ6ttwTZM/epxLcosRSnJFoqMVcVJwIAOusvO9mAgAA Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Maciek, Please test your solution thoroughly before submitting the next version. Writing to debugfs register attribute fails due to lack of proper copying from user memory, which makes testing impossible. Best Regards, Jacek Anaszewski On 09/28/2015 10:43 PM, Maciek Borzecki wrote: > Add debugfs entries for device activity trigger. Three entries are > created under /sys/kernel/debug/ledtrig-dev when the driver gets > loaded. These are: > > devices - cat'ing will produce a list of currently registered devices > in : format, a line for each device. > > register - echo'ing : will create a trigger for this > device > > trigger - echo'ing : will fire a trigger for this device > > Signed-off-by: Maciek Borzecki > --- > drivers/leds/trigger/ledtrig-device.c | 113 +++++++++++++++++++++++++++++++++- > 1 file changed, 111 insertions(+), 2 deletions(-) > > diff --git a/drivers/leds/trigger/ledtrig-device.c b/drivers/leds/trigger/ledtrig-device.c > index dbb8d7d2b4a0258149c581a040c416d412d9ceeb..6814db5e34d76974ae095d2e1c8f1f2e23dea79e 100644 > --- a/drivers/leds/trigger/ledtrig-device.c > +++ b/drivers/leds/trigger/ledtrig-device.c > @@ -16,15 +16,18 @@ > #include > #include > #include > +#include > +#include > +#include > > #define BLINK_DELAY 30 > static unsigned long blink_delay = BLINK_DELAY; > > static DECLARE_RWSEM(devs_list_lock); > static LIST_HEAD(devs_list); > +static struct dentry *debug_root; > > #define MAX_NAME_LEN 20 > - > struct ledtrig_dev_data { > char name[MAX_NAME_LEN]; > dev_t dev; > @@ -114,8 +117,11 @@ void ledtrig_dev_add(dev_t dev) > /* register with led triggers */ > led_trigger_register_simple(new_dev_trig->name, > &new_dev_trig->trig); > - else > + else { > + pr_warn("device %u:%u already registered\n", > + MAJOR(dev), MINOR(dev)); > kfree(new_dev_trig); > + } > } > EXPORT_SYMBOL(ledtrig_dev_add); > > @@ -167,13 +173,116 @@ static void ledtrig_dev_remove_all(void) > up_write(&devs_list_lock); > } > > +static int ledtrig_dev_devices_show(struct seq_file *s, void *unused) > +{ > + struct ledtrig_dev_data *dev_trig; > + > + down_read(&devs_list_lock); > + list_for_each_entry(dev_trig, &devs_list, node) { > + seq_printf(s, "%u:%u\n", MAJOR(dev_trig->dev), > + MINOR(dev_trig->dev)); > + } > + up_read(&devs_list_lock); > + return 0; > +} > + > +static int ledtrig_dev_devices_open(struct inode *inode, struct file *file) > +{ > + return single_open(file, ledtrig_dev_devices_show, > + &inode->i_private); > +} > + > +static const struct file_operations debug_devices_ops = { > + .owner = THIS_MODULE, > + .open = ledtrig_dev_devices_open, > + .read = seq_read, > + .llseek = seq_lseek, > + .release = single_release > +}; > + > +static int get_dev_from_user(const char __user *buf, size_t size, > + dev_t *dev) > +{ > + unsigned int major; > + unsigned int minor; > + int ret; > + > + WARN_ON(dev == NULL); > + if (dev == NULL) > + return -EINVAL; > + > + ret = sscanf(buf, "%u:%u", &major, &minor); > + if (ret < 2) > + return -EINVAL; > + > + *dev = MKDEV(major, minor); > + return 0; > +} > + > +static ssize_t ledtrig_dev_register_write(struct file *filp, > + const char __user *buf, > + size_t size, loff_t *off) > +{ > + dev_t dev; > + int ret; > + > + ret = get_dev_from_user(buf, size, &dev); > + if (ret < 0) > + return ret; > + > + pr_debug("got device %u:%u\n", MAJOR(dev), MINOR(dev)); > + ledtrig_dev_add(dev); > + > + return size; > +} > + > +static const struct file_operations debug_register_ops = { > + .owner = THIS_MODULE, > + .write = ledtrig_dev_register_write, > +}; > + > +static ssize_t ledtrig_dev_trigger_write(struct file *filp, > + const char __user *buf, > + size_t size, loff_t *off) > +{ > + dev_t dev; > + int ret; > + > + ret = get_dev_from_user(buf, size, &dev); > + if (ret < 0) > + return ret; > + > + pr_debug("trigger device %u:%u\n", MAJOR(dev), MINOR(dev)); > + ledtrig_dev_activity(dev); > + > + return size; > +} > + > +static const struct file_operations debug_trigger_ops = { > + .owner = THIS_MODULE, > + .write = ledtrig_dev_trigger_write, > +}; > + > static int __init ledtrig_dev_init(void) > { > + debug_root = debugfs_create_dir("ledtrig-dev", NULL); > + > + if (debug_root) { > + debugfs_create_file("devices", 0444, debug_root, NULL, > + &debug_devices_ops); > + debugfs_create_file("register", 0200, debug_root, NULL, > + &debug_register_ops); > + debugfs_create_file("trigger", 0200, debug_root, NULL, > + &debug_trigger_ops); > + } > + > return 0; > } > > static void __exit ledtrig_dev_exit(void) > { > + debugfs_remove_recursive(debug_root); > + > ledtrig_dev_remove_all(); > } > >