diff options
| author |   Stephen Hemminger <shemminger@osdl.org> | 2005-11-08 09:39:42 -0800 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2005-11-08 09:39:42 -0800 |
| commit | 9ee6b535af4c2c97b4e3b88f37f244bf1004ebd4 (patch) | |
| tree | c8a45fcf863e8d72b91fd91ef94fd1f2d70fecb3 | |
| parent | [PKT_SCHED]: Correctly handle empty ematch trees (diff) | |
| download | linux-dev-9ee6b535af4c2c97b4e3b88f37f244bf1004ebd4.tar.xz linux-dev-9ee6b535af4c2c97b4e3b88f37f244bf1004ebd4.zip | |
[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 e0498bd36004..ff13c4cc287a 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; \ |