diff options
| author | David S. Miller <davem@davemloft.net> | 2012-07-17 23:09:39 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2012-07-20 22:36:55 +0200 |
| commit | 93ac53410a82a4f1bf2baf9d65d95cc29f2774ca (patch) | |
| tree | 87f3f387777526c422c38d2fe5b400cc4d81bd55 | |
| parent | ipv4: Kill FLOWI_FLAG_RT_NOCACHE and associated code. (diff) | |
| download | linux-93ac53410a82a4f1bf2baf9d65d95cc29f2774ca.tar.xz linux-93ac53410a82a4f1bf2baf9d65d95cc29f2774ca.zip | |
ipv4: Dirty less cache lines in route caching paths.
Don't bother incrementing dst->__use and setting dst->lastuse,
they are completely pointless and just slow things down.
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | net/ipv4/route.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/net/ipv4/route.c b/net/ipv4/route.c index 7e1c0ed0ef70..b8707779b85d 100644 --- a/net/ipv4/route.c +++ b/net/ipv4/route.c @@ -1443,7 +1443,7 @@ static int __mkroute_input(struct sk_buff *skb, if (!(flags & RTCF_DIRECTSRC) && !itag) { rth = FIB_RES_NH(*res).nh_rth_input; if (rt_cache_valid(rth)) { - dst_use(&rth->dst, jiffies); + dst_hold(&rth->dst); goto out; } do_cache = true; @@ -1619,7 +1619,7 @@ local_input: if (!(flags & RTCF_DIRECTSRC) && !itag) { rth = FIB_RES_NH(res).nh_rth_input; if (rt_cache_valid(rth)) { - dst_use(&rth->dst, jiffies); + dst_hold(&rth->dst); goto set_and_out; } do_cache = true; @@ -1789,7 +1789,7 @@ static struct rtable *__mkroute_output(const struct fib_result *res, if (!fnhe) { rth = FIB_RES_NH(*res).nh_rth_output; if (rt_cache_valid(rth)) { - dst_use(&rth->dst, jiffies); + dst_hold(&rth->dst); return rth; } } |