关于对linus等人的不满意见

帅绝人寰

举个例子：


/**
* pci_find_capability - query for devices' capabilities
* @dev: PCI device to query
* @cap: capability code
*
* Tell if a device supports a given PCI capability.
* Returns the address of the requested capability structure within the
* device's PCI configuration space or 0 in case the device does not
* support it.  Possible values for @cap:
*
*  %PCI_CAP_ID_PM           Power Management
*  %PCI_CAP_ID_AGP          Accelerated Graphics Port
*  %PCI_CAP_ID_VPD          Vital Product Data
*  %PCI_CAP_ID_SLOTID       Slot Identification
*  %PCI_CAP_ID_MSI          Message Signalled Interrupts
*  %PCI_CAP_ID_CHSWP        CompactPCI HotSwap
*  %PCI_CAP_ID_PCIX         PCI-X
*  %PCI_CAP_ID_EXP          PCI Express
*/
int pci_find_capability(struct pci_dev *dev, int cap)

注释里一共列了8个可能的cap ID， 这估计是刚写该函数时写的注释。 后来kernel支持了更多的ID， 该注释却一直没有更新。 你查查kernel， 有很多传入其他ID的调用， 例如： pos = pci_find_capability(pdev, PCI_CAP_ID_MSIX);  这种写法，就是和注释矛盾的。

看代码时， 可以顺手发个小patch到 lkml

ljjk

albcamus

W.Z.T

写代码差劲的人才会抱怨没有注释， 楼主有这个时间都可以为kernel贡献好多补丁了

xjtuwjp

首先，有很多文档在 kernel/Document 里都有，不知道 LZ 发现过没有？
其次，人和人的理解能力不同，有的人 ...
platinum 发表于 2010-03-08 06:39

对的，其实我觉着linux虽然文档可能是不如windows之类的私有os完备，缺失也还是有很多资料的，主要是Document里面，在就是多看看lwn和邮件列表里的patch
一些重要的内核变化一般lwn都有介绍，patch里面也会说的比较明白的

帅绝人寰

写代码差劲的人才会抱怨没有注释， 楼主有这个时间都可以为kernel贡献好多补丁了
W.Z.T 发表于 2010-03-09 13:22

    2004/2005年Andrew Morton说， 他做的主要贡献都在ext3文件系统， 原因只有一个：ext3文件系统代码注释多。

他又说： 假如其他subsystem也能有这么多注释， 他可以作出更多的贡献。

大牛都这样说， 大家就别摆cool了。  就算没注释的代码你最终能读懂， 效率上也是低下的。

peimichael

无聊插两句。

1. 很多人都认为代码注释写多了会显得自己水平低，没脸见人一样！有个IBM的专家来给我们上课，怎么向我们强调代码注释的重要性啊？安装你们的观点，IBM这个专家呆在IBM肯定是个水货，是打造智慧的垃圾地球的！

没有几个人认为注释写多了显得自己水平低，这是你自己认为大家这么想

2. 好像很多人都认为自己不用注释就能轻松、闲庭信步一样理解kernel代码，我向你们表示仰慕！！

不用仰慕，这也是你自己想出来的，大家提到内核代码一般都会说他庞大复杂，难以阅读。

3. 我看很多人都看过赵炯的那边0.11代码完全注释，不都是冲着人家把每行代码都注释了吗？

没错，没有注释的话，我想大多数人在内核里都会完全摸不着北。至少我连从哪看起都不知道。

我想大家说这些话的意思是(至少我的意思是)内核开发者有很多都是无偿劳动，你能免费用上这么好的系统，学习这么优秀的代码应该心存感激，而不是上来就对他们进行攻击。
这就好像你好心义务帮别人打扫了卫生，结果却被别人骂打扫的不干净，你觉得那人骂的对么？

说完了，继续回去艰难的啃代码。。

platinum

PS， 有些同志说到Documentation，其实它往往是1）不实时的；2）不准确的。  而且它不够精确，和代码注释还不是一个level。
帅绝人寰 发表于 2010-03-09 11:25

兄弟说的不错，确实不一定准确，也不是很实时
但对于一个看过过时的、不一定精确的 Document 的人来说，如果之前掌握并熟悉了旧的实现方式，那么不难理解一些新的、Document 中没有跟上的内容

insmile

自己动手，丰衣足食