Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

virtio-fs: virtiofsd daemon is not in sandbox cgroup #1972

Closed
renzhengeek opened this issue Aug 19, 2019 · 1 comment
Closed

virtio-fs: virtiofsd daemon is not in sandbox cgroup #1972

renzhengeek opened this issue Aug 19, 2019 · 1 comment
Labels
bug Incorrect behaviour needs-review Needs to be assessed by the team.

Comments

@renzhengeek
Copy link

Description of problem

virtiofsd daemon is not in sandbox cgroup.

Expected result

qemu and virtiofsd are in the same cgroup.

Actual result

qemu pid: 44986
virtiofsd pid: 44981

qemu cgroups:

$cat /proc/44986/cgroup 
11:memory:/kata/default/kata_9f49ba797580a17bdc1ab6881af038246bb93fa3dbfaa5f91c5b4bdb2b785777
10:hugetlb:/
9:perf_event:/
8:blkio:/
7:pids:/
6:net_cgroup:/
5:freezer:/
4:net_cls,net_prio:/
3:devices:/
2:cpuset,cpu,cpuacct:/
1:name=systemd:/

virtiofsd cgroups:

$cat /proc/44981/cgroup 
11:memory:/
10:hugetlb:/
9:perf_event:/
8:blkio:/
7:pids:/
6:net_cgroup:/
5:freezer:/
4:net_cls,net_prio:/
3:devices:/
2:cpuset,cpu,cpuacct:/
1:name=systemd:/
@renzhengeek renzhengeek added bug Incorrect behaviour needs-review Needs to be assessed by the team. labels Aug 19, 2019
bergwolf added a commit to bergwolf/kata-runtime that referenced this issue Aug 19, 2019
So that we save it to the same cgroup as the qemu process.

Fixes: kata-containers#1972
Signed-off-by: Peng Tao <[email protected]>
bergwolf added a commit to bergwolf/kata-runtime that referenced this issue Aug 19, 2019
So that we save it to the same cgroup as the qemu process.

Fixes: kata-containers#1972
Signed-off-by: Peng Tao <[email protected]>
@renzhengeek
Copy link
Author

renzhengeek commented Aug 19, 2019

Thanks! Now qemu and virtiofsd are in the same cgroup:

# ps -o cgroup 85108
CGROUP
4:memory:/kata/default/kata_ce737c285b06a24a900dac0649d9a7ab6541ca89f878dd5e1ad0c23ca3a1c4ab
# ps -o cgroup 85101
CGROUP
4:memory:/kata/default/kata_ce737c285b06a24a900dac0649d9a7ab6541ca89f878dd5e1ad0c23ca3a1c4ab

bergwolf added a commit to bergwolf/kata-runtime that referenced this issue Aug 21, 2019
so that for qemu, we can save and export virtiofsd pid,
and put it to the same cgroup as the qemu process.

Fixes: kata-containers#1972
Signed-off-by: Peng Tao <[email protected]>
bergwolf added a commit to bergwolf/kata-runtime that referenced this issue Aug 21, 2019
so that for qemu, we can save and export virtiofsd pid,
and put it to the same cgroup as the qemu process.

Fixes: kata-containers#1972
Signed-off-by: Peng Tao <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Incorrect behaviour needs-review Needs to be assessed by the team.
Projects
None yet
Development

No branches or pull requests

1 participant