utterances-bot commented 3 years ago

ZH-comment-24749934

corvofeng commented 3 years ago

My two cents 我个人浅见

这个作为内核开发者的声音

相比于内核其他部分的代码质量，这个驱动很糟糕

然而， gpu的驱动不仅仅是开源的，并且对内核本身也是一件大事。内核开发者需要花费大量的实践， gpu驱动中有数百万的代码，如果他们每一行都需要长时间的review工作，它将不可能进入到内核树中。

因此，它看起来是十分邪恶的，我真希望可以把他们全部清除。我曾经为amd gpu的驱动提供补丁，需要在3个不同的文件提交相同的内容，它们太冗余了，这种事不可能在内核的其它地方发生。

corvofeng commented 3 years ago

一个小的挨冻在3处进行重复也是一种好的策略去保持分离 IMO 抽象是一种增加代码复杂性的主要来源

就像你开始使用了一个函数在3个地方, 然后增加了一个bool的参数, 那么它将会影响三个地方的函数, 最终增加量复杂性.

corvofeng / BlogCode