Skip to content

Conversation

@JulyWindK
Copy link
Collaborator

What type of PR is this?

What this PR does / why we need it:

Which issue(s) this PR fixes:

Special notes for your reviewer:

JulyWindK and others added 27 commits August 11, 2025 19:28
feat(irq-tuning): implement misc utils

Signed-off-by: 张浩宇 <[email protected]>
feat(irq-tuning): implement sys utils

Signed-off-by: 张浩宇 <[email protected]>
feat(irq-tuning): implement cpu utils

Signed-off-by: 张浩宇 <[email protected]>
feat(irq-tuning): implement net utils, and need to change
katalyst-adaptor accordingly.

Signed-off-by: 张浩宇 <[email protected]>
feat(irq-tuning): implement configurations

Signed-off-by: 张浩宇 <[email protected]>
feat(irq-tuning): implement controller

Signed-off-by: 张浩宇 <[email protected]>
chore(irq-tuning): exclude forbidden cores for balance-fair irq affinity

Signed-off-by: 张浩宇 <[email protected]>
refactor(irq-tuning): optimize qualified ccd criteria for fair-balance

Signed-off-by: 张浩宇 <[email protected]>
chore(irq-tuning): trivial fix

Signed-off-by: 张浩宇 <[email protected]>
enable/disable irq-tunner by env

Signed-off-by: 张浩宇 <[email protected]>
feat(irq-tuning): report nic's balance-fair irq tuning

Signed-off-by: 张浩宇 <[email protected]>
remove check if irqbalance-ng is running

Signed-off-by: 张浩宇 <[email protected]>
log info when irq tuning config changed

Signed-off-by: 张浩宇 <[email protected]>
feat(irq-tuning): introduce validation of irq tuning dynamic config

Signed-off-by: 张浩宇 <[email protected]>
fix cgroup v1 and v2 cgrouppath prefix

Signed-off-by: 张浩宇 <[email protected]>
feat(irq-tuning): support tx irq affinity tuning

Signed-off-by: 张浩宇 <[email protected]>
trivial fix

Signed-off-by: 张浩宇 <[email protected]>
@JulyWindK JulyWindK force-pushed the dev/support-irq-tuning-release-test-0813 branch from 1f719ef to d63cffa Compare August 14, 2025 03:11
@codecov
Copy link

codecov bot commented Aug 14, 2025

Codecov Report

❌ Patch coverage is 51.79063% with 1575 lines in your changes missing coverage. Please review.
✅ Project coverage is 58.09%. Comparing base (a1e195e) to head (f88f662).
⚠️ Report is 10 commits behind head on main.

Files with missing lines Patch % Lines
...lugins/cpu/dynamicpolicy/irqtuner/config/config.go 0.00% 422 Missing ⚠️
pkg/util/general/misc.go 10.74% 183 Missing and 8 partials ⚠️
pkg/util/machine/network_linux.go 80.87% 163 Missing and 26 partials ⚠️
.../qrm-plugins/cpu/dynamicpolicy/policy_irq_tuner.go 0.00% 161 Missing ⚠️
pkg/util/general/sys.go 0.00% 144 Missing ⚠️
...cpu/dynamicpolicy/irqtuner/tuner/irq_tuner_stub.go 0.00% 54 Missing ⚠️
pkg/util/procfs/manager/procfs_linux.go 62.50% 40 Missing and 14 partials ⚠️
pkg/util/machine/cpu.go 89.90% 29 Missing and 15 partials ⚠️
pkg/util/sysfs/manager/sys_linux.go 0.00% 43 Missing ⚠️
pkg/util/machine/topology.go 53.08% 34 Missing and 4 partials ⚠️
... and 35 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #941      +/-   ##
==========================================
- Coverage   61.84%   58.09%   -3.76%     
==========================================
  Files         642      668      +26     
  Lines       68182    75334    +7152     
==========================================
+ Hits        42166    43764    +1598     
- Misses      21839    27299    +5460     
- Partials     4177     4271      +94     
Flag Coverage Δ
unittest 58.09% <51.79%> (-3.76%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@JulyWindK JulyWindK force-pushed the dev/support-irq-tuning-release-test-0813 branch from 7fcfccf to f88f662 Compare August 14, 2025 09:18
@JulyWindK JulyWindK force-pushed the dev/support-irq-tuning-release-test-0813 branch from 589f35c to 4fdd878 Compare August 15, 2025 08:54
@JulyWindK JulyWindK force-pushed the dev/support-irq-tuning-release-test-0813 branch from 4fdd878 to 55e9bfd Compare August 15, 2025 09:18
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