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.

Sign up for GitHub

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

[core] Setup cgroup v2 in C++ #49416

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
+301 −0
Open

[core] Setup cgroup v2 in C++ #49416

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
a9e8dc6
setup cgroup v2 in C++
dentiny Dec 23, 2024
feb0282
strinv view for cgroup content
dentiny Dec 24, 2024
ce77002
rename cgroup -> cgroup v2
dentiny Dec 24, 2024
995dc0f
add comment for uuid
dentiny Dec 24, 2024
f54f2eb
minor update in ifdef macro
dentiny Dec 24, 2024
f3a4a9a
raii for cgroup v2
dentiny Dec 24, 2024
341317f
ifdef
dentiny Dec 26, 2024
50874cd
rename as id
dentiny Dec 26, 2024
3027ed8
comment on cgroup class
dentiny Dec 26, 2024
12f0cfe
fix comment
dentiny Dec 26, 2024
c5e3fd7
place process to default cgroup
dentiny Dec 26, 2024
b679ba7
comment to add process to application cgroup
dentiny Dec 27, 2024
7689889
add precondition requirement and fix cleanup logic
dentiny Dec 27, 2024
792fa54
leave TODO for string view
dentiny Dec 27, 2024
128bfdf
remove unused function
dentiny Dec 27, 2024
d3fdd2a
remove useless if branch
dentiny Dec 27, 2024
2dcd1e7
local manager change for cgroup
dentiny Dec 27, 2024
56aeea9
special handle EEXISTS
dentiny Jan 6, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
add comment for uuid 
Signed-off-by: dentiny <dentinyhao@gmail.com>
  • Loading branch information
@dentiny
dentiny committed Dec 24, 2024
commit 995dc0f6b6ed0bf4e907d9e4cb841c25c1a8f731
2 changes: 1 addition & 1 deletion src/ray/common/cgroup/cgroup_context.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ namespace ray {
struct PhysicalModeExecutionContext {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Q: do we need this separate config class from CgroupV2Setup?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These data fields are necessary to construct and destruct cgroup;
As of now the struct doesn't seem that necessary since it only contains 4 fields and we could directly pass them into the factory function, but we could have much more fields (i.e. cpu-related, resource min / high), better to have a struct.

// Directory for cgroup, which is applied to application process.
std::string_view cgroup_directory;
// UUID to indicate the current task / actor.
// A UUID to uniquely identity a certain task / actor attempt.
std::string uuid;
dentiny marked this conversation as resolved.
Show resolved Hide resolved
// PID for the process.
pid_t pid;
Expand Down
Loading