[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <AANLkTin3o_83kz1OxsN1Pme=0wv0g1zJ2F+FQ+U--u5e@mail.gmail.com>
Date: Wed, 6 Oct 2010 14:23:44 -0700
From: Chung-Yih Wang (王崇懿)
<cywang@...gle.com>
To: David Miller <davem@...emloft.net>
Cc: herbert@...dor.hengli.com.au, timo.teras@....fi,
netdev@...r.kernel.org
Subject: Re: Regarding to your linux kernel CL
Do you mean if we should move the genid/cookie logic into sk_dst_check
and remove the dst->ops->check()? If so, I dont think it will be a
good idea to remove 'dst->ops->check' since it is per dst entry's
check not just for ipv4 entry. At least, the fact of obsolete=2
reveals that the entry is freed and is ready for being released. That
will be weird if we still use the obsolete entry for routing the
socket's traffic(for sure, that's why the packets of the socket goes
nowhere but dropped).
On Wed, Oct 6, 2010 at 1:14 AM, David Miller <davem@...emloft.net> wrote:
> From: Herbert Xu <herbert@...dor.apana.org.au>
> Date: Wed, 6 Oct 2010 15:59:46 +0800
>
>> Perhaps we should move the genid/cookie logic into the dst so that
>> we can eliminate the dst->check call or at least make it a lot less
>> frequent.
>
> That sounds like a good idea.
>
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists