利用xhprof分析zabbix页面性能问题

xhprof是什么?

xhprof是一个php的性能分析利器,写过php的东西应该都比较清楚。

大致说下功能:

  • 统计函数调用的次数
  • 内存使用
  • CPU占用

zabbix页面的问题

zabbix 性能一直都是非常让人不满的地方,页面部分自然也跑不掉了。

页面的功能是非常的丰富和强大,但至于性能上着实不够。

最近我们遇到的一个问题就是发现screens.php这个页面打开速度非常慢,而且

在切换主机的时候同样非常非常慢。平均需要等待15s的样子吧,一大堆的同事均抱怨。

so,我们还是上xhprof来分析下问题吧。

定位问题

抓包分析

  • 首先是同时抱怨慢的时候,进行了抓包测试,发现消耗了大量的事件在等待服务器响应,但

页面刷新完成后发现每个图表的加载事件非常短,那说明问题不在数据加载,而在于整个页面

的处理逻辑上。

  • 其次,当时利用其他帐号测试的时候发现加载速度与帐号有关,更进一步的测试发现超级管理员的

速度非常块,而其他帐号速度基本一致,非常慢。

而帐号的差别最大的可能就是权限检查的部分不同–超级管理员不需要权限检查。

那我们暂时记住,先用xhprof分析下页面吧。

  • 抓包分析结果:

抓包分析

xhprof分析

配置、使用都不多说了,网上百度即可。

贴一个分析的结果:

xhprof分析结果

可以非常直观的看到大量的事件浪费在CScreen::get这个调用上,那问题就很简单了,我们去看下这部分的代码。

分析代码

Screen::get 这是一个api调用,所以代码路径在api/classes/CScreen.php中,我们看下get方法。

很快发现了疑似的地方:

               // editable + PERMISSION CHECK
                if ($userType == USER_TYPE_SUPER_ADMIN || $options['nopermissions']) {
                }   
                elseif ($result) {
                        $groupsToCheck = array();
                        $hostsToCheck = array();
                        $graphsToCheck = array();
                        $itemsToCheck = array();
                        $mapsToCheck = array();
                        $screensToCheck = array();
                        $screensItems = array();

                        $dbScreenItems = DBselect('SELECT si.* FROM screens_items si WHERE '.dbConditionInt('si.screenid', $screenIds));

                        while ($screenItem = DBfetch($dbScreenItems)) {

看到了吧,如果是超级用户则不进行权限检查,否则就进行一个非常复杂的权限检查过程。

我把对应的sql打印出来,发现非常非常长,执行长达4.8s,这和前面的xhprof图的中DBselect

速度慢也吻合。

修复方法

代码已经定位了,修复方法就不是问题了,简单的注释掉就可以了。

验证效果

重新抓包看下:

修复效果

Leave a Reply

Your email address will not be published. Required fields are marked *


To create code blocks or other preformatted text, indent by four spaces:

    This will be displayed in a monospaced font. The first four 
    spaces will be stripped off, but all other whitespace
    will be preserved.
    
    Markdown is turned off in code blocks:
     [This is not a link](http://example.com)

To create not a block, but an inline code span, use backticks:

Here is some inline `code`.

For more help see http://daringfireball.net/projects/markdown/syntax