diff options
author | Richard Palethorpe <rpalethorpe@suse.com> | 2023-10-25 11:26:08 +0100 |
---|---|---|
committer | Edward Liaw <edliaw@google.com> | 2024-01-18 18:21:16 +0000 |
commit | 454a4dbd424c2574345475e6f01ac4308a91b28b (patch) | |
tree | 8d955e58d7d3ded3a97df6cbf67ceaa5cb2ed61f /testcases/kernel/containers/Makefile.inc | |
parent | 50b54a5467e0159d6b0adf12a130d09499cac7fb (diff) | |
download | ltp-454a4dbd424c2574345475e6f01ac4308a91b28b.tar.gz |
cgroup: Handle trailing new line in cgroup.controllers
The last item in cgroup.controllers (misc or rdma in my case)
contained a new line character which caused the controller search to
fail.
This commit avoids including the newline character inside the name
comparison.
The search failure caused the "cgroup_regression_test.sh" test to fail
with a confusing error when it tries to mount a V1 subsys thus
removing the V2 and causing the available set of V2s to change between
scans.
According to the V2 docs subsys names can only include lowercase
characters and '_'. So we strictly look for those characters.
The newline (and delimiting space) is just what the kernel currently
prints. IDK if it is specified anywhere, but if it changes then the
error should be obvious.
Fixes: 310da3784 ("Add new CGroups APIs")
Signed-off-by: Richard Palethorpe <rpalethorpe@suse.com>
Reviewed-by: Petr Vorel <pvorel@suse.cz>
Reviewed-by: Marius Kittler <mkittler@suse.de>
(cherry picked from commit 5292c46e5032011337d73119f27d365f29140004)
Test: N/A
Signed-off-by: Edward Liaw <edliaw@google.com>
(cherry picked from https://android-review.googlesource.com/q/commit:6130f433e5225c4a534a7515571d5ea741f94c0e)
Bug: 305297408
Change-Id: Iad436c8bc7a854553da962f90d8623359a7654fb
Merged-In: Iad436c8bc7a854553da962f90d8623359a7654fb
Diffstat (limited to 'testcases/kernel/containers/Makefile.inc')
0 files changed, 0 insertions, 0 deletions