一个有趣的死循环问题

问题引出

最近在工作过程中,遇到一个很有意思的bug,问题出现概率很小,很难复现,但是特别严重,直接导致用户无法使用App,属于不可容忍的问题,因此必须解决。问题出现在Android-App跑压测的过程中,我们提供的第三方库会突然停止工作,但是App的UI还是能正常的运转,于是我以我的职业敏感性当即做出判断(害羞 <逃),导致这个bug是因为我们提供的动态库中的消息线程被堵塞了,从而导致整个库无法工作。
通常情况下,线程被堵塞住,有下面几个原因:

  • 执行耗时任务(如网络IO或者文件IO等,导致整个线程卡死)
  • 死锁(消息线程拿着A锁,等待B锁,数据处理线程拿着B锁,等待A锁,相互等待,导致卡死)
  • 死循环(程序卡在一个循环,CPU狂转,无法跳出)

问题定位

首先第一个思路,就是在App出现卡主问题后,导出所有线程的调用堆栈,查看我们自己的库线程,看哪个线程哪个函数调用存在问题,取线程堆栈,通常有两个方法:

  • 在Android端,程序ANR后,系统会保留该进程的所有线程堆栈的traces文件。但是有个前提,只有主线程(UI线程)卡主的时候,系统才会生成这个文件,其他子线程或者库线程卡主是不会生成traces文件的。
  • 出现问题后,让App强制crash,生成crash线程堆栈。我们尝试让UI线程强制crash后,却只生成了UI线程的调用栈,没有我们库线程的信息。

抓调用栈的思路否定之后,我们决定先进一步缩小问题范围,出现卡主问题后,我们对Android手机执行了如下命令,找出我们App所有线程的运行状况。

adb shell top -m 20 -t

巧合地发现每次采样数据中,CPU使用率最高都是我们App进程中的同一个线程,正是我们动态库的消息线程,而该线程的使用率竟然比图像绘制线程占比还要高了几倍,很明显的异常,因此几乎可以断定是该线程导致的卡死问题,同时也可以排除死锁的可能性,因为死锁会让线程wait住,不会过多的占用CPU,同时由于我们的AppIO操作很少,也就排除了文件IO和网络IO,基本可以确认是死循环导致。

那么这个线程到底是在哪个位置卡死呢?这时候最笨的方法,效果最明显,【二分加Log法】,加log的位置以代码行数进行二分切隔,如果第N行日志没有输出,那么肯定说明问题代码在第N-1~N行之间,于是乎最终定位到了这一段神奇的死循环代码问题。

void TransAngleTo360(double& dAngle)
{
    while (dAngle < 0)
        dAngle += 360;
    
    while (dAngle > 360)
        dAngle -= 360;
}

这段代码,通过两个while循环将输入角度归化到0~360度之间,问题就出在这个输入值这里。通过复现的日志发现,代码走到了异常分支,对某个静态数组 double arrAngele[8],执行了取下标-1的操作,arrAngele[-1],产生了一个未定义的极大值,8字节的double的取值范围是:

负值取值范围为 -1.79769313486231570E+308 到 -4.94065645841246544E-324;
正值取值范围为 4.94065645841246544E-324 到 1.79769313486231570E+308。

如果将1.79e+308这个数传给TransAngleTo360函数,那么函数需要执行5e+305次循环才能结束,假设我们的计算机每秒能执行10亿次机器指令,那执行完这个循环的时间,大约是1e+207秒。。。。

问题解决

对于函数输入参数,进行超大或超小值的范围控制,很明显这个方法只是暂时的补丁,且是在你知道入参大小的前提下,做的限制,如果不确定入参范围,很可能会出问题。
那么更好的做法其实是这样的,先看整数部分有多少个360,记为cnt,然后用原有角度减去cnt*360即可,为负数的时候对应处理下就行。

void TransAngleTo360(double& dAngle)
{
    int cnt = dAngle/360;
    if (time >= 0)
        dAngle = dAngle - cnt*360;
    else
        dAngle = dAngle - (cnt - 1)*360;
}
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 204,793评论 6 478
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 87,567评论 2 381
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 151,342评论 0 338
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 54,825评论 1 277
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 63,814评论 5 368
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 48,680评论 1 281
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 38,033评论 3 399
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 36,687评论 0 258
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 42,175评论 1 300
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 35,668评论 2 321
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 37,775评论 1 332
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 33,419评论 4 321
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 39,020评论 3 307
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 29,978评论 0 19
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 31,206评论 1 260
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 45,092评论 2 351
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 42,510评论 2 343

推荐阅读更多精彩内容

  • ORA-00001: 违反唯一约束条件 (.) 错误说明:当在唯一索引所对应的列上键入重复值时,会触发此异常。 O...
    我想起个好名字阅读 5,169评论 0 9
  • Swift1> Swift和OC的区别1.1> Swift没有地址/指针的概念1.2> 泛型1.3> 类型严谨 对...
    cosWriter阅读 11,084评论 1 32
  • 本文出自 Eddy Wiki ,转载请注明出处:http://eddy.wiki/interview-java.h...
    eddy_wiki阅读 2,054评论 0 14
  • 国贸162单凌云40 又是一年冬季即将到来,在这个时候我们最需要的是什么?也许...
    Amao666阅读 931评论 0 0
  • “喂,老公,出来了没,我已经到了。哦哦,好的,路上小心,待会见” 小凡挂下电话,先点了一杯蜜橘茶,满口满心甜蜜蜜。...
    zoe_1b5f阅读 1,041评论 0 0