datapath: Use 'bool' instead of 'int' where appropriate.
authorBen Pfaff <blp@nicira.com>
Fri, 13 Aug 2010 17:18:28 +0000 (10:18 -0700)
committerBen Pfaff <blp@nicira.com>
Thu, 26 Aug 2010 16:15:42 +0000 (09:15 -0700)
'bool' is better modern kernel style.

Signed-off-by: Ben Pfaff <blp@nicira.com>
datapath/flow.c

index f01071c61b6f4f1cc7e85ab201777ed30d7cb4e0..a64152b7fab927bd19621f7443276202b7a815f6 100644 (file)
@@ -49,13 +49,13 @@ struct arp_eth_header
        unsigned char       ar_tip[4];          /* target IP address        */
 } __attribute__((packed));
 
-static inline int arphdr_ok(struct sk_buff *skb)
+static inline bool arphdr_ok(struct sk_buff *skb)
 {
        int nh_ofs = skb_network_offset(skb);
        return pskb_may_pull(skb, nh_ofs + sizeof(struct arp_eth_header));
 }
 
-static inline int iphdr_ok(struct sk_buff *skb)
+static inline bool iphdr_ok(struct sk_buff *skb)
 {
        int nh_ofs = skb_network_offset(skb);
        if (skb->len >= nh_ofs + sizeof(struct iphdr)) {
@@ -63,10 +63,10 @@ static inline int iphdr_ok(struct sk_buff *skb)
                return (ip_len >= sizeof(struct iphdr)
                        && pskb_may_pull(skb, nh_ofs + ip_len));
        }
-       return 0;
+       return false;
 }
 
-static inline int tcphdr_ok(struct sk_buff *skb)
+static inline bool tcphdr_ok(struct sk_buff *skb)
 {
        int th_ofs = skb_transport_offset(skb);
        if (pskb_may_pull(skb, th_ofs + sizeof(struct tcphdr))) {
@@ -74,16 +74,16 @@ static inline int tcphdr_ok(struct sk_buff *skb)
                return (tcp_len >= sizeof(struct tcphdr)
                        && skb->len >= th_ofs + tcp_len);
        }
-       return 0;
+       return false;
 }
 
-static inline int udphdr_ok(struct sk_buff *skb)
+static inline bool udphdr_ok(struct sk_buff *skb)
 {
        int th_ofs = skb_transport_offset(skb);
        return pskb_may_pull(skb, th_ofs + sizeof(struct udphdr));
 }
 
-static inline int icmphdr_ok(struct sk_buff *skb)
+static inline bool icmphdr_ok(struct sk_buff *skb)
 {
        int th_ofs = skb_transport_offset(skb);
        return pskb_may_pull(skb, th_ofs + sizeof(struct icmphdr));