diff options
author | Stephen Hemminger <shemminger@osdl.org> | 2005-11-08 17:39:42 (GMT) |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2005-11-08 17:39:42 (GMT) |
commit | 9ee6b535af4c2c97b4e3b88f37f244bf1004ebd4 (patch) | |
tree | c8a45fcf863e8d72b91fd91ef94fd1f2d70fecb3 | |
parent | b541ca2c5a3f3f399d6f2ec9da33c1be5a8d8c19 (diff) | |
download | linux-fsl-qoriq-9ee6b535af4c2c97b4e3b88f37f244bf1004ebd4.tar.xz |
[NET]: sk_add_backlog convert from macro to inline
There is no reason for sk_add_backlog to be a macro. It can
just be an inline function and get type checking.
Signed-off-by: Stephen Hemminger <shemminger@osdl.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | include/net/sock.h | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/include/net/sock.h b/include/net/sock.h index e0498bd..ff13c4c 100644 --- a/include/net/sock.h +++ b/include/net/sock.h @@ -461,16 +461,16 @@ static inline void sk_stream_free_skb(struct sock *sk, struct sk_buff *skb) } /* The per-socket spinlock must be held here. */ -#define sk_add_backlog(__sk, __skb) \ -do { if (!(__sk)->sk_backlog.tail) { \ - (__sk)->sk_backlog.head = \ - (__sk)->sk_backlog.tail = (__skb); \ - } else { \ - ((__sk)->sk_backlog.tail)->next = (__skb); \ - (__sk)->sk_backlog.tail = (__skb); \ - } \ - (__skb)->next = NULL; \ -} while(0) +static inline void sk_add_backlog(struct sock *sk, struct sk_buff *skb) +{ + if (!sk->sk_backlog.tail) { + sk->sk_backlog.head = sk->sk_backlog.tail = skb; + } else { + sk->sk_backlog.tail->next = skb; + sk->sk_backlog.tail = skb; + } + skb->next = NULL; +} #define sk_wait_event(__sk, __timeo, __condition) \ ({ int rc; \ |