Skip to content
Snippets Groups Projects
Commit 97c18e2c authored by Herbert Xu's avatar Herbert Xu Committed by Linus Torvalds
Browse files

module: try_then_request_module must wait


Since the whole point of try_then_request_module is to retry
the operation after a module has been loaded, we must wait for
the module to fully load.

Otherwise all sort of things start breaking, e.g., you won't
be able to read your encrypted disks on the first attempt.

Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
Tested-by: default avatarMaciej Rutecki <maciej.rutecki@gmail.com>
Tested-by: default avatarPatrick McHardy <kaber@trash.net>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent f4efdd65
No related branches found
No related tags found
No related merge requests found
...@@ -34,7 +34,7 @@ extern int __request_module(bool wait, const char *name, ...) \ ...@@ -34,7 +34,7 @@ extern int __request_module(bool wait, const char *name, ...) \
#define request_module(mod...) __request_module(true, mod) #define request_module(mod...) __request_module(true, mod)
#define request_module_nowait(mod...) __request_module(false, mod) #define request_module_nowait(mod...) __request_module(false, mod)
#define try_then_request_module(x, mod...) \ #define try_then_request_module(x, mod...) \
((x) ?: (__request_module(false, mod), (x))) ((x) ?: (__request_module(true, mod), (x)))
#else #else
static inline int request_module(const char *name, ...) { return -ENOSYS; } static inline int request_module(const char *name, ...) { return -ENOSYS; }
static inline int request_module_nowait(const char *name, ...) { return -ENOSYS; } static inline int request_module_nowait(const char *name, ...) { return -ENOSYS; }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please to comment