diff options
author | Viresh Kumar <viresh.kumar@linaro.org> | 2015-09-26 18:04:07 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2015-10-04 06:36:07 -0400 |
commit | 621a5f7ad9cd1ce7933f1d302067cbd58354173c (patch) | |
tree | a9cd51d4b4e70286a339ffc463618d0f33279c67 /mm/failslab.c | |
parent | 6e58f752a6502b43e039fd7df2c7c5cde8dde437 (diff) |
debugfs: Pass bool pointer to debugfs_create_bool()
Its a bit odd that debugfs_create_bool() takes 'u32 *' as an argument,
when all it needs is a boolean pointer.
It would be better to update this API to make it accept 'bool *'
instead, as that will make it more consistent and often more convenient.
Over that bool takes just a byte.
That required updates to all user sites as well, in the same commit
updating the API. regmap core was also using
debugfs_{read|write}_file_bool(), directly and variable types were
updated for that to be bool as well.
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Acked-by: Mark Brown <broonie@kernel.org>
Acked-by: Charles Keepax <ckeepax@opensource.wolfsonmicro.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'mm/failslab.c')
-rw-r--r-- | mm/failslab.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/mm/failslab.c b/mm/failslab.c index fefaabaab76d..98fb490311eb 100644 --- a/mm/failslab.c +++ b/mm/failslab.c | |||
@@ -3,12 +3,12 @@ | |||
3 | 3 | ||
4 | static struct { | 4 | static struct { |
5 | struct fault_attr attr; | 5 | struct fault_attr attr; |
6 | u32 ignore_gfp_wait; | 6 | bool ignore_gfp_wait; |
7 | int cache_filter; | 7 | bool cache_filter; |
8 | } failslab = { | 8 | } failslab = { |
9 | .attr = FAULT_ATTR_INITIALIZER, | 9 | .attr = FAULT_ATTR_INITIALIZER, |
10 | .ignore_gfp_wait = 1, | 10 | .ignore_gfp_wait = true, |
11 | .cache_filter = 0, | 11 | .cache_filter = false, |
12 | }; | 12 | }; |
13 | 13 | ||
14 | bool should_failslab(size_t size, gfp_t gfpflags, unsigned long cache_flags) | 14 | bool should_failslab(size_t size, gfp_t gfpflags, unsigned long cache_flags) |