netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [RFC PATCH 0/5] Cache PMTU/redirects in inetpeer
@ 2011-02-10  6:12 David Miller
  2011-02-10 23:09 ` David Miller
  0 siblings, 1 reply; 3+ messages in thread
From: David Miller @ 2011-02-10  6:12 UTC (permalink / raw)
  To: netdev


This is what I've been working on for the past several days.

Right now if the routing cache is turned off (by setting
rt_cache_rebuild_count to "0") several things stop working.

We never make use of any PMTU or redirect information we learn
via ICMP packets.  This is because when the routing cache is
off, we can't "find" the existing cached routes that match
the ICMP because we don't add them to the hash table.

This functionality loss is also a blocker for eliminating the
routing cache entirely.

Solve this by remembering this state in the inetpeer entries.

PMTU information now self-expires.  It gets validated when
cached routes are sanity checked via dst_ops->check().  At
expiration, the original RTAX_MTU metric value is restored.
So we don't have to invalidate the entire cached route just
because it's PMTU learned value has expired.

Similarly, we store redirect information in inetpeer too.
Except that currently my patches don't remember the "original"
gateway the route had, so we have to kill the route off when
we get a dst_ops->negative_advice() call on a redirected route.

Avoid this is easy to fix and I might do that soon.

These patches implement the PMTU/redirect bits in ipv4 only at the
moment, but I do have ipv6 patches I'm in the process of finishing
up.  I just wanted people to see this as soon as possible so that
I can start getting feedback.

And hey if people can test this stuff out that'd be awesome!  If
you've used these changes in an environment where you did hit PMTU
and redirects, please do let me know.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [RFC PATCH 0/5] Cache PMTU/redirects in inetpeer
  2011-02-10  6:12 [RFC PATCH 0/5] Cache PMTU/redirects in inetpeer David Miller
@ 2011-02-10 23:09 ` David Miller
  2011-02-15  5:46   ` David Miller
  0 siblings, 1 reply; 3+ messages in thread
From: David Miller @ 2011-02-10 23:09 UTC (permalink / raw)
  To: netdev

From: David Miller <davem@davemloft.net>
Date: Wed, 09 Feb 2011 22:12:49 -0800 (PST)

> This is what I've been working on for the past several days.

Just FYI I've pushed just the infrastructure bits (patches 1, 2,
and 3) into net-next-2.6 since those have no side effects.

I won't push the other two until I can do some good testing on
them.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [RFC PATCH 0/5] Cache PMTU/redirects in inetpeer
  2011-02-10 23:09 ` David Miller
@ 2011-02-15  5:46   ` David Miller
  0 siblings, 0 replies; 3+ messages in thread
From: David Miller @ 2011-02-15  5:46 UTC (permalink / raw)
  To: netdev

From: David Miller <davem@davemloft.net>
Date: Thu, 10 Feb 2011 15:09:57 -0800 (PST)

> From: David Miller <davem@davemloft.net>
> Date: Wed, 09 Feb 2011 22:12:49 -0800 (PST)
> 
>> This is what I've been working on for the past several days.
> 
> Just FYI I've pushed just the infrastructure bits (patches 1, 2,
> and 3) into net-next-2.6 since those have no side effects.
> 
> I won't push the other two until I can do some good testing on
> them.

I've now pushed the final two patches from the original series
into net-next-2.6

Please if anyone runs into troubles, give a good bug report and I
will fix the problems as quickly as possible.

Thanks!

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2011-02-15  5:45 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-02-10  6:12 [RFC PATCH 0/5] Cache PMTU/redirects in inetpeer David Miller
2011-02-10 23:09 ` David Miller
2011-02-15  5:46   ` David Miller

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).