lwn 326552: 2 issues
unicornx opened this issue · 3 comments
unicornx commented
from rocklee
有两个地方有些小问题
第一点，是这里。原文的意思是最后一个活动的pdflush闲置超过1s，就会终止一个pdflush。而不是任意一个pdflush闲置超过1s。我核对了相关代码，这里每次检查的是队列中最后一个pdflush，也就是睡眠时间最长的那个pdflush
第二点，原文中明确指出回写那些弄脏时间超过一定时间的inode的脏页，但是在翻译的时候这部分有些模糊了，算个小bug
unicornx commented
unicornx commented
@Noctis-AKM 你提的两个问题我看过了。
第一个问题，你的理解是对的，我看了一下 2.6.31 的代码，也找到了代码中的明确注释 https://elixir.bootlin.com/linux/v2.6.31/source/mm/pdflush.c#L76
If the least-recently-went-to-sleep pdflush thread has been asleep
* for more than one second, terminate a thread.
第二个问题是翻译上我觉得 “按照一定的标准” 实际上是废话，可以删掉就好了，翻译中 “比较旧” 的描述已经表达了原文中 “older than a specified point in time”。“按照一定的标准” 属于画蛇添足。