diff options
author | Rusty Russell <rusty@rustcorp.com.au> | 2011-12-19 09:08:01 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2011-12-19 22:27:29 -0500 |
commit | eb93992207dadb946a3b5cf4544957dc924a6f58 (patch) | |
tree | 8d840ea0e0e5eb3cf6207a2608d434475a9076b4 /net/dccp | |
parent | 3db1cd5c05f35fb43eb134df6f321de4e63141f2 (diff) |
module_param: make bool parameters really bool (net & drivers/net)
module_param(bool) used to counter-intuitively take an int. In
fddd5201 (mid-2009) we allowed bool or int/unsigned int using a messy
trick.
It's time to remove the int/unsigned int option. For this version
it'll simply give a warning, but it'll break next kernel version.
(Thanks to Joe Perches for suggesting coccinelle for 0/1 -> true/false).
Cc: "David S. Miller" <davem@davemloft.net>
Cc: netdev@vger.kernel.org
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/dccp')
-rw-r--r-- | net/dccp/ccids/ccid2.c | 4 | ||||
-rw-r--r-- | net/dccp/ccids/ccid3.c | 2 | ||||
-rw-r--r-- | net/dccp/ccids/lib/tfrc.c | 2 | ||||
-rw-r--r-- | net/dccp/ccids/lib/tfrc.h | 2 | ||||
-rw-r--r-- | net/dccp/dccp.h | 2 | ||||
-rw-r--r-- | net/dccp/proto.c | 2 |
6 files changed, 7 insertions, 7 deletions
diff --git a/net/dccp/ccids/ccid2.c b/net/dccp/ccids/ccid2.c index 67164bb6ae4d..f053198e730c 100644 --- a/net/dccp/ccids/ccid2.c +++ b/net/dccp/ccids/ccid2.c | |||
@@ -29,7 +29,7 @@ | |||
29 | 29 | ||
30 | 30 | ||
31 | #ifdef CONFIG_IP_DCCP_CCID2_DEBUG | 31 | #ifdef CONFIG_IP_DCCP_CCID2_DEBUG |
32 | static int ccid2_debug; | 32 | static bool ccid2_debug; |
33 | #define ccid2_pr_debug(format, a...) DCCP_PR_DEBUG(ccid2_debug, format, ##a) | 33 | #define ccid2_pr_debug(format, a...) DCCP_PR_DEBUG(ccid2_debug, format, ##a) |
34 | #else | 34 | #else |
35 | #define ccid2_pr_debug(format, a...) | 35 | #define ccid2_pr_debug(format, a...) |
@@ -174,7 +174,7 @@ out: | |||
174 | /* | 174 | /* |
175 | * Congestion window validation (RFC 2861). | 175 | * Congestion window validation (RFC 2861). |
176 | */ | 176 | */ |
177 | static int ccid2_do_cwv = 1; | 177 | static bool ccid2_do_cwv = true; |
178 | module_param(ccid2_do_cwv, bool, 0644); | 178 | module_param(ccid2_do_cwv, bool, 0644); |
179 | MODULE_PARM_DESC(ccid2_do_cwv, "Perform RFC2861 Congestion Window Validation"); | 179 | MODULE_PARM_DESC(ccid2_do_cwv, "Perform RFC2861 Congestion Window Validation"); |
180 | 180 | ||
diff --git a/net/dccp/ccids/ccid3.c b/net/dccp/ccids/ccid3.c index 3d604e1349c0..560627307200 100644 --- a/net/dccp/ccids/ccid3.c +++ b/net/dccp/ccids/ccid3.c | |||
@@ -38,7 +38,7 @@ | |||
38 | #include <asm/unaligned.h> | 38 | #include <asm/unaligned.h> |
39 | 39 | ||
40 | #ifdef CONFIG_IP_DCCP_CCID3_DEBUG | 40 | #ifdef CONFIG_IP_DCCP_CCID3_DEBUG |
41 | static int ccid3_debug; | 41 | static bool ccid3_debug; |
42 | #define ccid3_pr_debug(format, a...) DCCP_PR_DEBUG(ccid3_debug, format, ##a) | 42 | #define ccid3_pr_debug(format, a...) DCCP_PR_DEBUG(ccid3_debug, format, ##a) |
43 | #else | 43 | #else |
44 | #define ccid3_pr_debug(format, a...) | 44 | #define ccid3_pr_debug(format, a...) |
diff --git a/net/dccp/ccids/lib/tfrc.c b/net/dccp/ccids/lib/tfrc.c index 1f94b7e01d39..62b5828acde0 100644 --- a/net/dccp/ccids/lib/tfrc.c +++ b/net/dccp/ccids/lib/tfrc.c | |||
@@ -8,7 +8,7 @@ | |||
8 | #include "tfrc.h" | 8 | #include "tfrc.h" |
9 | 9 | ||
10 | #ifdef CONFIG_IP_DCCP_TFRC_DEBUG | 10 | #ifdef CONFIG_IP_DCCP_TFRC_DEBUG |
11 | int tfrc_debug; | 11 | bool tfrc_debug; |
12 | module_param(tfrc_debug, bool, 0644); | 12 | module_param(tfrc_debug, bool, 0644); |
13 | MODULE_PARM_DESC(tfrc_debug, "Enable TFRC debug messages"); | 13 | MODULE_PARM_DESC(tfrc_debug, "Enable TFRC debug messages"); |
14 | #endif | 14 | #endif |
diff --git a/net/dccp/ccids/lib/tfrc.h b/net/dccp/ccids/lib/tfrc.h index f8ee3f549770..ed698c42a5fb 100644 --- a/net/dccp/ccids/lib/tfrc.h +++ b/net/dccp/ccids/lib/tfrc.h | |||
@@ -21,7 +21,7 @@ | |||
21 | #include "packet_history.h" | 21 | #include "packet_history.h" |
22 | 22 | ||
23 | #ifdef CONFIG_IP_DCCP_TFRC_DEBUG | 23 | #ifdef CONFIG_IP_DCCP_TFRC_DEBUG |
24 | extern int tfrc_debug; | 24 | extern bool tfrc_debug; |
25 | #define tfrc_pr_debug(format, a...) DCCP_PR_DEBUG(tfrc_debug, format, ##a) | 25 | #define tfrc_pr_debug(format, a...) DCCP_PR_DEBUG(tfrc_debug, format, ##a) |
26 | #else | 26 | #else |
27 | #define tfrc_pr_debug(format, a...) | 27 | #define tfrc_pr_debug(format, a...) |
diff --git a/net/dccp/dccp.h b/net/dccp/dccp.h index 5818032e35a9..29d6bb629a6c 100644 --- a/net/dccp/dccp.h +++ b/net/dccp/dccp.h | |||
@@ -39,7 +39,7 @@ | |||
39 | "%s: " fmt, __func__, ##a) | 39 | "%s: " fmt, __func__, ##a) |
40 | 40 | ||
41 | #ifdef CONFIG_IP_DCCP_DEBUG | 41 | #ifdef CONFIG_IP_DCCP_DEBUG |
42 | extern int dccp_debug; | 42 | extern bool dccp_debug; |
43 | #define dccp_pr_debug(format, a...) DCCP_PR_DEBUG(dccp_debug, format, ##a) | 43 | #define dccp_pr_debug(format, a...) DCCP_PR_DEBUG(dccp_debug, format, ##a) |
44 | #define dccp_pr_debug_cat(format, a...) DCCP_PRINTK(dccp_debug, format, ##a) | 44 | #define dccp_pr_debug_cat(format, a...) DCCP_PRINTK(dccp_debug, format, ##a) |
45 | #define dccp_debug(fmt, a...) dccp_pr_debug_cat(KERN_DEBUG fmt, ##a) | 45 | #define dccp_debug(fmt, a...) dccp_pr_debug_cat(KERN_DEBUG fmt, ##a) |
diff --git a/net/dccp/proto.c b/net/dccp/proto.c index e742f90a6858..7065c0ae1e7b 100644 --- a/net/dccp/proto.c +++ b/net/dccp/proto.c | |||
@@ -1099,7 +1099,7 @@ module_param(thash_entries, int, 0444); | |||
1099 | MODULE_PARM_DESC(thash_entries, "Number of ehash buckets"); | 1099 | MODULE_PARM_DESC(thash_entries, "Number of ehash buckets"); |
1100 | 1100 | ||
1101 | #ifdef CONFIG_IP_DCCP_DEBUG | 1101 | #ifdef CONFIG_IP_DCCP_DEBUG |
1102 | int dccp_debug; | 1102 | bool dccp_debug; |
1103 | module_param(dccp_debug, bool, 0644); | 1103 | module_param(dccp_debug, bool, 0644); |
1104 | MODULE_PARM_DESC(dccp_debug, "Enable debug messages"); | 1104 | MODULE_PARM_DESC(dccp_debug, "Enable debug messages"); |
1105 | 1105 | ||