Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed the issue where CPU usage only shows a % #11613

Closed
wants to merge 2 commits into from
Closed

Fixed the issue where CPU usage only shows a % #11613

wants to merge 2 commits into from

Conversation

cjpjxjx
Copy link
Contributor

@cjpjxjx cjpjxjx commented Oct 22, 2023

Q:你知道这是pull request吗?(使用 "x" 选择)

  • 我知道

之前提过 issues #11603

经过不断尝试,发现在 package/lean/autocore/files/x86/index.htm 文件中获取 CPU 使用率所在行的命令 top -n 1 | grep 'CPU:' 会获取到两行数据,而另一行是 grep CPU: 进程自身,最终导致 CPU 使用率计算报错,如下所示

root@OpenWrt:~# top -n 1 | grep 'CPU:' 
CPU:   0% usr   9% sys   0% nic  90% idle   0% io   0% irq   0% sirq
 2813  6672 root     S     1124   0%   0% grep CPU:

由于 x86 平台和 arm 平台使用的是不同的 index.htm 文件,获取 CPU 使用率的命令也不一样,再结合其他人的反馈,所以这个问题应该是只出现在 x86 平台上

虽然我也还是不知道为啥在编译时取消 autosamba 就会出现这个问题,但是修改匹配条件使用 grep -E '^CPU:' 匹配以 CPU: 开头的行可修复此问题

上次的 issues 问题:#11603

经过不断尝试,发现问题原因在 `package/lean/autocore/files/x86/index.htm` 文件

该文件中获取 CPU 使用率所在行的命令 `top -n 1 | grep 'CPU:'` 会获取到两行数据,另一行是 `grep CPU` 进程自身,导致计算报错,如下所示

```
root@OpenWrt:~# top -n 1 | grep 'CPU:' 
CPU:   0% usr   9% sys   0% nic  90% idle   0% io   0% irq   0% sirq
 2813  6672 root     S     1124   0%   0% grep CPU:
```

由于 x86 平台和 arm 平台使用的是不同的 index.htm 文件,获取 CPU 使用率的命令也不一样,再结合其他人的反馈,所以这个问题应该是只出现在 x86 平台上

虽然我也还是不知道为啥在编译时取消 autosamba 就会出现这个问题,但是修改匹配条件使用 `grep -E '^CPU:' ` 匹配以 `CPU:` 开头的行可修复此问题

在编译前修改 `package/lean/autocore/files/x86/index.htm`,找到 `grep 'CPU:'` 修改为 `grep -E '^CPU:'` 即可
@cjpjxjx cjpjxjx closed this Oct 22, 2023
@cjpjxjx
Copy link
Contributor Author

cjpjxjx commented Oct 22, 2023

已重新提交

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant