最近鬧的比較熱鬧的事情是，華為有人提交到Linux 上的代碼被審核員點名批評刷KPI，并且，這個郵件還上了頭條。

提交的代碼修改如下

From:?Zhen?Lei?<thunder.leizhen@huawei.com> To:?Kees?Cook?<keescook@chromium.org>,Anton?Vorontsov?<anton@enomsg.org>,Colin?Cross?<ccross@android.com>,?Tony?Luck?<tony.luck@intel.com>,linux-kernel?<linux-kernel@vger.kernel.org> Cc:?Zhen?Lei?<thunder.leizhen@huawei.com> Subject:?[PATCH?1/1]?pstore:?remove?unnecessary?oom?message Date:?Thu,?17?Jun?2021?17:10:54?+0800 Message-ID:?<20210617091054.1547-1-thunder.leizhen@huawei.com>?(raw)Fixes?scripts/checkpatch.pl?warning: WARNING:?Possible?unnecessary?'out?of?memory'?messageRemove?it?can?help?us?save?a?bit?of?memory.Signed-off-by:?Zhen?Lei?<thunder.leizhen@huawei.com> ---fs/pstore/platform.c?|??4?+---fs/pstore/ram_core.c?|?15?++++-----------2?files?changed,?5?insertions(+),?14?deletions(-)diff?--git?a/fs/pstore/platform.c?b/fs/pstore/platform.c index?b9614db48b1d..752c2338af6c?100644 ---?a/fs/pstore/platform.c +++?b/fs/pstore/platform.c @@?-752,10?+752,8?@@?void?pstore_get_backend_records(struct?pstore_info?*psi,int?rc;record?=?kzalloc(sizeof(*record),?GFP_KERNEL); -??if?(!record)?{ -???pr_err("out?of?memory?creating?record\n"); +??if?(!record)break; -??}pstore_record_init(record,?psi);record->size?=?psi->read(record); diff?--git?a/fs/pstore/ram_core.c?b/fs/pstore/ram_core.c index?fe5305028c6e..7da890505025?100644 ---?a/fs/pstore/ram_core.c +++?b/fs/pstore/ram_core.c @@?-301,10?+301,8?@@?void?persistent_ram_save_old(struct?persistent_ram_zone?*prz)if?(!prz->old_log)?{persistent_ram_ecc_old(prz);prz->old_log?=?kmalloc(size,?GFP_KERNEL); -?} -?if?(!prz->old_log)?{ -??pr_err("failed?to?allocate?buffer\n"); -??return; +??if?(!prz->old_log) +???return;}prz->old_log_size?=?size; @@?-429,11?+427,8?@@?static?void?*persistent_ram_vmap(phys_addr_t?start,?size_t?size,}pages?=?kmalloc_array(page_count,?sizeof(struct?page?*),?GFP_KERNEL); -?if?(!pages)?{ -??pr_err("%s:?Failed?to?allocate?array?for?%u?pages\n", -?????????__func__,?page_count); +?if?(!pages)return?NULL; -?}for?(i?=?0;?i?<?page_count;?i++)?{phys_addr_t?addr?=?page_start?+?i?*?PAGE_SIZE; @@?-578,10?+573,8?@@?struct?persistent_ram_zone?*persistent_ram_new(phys_addr_t?start,?size_t?size,int?ret?=?-ENOMEM;prz?=?kzalloc(sizeof(struct?persistent_ram_zone),?GFP_KERNEL); -?if?(!prz)?{ -??pr_err("failed?to?allocate?persistent?ram?zone\n"); +?if?(!prz)goto?err; -?}/*?Initialize?general?buffer?state.?*/raw_spin_lock_init(&prz->buffer_lock); --? 2.25.1

之后，華為的這位同學回復如下

對于這個回復，評審員依次回復如下

主要的意思是，你可以把這些多余的修改去除，我們也是很支持的，但是可以在某一個大的提交上去提交，不要依次提交一點點，這樣很不友好。

并提到一個事情，即使我不去查看git log，我也能知道你們對Linux內核的貢獻非常非常大，特別是在 EROFS 和 F2FS 上。

對于這件事情，我的看法是

國外的同學明顯沒有感受到KPI對員工的重要。

沒有KPI，那么年終獎就會很低，娃要吃飯喝奶，房子要還貸，作為一個賺錢的機器，多賺錢沒有錯，這個事情不只是在華為，如果在其他企業，我覺得也會存在，給Linux內核提交patch，那么在KPI上顯得多么優雅。

當然，作為內核維護者，他們是痛苦的，他們每天面對很多很多的郵件，要處理很多很多的問題，不能因為你的KPI問題，就沒有質量的提交代碼，這樣對我是不尊重的。

其他的沒啥問題，他們都是在自己的角度處理事情。

比較尷尬的是，這個是華為的人暴雷的，畢竟他是華為，這樣容易讓別人知道我們的實際情況。

對于網上的一些回復，就比較有意思了。

總結

以上是生活随笔為你收集整理的华为这个事，是不是刷KPI？的全部內容，希望文章能夠幫你解決所遇到的問題。

如果覺得生活随笔網站內容還不錯，歡迎將生活随笔推薦給好友。