前段时间，一哥们在论坛里对kernel route部分的一段代码有疑惑，于是就抽空帮忙看了下。
代码如下：
-
#ifdef CONFIG_IP_ROUTE_MULTIPATH
-
-
#define FIB_RES_NH(res) ((res).fi->fib_nh[(res).nh_sel])
-
-
#define FIB_TABLE_HASHSZ 2
-
-
#else /* CONFIG_IP_ROUTE_MULTIPATH */
-
-
#define FIB_RES_NH(res) ((res).fi->fib_nh[0])
-
-
#define FIB_TABLE_HASHSZ 256
-
-
#endif /* CONFIG_IP_ROUTE_MULTIPATH */
很简单的一个宏定义的问题。如果开启了多路路由，那么设置路由hash表的大小为2，否则为256。这显然是不合理的。因为如果开启了多路路由，应该将表增大，而不是将表减小。
为了稳妥起见，没有直接报bug，先跟作者沟通一下。
联系Alan cox，结果说他已经15年没动这块代码了，让我跟Alexey Kuzentsov沟通。于是跟Alexey 联系，作者又说这块代码是后来的patch改动的，让我跟Denis V.Lunev沟通。
于是跟Denis 沟通，这次总算找到根了。下面是Den的回复：
这个bug已经隐藏4年之久。正如作者所说，这个bug除了对性能有点影响，没有其他的问题。呵呵
虽然作者已经提交了FIX，但是真正到发行版中还需要一段时间，建议有条件的话，先自己FIX一下吧
阅读(7542) | 评论(5) | 转发(6) |
