Linus都要再三修正的max()宏是怎样演变的

发表于 1年以前  | 总阅读数:296 次

大家好,又与大家见面了,今天我们继续聊一聊关于Linux内核中的max()宏。

在前两篇文章中,小编的同门@梁金荣 同学已经非常贴心地将Linux内核中的max()宏为我们逐项拆解,今天我们在之前两篇文章的基础上继续探讨一下max()宏的发展历史。

ps:小编也是刚接触Linux内核的小白,如果有问题希望大家留言指正~

想必大家对老版本内核中的max()宏已经有了深刻印象:前面的文章已经详细地说明了一个max宏是如何从

#define max(x, y) (x) > (y) ? (x) : (y);

转换成下面这样的(内核版本2.6)

define max(x, y) ({ \
typeof(x) _max1 = (x); \
typeof(y) _max2 = (y); \
(void) (&_max1 == &_max2); \
_max1 > _max2 ? _max1 : _max2; })

具体请阅读【Linux内核中max()宏的奥妙何在?(一) 】 [点击访问]

其实很多资料显示max宏是这样的...

define max(x, y) ({ \
typeof(x) _x = (x); \
typeof(y) _y = (y); \
(void) (&_x == &_y); \
_x > _y2 ? _x : _y; })

看出来差距了吗?其实就是把变量 _ x换成了_ max1, _ y换成了_max2

为什么要这样做呢?

假如内核程序员调用max()宏来比较如下两个数的大小时,这会导致意想不到的后果:

nt x = 3; int _x =6; 
max(x, _x)

Linux内核在设计max()宏时使变量名尽量变得复杂,同时要保证"见名知义"。这样做确实很大程度上避免了一些重名导致的安全问题,因为很少有程序员会这样调用:

max(x, _max1)

但这明显属于"换汤不换药"的解决方案,内核设计者不应该苛求程序员定义一个变量名称时考虑这么多。

既然重名问题可能引起上述问题,那么有机制可以使得命名永远不会重复吗?内核开发者朝这个方向做出了努力,小编查阅了4.15.9版本内核源码,发现max()宏悄然发生了变化:

**
* max - return maximum of two values of the same or compatible types
* @x: first value
* @y: second value
*/
#define max(x, y) \
__max(typeof(x), typeof(y), \
__UNIQUE_ID(max1_), __UNIQUE_ID(max2_), \
x, y)

#define __max(t1, t2, max1, max2, x, y) ({ \
t1 max1 = (x); \
t2 max2 = (y); \
(void) (&max1 == &max2); \
max1 > max2 ? max1 : max2; })

<linux/gcc-compiler.h>中给出了 __UNIQUE_ID 的定义

* Indirect macros required for expanded argument pasting, eg. __LINE__. */
#define ___PASTE(a,b) a##b
#define __PASTE(a,b) ___PASTE(a,b)

#define __UNIQUE_ID(prefix) __PASTE \
(__PASTE(__UNIQUE_ID_, prefix), __COUNTER__)

通过代码,我们可以大概了解内核是如何PASTE一个ID给待比较大小的两个数。这里给出GCC文档相关原文

COUNTERThis macro expands to sequential integral values starting from 0. In conjunction with the ## operator, this provides a convenient means to generate unique identifiers. Care must be taken to ensure that COUNTER is not expanded prior to inclusion of precompiled headers which use it. Otherwise, the precompiled headers will not be used.

从GCC文档可以看出每个值"Unique"的名称是通过GCC编译器中的 COUNTER 和 运算符 ## 实现的。

至此,因为重名而引起max()宏异常已经成为了历史,因为每个数字都得到了独一无二的名称。

但是,这样并不完美

为什么呢?(这个关子卖的真是僵硬,上一篇大家都见识过了最新版本的max()宏了嘛,肯定知道介个并不是最新版本的max()宏啦)

了解过GNU C的同学应该知道有一个变长数组(VLA)这种神奇的存在,VLA在运行时其长度是不确定的,每次调用它都需要额外计算它的长度,增加了开销;更严重的是,内核的堆栈大小受限,而随意的使用VLA可能会使其长度飞速增长,攻击者如果可以以某种方式控制VLA大小,那么后果是可怕的。(想要深入了解GNU C的同学请访问http://www.nongnu.org/c-prog-book/online/)

所以内核设计者不鼓励在内核空间中使用VLA,但并没有禁止它。

不久之前,Linus宣称“使用VLA是愚蠢的!”并将VLA从内核移除提上了日程.…… 强大的GNU社区推出了Wvla工具提供GCC -Wvla选项,该选项可以在程序员声明VLA时发出警告

然而不久又有人发现了BUG。。。他给Linus发了邮件:

The "sym" calculation is actually a fixed size, but since the max() macro uses some extensive tricks for safety, it ends up looking like a variable size. This replaces max() with a simple max macro which is sufficient for the calculation of the array size

他在声明一个定长数组的时候(看起来像VLA)使用GCC -Wvla时得到了警告,并认为GCC不够聪明,无法分辨VLA和定长数组。

然而Linus就是Linus,大神就是大神,他迅速找出了问题所在,并回复了邮件:

C标准在“常量值”和“常量表达式”之间进行了区分。数组维度必须是常量表达式,但max()宏的设计不符合这个规范。

于是,修订版本的max()宏出现了,请欣赏(内核版本4.8.5)

define __single_eval_max(t1, t2, max1, max2, x, y) ({ \
t1 max1 = (x); \
t2 max2 = (y); \
(void) (&max1 == &max2); \
max1 > max2 ? max1 : max2; })

#define __max(t1, t2, x, y) \
__builtin_choose_expr(__builtin_constant_p(x) && \
__builtin_constant_p(y), \
(t1)(x) > (t2)(y) ? (t1)(x) : (t2)(y), \
__single_eval_max(t1, t2, \
__UNIQUE_ID(max1_), \
__UNIQUE_ID(max2_), \
x, y))

#define max(x, y) __max(typeof(x), typeof(y), x, y)

可以看到内核代码中含有 “__builtin_constant” 字样,这说明对判断常量这件事做出了足够的努力,由于这个版本的max()宏使用了太多“逻辑短路”的宏技巧,对旧的编译器没有很好的支持。所以还需要继续迭代,但经过内核开发者的不断提交patch,目前至少声明一个定长数组时不会再偶然地发出“VLA警告”。

(对 __builtin 有兴趣的小伙伴请访问 https://gcc.gnu.org/onlinedocs/gcc/Other-Builtins.html

最新的max()

在内核邮件列表中,有一篇主题为【3c8ba0d61d04ced9f8d9ff93977995a9e4e96e91 oopses on s390】 的邮件,Sebastian Ott发出抱怨,在他的机器上使用 min_not**_**zero()出现了问题

Kees Cook建议他使用适用于s390的GCC版本;而Linus看出了问题所在,他有过这方面的经验:又是老问题——重名!

随后Linus修复了该问题,然而在后续交流中Sebastian Ott继续吐槽:"......and our _UNIQUEID() macro is garbage anyway......" (小编吐槽环节:用了小众机器就不要乱骂别人garbage啊。。)

Linus作出回复,并深刻反思了错误出现的原因:

从此, _UNIQUE_ID()这个宏的使用需要变得谨慎,当然,max()也需要重写。

所以,最新的max()应运而生

最后请大家再欣赏一下max(),详细分解请参考公众号上一篇文章:

【[大神Linus对这个宏怎么看?] 】[点击访问]

#define __typecheck(x, y) \
    (!!(sizeof((typeof(x) *)1 == (typeof(y) *)1)))

#define __is_constexpr(x) \
    (sizeof(int) == sizeof(*(8 ? ((void *)((long)(x) * 0l)) : (int *)8)))

#define __no_side_effects(x, y) \
    (__is_constexpr(x) && __is_constexpr(y))

#define __safe_cmp(x, y) \
    (__typecheck(x, y) && __no_side_effects(x, y))

#define __cmp(x, y, op) ((x) op (y) ? (x) : (y))

#define __cmp_once(x, y, unique_x, unique_y, op) ({ \
    typeof(x) unique_x = (x);       \
    typeof(y) unique_y = (y);       \
    __cmp(unique_x, unique_y, op); })

#define __careful_cmp(x, y, op) \
    __builtin_choose_expr(__safe_cmp(x, y), \
    __cmp(x, y, op), \
    __cmp_once(x, y, __UNIQUE_ID(__x), __UNIQUE_ID(__y), op))


#define max(x, y)   __careful_cmp(x, y, >)

目前,Linux内核最新版本为5.3.1,该版本中max()似乎没有再进行修改。

结束语

从发现bug到修改bug,再上升到“机制”与“策略”,不断迭代,拥抱变化。Linux内核作为最大的开源项目,值得我们学习和探索。

本文由哈喽比特于1年以前收录,如有侵权请联系我们。
文章来源:https://mp.weixin.qq.com/s/TxYvntnPi7w5jnMcmWXltw

 相关推荐

刘强东夫妇:“移民美国”传言被驳斥

京东创始人刘强东和其妻子章泽天最近成为了互联网舆论关注的焦点。有关他们“移民美国”和在美国购买豪宅的传言在互联网上广泛传播。然而,京东官方通过微博发言人发布的消息澄清了这些传言,称这些言论纯属虚假信息和蓄意捏造。

发布于:8月以前  |  808次阅读  |  详细内容 »

博主曝三大运营商,将集体采购百万台华为Mate60系列

日前,据博主“@超能数码君老周”爆料,国内三大运营商中国移动、中国电信和中国联通预计将集体采购百万台规模的华为Mate60系列手机。

发布于:8月以前  |  770次阅读  |  详细内容 »

ASML CEO警告:出口管制不是可行做法,不要“逼迫中国大陆创新”

据报道,荷兰半导体设备公司ASML正看到美国对华遏制政策的负面影响。阿斯麦(ASML)CEO彼得·温宁克在一档电视节目中分享了他对中国大陆问题以及该公司面临的出口管制和保护主义的看法。彼得曾在多个场合表达了他对出口管制以及中荷经济关系的担忧。

发布于:8月以前  |  756次阅读  |  详细内容 »

抖音中长视频App青桃更名抖音精选,字节再发力对抗B站

今年早些时候,抖音悄然上线了一款名为“青桃”的 App,Slogan 为“看见你的热爱”,根据应用介绍可知,“青桃”是一个属于年轻人的兴趣知识视频平台,由抖音官方出品的中长视频关联版本,整体风格有些类似B站。

发布于:8月以前  |  648次阅读  |  详细内容 »

威马CDO:中国每百户家庭仅17户有车

日前,威马汽车首席数据官梅松林转发了一份“世界各国地区拥车率排行榜”,同时,他发文表示:中国汽车普及率低于非洲国家尼日利亚,每百户家庭仅17户有车。意大利世界排名第一,每十户中九户有车。

发布于:8月以前  |  589次阅读  |  详细内容 »

研究发现维生素 C 等抗氧化剂会刺激癌症生长和转移

近日,一项新的研究发现,维生素 C 和 E 等抗氧化剂会激活一种机制,刺激癌症肿瘤中新血管的生长,帮助它们生长和扩散。

发布于:8月以前  |  449次阅读  |  详细内容 »

苹果据称正引入3D打印技术,用以生产智能手表的钢质底盘

据媒体援引消息人士报道,苹果公司正在测试使用3D打印技术来生产其智能手表的钢质底盘。消息传出后,3D系统一度大涨超10%,不过截至周三收盘,该股涨幅回落至2%以内。

发布于:8月以前  |  446次阅读  |  详细内容 »

千万级抖音网红秀才账号被封禁

9月2日,坐拥千万粉丝的网红主播“秀才”账号被封禁,在社交媒体平台上引发热议。平台相关负责人表示,“秀才”账号违反平台相关规定,已封禁。据知情人士透露,秀才近期被举报存在违法行为,这可能是他被封禁的部分原因。据悉,“秀才”年龄39岁,是安徽省亳州市蒙城县人,抖音网红,粉丝数量超1200万。他曾被称为“中老年...

发布于:8月以前  |  445次阅读  |  详细内容 »

亚马逊股东起诉公司和贝索斯,称其在购买卫星发射服务时忽视了 SpaceX

9月3日消息,亚马逊的一些股东,包括持有该公司股票的一家养老基金,日前对亚马逊、其创始人贝索斯和其董事会提起诉讼,指控他们在为 Project Kuiper 卫星星座项目购买发射服务时“违反了信义义务”。

发布于:8月以前  |  444次阅读  |  详细内容 »

苹果上线AppsbyApple网站,以推广自家应用程序

据消息,为推广自家应用,苹果现推出了一个名为“Apps by Apple”的网站,展示了苹果为旗下产品(如 iPhone、iPad、Apple Watch、Mac 和 Apple TV)开发的各种应用程序。

发布于:8月以前  |  442次阅读  |  详细内容 »

特斯拉美国降价引发投资者不满:“这是短期麻醉剂”

特斯拉本周在美国大幅下调Model S和X售价,引发了该公司一些最坚定支持者的不满。知名特斯拉多头、未来基金(Future Fund)管理合伙人加里·布莱克发帖称,降价是一种“短期麻醉剂”,会让潜在客户等待进一步降价。

发布于:8月以前  |  441次阅读  |  详细内容 »

光刻机巨头阿斯麦:拿到许可,继续对华出口

据外媒9月2日报道,荷兰半导体设备制造商阿斯麦称,尽管荷兰政府颁布的半导体设备出口管制新规9月正式生效,但该公司已获得在2023年底以前向中国运送受限制芯片制造机器的许可。

发布于:8月以前  |  437次阅读  |  详细内容 »

马斯克与库克首次隔空合作:为苹果提供卫星服务

近日,根据美国证券交易委员会的文件显示,苹果卫星服务提供商 Globalstar 近期向马斯克旗下的 SpaceX 支付 6400 万美元(约 4.65 亿元人民币)。用于在 2023-2025 年期间,发射卫星,进一步扩展苹果 iPhone 系列的 SOS 卫星服务。

发布于:8月以前  |  430次阅读  |  详细内容 »

𝕏(推特)调整隐私政策,可拿用户发布的信息训练 AI 模型

据报道,马斯克旗下社交平台𝕏(推特)日前调整了隐私政策,允许 𝕏 使用用户发布的信息来训练其人工智能(AI)模型。新的隐私政策将于 9 月 29 日生效。新政策规定,𝕏可能会使用所收集到的平台信息和公开可用的信息,来帮助训练 𝕏 的机器学习或人工智能模型。

发布于:8月以前  |  428次阅读  |  详细内容 »

荣耀CEO谈华为手机回归:替老同事们高兴,对行业也是好事

9月2日,荣耀CEO赵明在采访中谈及华为手机回归时表示,替老同事们高兴,觉得手机行业,由于华为的回归,让竞争充满了更多的可能性和更多的魅力,对行业来说也是件好事。

发布于:8月以前  |  423次阅读  |  详细内容 »

AI操控无人机能力超越人类冠军

《自然》30日发表的一篇论文报道了一个名为Swift的人工智能(AI)系统,该系统驾驶无人机的能力可在真实世界中一对一冠军赛里战胜人类对手。

发布于:8月以前  |  423次阅读  |  详细内容 »

AI生成的蘑菇科普书存在可致命错误

近日,非营利组织纽约真菌学会(NYMS)发出警告,表示亚马逊为代表的电商平台上,充斥着各种AI生成的蘑菇觅食科普书籍,其中存在诸多错误。

发布于:8月以前  |  420次阅读  |  详细内容 »

社交媒体平台𝕏计划收集用户生物识别数据与工作教育经历

社交媒体平台𝕏(原推特)新隐私政策提到:“在您同意的情况下,我们可能出于安全、安保和身份识别目的收集和使用您的生物识别信息。”

发布于:8月以前  |  411次阅读  |  详细内容 »

国产扫地机器人热销欧洲,国产割草机器人抢占欧洲草坪

2023年德国柏林消费电子展上,各大企业都带来了最新的理念和产品,而高端化、本土化的中国产品正在不断吸引欧洲等国际市场的目光。

发布于:8月以前  |  406次阅读  |  详细内容 »

罗永浩吐槽iPhone15和14不会有区别,除了序列号变了

罗永浩日前在直播中吐槽苹果即将推出的 iPhone 新品,具体内容为:“以我对我‘子公司’的了解,我认为 iPhone 15 跟 iPhone 14 不会有什么区别的,除了序(列)号变了,这个‘不要脸’的东西,这个‘臭厨子’。

发布于:8月以前  |  398次阅读  |  详细内容 »
 相关文章
Android插件化方案 5年以前  |  236901次阅读
vscode超好用的代码书签插件Bookmarks 1年以前  |  7059次阅读
 目录