diff options
author | Herbert Xu <herbert@gondor.apana.org.au> | 2007-09-16 19:24:44 -0400 |
---|---|---|
committer | David S. Miller <davem@sunset.davemloft.net> | 2007-10-10 19:51:24 -0400 |
commit | 0cfad07555312468296ea3bbbcdf99038f58678b (patch) | |
tree | a95c432bcafaf22e965dcaf27bfd4a61723d1d64 /include/net/netlink.h | |
parent | 007e3936bdaaa012483c9fe06ca71c272458c710 (diff) |
[NETLINK]: Avoid pointer in netlink_run_queue
I was looking at Patrick's fix to inet_diag and it occured
to me that we're using a pointer argument to return values
unnecessarily in netlink_run_queue. Changing it to return
the value will allow the compiler to generate better code
since the value won't have to be memory-backed.
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/net/netlink.h')
-rw-r--r-- | include/net/netlink.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/net/netlink.h b/include/net/netlink.h index 695e613a207b..83113dfcbd04 100644 --- a/include/net/netlink.h +++ b/include/net/netlink.h | |||
@@ -220,7 +220,7 @@ struct nl_info { | |||
220 | u32 pid; | 220 | u32 pid; |
221 | }; | 221 | }; |
222 | 222 | ||
223 | extern void netlink_run_queue(struct sock *sk, unsigned int *qlen, | 223 | extern unsigned int netlink_run_queue(struct sock *sk, unsigned int qlen, |
224 | int (*cb)(struct sk_buff *, | 224 | int (*cb)(struct sk_buff *, |
225 | struct nlmsghdr *)); | 225 | struct nlmsghdr *)); |
226 | extern int nlmsg_notify(struct sock *sk, struct sk_buff *skb, | 226 | extern int nlmsg_notify(struct sock *sk, struct sk_buff *skb, |