summaryrefslogtreecommitdiffstats
path: root/mm/failslab.c
diff options
context:
space:
mode:
authorViresh Kumar <viresh.kumar@linaro.org>2015-09-26 18:04:07 -0400
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2015-10-04 06:36:07 -0400
commit621a5f7ad9cd1ce7933f1d302067cbd58354173c (patch)
treea9cd51d4b4e70286a339ffc463618d0f33279c67 /mm/failslab.c
parent6e58f752a6502b43e039fd7df2c7c5cde8dde437 (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.c8
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
4static struct { 4static 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
14bool should_failslab(size_t size, gfp_t gfpflags, unsigned long cache_flags) 14bool should_failslab(size_t size, gfp_t gfpflags, unsigned long cache_flags)