diff options
author | David S. Miller <davem@davemloft.net> | 2011-03-09 23:57:50 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2011-03-09 23:57:50 -0500 |
commit | cc7e17ea0427a5df319e43606a3d6c53b13a6e9c (patch) | |
tree | 3a4030df19e212619ce4437c2100a23a35e759dd /net/ipv4/fib_frontend.c | |
parent | 67e28ffd864eebbaf48b404d0a8cb4edd2bdc924 (diff) |
ipv4: Optimize flow initialization in fib_validate_source().
Like in commit 44713b67db10c774f14280c129b0d5fd13c70cf2
("ipv4: Optimize flow initialization in output route lookup."
we can optimize the on-stack flow setup to only initialize
the members which are actually used.
Otherwise we bzero the entire structure, then initialize
explicitly the first half of it.
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/fib_frontend.c')
-rw-r--r-- | net/ipv4/fib_frontend.c | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/net/ipv4/fib_frontend.c b/net/ipv4/fib_frontend.c index 1d2233cd99e6..fe10bcd0f307 100644 --- a/net/ipv4/fib_frontend.c +++ b/net/ipv4/fib_frontend.c | |||
@@ -193,19 +193,21 @@ int fib_validate_source(__be32 src, __be32 dst, u8 tos, int oif, | |||
193 | u32 *itag, u32 mark) | 193 | u32 *itag, u32 mark) |
194 | { | 194 | { |
195 | struct in_device *in_dev; | 195 | struct in_device *in_dev; |
196 | struct flowi fl = { | 196 | struct flowi fl; |
197 | .fl4_dst = src, | ||
198 | .fl4_src = dst, | ||
199 | .fl4_tos = tos, | ||
200 | .mark = mark, | ||
201 | .iif = oif | ||
202 | }; | ||
203 | struct fib_result res; | 197 | struct fib_result res; |
204 | int no_addr, rpf, accept_local; | 198 | int no_addr, rpf, accept_local; |
205 | bool dev_match; | 199 | bool dev_match; |
206 | int ret; | 200 | int ret; |
207 | struct net *net; | 201 | struct net *net; |
208 | 202 | ||
203 | fl.oif = 0; | ||
204 | fl.iif = oif; | ||
205 | fl.mark = mark; | ||
206 | fl.fl4_dst = src; | ||
207 | fl.fl4_src = dst; | ||
208 | fl.fl4_tos = tos; | ||
209 | fl.fl4_scope = RT_SCOPE_UNIVERSE; | ||
210 | |||
209 | no_addr = rpf = accept_local = 0; | 211 | no_addr = rpf = accept_local = 0; |
210 | in_dev = __in_dev_get_rcu(dev); | 212 | in_dev = __in_dev_get_rcu(dev); |
211 | if (in_dev) { | 213 | if (in_dev) { |