netdev
[Top] [All Lists]

Re: [2.6 PATCH] ipchains masquerade must select maddr correctly

To: "David S. Miller" <davem@xxxxxxxxxx>
Subject: Re: [2.6 PATCH] ipchains masquerade must select maddr correctly
From: Julian Anastasov <ja@xxxxxx>
Date: Wed, 3 Dec 2003 02:50:56 +0200 (EET)
Cc: netdev@xxxxxxxxxxx, <laforge@xxxxxxxxxxxx>
In-reply-to: <20031202162208.04629dab.davem@redhat.com>
Sender: netdev-bounce@xxxxxxxxxxx
        Hello,

On Tue, 2 Dec 2003, David S. Miller wrote:

> >     The attached patch fixes ipchains masquerade to use
> > correctly the routing. This bug-to-bug compatibility with 2.2
> > is not valid from long time. Also, a missing unlock is added.
>
> Slow down.
>
> I don't think it's always desirable to specify a specific TOS when
> we're working with an input packet.  In fact, what you're doing all
> over the tree is going to cause the routing cache size to explode in
> some very real usage.

        Yes, it can grow up to 8 times (IPTOS_RT_MASK is 3 bits) if
we detect different rt tos values. In fact, ipchains is the only
case where tos is not provided :) For some users may be this is
not only a maddr selection, may be they have real routes by tos
for this public IP. Perhaps, TOS matching and hash key should be
a sysctl/compile time option? Then a site that does not use tos
for routing can safely run PMTUD without problems. I think, it is
a common case not to route by tos.

        The good news is that for ipchains this is in->out traffic
and may be there is only one tos value per path.

Regards

--
Julian Anastasov <ja@xxxxxx>


<Prev in Thread] Current Thread [Next in Thread>