[<prev] [next>] [day] [month] [year] [list]
Message-ID: <AANLkTinoAh=MOOnPM1nJhq+tXAxFC2zia0YH49UwN9+b@mail.gmail.com>
Date: Tue, 5 Oct 2010 09:42:37 -0700
From: Chung-Yih Wang (王崇懿)
<cywang@...gle.com>
To: netdev@...r.kernel.org
Subject: Fwd: Regarding to your linux kernel CL
Hi,
We have an issue with the CL
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=d11a4dc18bf41719c9f0d7ed494d295dd2973b92.
Please see the original mail below. Any comment?
Thanks,
Chung-yih
---------- Forwarded message ----------
From: Chung-Yih Wang (王崇懿) <cywang@...gle.com>
Date: Mon, Oct 4, 2010 at 6:23 PM
Subject: Regarding to your linux kernel CL
To: timo.teras@....fi
Cc: herbert@...dor.apana.org.au, davem@...emloft.net
Hi Timo,
I encountered an issue with your CL
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=d11a4dc18bf41719c9f0d7ed494d295dd2973b92.
The cause is that we use a connected UDP socket for building the
l2tp/ipsec vpn connection. However, when the ipsec tunnel is built,
your CL made the sk_dst_check useless(since it always return the
'freed' dst_entry and can not reset the dst entry for the socket).
What is your comment to conquer this issue?
Solution 1. We could add a CL to change it to (dst && dst->obsolete &&
(dst->obsolete>0 || dst->ops->check(...)==NULL) in sk_dst_check()) ?
Solution 2. Revert the change?
Any comment?
Thanks,
Chung-yih
--
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