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 | |
| 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')
| -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, |