Skip to content

Introduce delay before reading CPU MHz - #6

Open
PutinYpa wants to merge 1 commit into
MiKing233:mainfrom
PutinYpa:patch-1
Open

Introduce delay before reading CPU MHz#6
PutinYpa wants to merge 1 commit into
MiKing233:mainfrom
PutinYpa:patch-1

Conversation

@PutinYpa

Copy link
Copy Markdown

Add a sleep delay before fetching CPU frequencies. 这是能找到的最好的PVE温度状态显示修改脚本了,但有一个问题就是页面后台刷新本身的资源占用导致显示的CPU线程频率偏高很多,经过多次尝试,在获取CPU频率之前 sleep 0.2秒,使当前进程冷却一小段时间再获取CPU频率即可相对准确。希望作者采纳 :)

Add a sleep delay before fetching CPU frequencies.
这是能找到的最好的PVE温度状态显示修改脚本了,但有一个问题就是页面后台刷新本身的资源占用导致显示的CPU线程频率偏高很多,经过多次尝试,在获取CPU频率之前 sleep 0.2秒,使当前进程冷却一小段时间再获取CPU频率即可相对准确。希望作者采纳 :)
@MiKing233

Copy link
Copy Markdown
Owner

功能确认生效, 的确可以更准确的显示设备的CPU频率, 但并非没有代价, 这将会带来额外 200ms 的网页刷新延迟,

目前在使用 pve-manager-status.sh 后, pve 管理页面的自动刷新在 GET https://x.x.x.x:8006/api2/json/nodes/xxx/status 时, 已经会因为新增的各项额外功能导致耗时来到了 200-400ms+, 这取决于具体的设备性能, 如果性能较弱甚至能超过500ms, 因此如果合并此更改将会又带来额外的 200ms 刷新延迟开销.

实测在我的一台配置为 Intel(R) N100 的设备上, 合并此更改后 GET status 的耗时已经达到了 680ms+ , 作为对比, 在未使用本项目时原生的 pve 界面在完成一次载入 status 的耗时仅为 30-40ms.

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.

2 participants