koordlet: fix not support cpu without L3 cache#960
Conversation
Signed-off-by: shelwin <weixf@szhcf.com.cn>
Codecov ReportBase: 66.96% // Head: 66.97% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #960 +/- ##
==========================================
+ Coverage 66.96% 66.97% +0.01%
==========================================
Files 243 243
Lines 28240 28244 +4
==========================================
+ Hits 18910 18917 +7
+ Misses 8035 8032 -3
Partials 1295 1295
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
e13006e to
71b0de7
Compare
|
@shelwinnn It seems to be a compatibility problem. Could you please add more information about your system environment and what the lscpu's output ( |
On my company's cloud, we use kvm as the cloud host and qemu-kvm not enable L3 cache, see: https://bugzilla.redhat.com/show_bug.cgi?id=1434537
CPU NODE SOCKET CORE L1d:L1i:L2 ONLINE
0 0 0 0 0:0:0 yes
1 0 0 1 1:1:1 yes
2 0 0 2 2:2:2 yes
3 0 0 3 3:3:3 yes
4 0 1 4 4:4:4 yes
5 0 1 5 5:5:5 yes
6 0 1 6 6:6:6 yes
7 0 1 7 7:7:7 yes |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: saintube The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Signed-off-by: shelwin <weixf@szhcf.com.cn> Co-authored-by: shelwin <weixf@szhcf.com.cn>
Ⅰ. Describe what this PR does
Fix
lscpuget cache info failed when the cpu has no L3 cache.Ⅱ. Does this pull request fix one issue?
Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist
make test