From: Thomas Graf <[EMAIL PROTECTED]>
Date: Thu, 27 Jul 2006 00:00:02 +0200
> Adds the framework to support multiple IPv6 routing tables.
> Currently all automatically generated routes are put into the
> same table. This could be changed at a later point after
> considering the produced locking overhead.
>
> When locating routes for redirects only the main table is
> searched for now. Since policy rules will not be reversible
> it is unclear whether it makes sense to change this.
>
> Signed-off-by: Thomas Graf <[EMAIL PROTECTED]>
This looks good, and it seems we even fixed a bug:
> @@ -586,7 +612,7 @@ restart:
>
> dst_hold(&rt->u.dst);
> if (nrt) {
> - err = ip6_ins_rt(nrt, NULL, NULL, &NETLINK_CB(skb));
> + err = ip6_ins_rt(nrt, NULL, NULL, NULL);
> if (!err)
> goto out2;
> }
Wow, were we corrupting the IP6CB() of input packets on every
route lookup that hit this path?
I'm probably the one who put that erroneous &NETLINK_CB(skb) there.
Sorry :)
Signed-off-by: David S. Miller <[EMAIL PROTECTED]>
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
