net/nfnetlink: lockdep_nfnl_is_held can be boolean
authorYaowei Bai <bywxiaobai@163.com>
Thu, 8 Oct 2015 13:28:56 +0000 (21:28 +0800)
committerDavid S. Miller <davem@davemloft.net>
Fri, 9 Oct 2015 14:49:00 +0000 (07:49 -0700)
This patch makes lockdep_nfnl_is_held return bool to improve
readability due to this particular function only using either
one or zero as its return value.

No functional change.

Signed-off-by: Yaowei Bai <bywxiaobai@163.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/linux/netfilter/nfnetlink.h
net/netfilter/nfnetlink.c

index e955d47306259c5867d80831bd78d99490849b0b..249d1bb01e03b0367f56f251c02269b24be0dac6 100644 (file)
@@ -45,11 +45,11 @@ int nfnetlink_unicast(struct sk_buff *skb, struct net *net, u32 portid,
 void nfnl_lock(__u8 subsys_id);
 void nfnl_unlock(__u8 subsys_id);
 #ifdef CONFIG_PROVE_LOCKING
-int lockdep_nfnl_is_held(__u8 subsys_id);
+bool lockdep_nfnl_is_held(__u8 subsys_id);
 #else
-static inline int lockdep_nfnl_is_held(__u8 subsys_id)
+static inline bool lockdep_nfnl_is_held(__u8 subsys_id)
 {
-       return 1;
+       return true;
 }
 #endif /* CONFIG_PROVE_LOCKING */
 
index 70277b11f742e8f0a2756c2ba54e1565419adc95..f1d9e887f5b157b58578a5a7244d65be788bb13b 100644 (file)
@@ -64,7 +64,7 @@ void nfnl_unlock(__u8 subsys_id)
 EXPORT_SYMBOL_GPL(nfnl_unlock);
 
 #ifdef CONFIG_PROVE_LOCKING
-int lockdep_nfnl_is_held(u8 subsys_id)
+bool lockdep_nfnl_is_held(u8 subsys_id)
 {
        return lockdep_is_held(&table[subsys_id].mutex);
 }