派对

sisi8408

帮你看了看，思路好像是想用更加细粒度的锁来提高双核下的并行效率。还使用了大量的__read_mostly。
///哈西表的本质不就是粒度？

对你不利的地方：
1)增加了双核下死锁的机率，两CPU各持一把锁，但又相互等待对方的锁。
///基本避免了死锁

2)失去了和广大Linux爱好者统一的编程接口，大家都有ip_conntrack_lock，你却用exp_lock，一个模块集成不小心，可能就game over、oops、外加panic了。
///这是HW头痛的问题，也是ct与rt的最大区别，尝试是应该地，也是艰苦地。

BTW：测试过前后的性能差异吗？
///如果方便，去你那里测桥。

sisi8408

1. 
   
2.         } else {
   
3.                 spin_lock(&br->hash_lock);
   
4. 
   
5.                 if (!fdb_find(head, addr)) {
   
6.                         /*
   
7.                          * nice game, ifuleu@
   
8.                          * better to alloc fdbe off lock
   
9.                          */
   
10.                         fdb_create(head, source, addr, 0);
    
11.                 }
    
12.                 /*
    
13.                  * else  we lose race and someone else inserts
    
14.                  * it first, don't bother updating
    
15.                  */
    
16.                 spin_unlock(&br->hash_lock);
    
17.         }
    
18. 
    

复制代码

漂亮的段子，不到400行，由于保守的逻辑，陷入明显的混乱，
期待仔细的测量结果。

[ 本帖最后由 sisi8408 于 2007-2-12 14:00 编辑 ]

sisi8408

1. 
   
2. static __inline__ void fdb_delete (struct net_bridge_fdb_entry *f)
   
3. {
   
4.         hlist_del_rcu(&f->hlist);
   
5.         /*
   
6.          * though I dont know where u go,
   
7.          * I do believe u exist.
   
8.          * what is the charm?
   
9.          */
   
10.         br_fdb_put(f);
    
11. }
    

复制代码

sisi8408

1. 
   
2. void br_fdb_cleanup (unsigned long _data)
   
3. {
   
4.         struct net_bridge * br = (struct net_bridge *)        _data;
   
5.         unsigned long delay = hold_time(br);
   
6.         int i;
   
7.         unsigned long now = jiffies;
   
8.         /*
   
9.          * of cough, rover is more appreciated.
   
10.          */
    
11.         unsigned long cnt = 0;
    
12.        
    
13.         //spin_lock_bh(&br->hash_lock);
    
14.         for (i = 0; i < BR_HASH_SIZE; i++) {
    
15.                 struct net_bridge_fdb_entry * f;
    
16.                 struct hlist_node *h, *n;
    
17. 
    
18.                 spin_lock_bh(slock(i));
    
19.                 hlist_for_each_entry_safe(f, h, n, &br->hash[i], hlist) {
    
20.                         ++cnt;
    
21.                         if (!f->is_static &&
    
22.                             time_before_eq(f->ageing_timer + delay, now /* jiffies */)) {
    
23.                                 --cnt;
    
24.                                 fdb_delete(f);
    
25.                         }
    
26.                 }
    
27.                 spin_unlock_bh(slock(i));
    
28.         }
    
29.         //spin_unlock_bh(&br->hash_lock);
    
30. 
    
31.         if ((cnt >> 3) > BR_HASH_SIZE)
    
32.                 cnt = HZ / 10;
    
33.         else if ((cnt >> 2) > BR_HASH_SIZE)
    
34.                 cnt = HZ / 2;
    
35.         else
    
36.                 cnt = HZ;
    
37.         mod_timer(&br->gc_timer, jiffies + cnt /* HZ/10 */);
    
38. }
    

复制代码

[ 本帖最后由 sisi8408 于 2007-2-12 16:00 编辑 ]

sisi8408

1. 
   
2. 刚才撞见straw，是不是当年的大侠straw@smth
   

复制代码

很疑惑的说。
不妨说说，当年为什么梁老大与武汉三剑客翻脸，至今没弄明白？
土狼哥哥，TND，也不知跑哪去了？

[ 本帖最后由 sisi8408 于 2007-2-13 11:06 编辑 ]

sisi8408

1. 
   
2. static int __init vlan_proto_init(void)
   
3. {
   
4.         int err;
   
5. 
   
6.         printk(VLAN_INF "%s v%s %s\n",
   
7.                vlan_fullname, vlan_version, vlan_copyright);
   
8.         printk(VLAN_INF "All bugs added by %s\n", vlan_buggyright);
   
9. 
   
10. .......
    
11. }
    

复制代码

真是强人的代码~~~

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各持一把锁，但又相互 ...

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

sisi8408

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

抱机器太麻烦，也用不着~~~

sisi8408

祝大家新年好，CUer，节后再见，
有事发邮件~~~