diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2013-02-20 01:10:26 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-02-20 01:10:26 -0500 |
commit | ece8e0b2f9c980e5511fe8db2d68c6f1859b9d83 (patch) | |
tree | 70f1e3363080884965686576d079d24da8863a58 /include | |
parent | 67cb104b4c30bd52292b6a7f526349aab2dd5cbd (diff) | |
parent | a0327ff0eda915be623658babacef706099c11a8 (diff) |
Merge branch 'for-3.9-async' of git://git.kernel.org/pub/scm/linux/kernel/git/tj/wq
Pull async changes from Tejun Heo:
"These are followups for the earlier deadlock issue involving async
ending up waiting for itself through block requesting module[1]. The
following changes are made by these commits.
- Instead of requesting default elevator on each request_queue init,
block now requests it once early during boot.
- Kmod triggers warning if invoked from an async worker.
- Async synchronization implementation has been reimplemented. It's
a lot simpler now."
* 'for-3.9-async' of git://git.kernel.org/pub/scm/linux/kernel/git/tj/wq:
async: initialise list heads to fix crash
async: replace list of active domains with global list of pending items
async: keep pending tasks on async_domain and remove async_pending
async: use ULLONG_MAX for infinity cookie value
async: bring sanity to the use of words domain and running
async, kmod: warn on synchronous request_module() from async workers
block: don't request module during elevator init
init, block: try to load default elevator module early during boot
Diffstat (limited to 'include')
-rw-r--r-- | include/linux/async.h | 9 | ||||
-rw-r--r-- | include/linux/elevator.h | 5 | ||||
-rw-r--r-- | include/linux/init.h | 1 |
3 files changed, 9 insertions, 6 deletions
diff --git a/include/linux/async.h b/include/linux/async.h index 345169cfa304..a2e3f18b2ad6 100644 --- a/include/linux/async.h +++ b/include/linux/async.h | |||
@@ -19,8 +19,7 @@ typedef u64 async_cookie_t; | |||
19 | typedef void (async_func_ptr) (void *data, async_cookie_t cookie); | 19 | typedef void (async_func_ptr) (void *data, async_cookie_t cookie); |
20 | struct async_domain { | 20 | struct async_domain { |
21 | struct list_head node; | 21 | struct list_head node; |
22 | struct list_head domain; | 22 | struct list_head pending; |
23 | int count; | ||
24 | unsigned registered:1; | 23 | unsigned registered:1; |
25 | }; | 24 | }; |
26 | 25 | ||
@@ -29,8 +28,7 @@ struct async_domain { | |||
29 | */ | 28 | */ |
30 | #define ASYNC_DOMAIN(_name) \ | 29 | #define ASYNC_DOMAIN(_name) \ |
31 | struct async_domain _name = { .node = LIST_HEAD_INIT(_name.node), \ | 30 | struct async_domain _name = { .node = LIST_HEAD_INIT(_name.node), \ |
32 | .domain = LIST_HEAD_INIT(_name.domain), \ | 31 | .pending = LIST_HEAD_INIT(_name.pending), \ |
33 | .count = 0, \ | ||
34 | .registered = 1 } | 32 | .registered = 1 } |
35 | 33 | ||
36 | /* | 34 | /* |
@@ -39,8 +37,7 @@ struct async_domain { | |||
39 | */ | 37 | */ |
40 | #define ASYNC_DOMAIN_EXCLUSIVE(_name) \ | 38 | #define ASYNC_DOMAIN_EXCLUSIVE(_name) \ |
41 | struct async_domain _name = { .node = LIST_HEAD_INIT(_name.node), \ | 39 | struct async_domain _name = { .node = LIST_HEAD_INIT(_name.node), \ |
42 | .domain = LIST_HEAD_INIT(_name.domain), \ | 40 | .pending = LIST_HEAD_INIT(_name.pending), \ |
43 | .count = 0, \ | ||
44 | .registered = 0 } | 41 | .registered = 0 } |
45 | 42 | ||
46 | extern async_cookie_t async_schedule(async_func_ptr *ptr, void *data); | 43 | extern async_cookie_t async_schedule(async_func_ptr *ptr, void *data); |
diff --git a/include/linux/elevator.h b/include/linux/elevator.h index c03af7687bb4..186620631750 100644 --- a/include/linux/elevator.h +++ b/include/linux/elevator.h | |||
@@ -138,6 +138,7 @@ extern void elv_drain_elevator(struct request_queue *); | |||
138 | /* | 138 | /* |
139 | * io scheduler registration | 139 | * io scheduler registration |
140 | */ | 140 | */ |
141 | extern void __init load_default_elevator_module(void); | ||
141 | extern int elv_register(struct elevator_type *); | 142 | extern int elv_register(struct elevator_type *); |
142 | extern void elv_unregister(struct elevator_type *); | 143 | extern void elv_unregister(struct elevator_type *); |
143 | 144 | ||
@@ -206,5 +207,9 @@ enum { | |||
206 | INIT_LIST_HEAD(&(rq)->csd.list); \ | 207 | INIT_LIST_HEAD(&(rq)->csd.list); \ |
207 | } while (0) | 208 | } while (0) |
208 | 209 | ||
210 | #else /* CONFIG_BLOCK */ | ||
211 | |||
212 | static inline void load_default_elevator_module(void) { } | ||
213 | |||
209 | #endif /* CONFIG_BLOCK */ | 214 | #endif /* CONFIG_BLOCK */ |
210 | #endif | 215 | #endif |
diff --git a/include/linux/init.h b/include/linux/init.h index 10ed4f436458..861814710d52 100644 --- a/include/linux/init.h +++ b/include/linux/init.h | |||
@@ -153,6 +153,7 @@ extern unsigned int reset_devices; | |||
153 | /* used by init/main.c */ | 153 | /* used by init/main.c */ |
154 | void setup_arch(char **); | 154 | void setup_arch(char **); |
155 | void prepare_namespace(void); | 155 | void prepare_namespace(void); |
156 | void __init load_default_modules(void); | ||
156 | 157 | ||
157 | extern void (*late_time_init)(void); | 158 | extern void (*late_time_init)(void); |
158 | 159 | ||