请教个linux源码的问题

b0207191

在看3.x中的sysctl.c这部分代码，有个函数看不懂

1. /* see if attaching q to p would be an improvement */
   
2. static void try_attach(struct ctl_table_header *p, struct ctl_table_header *q)
   
3. {
   
4.         struct ctl_table *to = p->ctl_table, *by = q->ctl_table;
   
5.         struct ctl_table *next;
   
6.         int is_better = 0;
   
7.         int not_in_parent = !p->attached_by;
   
8. 
   
9.         while ((next = is_branch_in(by, to)) != NULL) {
   
10.                 if (by == q->attached_by)
    
11.                         is_better = 1;
    
12.                 if (to == p->attached_by)
    
13.                         not_in_parent = 1;   《---- 这里为什么不添加个判断 if (is_better && not_in_parent)  就break； 提前结束循环
    
14.                 by = by->child;
    
15.                 to = next->child;
    
16.         }
    
17. 
    
18.         if (is_better && not_in_parent) {
    
19.                 q->attached_by = by;
    
20.                 q->attached_to = to;
    
21.                 q->parent = p;
    
22.         }
    
23. }

复制代码

另外，网络上关于sysctl这部分代码分析没有看见，有疑问不知道找谁交流，直接问源代码作者吗，

EraserKing

因为要继续遍历链表到头？下面有赋值

— from Xiaomi 23127PN0CC, Android 14 of S1 Next Goose v2.5.2-play

不要叫水狗做主人

先看最新版本有没有改这边的逻辑吧，说不定别人也觉得可以提前终止循环

精钢魔像

可能是循环里有两层if 不利于优化？

tsubasa9

没看懂你的改法，但 is_better && not_in_parent 显然不是while结束的条件
这里while结束的条件是is_branch_in函数返回值空

posthoc

虽然is_better 和 not_in_parent一旦赋值为1就不会变回去，但是最后的

1.                 q->attached_by = by;
   
2.                 q->attached_to = to;

复制代码

用到的是循环到最后的by和to，提前break的话显然语义不同了吧。