[google/syzkaller] ed11bb: executor: don't fail on cgroup mounting

39 views
Skip to first unread message

Dmitry Vyukov

unread,
Oct 13, 2021, 3:19:59 AM10/13/21
to syzk...@googlegroups.com
Branch: refs/heads/master
Home: https://github.com/google/syzkaller
Commit: ed11bb90f0a775e91f9ae64e3be951a47cd642c3
https://github.com/google/syzkaller/commit/ed11bb90f0a775e91f9ae64e3be951a47cd642c3
Author: Dmitry Vyukov <dvy...@google.com>
Date: 2021-10-13 (Wed, 13 Oct 2021)

Changed paths:
M executor/common_linux.h
M pkg/csource/generated.go

Log Message:
-----------
executor: don't fail on cgroup mounting

On stretch images setup_cgroups fails as:

mount(/syzcgroup/net, net) failed: 22
mount(/syzcgroup/net, net_cls) failed: 22
mount(/syzcgroup/net, net_prio) failed: 22
mount(/syzcgroup/net, blkio) failed: 22
SYZFAIL: mount cgroup failed
(/syzcgroup/net, devices,freezer): 16
(errno 16: Device or resource busy)

It seems that systemd starts messing with these mounts somehow
and repeated mounting fails with EBUSY.
Don't hard fail on that error.


Commit: 76a21659aa2233bae7f7323704a3ca50f0a43df6
https://github.com/google/syzkaller/commit/76a21659aa2233bae7f7323704a3ca50f0a43df6
Author: Dmitry Vyukov <dvy...@google.com>
Date: 2021-10-13 (Wed, 13 Oct 2021)

Changed paths:
M sys/linux/cgroup.txt
M sys/linux/filesystem.txt

Log Message:
-----------
sys/linux: enable controllers when mounting cgroups

Cgroup mounts also accept a list of subsystem controllers.


Commit: 9d56e7ddd67e5ec46588c6434db739d94a7d2aae
https://github.com/google/syzkaller/commit/9d56e7ddd67e5ec46588c6434db739d94a7d2aae
Author: Dmitry Vyukov <dvy...@google.com>
Date: 2021-10-13 (Wed, 13 Oct 2021)

Changed paths:
M executor/common_linux.h
M pkg/csource/generated.go

Log Message:
-----------
executor: don't mount some cgroup controllers during setup

Leave some controllers unbound so that the fuzzer can mount them during fuzzing.
This is suboptimal because all controllers are global (so different test
processes will collide, state accumulate, etc), but this still should give
at least some new coverage.


Compare: https://github.com/google/syzkaller/compare/118b58e6523c...9d56e7ddd67e
Reply all
Reply to author
Forward
0 new messages