派对

skipjack

原帖由 sisi8408 于 2007-2-6 17:10 发表于 49楼  

        helper = __ip_conntrack_helper_find(tuple);
        if (helper) {
                /* need to increase module usage count to assure helper will
                 * not go away while the caller is e.g. busy putting a
                 * conn ...

你以前说的是ip_nat_proto_find_get()函数中的__ip_nat_proto_find(),
而你现在又说是ip_conntrack_helper_find()函数中的__ip_conntrack_helper_find()
对不上号啊.
每个IP数据包都会有自己的proto(0~255),但并不一定每个应用层协议都有自己的helper(amanda ftp mms irc ...),所以前者不可能为null,后者可以为null.

skipjack

原帖由 sisi8408 于 2007-2-6 16:42 发表于 48楼  

你以前不是最看不起ddos设备嘛，怎么最近对这个话题这么感兴趣？
抗这个，不需要什么代码，都是现成的。防火墙都有这个模块
我只是说有，但好不好用，就不好说了。

另外，你说的有理论错误
1)修改必定在 ...

哇~你想干什么?

skipjack

原帖由 sisi8408 于 2007-2-8 16:19 发表于 55楼  
not only a nice game, but also almost realtime.

那里下载？coct.zip不是啊

skipjack

原帖由 sisi8408 于 2007-2-9 09:07 发表于 57楼  
请给个基本详细的点评，这是你擅长地，
然后再抱怨，这不是你擅长地~~~~~~~~~

帮你看了看，思路好像是想用更加细粒度的锁来提高双核下的并行效率。还使用了大量的__read_mostly。对你不利的地方：
1)增加了双核下死锁的机率，两CPU各持一把锁，但又相互等待对方的锁。
2)失去了和广大Linux爱好者统一的编程接口，大家都有ip_conntrack_lock，你却用exp_lock，一个模块集成不小心，可能就game over、oops、外加panic了。

BTW：测试过前后的性能差异吗？

skipjack

原帖由 sisi8408 于 2007-2-9 14:42 发表于 58楼  

        if (strchr(newname, '%')) {
                err = dev_alloc_name(dev, newname);
                if (err < 0)
                        return err;
               
                //no check ##########
                // if eth%d, eth12345
                strcpy(newname, dev->name);
         ...

代码有时没必要过分小心，否则冗余太多。
dev_alloc_name检查了边界，接下来的strcpy就应该是安全的。当然前提是newname和dev->name的空间用IFNAMSIZ宏来限定。

skipjack

原帖由 sisi8408 于 2007-2-13 10:48 发表于 66楼  

static int __init vlan_proto_init(void)
{
        int err;

        printk(VLAN_INF "%s v%s %s\n",
               vlan_fullname, vlan_version, vlan_copyright);
        printk(VLAN_INF "All bugs added  ...

没看懂

skipjack

原帖由 sisi8408 于 2007-2-12 10:25 发表于 61楼  
帮你看了看，思路好像是想用更加细粒度的锁来提高双核下的并行效率。还使用了大量的__read_mostly。
///哈西表的本质不就是粒度？

对你不利的地方：
1)增加了双核下死锁的机率，两CPU各持一把锁，但又相互 ...

可以，但你不会把机器抱过来吧