去年年底英特爾將 LAM(Linear Address Masking :線性地址掩碼) 功能提交到 Linux 6.2 的合并窗口,但該功能受到 Linus 的批評并拒絕合并。在經(jīng)歷了一段時間的代碼改進(jìn)后,Linus 終于同意將 LAM 代碼合并到 Linux 6.4 窗口。
但 Linus 似乎仍對英特爾工程師提交的代碼不太滿意,在合并了 LAM 代碼后,先是寫了一個使 access_ok () 獨立于 LAM 的新補(bǔ)丁,而后又親手寫了多個補(bǔ)丁對 LAM 代碼進(jìn)行了優(yōu)化。
在最新提交的 LAM 優(yōu)化補(bǔ)丁中,Linus 解釋了自己的動機(jī):
我對此版本中的 LAM(“線性地址掩碼”)的 “access_ok ()” 的完成方式感到很不爽,而且它實際上也有一些小 Bug ,所以我動手清理了代碼。
改動主要集中在以下幾方面:
使用 __user 指針的符號位而不是屏蔽地址,并根據(jù) TASK_SIZE 范圍檢查它。 get/put_user () 端做了這部分,但是 ‘a(chǎn)ccess_ok ()’ 做了天真的 “掩碼和范圍檢查”,它不僅生成多余的代碼,還意味著 __access_ok 本身的任務(wù)做得不好, copy_from_user_nmi () 沒有得到正確的檢查。
將所有 64 位代碼僅移動到 64 位版本的頭文件中,這樣就不會污染共享的 x86 代碼,也不會誤導(dǎo)用戶 LAM 可以在 32 位環(huán)境中工作。
修復(fù)地址掩碼中的 Bug(這不重要,只是完全刪除了錯誤的代碼)。
幾個簡單的清理,并添加了關(guān)于 access_ok () 規(guī)則的注釋。
Linus 重新編寫了約一百行代碼來清理 LAM ,這意味著如果測試沒問題, 就可以在 Linux 6.4 中順利啟用 LAM 功能。不過這次 Linus 竟然親自動手為英特爾工程師修改 “有瑕疵的代碼”,這種情況相當(dāng)少見。
審核編輯 :李倩
-
英特爾
+關(guān)注
關(guān)注
61文章
10188瀏覽量
174266 -
Linux
+關(guān)注
關(guān)注
87文章
11495瀏覽量
213207 -
代碼
+關(guān)注
關(guān)注
30文章
4893瀏覽量
70434
原文標(biāo)題:Linus親自修改英特爾的“屎山”代碼
文章出處:【微信號:OSC開源社區(qū),微信公眾號:OSC開源社區(qū)】歡迎添加關(guān)注!文章轉(zhuǎn)載請注明出處。
發(fā)布評論請先 登錄
世紀(jì)大并購!傳高通有意整體收購英特爾,英特爾最新回應(yīng)

為什么無法檢測到OpenVINO?工具套件中的英特爾?集成圖形處理單元?
請問OpenVINO?工具套件英特爾?Distribution是否與Windows? 10物聯(lián)網(wǎng)企業(yè)版兼容?
英特爾任命王稚聰擔(dān)任中國區(qū)副董事長
英特爾推出全新英特爾銳炫B系列顯卡

英特爾CEO Gelsinger宣布退休
英特爾考慮出售Altera股權(quán)
剛剛!英特爾最新回應(yīng)

英特爾將攜手AMD共同捍衛(wèi)x86生態(tài)
英特爾至強(qiáng)品牌新戰(zhàn)略發(fā)布
面對高通收購,Apollo 50億美元投資,你該買入英特爾股票嗎?

評論