lkml.org 
[lkml]   [2014]   [Feb]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH 1/2] async: async device driver probing
Date
From: Wu Zhangjin <wuzhangjin@gmail.com>

[*Note*: NOT applicable, only for comments.]

To async the slow driver probing function of some devices, the device probing
support is modified to support async scheduling.

In order to async your driver probing function, please mask the async_probe
flag to 1, and to make sure one asynced probing is executed before an specified
point, please call async_synchronize_full() in that point..

Usage:

static struct i2c_driver test_driver = {
.driver = {
.name = TEST_DEV_NAME,
.owner = THIS_MODULE,
+ .async_probe = 1,
},

Signed-off-by: Wu Zhangjin <falcon@meizu.com>
---
drivers/base/dd.c | 36 +++++++++++++++++++++++++++++++++---
include/linux/device.h | 2 ++
2 files changed, 35 insertions(+), 3 deletions(-)

diff --git a/drivers/base/dd.c b/drivers/base/dd.c
index 0605176..357f36e 100644
--- a/drivers/base/dd.c
+++ b/drivers/base/dd.c
@@ -23,6 +23,7 @@
#include <linux/kthread.h>
#include <linux/wait.h>
#include <linux/async.h>
+#include <linux/slab.h>
#include <linux/pm_runtime.h>
#include <linux/pinctrl/devinfo.h>

@@ -357,6 +358,11 @@ void wait_for_device_probe(void)
}
EXPORT_SYMBOL_GPL(wait_for_device_probe);

+struct stupid_thread_structure {
+ struct device_driver *drv;
+ struct device *dev;
+};
+
/**
* driver_probe_device - attempt to bind device & driver together
* @drv: driver to bind a device to
@@ -368,8 +374,23 @@ EXPORT_SYMBOL_GPL(wait_for_device_probe);
* This function must be called with @dev lock held. When called for a
* USB interface, @dev->parent lock must be held as well.
*/
+static void __driver_probe_device(void *void_data, async_cookie_t cookie)
+{
+ struct stupid_thread_structure *data = void_data;
+ struct device_driver *drv = data->drv;
+ struct device *dev = data->dev;
+
+ pm_runtime_barrier(dev);
+ really_probe(dev, drv);
+ pm_request_idle(dev);
+
+ kfree(data);
+}
+
int driver_probe_device(struct device_driver *drv, struct device *dev)
{
+ struct stupid_thread_structure *data;
+ async_cookie_t cookie;
int ret = 0;

if (!device_is_registered(dev))
@@ -378,9 +399,18 @@ int driver_probe_device(struct device_driver *drv, struct device *dev)
pr_debug("bus: '%s': %s: matched device %s with driver %s\n",
drv->bus->name, __func__, dev_name(dev), drv->name);

- pm_runtime_barrier(dev);
- ret = really_probe(dev, drv);
- pm_request_idle(dev);
+ if (drv->async_probe) {
+ data = kmalloc(sizeof(*data), GFP_KERNEL);
+ data->drv = drv;
+ data->dev = dev;
+
+ cookie = async_schedule(__driver_probe_device, data);
+ pr_info("%s: async call %s driver, cookie is %llu\n", __func__, drv->name, cookie);
+ } else {
+ pm_runtime_barrier(dev);
+ ret = really_probe(dev, drv);
+ pm_request_idle(dev);
+ }

return ret;
}
diff --git a/include/linux/device.h b/include/linux/device.h
index 952b010..f39ee48 100644
--- a/include/linux/device.h
+++ b/include/linux/device.h
@@ -247,6 +247,8 @@ struct device_driver {
const struct dev_pm_ops *pm;

struct driver_private *p;
+
+ unsigned int async_probe:1;
};


--
1.7.10.4


\
 
 \ /
  Last update: 2014-02-08 12:41    [W:0.053 / U:0.100 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site