前段时间,一哥们在论坛里对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一下吧
阅读(7548) | 评论(5) | 转发(6) |