华为这个事,是不是刷KPI?
最近闹的比较热闹的事情是,华为有人提交到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?相关推荐
- 华为开发者被批评在 Linux 内核刷 KPI
来源:cnBeta + Solidot + 程序员的那些事 + Linux爱好者 Linux 内核维护者.发行版 SuSE 的开发者 Qu Wenruo 在邮件列表指出来自华为的开发者 Leizhen ...
- 华为 Linux 内核贡献者被质疑刷 KPI
本文转载自IT之家 IT之家 6 月 21 日消息 近日,Linux 内核邮件列表出现了一封特殊的邮件,该邮件名为<Please don't waste maintainers' time on ...
- 07-20210305在WIN10下通过USB口给华为海思Hi3516DV300刷机(鸿蒙系统)
07-20210305在WIN10下通过USB口给华为海思Hi3516DV300刷机(鸿蒙系统) 2021/3/5 17:26 https://blog.csdn.net/cocoron/articl ...
- 08-20210305在WIN10下通过网口给华为海思Hi3516DV300刷机(鸿蒙系统)
08-20210301在WIN10下通过网口给华为海思Hi3516DV300刷机(鸿蒙系统) 2021/3/5 11:03 https://bbs.elecfans.com/jishu_2002000 ...
- 在WIN10下通过网口给华为海思Hi3516DV300刷机(鸿蒙系统)
08-20210301在WIN10下通过网口给华为海思Hi3516DV300刷机(鸿蒙系统) 2021/3/5 11:03 https://bbs.elecfans.com/jishu_2002000 ...
- 华为G520联通版刷机包 高仿三星S5 UI风格美化 B老师作品
ROM介绍 基于官方EMUI固件制作,root权限 加入三星框架,TouchWiz启动器 修改默认程序图标为S5风格 默认字体使用更加明晰的微软雅黑 移植三星风格透明时钟插件 移植三星风格新浪天气插件 ...
- 华为平板M5一键刷ji教程--精简ROOT工具分享
目前来说,平板的使用场景锐减,在大多数时候,手机已经完全能够代替平板,而且手机更便捷,在大屏幕优势方面,机身越来越轻薄的笔记本也正在威胁着平板.但在全球销量下滑的大背景下,依然有个平板品牌逆势增长,那 ...
- 华为G520联通版刷机包 新蜂V2.3 基于官方精简优化 稳定流畅
ROM介绍 [出品]:新蜂工作室(基于官方) 1.国行 2.源于官方:基于G520官方B203包提取 3.深度精简:无预装APK,自带APK数量从原厂包的114个减少到78个,精简31.5% ; 4. ...
- 《华为机试》刷题之HJ84 统计大写字母个数
<华为机试>刷题之统计大写字母个数 我不知道将去向何方,但我已在路上! 时光匆匆,虽未曾谋面,却相遇于斯,实在是莫大的缘分,感谢您的到访 ! 题目: 找出给定字符串中大写字符(即'A'-' ...
最新文章
- -i 可以编译添加多个_C语言的编译过程
- java 正则 捕获_Java通过正则表达式捕获组中的文本
- C++ Map Source
- Renascence架构原理——最优化算法
- python 更新数据库历史_python 实现数据库中数据添加、查询与更新的示例代码
- pat 乙级 1012 数字分类(C++)
- Java线程中断的理解(转载)
- 萌新的Python练习实例100例(二)根据企业的利润,计算企业的方法奖金
- yolov3从头实现(四)-- darknet53网络tf.keras搭建
- stm8s电机库vtimer_SetTimer()函数的使用
- Axure中使用fontawesome字体
- ps安装 Photoshop CC2017 安装,cutterman下载安装,ps简单设置
- 更新glibc版本-GLIBC升级有风险,操作前最好请预先写好离职申请书
- HTML实现W3school导航栏(附带重置样式表reset.css)
- 知识付费平台现状:供应商、购买者议价能力较
- java读取欧姆龙plc,寄存器读取命令响应超时 欧姆龙plc寄存器读取命
- [953]mongo创建索引及索引相关方法
- mac android 模拟器地址,在mac上运行android模拟器
- Cesium.js解析第一篇 Cartesian3
- Java中对象属性的初始化顺序
热门文章
- 2016年光伏电站交易和融资的十大猜想
- SpringMVC实战(注解)
- Weblogic的管理服务器与受管服务器
- 属性动画基础之ValueAnimator
- JPA字段长度 Mysql数据库
- 【运动快乐】享受赤脚慢跑 收获健康快乐
- TCP三次握手及四次挥手详解
- CharSequence类
- aix oracle 10.2.0.1 升级 10.2.0.4,AIX Oracle RAC 升级到10.2.0.4.0要特别注意的问题 - 爱肯的专栏 ......
- java sunjce,无法初始化类javax.crypto.SunJCE_b