resource package support pin cpuset#1074
Open
luomingmeng wants to merge 10 commits intokubewharf:mainfrom
Open
resource package support pin cpuset#1074luomingmeng wants to merge 10 commits intokubewharf:mainfrom
luomingmeng wants to merge 10 commits intokubewharf:mainfrom
Conversation
luomingmeng
requested review from
cheney-lin,
sun-yuliang,
waynepeking348 and
xu282934741
as code owners
luomingmeng
force-pushed
the
dev/resource-package-support-pin-cpuset
branch
7 times, most recently
from
566dd0c to
b609a22
Compare
luomingmeng
force-pushed
the
dev/resource-package-support-pin-cpuset
branch
from
8167ee4 to
fabdd52
Compare
luomingmeng
force-pushed
the
dev/resource-package-support-pin-cpuset
branch
4 times, most recently
from
c5d3e65 to
5279657
Compare
luomingmeng
force-pushed
the
dev/resource-package-support-pin-cpuset
branch
12 times, most recently
from
042ac3a to
6f77250
Compare
luomingmeng
force-pushed
the
dev/resource-package-support-pin-cpuset
branch
from
6f77250 to
bf27756
Compare
luomingmeng
force-pushed
the
dev/resource-package-support-pin-cpuset
branch
2 times, most recently
from
d499595 to
2e422cb
Compare
…pool to boolean Update the resource package conversion logic to represent pinned cpuset as a boolean flag instead of a pool name Modify test cases to reflect the new pinned cpuset representation
- Fix race condition in Start() by properly handling started flag and stop channel - Replace sleep-based test synchronization with channel signaling - Ensure cpuPressureEviction only runs when policy is started
…e PinnedCPUSetInfo - Remove PinnedCPUSetInfo and related fields from QoSRegion interfaces - Add GetResourcePackageName method to QoSRegion interface - Introduce ResourcePackageConfig type for storing pinned cpuset info - Update CPU server to handle resource package config from advisor requests - Refactor region assembly to use resource package config from metacache - Simplify region creation by removing pinnedCPUSetInfo parameter - Add comprehensive tests for resource package config handling
Move zone attribute names used in topology adapter to a centralized consts package to maintain consistency between CNR producers and consumers. Update all references to use the new constants from pkg/consts.
…n pools split the pool allocation logic into separate functions for pinned and common pools add comprehensive tests for various allocation scenarios remove redundant pinned CPU set parameter from generatePoolsAndIsolation
…support Add resource package name handling in region creation and metrics Modify CPU allocation logic to consider resource package pinned CPUs
… package Consider resource package when setting origin owner pool name for numa binding containers. This ensures proper pool name wrapping when a resource package is specified and valid.
…pu allocation logic remove redundant unwrap operation in GetResourcePackageName and use annotations directly modify cpu allocation to consider pinned cpusets for better resource isolation
…utes add attribute selector to filter resource packages for IRQ forbidden CPUs add helper function to convert CNR attributes to string map add utility function to aggregate pinned CPU sets from matching resource packages update IRQ forbidden core logic to include resource package pinned CPUs add tests for new functionality
luomingmeng
force-pushed
the
dev/resource-package-support-pin-cpuset
branch
from
2e422cb to
f0db5be
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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: