* [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests
@ 2022-07-21 20:52 Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH v3 01/18] API/cgroup: Modify tst_cg_print_config for parsing and consumption Luke Nowakowski-Krijger
` (19 more replies)
0 siblings, 20 replies; 33+ messages in thread
From: Luke Nowakowski-Krijger @ 2022-07-21 20:52 UTC (permalink / raw)
To: ltp, rpalethorpe, liwang, pvorel, chrubis
This patchset aims to expand the cgroup_lib shell library to simplify
and centralize the whole mounting and cleanup process that can get
rather confusing and redundant when writing cgroup controller tests from
a shell environment. This is done by having the shell library make calls
to the C cgroup API from a binary utility.
In this patch set there are a few tests that have been extensively
rewritten to work with the new test API and to use the new functionality
from the cgroup lib. Because the test Cgroup lib handles mounting for v1
and v2 controllers, some tests were modified to also work under cgroup
v2. Some tests that were written for v1 controller also effictively test
v2 controllers, while others were written to test v2 controllers in the
spirit of the test or skipped outright.
Luke Nowakowski-Krijger (18):
API/cgroup: Modify tst_cg_print_config for parsing and consumption
API/cgroup: Add option for specific pid to tst_cg_opts
API/cgroup: Add cgroup_find_root helper function
API/cgroup: Add CTRL_NAME_MAX define
tst_test_macros: Add TST_TO_STR and TST_STR macro
API/cgroup: Implement tst_cg_load_config
API/cgroup: Add more controllers to tst_cgroup
API/cgroup: refuse to mount blkio when io controller is mounted
testcases/lib: Implement tst_cgctl binary
controllers: Expand cgroup_lib shell library
controllers: Update cgroup_fj_* to use newer cgroup lib and test lib
controllers: Update memcg_control_test to newer test lib and cgroup
lib
controllers: Update memcg/regression/* to new test and cgroup lib
controllers: Update memcg_stress_test to use newer cgroup lib
controllers: update memcg/functional to use newer cgroup lib
controllers: Update pids.sh to use newer cgroup lib
controllers: update cpuset_regression_test.sh to use newer cgroup lib
controllers: update cgroup_regression_test to use newer cgroup lib
include/tst_cgroup.h | 17 +-
include/tst_test_macros.h | 3 +
lib/tst_cgroup.c | 216 +++++++++++++++++-
.../cgroup/cgroup_regression_test.sh | 31 +--
.../controllers/cgroup_fj/cgroup_fj_common.sh | 115 +++-------
.../cgroup_fj/cgroup_fj_function.sh | 172 ++++++++------
.../controllers/cgroup_fj/cgroup_fj_proc.c | 24 +-
.../controllers/cgroup_fj/cgroup_fj_stress.sh | 171 +++++++-------
testcases/kernel/controllers/cgroup_lib.sh | 133 +++++++++--
.../cpuset/cpuset_regression_test.sh | 26 +--
.../controllers/memcg/control/mem_process.c | 28 +--
.../memcg/control/memcg_control_test.sh | 149 +++---------
.../memcg/functional/memcg_force_empty.sh | 2 +-
.../controllers/memcg/functional/memcg_lib.sh | 54 +++--
.../memcg/regression/memcg_regression_test.sh | 203 ++++++++--------
.../memcg/regression/memcg_test_1.c | 40 ++--
.../memcg/regression/memcg_test_2.c | 24 +-
.../memcg/regression/memcg_test_3.c | 38 ++-
.../memcg/regression/memcg_test_4.c | 24 +-
.../memcg/regression/memcg_test_4.sh | 50 ++--
.../memcg/stress/memcg_stress_test.sh | 32 +--
testcases/kernel/controllers/pids/pids.sh | 67 +-----
testcases/lib/Makefile | 2 +-
testcases/lib/tst_cgctl.c | 87 +++++++
24 files changed, 909 insertions(+), 799 deletions(-)
create mode 100644 testcases/lib/tst_cgctl.c
--
2.34.1
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 33+ messages in thread
* [LTP] [PATCH v3 01/18] API/cgroup: Modify tst_cg_print_config for parsing and consumption
2022-07-21 20:52 [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests Luke Nowakowski-Krijger
@ 2022-07-21 20:52 ` Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH 02/18] API/cgroup: Add option for specific pid to tst_cg_opts Luke Nowakowski-Krijger
` (18 subsequent siblings)
19 siblings, 0 replies; 33+ messages in thread
From: Luke Nowakowski-Krijger @ 2022-07-21 20:52 UTC (permalink / raw)
To: ltp, rpalethorpe, liwang, pvorel, chrubis
Modify tst_cg_print_config to be easily parsed by sscanf so that it can
later be consumed by tst_cg_load_config.
Prints out root info of each controller found as well as the minimal
associated state needed to cleanup mounts and dirs made my tst_cg_require.
Signed-off-by: Luke Nowakowski-Krijger <luke.nowakowskikrijger@canonical.com>
---
v2: Remove "mounted_drain_dir" as mounting ltp dir and drain dir happen
at the same time
v3: Print config to be scanf friendly to be much easier to consume.
Rename variables to reflect the variables used in the library for
simplicity.
include/tst_cgroup.h | 6 +++++-
lib/tst_cgroup.c | 25 ++++++++++++++++++-------
2 files changed, 23 insertions(+), 8 deletions(-)
diff --git a/include/tst_cgroup.h b/include/tst_cgroup.h
index c4bc7ae1c..7ebc2a624 100644
--- a/include/tst_cgroup.h
+++ b/include/tst_cgroup.h
@@ -110,7 +110,11 @@ extern const struct tst_cg_group *const tst_cg_drain;
* controllers. Called automatically by tst_cg_require.
*/
void tst_cg_scan(void);
-/* Print the config detected by tst_cg_scan */
+/* Print the config detected by tst_cg_scan and print the internal
+ * state associated with each controller. Output can be passed to
+ * tst_cg_load_config to configure the internal state to that of the
+ * config between program invocations.
+ */
void tst_cg_print_config(void);
/* Ensure the specified controller is available in the test's default
diff --git a/lib/tst_cgroup.c b/lib/tst_cgroup.c
index 081dfa9c6..c56a7c6b9 100644
--- a/lib/tst_cgroup.c
+++ b/lib/tst_cgroup.c
@@ -317,24 +317,35 @@ opendir:
O_PATH | O_DIRECTORY);
}
+#define CONFIG_HEADER "ctrl_name ver we_require_it mnt_path we_mounted_it ltp_dir.we_created_it test_dir.dir_name"
+#define CONFIG_FORMAT "%s\t%d\t%d\t%s\t%d\t%d\t%s"
+/* Prints out the state associated with each controller to be consumed by
+ * tst_cg_load_config.
+ *
+ * The config keeps track of the minimal state needed for tst_cg_cleanup
+ * to cleanup mounts and directories made by tst_cg_require.
+ */
void tst_cg_print_config(void)
{
- struct cgroup_root *root;
const struct cgroup_ctrl *ctrl;
- tst_res(TINFO, "Detected Controllers:");
+ printf("%s\n", CONFIG_HEADER);
for_each_ctrl(ctrl) {
- root = ctrl->ctrl_root;
+ struct cgroup_root *root = ctrl->ctrl_root;
if (!root)
continue;
- tst_res(TINFO, "\t%.10s %s @ %s:%s",
+ printf(CONFIG_FORMAT,
ctrl->ctrl_name,
- root->no_cpuset_prefix ? "[noprefix]" : "",
- root->ver == TST_CG_V1 ? "V1" : "V2",
- root->mnt_path);
+ root->ver,
+ ctrl->we_require_it,
+ root->mnt_path,
+ root->we_mounted_it,
+ root->ltp_dir.we_created_it,
+ root->test_dir.dir_name ? root->test_dir.dir_name : "NULL");
+ printf("\n");
}
}
--
2.34.1
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [LTP] [PATCH 02/18] API/cgroup: Add option for specific pid to tst_cg_opts
2022-07-21 20:52 [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH v3 01/18] API/cgroup: Modify tst_cg_print_config for parsing and consumption Luke Nowakowski-Krijger
@ 2022-07-21 20:52 ` Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH v2 03/18] API/cgroup: Add cgroup_find_root helper function Luke Nowakowski-Krijger
` (17 subsequent siblings)
19 siblings, 0 replies; 33+ messages in thread
From: Luke Nowakowski-Krijger @ 2022-07-21 20:52 UTC (permalink / raw)
To: ltp, rpalethorpe, liwang, pvorel, chrubis
Add an option that would allow to create a test directory with a
specified pid, as opposed to the calling processes pid.
Signed-off-by: Luke Nowakowski-Krijger <luke.nowakowskikrijger@canonical.com>
---
include/tst_cgroup.h | 4 ++++
lib/tst_cgroup.c | 6 +++++-
2 files changed, 9 insertions(+), 1 deletion(-)
diff --git a/include/tst_cgroup.h b/include/tst_cgroup.h
index 7ebc2a624..aa70dc27a 100644
--- a/include/tst_cgroup.h
+++ b/include/tst_cgroup.h
@@ -97,6 +97,10 @@ struct tst_cg_opts {
* version. Defautls to zero to accept any version.
*/
enum tst_cg_ver needs_ver;
+ /* Pass in a specific pid to create and identify the test
+ * directory as opposed to the default pid of the calling process.
+ */
+ int test_pid;
};
/* A Control Group in LTP's aggregated hierarchy */
diff --git a/lib/tst_cgroup.c b/lib/tst_cgroup.c
index c56a7c6b9..d757c832f 100644
--- a/lib/tst_cgroup.c
+++ b/lib/tst_cgroup.c
@@ -713,7 +713,11 @@ mkdirs:
cgroup_dir_mk(&root->ltp_dir, cgroup_ltp_drain_dir, &root->drain_dir);
- sprintf(cgroup_test_dir, "test-%d", getpid());
+ if (options->test_pid)
+ sprintf(cgroup_test_dir, "test-%d", options->test_pid);
+ else
+ sprintf(cgroup_test_dir, "test-%d", getpid());
+
cgroup_dir_mk(&root->ltp_dir, cgroup_test_dir, &root->test_dir);
}
--
2.34.1
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [LTP] [PATCH v2 03/18] API/cgroup: Add cgroup_find_root helper function
2022-07-21 20:52 [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH v3 01/18] API/cgroup: Modify tst_cg_print_config for parsing and consumption Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH 02/18] API/cgroup: Add option for specific pid to tst_cg_opts Luke Nowakowski-Krijger
@ 2022-07-21 20:52 ` Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH 04/18] API/cgroup: Add CTRL_NAME_MAX define Luke Nowakowski-Krijger
` (16 subsequent siblings)
19 siblings, 0 replies; 33+ messages in thread
From: Luke Nowakowski-Krijger @ 2022-07-21 20:52 UTC (permalink / raw)
To: ltp, rpalethorpe, liwang, pvorel, chrubis
Add a helper function similar to cgroup_find_ctrl to make matching paths
to roots more convenient.
Signed-off-by: Luke Nowakowski-Krijger <luke.nowakowskikrijger@canonical.com>
---
v2: Use the for_each_root() macro to properly loop over v1 roots
lib/tst_cgroup.c | 13 +++++++++++++
1 file changed, 13 insertions(+)
diff --git a/lib/tst_cgroup.c b/lib/tst_cgroup.c
index d757c832f..677a6f318 100644
--- a/lib/tst_cgroup.c
+++ b/lib/tst_cgroup.c
@@ -362,6 +362,19 @@ static struct cgroup_ctrl *cgroup_find_ctrl(const char *const ctrl_name)
return NULL;
}
+static struct cgroup_root *cgroup_find_root(const char *const mnt_path)
+{
+ struct cgroup_root *root;
+
+ for_each_root(root) {
+ if (!strcmp(root->mnt_path, mnt_path))
+ return root;
+ }
+
+ return NULL;
+}
+
+
/* Determine if a mounted cgroup hierarchy is unique and record it if so.
*
* For CGroups V2 this is very simple as there is only one
--
2.34.1
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [LTP] [PATCH 04/18] API/cgroup: Add CTRL_NAME_MAX define
2022-07-21 20:52 [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests Luke Nowakowski-Krijger
` (2 preceding siblings ...)
2022-07-21 20:52 ` [LTP] [PATCH v2 03/18] API/cgroup: Add cgroup_find_root helper function Luke Nowakowski-Krijger
@ 2022-07-21 20:52 ` Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH v2 05/18] tst_test_macros: Add TST_TO_STR and TST_STR macro Luke Nowakowski-Krijger
` (15 subsequent siblings)
19 siblings, 0 replies; 33+ messages in thread
From: Luke Nowakowski-Krijger @ 2022-07-21 20:52 UTC (permalink / raw)
To: ltp, rpalethorpe, liwang, pvorel, chrubis
Add a define for max controller name that will be useful for parsing
controller names from strings and helps avoid having unexplained numbers
in the codebase.
Signed-off-by: Luke Nowakowski-Krijger <luke.nowakowskikrijger@canonical.com>
---
lib/tst_cgroup.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/lib/tst_cgroup.c b/lib/tst_cgroup.c
index 677a6f318..e9652703f 100644
--- a/lib/tst_cgroup.c
+++ b/lib/tst_cgroup.c
@@ -196,6 +196,7 @@ static const struct cgroup_file io_ctrl_files[] = {
{ }
};
+#define CTRL_NAME_MAX 31
/* Lookup tree for item names. */
static struct cgroup_ctrl controllers[] = {
[0] = { "cgroup", cgroup_ctrl_files, 0, NULL, 0 },
@@ -966,7 +967,7 @@ static const struct cgroup_file *cgroup_file_find(const char *const file,
{
const struct cgroup_file *cfile;
const struct cgroup_ctrl *ctrl;
- char ctrl_name[32];
+ char ctrl_name[CTRL_NAME_MAX + 1];
const char *const sep = strchr(file_name, '.');
size_t len;
--
2.34.1
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [LTP] [PATCH v2 05/18] tst_test_macros: Add TST_TO_STR and TST_STR macro
2022-07-21 20:52 [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests Luke Nowakowski-Krijger
` (3 preceding siblings ...)
2022-07-21 20:52 ` [LTP] [PATCH 04/18] API/cgroup: Add CTRL_NAME_MAX define Luke Nowakowski-Krijger
@ 2022-07-21 20:52 ` Luke Nowakowski-Krijger
2022-07-26 14:44 ` Petr Vorel
2022-07-21 20:52 ` [LTP] [PATCH v5 06/18] API/cgroup: Implement tst_cg_load_config Luke Nowakowski-Krijger
` (14 subsequent siblings)
19 siblings, 1 reply; 33+ messages in thread
From: Luke Nowakowski-Krijger @ 2022-07-21 20:52 UTC (permalink / raw)
To: ltp, rpalethorpe, liwang, pvorel, chrubis
Add the TST_TO_STR macro which uses a preprocessor trick to concetenate
variables into strings.
Useful when needing to create strings from other #define variables.
Also define TST_STR as an intermediate macro that could be used when
defining string literals.
Signed-off-by: Luke Nowakowski-Krijger <luke.nowakowskikrijger@canonical.com>
---
v2: Change names to TST_STR and TST_TO_STR
Treat TST_STR as its own macro because it could be useful.
include/tst_test_macros.h | 3 +++
1 file changed, 3 insertions(+)
diff --git a/include/tst_test_macros.h b/include/tst_test_macros.h
index 2e7b7871c..58b965f78 100644
--- a/include/tst_test_macros.h
+++ b/include/tst_test_macros.h
@@ -36,6 +36,9 @@ extern void *TST_RET_PTR;
#define TST_2_(_1, _2, ...) _2
+#define TST_STR(s) #s
+#define TST_TO_STR(s) TST_STR(s)
+
#define TST_FMT_(FMT, _1, ...) FMT, ##__VA_ARGS__
#define TST_MSG_(RES, FMT, SCALL, ...) \
--
2.34.1
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [LTP] [PATCH v5 06/18] API/cgroup: Implement tst_cg_load_config
2022-07-21 20:52 [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests Luke Nowakowski-Krijger
` (4 preceding siblings ...)
2022-07-21 20:52 ` [LTP] [PATCH v2 05/18] tst_test_macros: Add TST_TO_STR and TST_STR macro Luke Nowakowski-Krijger
@ 2022-07-21 20:52 ` Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH v3 07/18] API/cgroup: Add more controllers to tst_cgroup Luke Nowakowski-Krijger
` (13 subsequent siblings)
19 siblings, 0 replies; 33+ messages in thread
From: Luke Nowakowski-Krijger @ 2022-07-21 20:52 UTC (permalink / raw)
To: ltp, rpalethorpe, liwang, pvorel, chrubis
Implement tst_cg_load_config which consumes the state given by
tst_cg_print_config to update the internal test state to reflect
the given config.
This allows for programs using the cgroup C API to load and reload
state, allowing functionality such as calling tst_cg_require and
tst_cg_cleanup to function properly between programs or between
invocations of a binary using the C API.
Signed-off-by: Luke Nowakowski-Krijger <luke.nowakowskikrijger@canonical.com>
---
v2: Add root null check in parse_root_config.
Remove checking for ltp_drain_dir key from config as it was
redundant.
Remove unsued variable in parse_ctrl_config.
Cleanup some compiler warnings.
v3: Rewrite to consume each line of the config with a scanf to make
the parsing much simpler while using new config variables.
v4: Modify CONFIG_FORMAT to have variable delimits for sscanf to make it
more safe. Add extra checks for sscanf reading.
v5: Change to use TST_TO_STR macro
include/tst_cgroup.h | 7 ++++
lib/tst_cgroup.c | 77 ++++++++++++++++++++++++++++++++++++++++++--
2 files changed, 81 insertions(+), 3 deletions(-)
diff --git a/include/tst_cgroup.h b/include/tst_cgroup.h
index aa70dc27a..d06847cc6 100644
--- a/include/tst_cgroup.h
+++ b/include/tst_cgroup.h
@@ -121,6 +121,13 @@ void tst_cg_scan(void);
*/
void tst_cg_print_config(void);
+/* Load the config printed out by tst_cg_print_config and configure the internal
+ * libary state to match the config. Used to allow tst_cg_cleanup to properly
+ * cleanup mounts and directories created by tst_cg_require between program
+ * invocations.
+ */
+void tst_cg_load_config(const char *const config);
+
/* Ensure the specified controller is available in the test's default
* CGroup, mounting/enabling it if necessary. Usually this is not
* necessary use tst_test.needs_cgroup_ctrls instead.
diff --git a/lib/tst_cgroup.c b/lib/tst_cgroup.c
index e9652703f..27e3f5f0d 100644
--- a/lib/tst_cgroup.c
+++ b/lib/tst_cgroup.c
@@ -318,8 +318,9 @@ opendir:
O_PATH | O_DIRECTORY);
}
+#define PATH_MAX_STRLEN 4095
#define CONFIG_HEADER "ctrl_name ver we_require_it mnt_path we_mounted_it ltp_dir.we_created_it test_dir.dir_name"
-#define CONFIG_FORMAT "%s\t%d\t%d\t%s\t%d\t%d\t%s"
+#define CONFIG_FORMAT "%" TST_TO_STR(CTRL_NAME_MAX) "s\t%d\t%d\t%" TST_TO_STR(PATH_MAX_STRLEN) "s\t%d\t%d\t%" TST_TO_STR(NAME_MAX) "s"
/* Prints out the state associated with each controller to be consumed by
* tst_cg_load_config.
*
@@ -338,7 +339,7 @@ void tst_cg_print_config(void)
if (!root)
continue;
- printf(CONFIG_FORMAT,
+ printf("%s\t%d\t%d\t%s\t%d\t%d\t%s\n",
ctrl->ctrl_name,
root->ver,
ctrl->we_require_it,
@@ -346,7 +347,6 @@ void tst_cg_print_config(void)
root->we_mounted_it,
root->ltp_dir.we_created_it,
root->test_dir.dir_name ? root->test_dir.dir_name : "NULL");
- printf("\n");
}
}
@@ -375,6 +375,77 @@ static struct cgroup_root *cgroup_find_root(const char *const mnt_path)
return NULL;
}
+static void cgroup_parse_config_line(const char *const config_entry)
+{
+ char ctrl_name[CTRL_NAME_MAX + 1], mnt_path[PATH_MAX_STRLEN + 1], test_dir_name[NAME_MAX + 1];
+ int ver, we_require_it, we_mounted_it, ltp_dir_we_created_it, vars_read;
+ size_t len;
+ struct cgroup_root *root;
+ struct cgroup_ctrl *ctrl;
+
+ vars_read = sscanf(config_entry, CONFIG_FORMAT,
+ ctrl_name, &ver, &we_require_it, mnt_path, &we_mounted_it,
+ <p_dir_we_created_it, test_dir_name);
+
+ if (vars_read != 7)
+ tst_brk(TBROK, "Incorrect number of vars read from config. Config possibly malformed?");
+
+ ctrl = cgroup_find_ctrl(ctrl_name);
+ if (!ctrl)
+ tst_brk(TBROK, "Could not find ctrl from config. Ctrls changing between calls?");
+
+ ctrl->we_require_it = we_require_it;
+
+ root = cgroup_find_root(mnt_path);
+ if (!root)
+ tst_brk(TBROK, "Could not find root from config. Config possibly malformed?");
+
+ if (we_mounted_it)
+ root->we_mounted_it = 1;
+
+ if (!root->ltp_dir.dir_name) {
+ cgroup_dir_mk(&root->mnt_dir, cgroup_ltp_dir, &root->ltp_dir);
+ cgroup_dir_mk(&root->ltp_dir, cgroup_ltp_drain_dir, &root->drain_dir);
+ if (ltp_dir_we_created_it) {
+ root->ltp_dir.we_created_it = 1;
+ root->drain_dir.we_created_it = 1;
+ }
+ }
+
+ if (!root->test_dir.dir_name && strcmp(test_dir_name, "NULL")) {
+ strncpy(cgroup_test_dir, test_dir_name, NAME_MAX);
+ cgroup_dir_mk(&root->ltp_dir, cgroup_test_dir, &root->test_dir);
+ root->test_dir.we_created_it = 1;
+ }
+}
+
+/* Load the test state config provided by tst_cg_print_config
+ *
+ * This will reload some internal tst_cgroup state given by the config
+ * that might otherwise have been lost between calls or between different
+ * processes. In particular this is used by testcases/lib/tst_cgctl to
+ * provide access to this C api to shell scripts.
+ *
+ * The config keeps track of the minimal state needed for tst_cg_cleanup
+ * to cleanup mounts and directories created by tst_cg_require.
+ */
+void tst_cg_load_config(const char *const config)
+{
+ char temp_config[BUFSIZ];
+ char *line;
+ const size_t config_len = strlen(config) + 1;
+
+ if (config_len >= BUFSIZ)
+ tst_brk(TBROK, "Config has exceeded buffer size?");
+
+ memcpy(temp_config, config, config_len);
+ temp_config[config_len] = '\0';
+
+ line = strtok(temp_config, "\n");
+ /* Make sure to consume the header. */
+ for (line = strtok(NULL, "\n"); line; line = strtok(NULL, "\n"))
+ cgroup_parse_config_line(line);
+}
/* Determine if a mounted cgroup hierarchy is unique and record it if so.
*
--
2.34.1
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [LTP] [PATCH v3 07/18] API/cgroup: Add more controllers to tst_cgroup
2022-07-21 20:52 [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests Luke Nowakowski-Krijger
` (5 preceding siblings ...)
2022-07-21 20:52 ` [LTP] [PATCH v5 06/18] API/cgroup: Implement tst_cg_load_config Luke Nowakowski-Krijger
@ 2022-07-21 20:52 ` Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH v2 08/18] API/cgroup: refuse to mount blkio when io controller is mounted Luke Nowakowski-Krijger
` (12 subsequent siblings)
19 siblings, 0 replies; 33+ messages in thread
From: Luke Nowakowski-Krijger @ 2022-07-21 20:52 UTC (permalink / raw)
To: ltp, rpalethorpe, liwang, pvorel, chrubis
Add the rest of the controllers that are tested in LTP controller tests
so that they can be mounted using the cgroup api.
Control files should be added on an as needed basis.
Signed-off-by: Luke Nowakowski-Krijger <luke.nowakowskikrijger@canonical.com>
---
v2: remove rdma controller as that is not used in any LTP tests.
Remove control files that should be added on an as needed basis.
v3: Add debug controllers
lib/tst_cgroup.c | 92 +++++++++++++++++++++++++++++++++++++++++++++++-
1 file changed, 91 insertions(+), 1 deletion(-)
diff --git a/lib/tst_cgroup.c b/lib/tst_cgroup.c
index 27e3f5f0d..16cb233a8 100644
--- a/lib/tst_cgroup.c
+++ b/lib/tst_cgroup.c
@@ -83,8 +83,19 @@ enum cgroup_ctrl_indx {
CTRL_CPU,
CTRL_CPUSET,
CTRL_IO,
+ CTRL_PIDS,
+ CTRL_HUGETLB,
+ CTRL_CPUACCT,
+ CTRL_DEVICES,
+ CTRL_FREEZER,
+ CTRL_NETCLS,
+ CTRL_NETPRIO,
+ CTRL_BLKIO,
+ CTRL_MISC,
+ CTRL_PERFEVENT,
+ CTRL_DEBUG
};
-#define CTRLS_MAX CTRL_CPUSET
+#define CTRLS_MAX CTRL_DEBUG
/* At most we can have one cgroup V1 tree for each controller and one
* (empty) v2 tree.
@@ -196,6 +207,52 @@ static const struct cgroup_file io_ctrl_files[] = {
{ }
};
+static const struct cgroup_file pids_ctrl_files[] = {
+ { "pids.max", "pids.max", CTRL_PIDS },
+ { "pids.current", "pids.current", CTRL_PIDS },
+ { }
+};
+
+static const struct cgroup_file hugetlb_ctrl_files[] = {
+ { }
+};
+
+static const struct cgroup_file cpuacct_ctrl_files[] = {
+ { }
+};
+
+static const struct cgroup_file devices_ctrl_files[] = {
+ { }
+};
+
+static const struct cgroup_file freezer_ctrl_files[] = {
+ { }
+};
+
+static const struct cgroup_file netcls_ctrl_files[] = {
+ { }
+};
+
+static const struct cgroup_file netprio_ctrl_files[] = {
+ { }
+};
+
+static const struct cgroup_file blkio_ctrl_files[] = {
+ { }
+};
+
+static const struct cgroup_file misc_ctrl_files[] = {
+ { }
+};
+
+static const struct cgroup_file perf_event_ctrl_files[] = {
+ { }
+};
+
+static const struct cgroup_file debug_ctrl_files[] = {
+ { }
+};
+
#define CTRL_NAME_MAX 31
/* Lookup tree for item names. */
static struct cgroup_ctrl controllers[] = {
@@ -212,6 +269,39 @@ static struct cgroup_ctrl controllers[] = {
[CTRL_IO] = {
"io", io_ctrl_files, CTRL_IO, NULL, 0
},
+ [CTRL_PIDS] = {
+ "pids", pids_ctrl_files, CTRL_PIDS, NULL, 0
+ },
+ [CTRL_HUGETLB] = {
+ "hugetlb", hugetlb_ctrl_files, CTRL_HUGETLB, NULL, 0
+ },
+ [CTRL_CPUACCT] = {
+ "cpuacct", cpuacct_ctrl_files, CTRL_CPUACCT, NULL, 0
+ },
+ [CTRL_DEVICES] = {
+ "devices", devices_ctrl_files, CTRL_DEVICES, NULL, 0
+ },
+ [CTRL_FREEZER] = {
+ "freezer", freezer_ctrl_files, CTRL_FREEZER, NULL, 0
+ },
+ [CTRL_NETCLS] = {
+ "net_cls", netcls_ctrl_files, CTRL_NETCLS, NULL, 0
+ },
+ [CTRL_NETPRIO] = {
+ "net_prio", netprio_ctrl_files, CTRL_NETPRIO, NULL, 0
+ },
+ [CTRL_BLKIO] = {
+ "blkio", blkio_ctrl_files, CTRL_BLKIO, NULL, 0
+ },
+ [CTRL_MISC] = {
+ "misc", misc_ctrl_files, CTRL_MISC, NULL, 0
+ },
+ [CTRL_PERFEVENT] = {
+ "perf_event", perf_event_ctrl_files, CTRL_PERFEVENT, NULL, 0
+ },
+ [CTRL_DEBUG] = {
+ "debug", debug_ctrl_files, CTRL_DEBUG, NULL, 0
+ },
{ }
};
--
2.34.1
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [LTP] [PATCH v2 08/18] API/cgroup: refuse to mount blkio when io controller is mounted
2022-07-21 20:52 [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests Luke Nowakowski-Krijger
` (6 preceding siblings ...)
2022-07-21 20:52 ` [LTP] [PATCH v3 07/18] API/cgroup: Add more controllers to tst_cgroup Luke Nowakowski-Krijger
@ 2022-07-21 20:52 ` Luke Nowakowski-Krijger
2022-07-26 14:40 ` Petr Vorel
2022-07-21 20:52 ` [LTP] [PATCH v2 09/18] testcases/lib: Implement tst_cgctl binary Luke Nowakowski-Krijger
` (11 subsequent siblings)
19 siblings, 1 reply; 33+ messages in thread
From: Luke Nowakowski-Krijger @ 2022-07-21 20:52 UTC (permalink / raw)
To: ltp, rpalethorpe, liwang, pvorel, chrubis
Mounting the v1 blkio controller while v2 io controller is mounted
unmounts the io controller, triggering a tst_brk that the number of
controller has gone down.
Because these controllers don't seem to be compatible, tst_res with
TCONF and report that we refused to mount the blkio controller while the
io controller is mounted.
Signed-off-by: Luke Nowakowski-Krijger <luke.nowakowskikrijger@canonical.com>
---
v2: Change from tst_brk to tst_res TCONF and return
lib/tst_cgroup.c | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/lib/tst_cgroup.c b/lib/tst_cgroup.c
index 16cb233a8..d29279838 100644
--- a/lib/tst_cgroup.c
+++ b/lib/tst_cgroup.c
@@ -716,6 +716,12 @@ static void cgroup_mount_v1(struct cgroup_ctrl *const ctrl)
char mnt_path[PATH_MAX];
int made_dir = 0;
+ if (ctrl->ctrl_indx == CTRL_BLKIO && controllers[CTRL_IO].ctrl_root) {
+ tst_res(TCONF,
+ "IO controller found on V2 root, skipping blkio mount that would unmount IO controller");
+ return;
+ }
+
sprintf(mnt_path, "%s%s", cgroup_mount_ltp_prefix, ctrl->ctrl_name);
if (!mkdir(mnt_path, 0777)) {
--
2.34.1
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [LTP] [PATCH v2 09/18] testcases/lib: Implement tst_cgctl binary
2022-07-21 20:52 [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests Luke Nowakowski-Krijger
` (7 preceding siblings ...)
2022-07-21 20:52 ` [LTP] [PATCH v2 08/18] API/cgroup: refuse to mount blkio when io controller is mounted Luke Nowakowski-Krijger
@ 2022-07-21 20:52 ` Luke Nowakowski-Krijger
2022-07-26 12:48 ` Petr Vorel
2022-07-21 20:52 ` [LTP] [PATCH v5 10/18] controllers: Expand cgroup_lib shell library Luke Nowakowski-Krijger
` (10 subsequent siblings)
19 siblings, 1 reply; 33+ messages in thread
From: Luke Nowakowski-Krijger @ 2022-07-21 20:52 UTC (permalink / raw)
To: ltp, rpalethorpe, liwang, pvorel, chrubis
Implement a binary utility that creates an interface to make calls to
the cgroup C API.
This will effectively allow shell scripts to make calls to the cgroup C
api.
Signed-off-by: Luke Nowakowski-Krijger <luke.nowakowskikrijger@canonical.com>
---
v2: Add license identifier and copyright.
Reformat with tabs instead of spaces.
Add help format message and help function.
Add error gotos to streamline error messaging.
testcases/lib/Makefile | 2 +-
testcases/lib/tst_cgctl.c | 87 +++++++++++++++++++++++++++++++++++++++
2 files changed, 88 insertions(+), 1 deletion(-)
create mode 100644 testcases/lib/tst_cgctl.c
diff --git a/testcases/lib/Makefile b/testcases/lib/Makefile
index f2de0c832..f4f8c8524 100644
--- a/testcases/lib/Makefile
+++ b/testcases/lib/Makefile
@@ -12,6 +12,6 @@ MAKE_TARGETS := tst_sleep tst_random tst_checkpoint tst_rod tst_kvcmp\
tst_device tst_net_iface_prefix tst_net_ip_prefix tst_net_vars\
tst_getconf tst_supported_fs tst_check_drivers tst_get_unused_port\
tst_get_median tst_hexdump tst_get_free_pids tst_timeout_kill\
- tst_check_kconfigs
+ tst_check_kconfigs tst_cgctl
include $(top_srcdir)/include/mk/generic_leaf_target.mk
diff --git a/testcases/lib/tst_cgctl.c b/testcases/lib/tst_cgctl.c
new file mode 100644
index 000000000..4f4fe8542
--- /dev/null
+++ b/testcases/lib/tst_cgctl.c
@@ -0,0 +1,87 @@
+// SPDX-License-Identifier: GPL-2.0-or-later
+/*
+ * Copyright (c) 2022 Canonical Ltd.
+ */
+
+#include <stdio.h>
+#include <string.h>
+#include <stdlib.h>
+#include <stdint.h>
+#include <unistd.h>
+#include "tst_cgroup.h"
+
+#define USAGE "Usage: tst_cgctl require [controller] [test_pid]\n\
+ \t cleanup [config (output of tst_cg_print_config)]\n\
+ \t print\n\
+ \t help\n"
+
+static int cgctl_require(const char *ctrl, int test_pid)
+{
+ struct tst_cg_opts opts;
+
+ memset(&opts, 0, sizeof(opts));
+ opts.test_pid = test_pid;
+
+ tst_cg_require(ctrl, &opts);
+ tst_cg_print_config();
+
+ return 0;
+}
+
+static int cgctl_cleanup(const char *const config)
+{
+ tst_cg_scan();
+ tst_cg_load_config(config);
+ tst_cg_cleanup();
+
+ return 0;
+}
+
+static int cgctl_print(void)
+{
+ tst_cg_scan();
+ tst_cg_print_config();
+
+ return 0;
+}
+
+int main(int argc, char *argv[])
+{
+ int test_pid;
+ const char *cmd_name = argv[1];
+
+ if (argc < 2)
+ goto error;
+
+ if (!strcmp(cmd_name, "require")) {
+ if (argc != 4)
+ goto arg_num_error;
+ test_pid = atoi(argv[3]);
+ if (!test_pid) {
+ fprintf(stderr, "tst_cgctl: Invalid test_pid '%s' given\n",
+ argv[3]);
+ goto error;
+ }
+ return cgctl_require(argv[2], test_pid);
+ } else if (!strcmp(cmd_name, "cleanup")) {
+ if (argc != 3)
+ goto arg_num_error;
+ return cgctl_cleanup(argv[2]);
+ } else if (!strcmp(cmd_name, "print")) {
+ return cgctl_print();
+ } else if (!strcmp(cmd_name, "help")) {
+ printf(USAGE);
+ return 0;
+ }
+
+ fprintf(stderr, "tst_cgctl: Unknown command '%s' given\n", cmd_name);
+ goto error;
+
+arg_num_error:
+ fprintf(stderr,
+ "tst_cgctl: Invalid number of arguments given for command '%s'\n",
+ cmd_name);
+error:
+ fprintf(stderr, USAGE);
+ return 1;
+}
--
2.34.1
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [LTP] [PATCH v5 10/18] controllers: Expand cgroup_lib shell library
2022-07-21 20:52 [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests Luke Nowakowski-Krijger
` (8 preceding siblings ...)
2022-07-21 20:52 ` [LTP] [PATCH v2 09/18] testcases/lib: Implement tst_cgctl binary Luke Nowakowski-Krijger
@ 2022-07-21 20:52 ` Luke Nowakowski-Krijger
2022-07-26 13:12 ` Petr Vorel
2022-07-21 20:52 ` [LTP] [PATCH v3 11/18] controllers: Update cgroup_fj_* to use newer cgroup lib and test lib Luke Nowakowski-Krijger
` (9 subsequent siblings)
19 siblings, 1 reply; 33+ messages in thread
From: Luke Nowakowski-Krijger @ 2022-07-21 20:52 UTC (permalink / raw)
To: ltp, rpalethorpe, liwang, pvorel, chrubis
Expand the cgroup_lib library by using the tst_cgctl binary
utility to make calls to the Cgroup C API to simplify and centralize the
mounting and cleanup process of Cgroups
Signed-off-by: Luke Nowakowski-Krijger <luke.nowakowskikrijger@canonical.com>
---
v2: Add "^" to propery grep the correct mountpoint.
Removed is_cgroup_enabled_and_available function and put the check
in cgroup_require().
Check if /proc/cgroups exists in cgroup_require().
Change to TCONF if controllers not available.
v3: Add copyright.
Change awks to grab new config variables.
Change version from v2 to just 2 to match config variables.
v4: add _cgroup_check_return to repeat return value of tst_cgctl
v5: Instead of _cgroup_check_return, use ROD in cgroup_cleanup
Also manually parse return value in cgroup_require
Change some minor nits and quote a variable
testcases/kernel/controllers/cgroup_lib.sh | 133 +++++++++++++++++----
1 file changed, 113 insertions(+), 20 deletions(-)
diff --git a/testcases/kernel/controllers/cgroup_lib.sh b/testcases/kernel/controllers/cgroup_lib.sh
index 56c9a7521..3e9a9ea9b 100644
--- a/testcases/kernel/controllers/cgroup_lib.sh
+++ b/testcases/kernel/controllers/cgroup_lib.sh
@@ -2,38 +2,131 @@
# SPDX-License-Identifier: GPL-2.0-or-later
# Copyright (c) 2019-2022 Petr Vorel <pvorel@suse.cz>
# Copyright (c) 2018-2019 ARM Ltd. All Rights Reserved.
+# Copyright (c) 2022 Canonical Ltd.
-# Find mountpoint to given subsystem
-# get_cgroup_mountpoint SUBSYSTEM
-# RETURN: 0 if mountpoint found, otherwise 1
-get_cgroup_mountpoint()
+_cgroup_state=
+
+# Find mountpoint of the given controller
+# USAGE: cgroup_get_mountpoint CONTROLLER
+# RETURNS: Prints the mountpoint of the given controller
+# Must call cgroup_require before calling
+cgroup_get_mountpoint()
+{
+ local ctrl="$1"
+ local mountpoint
+
+ [ $# -eq 0 ] && tst_brk TBROK "cgroup_get_mountpoint: controller not defined"
+ [ "$_cgroup_state" = "" ] && tst_brk TBROK "cgroup_get_mountpoint: No previous state found. Forgot to call cgroup_require?"
+
+ mountpoint=$(echo "$_cgroup_state" | grep -w "^$ctrl" | awk '{ print $4 }')
+ echo "$mountpoint"
+
+ return 0
+}
+
+# Get the test path of a given controller that has been created by the cgroup C API
+# USAGE: cgroup_get_test_path CONTROLLER
+# RETURNS: Prints the path to the test direcory
+# Must call cgroup_require before calling
+cgroup_get_test_path()
+{
+ local ctrl="$1"
+ local mountpoint
+ local test_path
+
+ [ $# -eq 0 ] && tst_brk TBROK "cgroup_get_test_path: controller not defined"
+ [ "$_cgroup_state" = "" ] && tst_brk TBROK "cgroup_get_test_path: No previous state found. Forgot to call cgroup_require?"
+
+ mountpoint=$(cgroup_get_mountpoint "$ctrl")
+
+ test_path="$mountpoint/ltp/test-$$"
+
+ [ ! -e "$test_path" ] && tst_brk TBROK "cgroup_get_test_path: No test path found. Forgot to call cgroup_require?"
+
+ echo "$test_path"
+
+ return 0
+}
+
+# Gets the cgroup version of the given controller
+# USAGE: cgroup_get_version CONTROLLER
+# RETURNS: "1" if version 1 and "2" if version 2
+# Must call cgroup_require before calling
+cgroup_get_version()
{
- local subsystem=$1
- local mntpoint
+ local ctrl="$1"
+ local version
- [ $# -eq 0 ] && tst_brk TBROK "get_cgroup_mountpoint: subsystem not defined"
+ [ $# -eq 0 ] && tst_brk TBROK "cgroup_get_version: controller not defined"
+ [ "$_cgroup_state" = "" ] && tst_brk TBROK "cgroup_get_version: No previous state found. Forgot to call cgroup_require?"
- mntpoint=$(grep cgroup /proc/mounts | grep -w $subsystem | awk '{ print $2 }')
- [ -z "$mntpoint" ] && return 1
+ version=$(echo "$_cgroup_state" | grep -w "^$ctrl" | awk '{ print $2 }')
+ [ "$version" = "" ] && tst_brk TBROK "cgroup_get_version: Could not find controller $ctrl"
+
+ echo "$version"
- echo $mntpoint
return 0
}
-# Check if given subsystem is supported and enabled
-# is_cgroup_subsystem_available_and_enabled SUBSYSTEM
-# RETURN: 0 if subsystem supported and enabled, otherwise 1
-is_cgroup_subsystem_available_and_enabled()
+# Cleans up any setup done by calling cgroup_require.
+# USAGE: cgroup_cleanup
+# Can be safely called even when no setup has been done
+cgroup_cleanup()
{
- local val
- local subsystem=$1
+ [ "$_cgroup_state" = "" ] && return 0
- [ $# -eq 0 ] && tst_brk TBROK "is_cgroup_subsystem_available_and_enabled: subsystem not defined"
+ ROD tst_cgctl cleanup "$_cgroup_state"
- val=$(grep -w $subsystem /proc/cgroups | awk '{ print $4 }')
- [ "$val" = "1" ] && return 0
+ _cgroup_state=
- return 1
+ return 0
+}
+
+# Get the task list of the given controller
+# USAGE: cgroup_get_task_list CONTROLLER
+# RETURNS: prints out "cgroup.procs" if version 2 otherwise "tasks"
+# Must call cgroup_require before calling
+cgroup_get_task_list()
+{
+ local ctrl="$1"
+ local version
+
+ [ $# -eq 0 ] && tst_brk TBROK "cgroup_get_task_list: controller not defined"
+
+ version=$(cgroup_get_version "$ctrl")
+
+ if [ "$version" = "2" ]; then
+ echo "cgroup.procs"
+ else
+ echo "tasks"
+ fi
+
+ return 0
+}
+
+# Mounts and configures the given controller
+# USAGE: cgroup_require CONTROLLER
+cgroup_require()
+{
+ local ctrl="$1"
+
+ [ $# -eq 0 ] && tst_brk TBROK "cgroup_require: controller not defined"
+
+ [ ! -f /proc/cgroups ] && tst_brk TCONF "Kernel does not support control groups"
+
+ _cgroup_state=$(tst_cgctl require "$ctrl" $$)
+
+ if [ $? -eq 32 ]; then
+ tst_brk TCONF "'tst_cgctl require' exited. Controller is probably not available?"
+ fi
+
+ if [ $? -ne 0 ]; then
+ tst_brk TBROK "'tst_cgctl require' exited."
+ fi
+
+ [ "$_cgroup_state" = "" ] && tst_brk TBROK "cgroup_require: No state was set after call to tst_cgctl require?"
+
+ return 0
}
. tst_test.sh
--
2.34.1
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [LTP] [PATCH v3 11/18] controllers: Update cgroup_fj_* to use newer cgroup lib and test lib
2022-07-21 20:52 [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests Luke Nowakowski-Krijger
` (9 preceding siblings ...)
2022-07-21 20:52 ` [LTP] [PATCH v5 10/18] controllers: Expand cgroup_lib shell library Luke Nowakowski-Krijger
@ 2022-07-21 20:52 ` Luke Nowakowski-Krijger
2022-07-25 11:41 ` Richard Palethorpe
2022-07-21 20:52 ` [LTP] [PATCH v3 12/18] controllers: Update memcg_control_test to newer test lib and cgroup lib Luke Nowakowski-Krijger
` (8 subsequent siblings)
19 siblings, 1 reply; 33+ messages in thread
From: Luke Nowakowski-Krijger @ 2022-07-21 20:52 UTC (permalink / raw)
To: ltp, rpalethorpe, liwang, pvorel, chrubis
Update the cgroup_fj_* tests to use the newer test lib and to use the
updated version of the cgroup lib which handles mounting and unmounting
for both v1 and v2 controllers.
The tests were modified to accommodate testing the v2 controller
interfaces where it still made sense, and in other places tests were
skipped as they were testing using specific parts of the v1 interface
that doesn't exist on v2 controllers.
Also updated the licensing info at the beginning of the file with SPDX
license identifier.
Signed-off-by: Luke Nowakowski-Krijger <luke.nowakowskikrijger@canonical.com>
---
v2: Remove exist_subsystem() function.
Correct spelling mistakes in commit message.
Change version numbers to "2" instead of "v2"
v3: put cgroup_lib.sh include in bottom of file
.../controllers/cgroup_fj/cgroup_fj_common.sh | 115 ++++--------
.../cgroup_fj/cgroup_fj_function.sh | 172 ++++++++++--------
.../controllers/cgroup_fj/cgroup_fj_proc.c | 24 +--
.../controllers/cgroup_fj/cgroup_fj_stress.sh | 171 +++++++++--------
4 files changed, 217 insertions(+), 265 deletions(-)
diff --git a/testcases/kernel/controllers/cgroup_fj/cgroup_fj_common.sh b/testcases/kernel/controllers/cgroup_fj/cgroup_fj_common.sh
index 53ab637e8..6d5586534 100755
--- a/testcases/kernel/controllers/cgroup_fj/cgroup_fj_common.sh
+++ b/testcases/kernel/controllers/cgroup_fj/cgroup_fj_common.sh
@@ -1,43 +1,17 @@
#!/bin/sh
-
-################################################################################
-## ##
-## Copyright (c) 2009 FUJITSU LIMITED ##
-## Author: Shi Weihua <shiwh@cn.fujitsu.com> ##
-## Copyright (c) 2015 Cedric Hnyda <chnyda@suse.com> ##
-## Copyright (c) 2015-2016 Cyril Hrubis <chrubis@suse.cz> ##
-## ##
-## This program is free software; you can redistribute it and#or modify ##
-## it under the terms of the GNU General Public License as published by ##
-## the Free Software Foundation; either version 2 of the License, or ##
-## (at your option) any later version. ##
-## ##
-## This program is distributed in the hope that it will be useful, but ##
-## WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY ##
-## or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License ##
-## for more details. ##
-## ##
-## You should have received a copy of the GNU General Public License ##
-## along with this program; if not, write to the Free Software Foundation, ##
-## Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA ##
-## ##
-################################################################################
+# SPDX-License-Identifier: GPL-2.0-or-later
+# Copyright (c) 2009 FUJITSU LIMITED
+# Copyright (c) 2015 Cedric Hnyda <chnyda@suse.com>
+# Copyright (c) 2015-2016 Cyril Hrubis <chrubis@suse.cz>
+# Author: Shi Weihua <shiwh@cn.fujitsu.com>
for arg; do
TCID="${TCID}_$arg"
done
-. test.sh
-
-exist_subsystem()
-{
- local subsystem="$1"
- local exist=`grep -w $subsystem /proc/cgroups | cut -f1`
-
- if [ -z "$exist" ]; then
- tst_brkm TCONF "Subsystem $subsystem not supported"
- fi
-}
+TST_NEEDS_CMDS="rmdir killall"
+TST_NEEDS_ROOT=1
+TST_NEEDS_TMPDIR=1
attach_and_check()
{
@@ -46,13 +20,13 @@ attach_and_check()
local task
shift
- tst_resm TINFO "Attaching task $pid to $path"
+ tst_res TINFO "Attaching task $pid to $path"
- ROD echo "$pid" \> "$path/tasks"
+ ROD echo "$pid" \> "$path/$task_list"
- for task in $(cat "$path/tasks"); do
+ for task in $(cat "$path/$task_list"); do
if [ "$task" -ne "$pid" ]; then
- tst_resm TINFO "Unexpected pid $task in $path/tasks, expected $pid"
+ tst_res TINFO "Unexpected pid $task in $path/$task_list, expected $pid"
return 1
fi
done
@@ -64,11 +38,13 @@ create_subgroup()
{
path="$1"
- ROD mkdir "$path"
+ [ ! -d "$path" ] && ROD mkdir "$path"
# cpuset.cpus and cpuset.mems must be initialized with suitable value
- # before any pids are attached
- if [ "$subsystem" = "cpuset" ]; then
+ # before any pids are attached.
+ # Only needs to be done for cgroup v1 as sets are inherited from parents
+ # by default in cgroup v2.
+ if [ "$cgroup_version" = "1" ] && [ "$subsystem" = "cpuset" ]; then
if [ -e "$mount_point/cpus" ]; then
ROD cat "$mount_point/cpus" \> "$path/cpus"
ROD cat "$mount_point/mems" \> "$path/mems"
@@ -79,54 +55,27 @@ create_subgroup()
fi
}
-
-setup()
+common_setup()
{
- tst_require_root
- tst_require_cmds killall
-
- if [ ! -f /proc/cgroups ]; then
- tst_brkm TCONF "Kernel does not support for control groups"
- fi
-
- exist_subsystem "$subsystem"
-
- tst_tmpdir
- TST_CLEANUP=cleanup
-
- mount_point=`grep -w $subsystem /proc/mounts | grep -w "cgroup" | \
- cut -f 2 | cut -d " " -f2`
-
- if [ -z "$mount_point" ]; then
- try_umount=1
- mount_point="/dev/cgroup"
- tst_resm TINFO "Subsystem $subsystem is not mounted, mounting it at $mount_point"
- ROD mkdir $mount_point
- ROD mount -t cgroup -o "$subsystem" "ltp_cgroup" "$mount_point"
- else
- tst_resm TINFO "Subsystem $subsystem is mounted at $mount_point"
- fi
+ cgroup_require "$subsystem"
+ mount_point=$(cgroup_get_mountpoint "$subsystem")
+ start_path=$(cgroup_get_test_path "$subsystem")
+ cgroup_version=$(cgroup_get_version "$subsystem")
+ task_list=$(cgroup_get_task_list "$subsystem")
+
+ [ "$cgroup_version" = "2" ] && ROD echo "+$subsystem" \> "$start_path/cgroup.subtree_control"
+ tst_res TINFO "test starts with cgroup version $cgroup_version"
}
-cleanup()
+common_cleanup()
{
- tst_rmdir
-
killall -9 cgroup_fj_proc >/dev/null 2>&1
- tst_resm TINFO "Removing all ltp subgroups..."
-
- find "$mount_point/ltp/" -depth -type d -exec rmdir '{}' \;
+ tst_res TINFO "Removing all ltp subgroups..."
- if [ -z "$try_umount" ]; then
- return
- fi
-
- if grep -q "$mount_point" /proc/mounts; then
- EXPECT_PASS umount "$mount_point"
- fi
+ [ -d "$start_path" ] && find "$start_path" -depth -type d -exec rmdir '{}' \;
- if [ -e "$mount_point" ]; then
- EXPECT_PASS rmdir "$mount_point"
- fi
+ cgroup_cleanup
}
+
+. cgroup_lib.sh
diff --git a/testcases/kernel/controllers/cgroup_fj/cgroup_fj_function.sh b/testcases/kernel/controllers/cgroup_fj/cgroup_fj_function.sh
index fc3ad1b63..0fde2d9c0 100755
--- a/testcases/kernel/controllers/cgroup_fj/cgroup_fj_function.sh
+++ b/testcases/kernel/controllers/cgroup_fj/cgroup_fj_function.sh
@@ -1,32 +1,16 @@
#!/bin/sh
-
-################################################################################
-## ##
-## Copyright (c) 2009 FUJITSU LIMITED ##
-## Author: Shi Weihua <shiwh@cn.fujitsu.com> ##
-## Copyright (c) 2015 Cedric Hnyda <chnyda@suse.com> ##
-## Copyright (c) 2015-2016 Cyril Hrubis <chrubis@suse.cz> ##
-## ##
-## This program is free software; you can redistribute it and#or modify ##
-## it under the terms of the GNU General Public License as published by ##
-## the Free Software Foundation; either version 2 of the License, or ##
-## (at your option) any later version. ##
-## ##
-## This program is distributed in the hope that it will be useful, but ##
-## WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY ##
-## or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License ##
-## for more details. ##
-## ##
-## You should have received a copy of the GNU General Public License ##
-## along with this program; if not, write to the Free Software Foundation, ##
-## Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA ##
-## ##
-################################################################################
+# SPDX-License-Identifier: GPL-2.0-or-later
+# Copyright (c) 2009 FUJITSU LIMITED
+# Copyright (c) 2015 Cedric Hnyda <chnyda@suse.com>
+# Copyright (c) 2015-2016 Cyril Hrubis <chrubis@suse.cz>
+# Author: Shi Weihua <shiwh@cn.fujitsu.com>
TCID="cgroup_fj_function2"
-TST_TOTAL=7
-
-. cgroup_fj_common.sh
+TST_TESTFUNC=test
+TST_SETUP=setup
+TST_CLEANUP=cleanup
+TST_CNT=9
+TST_POS_ARGS=1
subsystem=$1
@@ -36,7 +20,7 @@ usage_and_exit()
echo " ./cgroup_fj_function2.sh subsystem"
echo "example: ./cgroup_fj_function2.sh cpuset"
- tst_brkm TBROK "$1"
+ tst_brk TBROK "$1"
}
if [ "$#" -ne "1" ]; then
@@ -46,49 +30,67 @@ fi
# Move a task from group to group
test1()
{
+ # mv'ing cgroups is not available in cgroup2
+ if [ "$cgroup_version" = "2" ]; then
+ tst_res TCONF "Controller mounted on cgroup2 hierachy, skipping test"
+ return
+ fi
+
if ! attach_and_check "$pid" "$start_path/ltp_1"; then
- tst_resm TFAIL "Failed to attach task"
+ tst_res TFAIL "Failed to attach task"
return
fi
if ! attach_and_check "$pid" "$start_path"; then
- tst_resm TFAIL "Failed to attach task"
+ tst_res TFAIL "Failed to attach task"
return
fi
- tst_resm TPASS "Task attached succesfully"
+ tst_res TPASS "Task attached succesfully"
}
# Group can be renamed with mv
test2()
{
+ # mv'ing cgroups is not available in cgroup2
+ if [ "$cgroup_version" = "2" ]; then
+ tst_res TCONF "Controller mounted on cgroup2 hierachy, skipping test"
+ return
+ fi
+
create_subgroup "$start_path/ltp_2"
if ! mv "$start_path/ltp_2" "$start_path/ltp_3"; then
- tst_resm TFAIL "Failed to move $start_path/ltp_2 to $start_path/ltp_3"
+ tst_res TFAIL "Failed to move $start_path/ltp_2 to $start_path/ltp_3"
rmdir "$start_path/ltp_2"
return
fi
if ! rmdir "$start_path/ltp_3"; then
- tst_resm TFAIL "Failed to remove $start_path/ltp_3"
+ tst_res TFAIL "Failed to remove $start_path/ltp_3"
return
fi
- tst_resm TPASS "Successfully moved $start_path/ltp_2 to $start_path/ltp_3"
+ tst_res TPASS "Successfully moved $start_path/ltp_2 to $start_path/ltp_3"
}
# Group can be renamed with mv unless the target name exists
test3()
{
+ # mv'ing cgroups is not available in cgroup2
+ if [ "$cgroup_version" = "2" ]; then
+ tst_res TCONF "Controller mounted on cgroup2 hierachy, skipping test"
+ return
+ fi
+
create_subgroup "$start_path/ltp_2"
if mv "$start_path/ltp_2" "$start_path/ltp_1" > /dev/null 2>&1; then
- tst_resm TFAIL "Moved $start_path/ltp_2 over existing $start_path/ltp_1"
+ tst_res TFAIL "Moved $start_path/ltp_2 over existing $start_path/ltp_1"
return
fi
- tst_resm TPASS "Failed to move $start_path/ltp_2 over existing $start_path/ltp_1"
+ tst_res TPASS "Failed to move $start_path/ltp_2 over existing $start_path/ltp_1"
ROD rmdir "$start_path/ltp_2"
}
@@ -97,77 +99,104 @@ test3()
test4()
{
if ! attach_and_check "$pid" "$start_path/ltp_1"; then
- tst_resm TFAIL "Failed to attach $pid to $start_path/ltp_1"
+ tst_res TFAIL "Failed to attach $pid to $start_path/ltp_1"
return
fi
if rmdir "$start_path/ltp_1" > /dev/null 2>&1; then
- tst_resm TFAIL "Removed $start_path/ltp_1 which contains task $pid"
- create_subgroup "$start_path/ltp_1"
+ tst_res TFAIL "Removed $start_path/ltp_1 which contains task $pid"
return
fi
- tst_resm TPASS "Group $start_path/ltp_1 with task $pid cannot be removed"
+ tst_res TPASS "Group $start_path/ltp_1 with task $pid cannot be removed"
}
# Group with a subgroup cannot be removed
test5()
{
+ # We need to move the tasks back to root to create a subgroup
+ if [ "$cgroup_version" = "2" ]; then
+ for pid in $(cat "$start_path/ltp_1/$task_list"); do
+ echo $pid > "$mount_point/$task_list" 2> /dev/null
+ done
+
+ ROD echo "+$subsystem" \> "$start_path/ltp_1/cgroup.subtree_control"
+ fi
+
create_subgroup "$start_path/ltp_1/a"
if rmdir "$start_path/ltp_1" > /dev/null 2>&1; then
- tst_resm TFAIL "Removed $start_path/ltp_1 which contains subdir 'a'"
+ tst_res TFAIL "Removed $start_path/ltp_1 which contains subdir 'a'"
return
fi
- tst_resm TPASS "Dir $start_path/ltp_1 with subdir 'a' cannot be removed"
+ tst_res TPASS "Dir $start_path/ltp_1 with subdir 'a' cannot be removed"
ROD rmdir "$start_path/ltp_1/a"
- ROD echo "$pid" \> "$start_path/tasks"
+ [ "$cgroup_version" = "2" ] && ROD echo "-$subsystem" \> "$start_path/ltp_1/cgroup.subtree_control"
+ ROD echo "$pid" \> "$start_path/ltp_1/$task_list"
}
# Group cannot be moved outside of hierarchy
test6()
{
+ # mv'ing cgroups is not available in cgroup2
+ if [ "$cgroup_version" = "2" ]; then
+ tst_res TCONF "Controller mounted on cgroup2 hierachy, skipping test"
+ return
+ fi
+
if mv "$start_path/ltp_1" "$PWD/ltp" > /dev/null 2>&1; then
- tst_resm TFAIL "Subgroup $start_path/ltp_1 outside hierarchy to $PWD/ltp"
+ tst_res TFAIL "Subgroup $start_path/ltp_1 outside hierarchy to $PWD/ltp"
return
fi
- tst_resm TPASS "Subgroup $start_path/ltp_1 cannot be moved to $PWD/ltp"
+ tst_res TPASS "Subgroup $start_path/ltp_1 cannot be moved to $PWD/ltp"
}
# Tasks file cannot be removed
test7()
{
- if rm "$start_path/ltp_1/tasks" > /dev/null 2>&1; then
- tst_resm TFAIL "Tasks file $start_path/ltp_1/tasks could be removed"
+ if rm "$start_path/ltp_1/$task_list" > /dev/null 2>&1; then
+ tst_res TFAIL "Tasks file $start_path/ltp_1/$task_list could be removed"
return
fi
- tst_resm TPASS "Tasks file $start_path/ltp_1/tasks cannot be removed"
+ tst_res TPASS "Tasks file $start_path/ltp_1/tasks cannot be removed"
}
# Test notify_on_release with invalid inputs
test8()
{
+ # notify_on_release is not available in cgroup2 so skip the test
+ if [ "$cgroup_version" = "2" ]; then
+ tst_res TCONF "Controller mounted on cgroup2 hierachy, skipping test"
+ return
+ fi
+
if echo "-1" > "$start_path/ltp_1/notify_on_release" 2>/dev/null; then
- tst_resm TFAIL "Can write -1 to $start_path/ltp_1/notify_on_release"
+ tst_res TFAIL "Can write -1 to $start_path/ltp_1/notify_on_release"
return
fi
if echo "ltp" > "$start_path/ltp_1/notify_on_release" 2>/dev/null; then
- tst_resm TFAIL "Can write ltp to $start_path/ltp_1/notify_on_release"
+ tst_res TFAIL "Can write ltp to $start_path/ltp_1/notify_on_release"
return
fi
- tst_resm TPASS "Cannot write invalid values to $start_path/ltp_1/notify_on_release"
+ tst_res TPASS "Cannot write invalid values to $start_path/ltp_1/notify_on_release"
}
# Test that notify_on_release can be changed
test9()
{
+ # notify_on_release is not available in cgroup2 so skip the test
+ if [ "$cgroup_version" = "2" ]; then
+ tst_res TCONF "Controller mounted on cgroup2 hierachy, skipping test"
+ return
+ fi
+
local notify=$(ROD cat "$start_path/ltp_1/notify_on_release")
local value
@@ -178,37 +207,30 @@ test9()
fi
if ! echo "$value" > "$start_path/ltp_1/notify_on_release"; then
- tst_resm TFAIL "Failed to set $start_path/ltp_1/notify_on_release to $value"
+ tst_res TFAIL "Failed to set $start_path/ltp_1/notify_on_release to $value"
return
fi
ROD echo "$notify" \> "$start_path/ltp_1/notify_on_release"
- tst_resm TPASS "Set $start_path/ltp_1/notify_on_release to $value"
+ tst_res TPASS "Set $start_path/ltp_1/notify_on_release to $value"
}
-setup
-
-cgroup_fj_proc&
-pid=$!
-
-start_path="$mount_point/ltp"
-
-create_subgroup "$start_path"
-create_subgroup "$start_path/ltp_1"
-
-test1
-test2
-test3
-test4
-test5
-test6
-test7
-test8
-test9
+setup()
+{
+ common_setup
+ cgroup_fj_proc&
+ pid=$!
+ create_subgroup "$start_path/ltp_1"
+}
-ROD kill -9 $pid
-wait $pid
-ROD rmdir "$start_path/ltp_1"
+cleanup()
+{
+ kill -9 $pid >/dev/null 2>&1
+ wait $pid >/dev/null 2>&1
+ rmdir "$start_path/ltp_1" >/dev/null 2>&1
+ common_cleanup
+}
-tst_exit
+. cgroup_fj_common.sh
+tst_run
diff --git a/testcases/kernel/controllers/cgroup_fj/cgroup_fj_proc.c b/testcases/kernel/controllers/cgroup_fj/cgroup_fj_proc.c
index 93bc8b744..e3c1153cb 100644
--- a/testcases/kernel/controllers/cgroup_fj/cgroup_fj_proc.c
+++ b/testcases/kernel/controllers/cgroup_fj/cgroup_fj_proc.c
@@ -1,24 +1,6 @@
-/******************************************************************************/
-/* */
-/* Copyright (c) 2009 FUJITSU LIMITED */
-/* */
-/* This program is free software; you can redistribute it and/or modify */
-/* it under the terms of the GNU General Public License as published by */
-/* the Free Software Foundation; either version 2 of the License, or */
-/* (at your option) any later version. */
-/* */
-/* This program is distributed in the hope that it will be useful, */
-/* but WITHOUT ANY WARRANTY; without even the implied warranty of */
-/* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See */
-/* the GNU General Public License for more details. */
-/* */
-/* You should have received a copy of the GNU General Public License */
-/* along with this program; if not, write to the Free Software */
-/* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */
-/* */
-/* Author: Shi Weihua <shiwh@cn.fujitsu.com> */
-/* */
-/******************************************************************************/
+// SPDX-License-Identifier: GPL-2.0-or-later
+// Copyright (c) 2009 FUJITSU LIMITED
+// Author: Shi Weihua <shiwh@cn.fujitsu.com>
#include <sys/types.h>
#include <sys/wait.h>
diff --git a/testcases/kernel/controllers/cgroup_fj/cgroup_fj_stress.sh b/testcases/kernel/controllers/cgroup_fj/cgroup_fj_stress.sh
index 01c2517c9..d80b83bbd 100755
--- a/testcases/kernel/controllers/cgroup_fj/cgroup_fj_stress.sh
+++ b/testcases/kernel/controllers/cgroup_fj/cgroup_fj_stress.sh
@@ -1,32 +1,16 @@
#!/bin/sh
-
-################################################################################
-## ##
-## Copyright (c) 2009 FUJITSU LIMITED ##
-## Author: Shi Weihua <shiwh@cn.fujitsu.com> ##
-## Copyright (c) 2015 Cedric Hnyda <chnyda@suse.com> ##
-## Copyright (c) 2015-2016 Cyril Hrubis <chrubis@suse.cz> ##
-## ##
-## This program is free software; you can redistribute it and#or modify ##
-## it under the terms of the GNU General Public License as published by ##
-## the Free Software Foundation; either version 2 of the License, or ##
-## (at your option) any later version. ##
-## ##
-## This program is distributed in the hope that it will be useful, but ##
-## WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY ##
-## or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License ##
-## for more details. ##
-## ##
-## You should have received a copy of the GNU General Public License ##
-## along with this program; if not, write to the Free Software Foundation, ##
-## Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA ##
-## ##
-################################################################################
+# SPDX-License-Identifier: GPL-2.0-or-later
+# Copyright (c) 2009 FUJITSU LIMITED
+# Copyright (c) 2015 Cedric Hnyda <chnyda@suse.com>
+# Copyright (c) 2015-2016 Cyril Hrubis <chrubis@suse.cz>
+# Author: Shi Weihua <shiwh@cn.fujitsu.com>
TCID="cgroup_fj_stress"
-TST_TOTAL=1
-
-. cgroup_fj_common.sh
+TST_CNT=1
+TST_TESTFUNC=test
+TST_SETUP=setup
+TST_CLEANUP=cleanup
+TST_POS_ARGS=4
subsystem="$1"
subgroup_num="$2"
@@ -47,35 +31,9 @@ usage_and_exit()
echo " each - attach process to each subgroup"
echo "example: ./cgroup_fj_stress.sh cpuset 1 1 one"
echo
- tst_brkm TBROK "$1"
+ tst_brk TBROK "$1"
}
-if [ "$#" -ne "4" ]; then
- usage_and_exit "Wrong number of parameters, expected 4"
-fi
-
-case $subgroup_num in
- ''|*[!0-9]*) usage_and_exit "Number of subgroups must be positive integer";;
- *) ;;
-esac
-
-case $subgroup_depth in
- ''|*[!0-9]*) usage_and_exit "Depth of the subgroup tree must be positive integer";;
- *) ;;
-esac
-
-case $attach_operation in
- 'none'|'one'|'each');;
- *) usage_and_exit "Invalid attach operation: $attach_operation";;
-esac
-
-setup
-
-export TMPFILE=./tmp_tasks.$$
-
-count=0
-collected_pids=
-
build_subgroups()
{
local cur_path="$1"
@@ -87,6 +45,12 @@ build_subgroups()
fi
create_subgroup "$cur_path"
+
+ # We can only attach processes to the leaves of the tree in cgroup v2 which
+ # means we need to enable the controllers everywhere inbetween.
+ if [ "$cgroup_version" = "2" ] && [ "$cur_depth" -ne "$subgroup_depth" ]; then
+ ROD echo "+$subsystem" \> "$cur_path/cgroup.subtree_control"
+ fi
count=$((count+1))
for i in $(seq 1 $subgroup_num); do
@@ -113,8 +77,10 @@ attach_task()
pid="$ppid"
fi
- if ! attach_and_check "$pid" "$cur_path"; then
+ if [ "$cgroup_version" = "2" ] && [ $cur_depth -eq $subgroup_depth ] || [ "$cgroup_version" = "1" ]; then
+ if ! attach_and_check "$pid" "$cur_path"; then
fail=1
+ fi
fi
for i in $(seq 1 $subgroup_num); do
@@ -123,46 +89,79 @@ attach_task()
done
if [ -n "$ppid" ]; then
- if ! attach_and_check "$pid" "$cur_path"; then
- fail=1
+ if [ "$cgroup_version" = "2" ] && [ $cur_depth -eq $subgroup_depth ] || [ "$cgroup_version" = "1" ]; then
+ if ! attach_and_check "$pid" "$cur_path"; then
+ fail=1
+ fi
fi
fi
}
-start_path="$mount_point/ltp"
+setup()
+{
+ export TMPFILE=./tmp_tasks.$$
+ count=0
+ collected_pids=
+
+ case $subgroup_num in
+ ''|*[!0-9]*) usage_and_exit "Number of subgroups must be possitive integer";;
+ *) ;;
+ esac
+
+ case $subgroup_depth in
+ ''|*[!0-9]*) usage_and_exit "Depth of the subgroup tree must be possitive integer";;
+ *) ;;
+ esac
+
+ case $attach_operation in
+ 'none'|'one'|'each');;
+ *) usage_and_exit "Invalid attach operation: $attach_operation";;
+ esac
+
+ common_setup
+}
-tst_resm TINFO "Creating subgroups ..."
+cleanup()
+{
+ common_cleanup
+}
+
+test()
+{
+ tst_res TINFO "Creating subgroups ..."
-build_subgroups "$start_path" 0
+ build_subgroups "$start_path" 0
-tst_resm TINFO "... mkdired $count times"
+ tst_res TINFO "... mkdired $count times"
-case $attach_operation in
-"one" )
- cgroup_fj_proc &
- pid=$!
+ case $attach_operation in
+ "one" )
+ cgroup_fj_proc &
+ pid=$!
- tst_resm TINFO "Moving one task around"
- attach_task "$start_path" 0 "$pid"
- ROD kill -9 "$pid"
- wait "$pid"
- ;;
-"each" )
- tst_resm TINFO "Attaching task to each subgroup"
- attach_task "$start_path" 0
- for pid in $collected_pids; do
+ tst_res TINFO "Moving one task around"
+ attach_task "$start_path" 0 "$pid"
ROD kill -9 "$pid"
wait "$pid"
- done
- ;;
-* )
- ;;
-esac
-
-if [ -n "$fail" ]; then
- tst_resm TFAIL "Attaching tasks failed!"
-else
- tst_resm TPASS "All done!"
-fi
-
-tst_exit
+ ;;
+ "each" )
+ tst_res TINFO "Attaching task to each subgroup"
+ attach_task "$start_path" 0
+ for pid in $collected_pids; do
+ ROD kill -9 "$pid"
+ wait "$pid"
+ done
+ ;;
+ * )
+ ;;
+ esac
+
+ if [ -n "$fail" ]; then
+ tst_res TFAIL "Attaching tasks failed!"
+ else
+ tst_res TPASS "All done!"
+ fi
+}
+
+. cgroup_fj_common.sh
+tst_run
--
2.34.1
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [LTP] [PATCH v3 12/18] controllers: Update memcg_control_test to newer test lib and cgroup lib
2022-07-21 20:52 [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests Luke Nowakowski-Krijger
` (10 preceding siblings ...)
2022-07-21 20:52 ` [LTP] [PATCH v3 11/18] controllers: Update cgroup_fj_* to use newer cgroup lib and test lib Luke Nowakowski-Krijger
@ 2022-07-21 20:52 ` Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH v3 13/18] controllers: Update memcg/regression/* to new test " Luke Nowakowski-Krijger
` (7 subsequent siblings)
19 siblings, 0 replies; 33+ messages in thread
From: Luke Nowakowski-Krijger @ 2022-07-21 20:52 UTC (permalink / raw)
To: ltp, rpalethorpe, liwang, pvorel, chrubis
Update memcg_control_test to use the newer test lib and to use the newer
cgroup lib which enables the memory v1 and v2 controller to be tested.
Also updated to newer SPDX license identifier.
Signed-off-by: Luke Nowakowski-Krijger <luke.nowakowskikrijger@canonical.com>
---
v2: Change version numbers to 2 instead of v2
v3: put cgroup_lib.sh include in bottom of file
.../controllers/memcg/control/mem_process.c | 28 +---
.../memcg/control/memcg_control_test.sh | 149 +++++-------------
2 files changed, 39 insertions(+), 138 deletions(-)
diff --git a/testcases/kernel/controllers/memcg/control/mem_process.c b/testcases/kernel/controllers/memcg/control/mem_process.c
index 6c1b36ca6..8ecabb272 100644
--- a/testcases/kernel/controllers/memcg/control/mem_process.c
+++ b/testcases/kernel/controllers/memcg/control/mem_process.c
@@ -1,28 +1,6 @@
-/*****************************************************************************/
-/* */
-/* Copyright (c) 2010 Mohamed Naufal Basheer */
-/* */
-/* This program is free software; you can redistribute it and/or modify */
-/* it under the terms of the GNU General Public License as published by */
-/* the Free Software Foundation; either version 2 of the License, or */
-/* (at your option) any later version. */
-/* */
-/* This program is distributed in the hope that it will be useful, */
-/* but WITHOUT ANY WARRANTY; without even the implied warranty of */
-/* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See */
-/* the GNU General Public License for more details. */
-/* */
-/* You should have received a copy of the GNU General Public License */
-/* along with this program; if not, write to the Free Software */
-/* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */
-/* */
-/* File: mem_process.c */
-/* */
-/* Purpose: act as a memory hog for the memcg_control tests */
-/* */
-/* Author: Mohamed Naufal Basheer <naufal11@gmail.com > */
-/* */
-/*****************************************************************************/
+// SPDX-License-Identifier: GPL-2.0-or-later
+// Copyright (c) 2010 Mohamed Naufal Basheer
+// Author: Mohamed Naufal Basheer
#include <sys/types.h>
#include <sys/mman.h>
diff --git a/testcases/kernel/controllers/memcg/control/memcg_control_test.sh b/testcases/kernel/controllers/memcg/control/memcg_control_test.sh
index 4d9f1bb5d..f96ed3abb 100644
--- a/testcases/kernel/controllers/memcg/control/memcg_control_test.sh
+++ b/testcases/kernel/controllers/memcg/control/memcg_control_test.sh
@@ -1,45 +1,14 @@
#!/bin/sh
+# SPDX-License-Identifier: GPL-2.0-or-later
+# Copyright (c) 2010 Mohamed Naufal Basheer
+# Author: Mohamed Naufal Basheer
-################################################################################
-## ##
-## Copyright (c) 2010 Mohamed Naufal Basheer ##
-## ##
-## This program is free software; you can redistribute it and/or modify ##
-## it under the terms of the GNU General Public License as published by ##
-## the Free Software Foundation; either version 2 of the License, or ##
-## (at your option) any later version. ##
-## ##
-## This program is distributed in the hope that it will be useful, ##
-## but WITHOUT ANY WARRANTY; without even the implied warranty of ##
-## MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See ##
-## the GNU General Public License for more details. ##
-## ##
-## You should have received a copy of the GNU General Public License ##
-## along with this program; if not, write to the Free Software ##
-## Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA ##
-## ##
-## ##
-## File: memcg_control_test.sh ##
-## ##
-## Purpose: Implement various memory controller tests ##
-## ##
-## Author: Mohamed Naufal Basheer <naufal11@gmail.com> ##
-## ##
-################################################################################
-
-if [ "x$(grep -w memory /proc/cgroups | cut -f4)" != "x1" ]; then
- echo "WARNING:"
- echo "Either kernel does not support memory resource controller or feature not enabled"
- echo "Skipping all memcg_control testcases...."
- exit 0
-fi
-
-export TCID="memcg_control"
-export TST_TOTAL=1
-export TST_COUNT=0
-
-export TMP=${TMP:-/tmp}
-cd $TMP
+TST_TESTFUNC=test
+TST_SETUP=setup
+TST_CLEANUP=cleanup
+TST_CNT=1
+TST_NEEDS_ROOT=1
+TST_NEEDS_TMPDIR=1
PAGE_SIZE=$(tst_getconf PAGESIZE)
@@ -47,20 +16,14 @@ TOT_MEM_LIMIT=$PAGE_SIZE
ACTIVE_MEM_LIMIT=$PAGE_SIZE
PROC_MEM=$((PAGE_SIZE * 2))
-TST_PATH=$PWD
-STATUS_PIPE="$TMP/status_pipe"
-
-PASS=0
-FAIL=1
+STATUS_PIPE="status_pipe"
# Check if the test process is killed on crossing boundary
test_proc_kill()
{
- cd $TMP
mem_process -m $PROC_MEM &
- cd $OLDPWD
sleep 1
- echo $! > tasks
+ ROD echo $! > "$test_dir/$task_list"
#Instruct the test process to start acquiring memory
echo m > $STATUS_PIPE
@@ -77,87 +40,47 @@ test_proc_kill()
}
# Validate the memory usage limit imposed by the hierarchically topmost group
-testcase_1()
+test1()
{
- TST_COUNT=1
- tst_resm TINFO "Test #1: Checking if the memory usage limit imposed by the topmost group is enforced"
+ cd $TST_TMPDIR
+
+ tst_res TINFO "Test #1: Checking if the memory usage limit imposed by the topmost group is enforced"
- echo "$ACTIVE_MEM_LIMIT" > $TST_PATH/mnt/$TST_NUM/memory.limit_in_bytes
- echo "$TOT_MEM_LIMIT" > $TST_PATH/mnt/$TST_NUM/memory.memsw.limit_in_bytes
+ ROD echo "$ACTIVE_MEM_LIMIT" > "$test_dir/$memory_limit"
+ ROD echo "$TOT_MEM_LIMIT" > "$test_dir/$memsw_memory_limit"
- mkdir sub
- (cd sub
KILLED_CNT=0
test_proc_kill
if [ $PROC_MEM -gt $TOT_MEM_LIMIT ] && [ $KILLED_CNT -eq 0 ]; then
- result $FAIL "Test #1: failed"
+ tst_res TFAIL "Test #1: failed"
else
- result $PASS "Test #1: passed"
- fi)
- rmdir sub
+ tst_res TPASS "Test #1: passed"
+ fi
}
-# Record the test results
-#
-# $1: Result of the test case, $PASS or $FAIL
-# $2: Output information
-result()
+setup()
{
- RES=$1
- INFO=$2
-
- if [ $RES -eq $PASS ]; then
- tst_resm TPASS "$INFO"
+ cgroup_require "memory"
+ cgroup_version=$(cgroup_get_version "memory")
+ test_dir=$(cgroup_get_test_path "memory")
+ task_list=$(cgroup_get_task_list "memory")
+
+ if [ "$cgroup_version" = "2" ]; then
+ memory_limit="memory.max"
+ memsw_memory_limit="memory.swap.max"
else
- : $((FAILED_CNT += 1))
- tst_resm TFAIL "$INFO"
+ memory_limit="memory.limit_in_bytes"
+ memsw_memory_limit="memory.memsw.limit_in_bytes"
fi
-}
-cleanup()
-{
- if [ -e $TST_PATH/mnt ]; then
- umount $TST_PATH/mnt 2> /dev/null
- rm -rf $TST_PATH/mnt
- fi
+ tst_res TINFO "Test starts with cgroup version $cgroup_version"
}
-do_mount()
+cleanup()
{
- cleanup
-
- mkdir $TST_PATH/mnt
- mount -t cgroup -o memory cgroup $TST_PATH/mnt 2> /dev/null
- if [ $? -ne 0 ]; then
- tst_brkm TBROK NULL "Mounting cgroup to temp dir failed"
- rmdir $TST_PATH/mnt
- exit 1
- fi
+ cgroup_cleanup
}
-do_mount
-
-echo 1 > mnt/memory.use_hierarchy 2> /dev/null
-
-FAILED_CNT=0
-
-TST_NUM=1
-while [ $TST_NUM -le $TST_TOTAL ]; do
- mkdir $TST_PATH/mnt/$TST_NUM
- (cd $TST_PATH/mnt/$TST_NUM && testcase_$TST_NUM)
- rmdir $TST_PATH/mnt/$TST_NUM
- : $((TST_NUM += 1))
-done
-
-echo 0 > mnt/memory.use_hierarchy 2> /dev/null
-
-cleanup
-
-if [ "$FAILED_CNT" -ne 0 ]; then
- tst_resm TFAIL "memcg_control: failed"
- exit 1
-else
- tst_resm TPASS "memcg_control: passed"
- exit 0
-fi
+. cgroup_lib.sh
+tst_run
--
2.34.1
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [LTP] [PATCH v3 13/18] controllers: Update memcg/regression/* to new test and cgroup lib
2022-07-21 20:52 [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests Luke Nowakowski-Krijger
` (11 preceding siblings ...)
2022-07-21 20:52 ` [LTP] [PATCH v3 12/18] controllers: Update memcg_control_test to newer test lib and cgroup lib Luke Nowakowski-Krijger
@ 2022-07-21 20:52 ` Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH v2 14/18] controllers: Update memcg_stress_test to use newer " Luke Nowakowski-Krijger
` (6 subsequent siblings)
19 siblings, 0 replies; 33+ messages in thread
From: Luke Nowakowski-Krijger @ 2022-07-21 20:52 UTC (permalink / raw)
To: ltp, rpalethorpe, liwang, pvorel, chrubis
Update the tests in memcg/regression/* to use the new test lib and the
newer cgroup lib to enable testing when either v1 or v2 memory
controller is mounted. Only some tests still made sense to be testing
with the v2 memory controller, otherwise they were skipped.
Also updated to the newer SPDX license identifier.
Signed-off-by: Luke Nowakowski-Krijger <luke.nowakowskikrijger@canonical.com>
---
v2: change version numbers to 2 instead of v2.
Update to use new cg_init() and test_cg variable in memcg_test_3.c
v3: put cgroup_lib.sh include at bottom of file
.../memcg/regression/memcg_regression_test.sh | 203 +++++++++---------
.../memcg/regression/memcg_test_1.c | 40 ++--
.../memcg/regression/memcg_test_2.c | 24 +--
.../memcg/regression/memcg_test_3.c | 38 ++--
.../memcg/regression/memcg_test_4.c | 24 +--
.../memcg/regression/memcg_test_4.sh | 50 ++---
6 files changed, 160 insertions(+), 219 deletions(-)
diff --git a/testcases/kernel/controllers/memcg/regression/memcg_regression_test.sh b/testcases/kernel/controllers/memcg/regression/memcg_regression_test.sh
index c91a4069e..bc8a1c661 100755
--- a/testcases/kernel/controllers/memcg/regression/memcg_regression_test.sh
+++ b/testcases/kernel/controllers/memcg/regression/memcg_regression_test.sh
@@ -1,50 +1,16 @@
#! /bin/sh
-
-################################################################################
-## ##
-## Copyright (c) 2009 FUJITSU LIMITED ##
-## ##
-## This program is free software; you can redistribute it and#or modify ##
-## it under the terms of the GNU General Public License as published by ##
-## the Free Software Foundation; either version 2 of the License, or ##
-## (at your option) any later version. ##
-## ##
-## This program is distributed in the hope that it will be useful, but ##
-## WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY ##
-## or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License ##
-## for more details. ##
-## ##
-## You should have received a copy of the GNU General Public License ##
-## along with this program; if not, write to the Free Software ##
-## Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA ##
-## ##
-## Author: Li Zefan <lizf@cn.fujitsu.com> ##
-## Added memcg enable/disable functinality: Rishikesh K Rajak ##
-## <risrajak@linux.vnet.ibm.com ##
-## ##
-################################################################################
-
-cd $LTPROOT/testcases/bin
-
-export TCID="memcg_regression_test"
-export TST_TOTAL=4
-export TST_COUNT=1
-
-if [ "$(id -ru)" != 0 ]; then
- tst_brkm TBROK ignored "Test must be run as root"
- exit 0
-fi
-
-if [ "x$(grep -w memory /proc/cgroups | cut -f4)" != "x1" ]; then
- tst_resm TCONF "Either memory resource controller kernel support absent"
- tst_resm TCONF "or feature is not enabled; skipping all memcgroup testcases."
- exit 0
-fi
-
-if tst_kvcmp -lt "2.6.30"; then
- tst_brkm TBROK ignored "Test should be run with kernel 2.6.30 or newer"
- exit 0
-fi
+# SPDX-License-Identifier: GPL-2.0-or-later
+# Copyright (c) 2009 FUJITSU LIMITED
+# Author: Li Zefan <lizf@cn.fujitsu.com>
+# Added memcg enable/disable functionality: Rishikesh K Rajak <risrajak@linux.vnet.ibm.com>
+
+TST_ID="memcg_regression_test"
+TST_CLEANUP=cleanup
+TST_SETUP=setup
+TST_TESTFUNC=test_
+TST_NEEDS_ROOT=1
+TST_NEEDS_CMDS="killall kill"
+TST_CNT=4
#buffer can rotate and number of found bugs can actually go down
#so clear the buffer to avoid this
@@ -70,16 +36,16 @@ check_kernel_bug()
# some kernel bug is detected
if [ $new_bug -gt $nr_bug ]; then
- tst_resm TFAIL "kernel BUG was detected!"
+ tst_res TFAIL "kernel BUG was detected!"
fi
if [ $new_warning -gt $nr_warning ]; then
- tst_resm TFAIL "kernel WARNING was detected!"
+ tst_res TFAIL "kernel WARNING was detected!"
fi
if [ $new_null -gt $nr_null ]; then
- tst_resm "kernel NULL pointer dereference!"
+ tst_res TWARN "kernel NULL pointer dereference!"
fi
if [ $new_lockdep -gt $nr_lockdep ]; then
- tst_resm "kernel lockdep warning was detected!"
+ tst_res TWARN "kernel lockdep warning was detected!"
fi
nr_bug=$new_bug
@@ -89,10 +55,49 @@ check_kernel_bug()
echo "check_kernel_bug found something!"
dmesg
- failed=1
return 0
}
+setup()
+{
+ if tst_kvcmp -lt "2.6.30"; then
+ tst_brk TBROK "Test should be run with kernel 2.6.30 or newer"
+ fi
+
+ cgroup_require "memory"
+ cgroup_version=$(cgroup_get_version "memory")
+ mount_point=$(cgroup_get_mountpoint "memory")
+ test_dir=$(cgroup_get_test_path "memory")
+ task_list=$(cgroup_get_task_list "memory")
+ if [ "$cgroup_version" = "2" ]; then
+ memory_limit="memory.max"
+ else
+ memory_limit="memory.limit_in_bytes"
+ fi
+
+ [ "$cgroup_version" = "2" ] && ROD echo "+memory" \> "$test_dir/cgroup.subtree_control"
+
+ tst_res TINFO "test starts with cgroup version $cgroup_version"
+}
+
+cleanup()
+{
+ cleanup_testpath "$test_dir/0"
+ cgroup_cleanup
+}
+
+create_testpath()
+{
+ local path="$1"
+ [ ! -e "$path" ] && ROD mkdir "$path"
+}
+
+cleanup_testpath()
+{
+ local path="$1"
+ [ -e "$path" ] && ROD rmdir "$path"
+}
+
#---------------------------------------------------------------------------
# Bug: The bug was, while forking mass processes, trigger memcgroup OOM,
# then NULL pointer dereference may be hit.
@@ -102,16 +107,19 @@ check_kernel_bug()
#---------------------------------------------------------------------------
test_1()
{
- mkdir memcg/0/
- echo 0 > memcg/0/memory.limit_in_bytes
+ local test_path
+ test_path="$test_dir/0"
- ./memcg_test_1
+ create_testpath "$test_path"
+ ROD echo 0 > "$test_path/$memory_limit"
- rmdir memcg/0/
+ ./memcg_test_1 "$test_path/$task_list"
+
+ cleanup_testpath "$test_path"
check_kernel_bug
if [ $? -eq 1 ]; then
- tst_resm TPASS "no kernel bug was found"
+ tst_res TPASS "no kernel bug was found"
fi
}
@@ -124,19 +132,31 @@ test_1()
#---------------------------------------------------------------------------
test_2()
{
+ local test_path
+
+ # for cgroup2 writing to memory.max first checks the new limit against the
+ # current usage and will start killing processes if oom, therefore we do not
+ # expect EBUSY to be returned by the shrink operation.
+ if [ "$cgroup_version" = "2" ]; then
+ tst_res TCONF "Cgroup v2 found, skipping test"
+ return
+ fi
+
+ test_path="$test_dir/0"
+
./memcg_test_2 &
pid1=$!
sleep 1
- mkdir memcg/0
- echo $pid1 > memcg/0/tasks
+ create_testpath "$test_path"
+ ROD echo $pid1 > "$test_path"/tasks
# let pid1 'test_2' allocate memory
/bin/kill -SIGUSR1 $pid1
sleep 1
# shrink memory
- echo 1 > memcg/0/memory.limit_in_bytes 2>&1 &
+ echo 1 > "$test_path"/memory.limit_in_bytes 2>&1 &
pid2=$!
# check if 'echo' will exit and exit with failure
@@ -146,26 +166,25 @@ test_2()
if [ $? -ne 0 ]; then
wait $pid2
if [ $? -eq 0 ]; then
- tst_resm TFAIL "echo should return failure"
- failed=1
+ tst_res TFAIL "echo should return failure"
kill -9 $pid1 $pid2 > /dev/null 2>&1
wait $pid1 $pid2
- rmdir memcg/0
+ cleanup_testpath "$test_path"
+ return
fi
break
fi
done
if [ $tmp -eq 5 ]; then
- tst_resm TFAIL "'echo' doesn't exit!"
- failed=1
+ tst_res TFAIL "'echo' doesn't exit!"
else
- tst_resm TPASS "EBUSY was returned as expected"
+ tst_res TPASS "EBUSY was returned as expected"
fi
kill -9 $pid1 $pid2 > /dev/null 2>&1
wait $pid1 $pid2 > /dev/null 2>&1
- rmdir memcg/0
+ cleanup_testpath "$test_path"
}
#---------------------------------------------------------------------------
@@ -176,19 +195,22 @@ test_2()
#---------------------------------------------------------------------------
test_3()
{
- mkdir memcg/0
- for pid in `cat memcg/tasks`; do
- echo $pid > memcg/0/tasks 2> /dev/null
+ local test_path
+ test_path="$test_dir/0"
+ create_testpath "$test_path"
+
+ for pid in $(cat "$mount_point/$task_list"); do
+ echo $pid > "$test_path/$task_list" 2> /dev/null
done
- for pid in `cat memcg/0/tasks`; do
- echo $pid > memcg/tasks 2> /dev/null
+ for pid in $(cat "$test_path/$task_list"); do
+ echo $pid > "$mount_point/$task_list" 2> /dev/null
done
- rmdir memcg/0
+ cleanup_testpath "$test_path"
check_kernel_bug
if [ $? -eq 1 ]; then
- tst_resm TPASS "no kernel bug was found"
+ tst_res TPASS "no kernel bug was found"
fi
}
@@ -200,11 +222,15 @@ test_3()
#---------------------------------------------------------------------------
test_4()
{
- ./memcg_test_4.sh
+ local test_path
+ test_path="$test_dir/0"
+ create_testpath "$test_path"
+
+ ./memcg_test_4.sh "$cgroup_version" "$mount_point" "$test_path"
check_kernel_bug
if [ $? -eq 1 ]; then
- tst_resm TPASS "no kernel bug was found"
+ tst_res TPASS "no kernel bug was found"
fi
# test_4.sh might be killed by oom, so do clean up here
@@ -212,31 +238,10 @@ test_4()
killall -9 memcg_test_4.sh 2> /dev/null
# if test_4.sh gets killed, it won't clean cgroup it created
- rmdir memcg/0 2> /dev/null
+ cleanup_testpath "$test_path"
swapon -a
}
-# main
-failed=0
-mkdir memcg/
-
-for cur in $(seq 1 $TST_TOTAL); do
- export TST_COUNT=$cur
-
- mount -t cgroup -o memory xxx memcg/
- if [ $? -ne 0 ]; then
- tst_resm TFAIL "failed to mount memory subsystem"
- failed=1
- continue
- fi
-
- test_$cur
-
- umount memcg/
-done
-
-rmdir memcg/
-
-exit $failed
-
+. cgroup_lib.sh
+tst_run
diff --git a/testcases/kernel/controllers/memcg/regression/memcg_test_1.c b/testcases/kernel/controllers/memcg/regression/memcg_test_1.c
index c7fb948fe..95f1aabb3 100644
--- a/testcases/kernel/controllers/memcg/regression/memcg_test_1.c
+++ b/testcases/kernel/controllers/memcg/regression/memcg_test_1.c
@@ -1,24 +1,6 @@
-/******************************************************************************/
-/* */
-/* Copyright (c) 2009 FUJITSU LIMITED */
-/* */
-/* This program is free software; you can redistribute it and/or modify */
-/* it under the terms of the GNU General Public License as published by */
-/* the Free Software Foundation; either version 2 of the License, or */
-/* (at your option) any later version. */
-/* */
-/* This program is distributed in the hope that it will be useful, */
-/* but WITHOUT ANY WARRANTY; without even the implied warranty of */
-/* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See */
-/* the GNU General Public License for more details. */
-/* */
-/* You should have received a copy of the GNU General Public License */
-/* along with this program; if not, write to the Free Software */
-/* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */
-/* */
-/* Author: Li Zefan <lizf@cn.fujitsu.com> */
-/* */
-/******************************************************************************/
+// SPDX-License-Identifier: GPL-2.0-or-later
+// Copyright (c) 2009 FUJITSU LIMITED
+// Author: Li Zefan <lizf@cn.fujitsu.com>
#include <stdio.h>
#include <stdlib.h>
@@ -33,17 +15,25 @@
#define FORKED_PROC_COUNT 10
-int main(void)
+int main(int argc, char *argv[])
{
char buf[10];
int i;
int loop;
int pid;
+ int fd;
int size = getpagesize();
- int fd = open("memcg/0/tasks", O_WRONLY);
- if (fd < 0)
- return 1;
+ if (argc != 2) {
+ perror("Invalid num of args");
+ exit(1);
+ }
+
+ fd = open(argv[1], O_WRONLY);
+ if (fd < 0) {
+ perror("Could not open tasklist");
+ exit(1);
+ }
for (loop = 0; loop < LOOP; loop++) {
for (i = 0; i < FORKED_PROC_COUNT; i++) {
diff --git a/testcases/kernel/controllers/memcg/regression/memcg_test_2.c b/testcases/kernel/controllers/memcg/regression/memcg_test_2.c
index 843b07816..c118d4559 100644
--- a/testcases/kernel/controllers/memcg/regression/memcg_test_2.c
+++ b/testcases/kernel/controllers/memcg/regression/memcg_test_2.c
@@ -1,24 +1,6 @@
-/******************************************************************************/
-/* */
-/* Copyright (c) 2009 FUJITSU LIMITED */
-/* */
-/* This program is free software; you can redistribute it and/or modify */
-/* it under the terms of the GNU General Public License as published by */
-/* the Free Software Foundation; either version 2 of the License, or */
-/* (at your option) any later version. */
-/* */
-/* This program is distributed in the hope that it will be useful, */
-/* but WITHOUT ANY WARRANTY; without even the implied warranty of */
-/* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See */
-/* the GNU General Public License for more details. */
-/* */
-/* You should have received a copy of the GNU General Public License */
-/* along with this program; if not, write to the Free Software */
-/* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */
-/* */
-/* Author: Li Zefan <lizf@cn.fujitsu.com> */
-/* */
-/******************************************************************************/
+// SPDX-License-Identifier: GPL-2.0-or-later
+// Copyright (c) 2009 FUJITSU LIMITED
+// Author: Li Zefan <lizf@cn.fujitsu.com>
#include <sys/mman.h>
#include <signal.h>
diff --git a/testcases/kernel/controllers/memcg/regression/memcg_test_3.c b/testcases/kernel/controllers/memcg/regression/memcg_test_3.c
index 75a6e1545..81e1fc161 100644
--- a/testcases/kernel/controllers/memcg/regression/memcg_test_3.c
+++ b/testcases/kernel/controllers/memcg/regression/memcg_test_3.c
@@ -17,12 +17,10 @@
#include <sys/types.h>
#include <sys/mount.h>
#include "tst_test.h"
+#include "tst_cgroup.h"
-#define MNTPOINT "memcg"
-#define SUBDIR "memcg/testdir"
-
-static int mount_flag;
static volatile int sigcounter;
+static struct tst_cg_group *test_cg;
static void sighandler(int sig LTP_ATTRIBUTE_UNUSED)
{
@@ -48,9 +46,10 @@ static void do_test(void)
do_child();
while (sigcounter < 50000) {
- if (access(SUBDIR, F_OK))
- SAFE_MKDIR(SUBDIR, 0644);
- rmdir(SUBDIR);
+ test_cg = tst_cg_group_mk(tst_cg, "test");
+
+ if (test_cg)
+ test_cg = tst_cg_group_rm(test_cg);
}
SAFE_KILL(cpid, SIGKILL);
@@ -61,32 +60,25 @@ static void do_test(void)
static void setup(void)
{
- int ret;
-
- SAFE_MKDIR(MNTPOINT, 0644);
+ struct tst_cg_opts opts;
+ memset(&opts, 0, sizeof(opts));
- ret = mount("memcg", MNTPOINT, "cgroup", 0, "memory");
- if (ret) {
- if (errno == ENOENT)
- tst_brk(TCONF | TERRNO, "memcg not supported");
-
- tst_brk(TCONF | TERRNO, "mounting memcg failed");
- }
- mount_flag = 1;
+ tst_cg_require("memory", &opts);
+ tst_cg_init();
+ if (TST_CG_VER(tst_cg, "memory") != TST_CG_V1)
+ SAFE_CG_PRINT(tst_cg, "cgroup.subtree_control", "+memory");
}
static void cleanup(void)
{
- if (!access(SUBDIR, F_OK))
- SAFE_RMDIR(SUBDIR);
+ if (test_cg)
+ test_cg = tst_cg_group_rm(test_cg);
- if (mount_flag)
- tst_umount(MNTPOINT);
+ tst_cg_cleanup();
}
static struct tst_test test = {
.needs_root = 1,
- .needs_tmpdir = 1,
.forks_child = 1,
.min_kver = "2.6.24",
.setup = setup,
diff --git a/testcases/kernel/controllers/memcg/regression/memcg_test_4.c b/testcases/kernel/controllers/memcg/regression/memcg_test_4.c
index d714561ed..743c84108 100644
--- a/testcases/kernel/controllers/memcg/regression/memcg_test_4.c
+++ b/testcases/kernel/controllers/memcg/regression/memcg_test_4.c
@@ -1,24 +1,6 @@
-/******************************************************************************/
-/* */
-/* Copyright (c) 2009 FUJITSU LIMITED */
-/* */
-/* This program is free software; you can redistribute it and/or modify */
-/* it under the terms of the GNU General Public License as published by */
-/* the Free Software Foundation; either version 2 of the License, or */
-/* (at your option) any later version. */
-/* */
-/* This program is distributed in the hope that it will be useful, */
-/* but WITHOUT ANY WARRANTY; without even the implied warranty of */
-/* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See */
-/* the GNU General Public License for more details. */
-/* */
-/* You should have received a copy of the GNU General Public License */
-/* along with this program; if not, write to the Free Software */
-/* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */
-/* */
-/* Author: Li Zefan <lizf@cn.fujitsu.com> */
-/* */
-/******************************************************************************/
+// SPDX-License-Identifier: GPL-2.0-or-later
+// Copyright (c) 2009 FUJITSU LIMITED
+// Author: Li Zefan <lizf@cn.fujitsu.com>
#include <sys/mman.h>
#include <err.h>
diff --git a/testcases/kernel/controllers/memcg/regression/memcg_test_4.sh b/testcases/kernel/controllers/memcg/regression/memcg_test_4.sh
index 620031366..8723abb2f 100755
--- a/testcases/kernel/controllers/memcg/regression/memcg_test_4.sh
+++ b/testcases/kernel/controllers/memcg/regression/memcg_test_4.sh
@@ -1,30 +1,21 @@
#! /bin/sh
+# SPDX-License-Identifier: GPL-2.0-or-later
+# Copyright (c) 2009 FUJITSU LIMITED
+# Author: Li Zefan <lizf@cn.fujitsu.com>
-################################################################################
-## ##
-## Copyright (c) 2009 FUJITSU LIMITED ##
-## ##
-## This program is free software; you can redistribute it and#or modify ##
-## it under the terms of the GNU General Public License as published by ##
-## the Free Software Foundation; either version 2 of the License, or ##
-## (at your option) any later version. ##
-## ##
-## This program is distributed in the hope that it will be useful, but ##
-## WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY ##
-## or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License ##
-## for more details. ##
-## ##
-## You should have received a copy of the GNU General Public License ##
-## along with this program; if not, write to the Free Software ##
-## Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA ##
-## ##
-## Author: Li Zefan <lizf@cn.fujitsu.com> ##
-## ##
-################################################################################
-
-# attach current task to memcg/0/
-mkdir memcg/0
-echo $$ > memcg/0/tasks
+cgroup_version="$1"
+mount_point="$2"
+test_path="$3"
+
+if [ "$cgroup_version" = "2" ]; then
+ task_list="cgroup.procs"
+ memory_limit="memory.max"
+else
+ task_list="tasks"
+ memory_limit="memory.limit_in_bytes"
+fi
+
+echo $$ > "$test_path/$task_list"
./memcg_test_4 &
pid=$!
@@ -35,14 +26,13 @@ sleep 1
sleep 1
# shrink memory, and then 80M will be swapped
-echo 40M > memcg/0/memory.limit_in_bytes
+echo 40M > "$test_path/$memory_limit"
# turn off swap, and swapoff will be killed
swapoff -a
sleep 1
-echo $pid > memcg/tasks 2> /dev/null
-echo $$ > memcg/tasks 2> /dev/null
+echo $pid > "$mount_point/$task_list" 2> /dev/null
+echo $$ > "$mount_point/$task_list" 2> /dev/null
# now remove the cgroup
-rmdir memcg/0
-
+rmdir "$test_path"
--
2.34.1
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [LTP] [PATCH v2 14/18] controllers: Update memcg_stress_test to use newer cgroup lib
2022-07-21 20:52 [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests Luke Nowakowski-Krijger
` (12 preceding siblings ...)
2022-07-21 20:52 ` [LTP] [PATCH v3 13/18] controllers: Update memcg/regression/* to new test " Luke Nowakowski-Krijger
@ 2022-07-21 20:52 ` Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH v2 15/18] controllers: update memcg/functional " Luke Nowakowski-Krijger
` (5 subsequent siblings)
19 siblings, 0 replies; 33+ messages in thread
From: Luke Nowakowski-Krijger @ 2022-07-21 20:52 UTC (permalink / raw)
To: ltp, rpalethorpe, liwang, pvorel, chrubis
Update the test to use the newer cgroup lib which enables the testing of
both v1 and v2 memory controllers and makes the setup and cleanup much
simpler.
Signed-off-by: Luke Nowakowski-Krijger <luke.nowakowskikrijger@canonical.com>
---
v2: Use version number 2 instead of v2
.../memcg/stress/memcg_stress_test.sh | 32 +++++++------------
1 file changed, 12 insertions(+), 20 deletions(-)
diff --git a/testcases/kernel/controllers/memcg/stress/memcg_stress_test.sh b/testcases/kernel/controllers/memcg/stress/memcg_stress_test.sh
index fa0df3196..d38c650ea 100755
--- a/testcases/kernel/controllers/memcg/stress/memcg_stress_test.sh
+++ b/testcases/kernel/controllers/memcg/stress/memcg_stress_test.sh
@@ -21,10 +21,16 @@ TST_TIMEOUT=2100
setup()
{
- if ! is_cgroup_subsystem_available_and_enabled "memory"; then
- tst_brk TCONF "Either kernel does not support Memory Resource Controller or feature not enabled"
+ cgroup_require "memory"
+ cgroup_version=$(cgroup_get_version "memory")
+ test_path=$(cgroup_get_test_path "memory")
+ task_list=$(cgroup_get_task_list "memory")
+ if [ "$cgroup_version" = "2" ]; then
+ ROD echo "+memory" > "$test_path/cgroup.subtree_control"
fi
+ tst_res TINFO "test starts with cgroup version $cgroup_version"
+
echo 3 > /proc/sys/vm/drop_caches
sleep 2
local mem_free=`cat /proc/meminfo | grep MemFree | awk '{ print $2 }'`
@@ -41,18 +47,7 @@ setup()
cleanup()
{
- if [ -e /dev/memcg ]; then
- EXPECT_PASS umount /dev/memcg
- EXPECT_PASS rmdir /dev/memcg
- fi
-}
-
-do_mount()
-{
- cleanup
-
- EXPECT_PASS mkdir /dev/memcg
- EXPECT_PASS mount -t cgroup -omemory memcg /dev/memcg
+ cgroup_cleanup
}
# $1 Number of cgroups
@@ -69,13 +64,11 @@ run_stress()
tst_res TINFO "Testing $cgroups cgroups, using $mem_size MB, interval $interval"
- do_mount
-
tst_res TINFO "Starting cgroups"
for i in $(seq 0 $(($cgroups-1))); do
- mkdir /dev/memcg/$i 2> /dev/null
+ ROD mkdir "$test_path/$i"
memcg_process_stress $mem_size $interval &
- echo $! > /dev/memcg/$i/tasks
+ ROD echo $! > "$test_path/$i/$task_list"
pids="$pids $!"
done
@@ -91,12 +84,11 @@ run_stress()
for pid in $pids; do
kill -KILL $pid 2> /dev/null
wait $pid 2> /dev/null
- rmdir /dev/memcg/$i 2> /dev/null
+ ROD rmdir "$test_path/$i"
i=$((i+1))
done
tst_res TPASS "Test passed"
- cleanup
}
test1()
--
2.34.1
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [LTP] [PATCH v2 15/18] controllers: update memcg/functional to use newer cgroup lib
2022-07-21 20:52 [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests Luke Nowakowski-Krijger
` (13 preceding siblings ...)
2022-07-21 20:52 ` [LTP] [PATCH v2 14/18] controllers: Update memcg_stress_test to use newer " Luke Nowakowski-Krijger
@ 2022-07-21 20:52 ` Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH v3 16/18] controllers: Update pids.sh " Luke Nowakowski-Krijger
` (4 subsequent siblings)
19 siblings, 0 replies; 33+ messages in thread
From: Luke Nowakowski-Krijger @ 2022-07-21 20:52 UTC (permalink / raw)
To: ltp, rpalethorpe, liwang, pvorel, chrubis
Update memcg_lib.sh to uses the newer cgroup lib to cleanup the mounting
and checking of the memory controller.
There are still some tests that make sense for v2 and should be modified
in a future patch, but since most of the tests are testing specific v1
memory controller features lets just skip it for now if v2 is mounted.
Signed-off-by: Luke Nowakowski-Krijger <luke.nowakowskikrijger@canonical.com>
---
v2: Use version number 2 instead of v2
.../memcg/functional/memcg_force_empty.sh | 2 +-
.../controllers/memcg/functional/memcg_lib.sh | 54 ++++++++++---------
2 files changed, 30 insertions(+), 26 deletions(-)
diff --git a/testcases/kernel/controllers/memcg/functional/memcg_force_empty.sh b/testcases/kernel/controllers/memcg/functional/memcg_force_empty.sh
index 23a4bd04d..580468784 100755
--- a/testcases/kernel/controllers/memcg/functional/memcg_force_empty.sh
+++ b/testcases/kernel/controllers/memcg/functional/memcg_force_empty.sh
@@ -50,7 +50,7 @@ test6()
{
# writing to non-empty top mem cgroup's force_empty
# should return failure
- EXPECT_FAIL echo 1 \> /dev/memcg/memory.force_empty
+ EXPECT_FAIL echo 1 \> "$mount_point/memory.force_empty"
}
. memcg_lib.sh
diff --git a/testcases/kernel/controllers/memcg/functional/memcg_lib.sh b/testcases/kernel/controllers/memcg/functional/memcg_lib.sh
index 96d26ef2d..a89e24485 100755
--- a/testcases/kernel/controllers/memcg/functional/memcg_lib.sh
+++ b/testcases/kernel/controllers/memcg/functional/memcg_lib.sh
@@ -16,20 +16,20 @@ TST_TESTFUNC=memcg_testfunc
memcg_require_memsw()
{
- if ! [ -e /dev/memcg/memory.limit_in_bytes ]; then
- tst_brk TBROK "/dev/memcg must be mounted before calling memcg_require_memsw"
+ if ! [ -e "$mount_point/memory.limit_in_bytes" ]; then
+ tst_brk TBROK "$mount_point must be mounted before calling memcg_require_memsw"
fi
- if ! [ -e /dev/memcg/memory.memsw.limit_in_bytes ]; then
+ if ! [ -e "$mount_point/memory.memsw.limit_in_bytes" ]; then
tst_brk TCONF "mem+swap is not enabled"
fi
}
memcg_require_hierarchy_disabled()
{
- if [ ! -e "/dev/memcg/memory.use_hierarchy" ]; then
- tst_brk TBROK "/dev/memcg must be mounted before calling memcg_require_hierarchy_disabled"
+ if [ ! -e "$mount_point/memory.use_hierarchy" ]; then
+ tst_brk TBROK "$mount_point must be mounted before calling memcg_require_hierarchy_disabled"
fi
- if [ $(cat /dev/memcg/memory.use_hierarchy) -eq 1 ]; then
+ if [ "$(cat "$mount_point/memory.use_hierarchy")" -eq 1 ]; then
tst_brk TCONF "Test requires root cgroup memory.use_hierarchy=0"
fi
}
@@ -71,12 +71,19 @@ memcg_adjust_limit_for_kmem()
memcg_setup()
{
- if ! is_cgroup_subsystem_available_and_enabled "memory"; then
- tst_brk TCONF "Either kernel does not support Memory Resource Controller or feature not enabled"
+ cgroup_require "memory"
+ cgroup_version=$(cgroup_get_version "memory")
+
+ # Most of the tests here are testing specific parts of the cgroup v1 memory interface that is
+ # not present for cgroup2, so if it is already mounted on a cgroup v2 hierarchy we should skip
+ # the test.
+ # Some tests still make sense in v2 and should be modified in a future patch
+ if [ "$cgroup_version" = "2" ]; then
+ tst_brk TCONF "memory controller mounted on cgroup v2 hierarchy, skipping test."
fi
- ROD mkdir /dev/memcg
- ROD mount -t cgroup -omemory memcg /dev/memcg
+ mount_point=$(cgroup_get_mountpoint "memory")
+ test_dir=$(cgroup_get_test_path "memory")
# For kernels older than v5.11 the default value for
# memory.use_hierarchy is 0 and some of tests (memcg_stat_test.sh and
@@ -89,15 +96,15 @@ memcg_setup()
# Starting with kernel v5.11, the non-hierarchical mode is not
# available. See Linux kernel commit bef8620cd8e0 ("mm: memcg:
# deprecate the non-hierarchical mode").
- orig_memory_use_hierarchy=$(cat /dev/memcg/memory.use_hierarchy)
+ orig_memory_use_hierarchy=$(cat "$mount_point/memory.use_hierarchy")
if [ -z "$orig_memory_use_hierarchy" ];then
- tst_res TINFO "cat /dev/memcg/ failed"
+ tst_res TINFO "cat $mount_point failed"
elif [ "$orig_memory_use_hierarchy" = "0" ];then
orig_memory_use_hierarchy=""
else
- echo 0 > /dev/memcg/memory.use_hierarchy 2>/dev/null
+ echo 0 > "$mount_point/memory.use_hierarchy" 2>/dev/null
if [ $? -ne 0 ];then
- tst_res TINFO "set /dev/memcg/memory.use_hierarchy to 0 failed"
+ tst_res TINFO "set $mount_point/memory.use_hierarchy to 0 failed"
fi
fi
@@ -110,22 +117,19 @@ memcg_cleanup()
cd $TST_TMPDIR
# In order to remove all subgroups, we have to remove them recursively
- if [ -e /dev/memcg/ltp_$$ ]; then
- ROD find /dev/memcg/ltp_$$ -depth -type d -delete
+ if [ -e $test_dir ]; then
+ ROD find $test_dir -depth -type d -delete
fi
if [ -n "$orig_memory_use_hierarchy" ];then
- echo $orig_memory_use_hierarchy > /dev/memcg/memory.use_hierarchy
+ echo $orig_memory_use_hierarchy > $mount_point/memory.use_hierarchy
if [ $? -ne 0 ];then
- tst_res TINFO "restore /dev/memcg/memory.use_hierarchy failed"
+ tst_res TINFO "restore $mount_point/memory.use_hierarchy failed"
fi
orig_memory_use_hierarchy=""
fi
- if [ -e "/dev/memcg" ]; then
- umount /dev/memcg
- rmdir /dev/memcg
- fi
+ cgroup_cleanup
[ "$MEMCG_SHMMAX" = "1" ] && shmmax_cleanup
}
@@ -369,8 +373,8 @@ test_limit_in_bytes()
memcg_testfunc()
{
- ROD mkdir /dev/memcg/ltp_$$
- cd /dev/memcg/ltp_$$
+ ROD mkdir $test_dir/ltp_$$
+ cd $test_dir/ltp_$$
if type ${MEMCG_TESTFUNC}1 > /dev/null 2>&1; then
${MEMCG_TESTFUNC}$1 $1 "$2"
@@ -379,7 +383,7 @@ memcg_testfunc()
fi
cd $TST_TMPDIR
- ROD rmdir /dev/memcg/ltp_$$
+ ROD rmdir $test_dir/ltp_$$
}
memcg_no_testfunc()
--
2.34.1
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [LTP] [PATCH v3 16/18] controllers: Update pids.sh to use newer cgroup lib
2022-07-21 20:52 [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests Luke Nowakowski-Krijger
` (14 preceding siblings ...)
2022-07-21 20:52 ` [LTP] [PATCH v2 15/18] controllers: update memcg/functional " Luke Nowakowski-Krijger
@ 2022-07-21 20:52 ` Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH v2 17/18] controllers: update cpuset_regression_test.sh " Luke Nowakowski-Krijger
` (3 subsequent siblings)
19 siblings, 0 replies; 33+ messages in thread
From: Luke Nowakowski-Krijger @ 2022-07-21 20:52 UTC (permalink / raw)
To: ltp, rpalethorpe, liwang, pvorel, chrubis
Updated to use the newer cgroup_lib to make mounting and cleanup
nicer.
Signed-off-by: Luke Nowakowski-Krijger <luke.nowakowskikrijger@canonical.com>
---
v2: Use version number 2 instead of v2
v3: put cgroup_lib.sh include in bottom of file
testcases/kernel/controllers/pids/pids.sh | 67 +++--------------------
1 file changed, 9 insertions(+), 58 deletions(-)
diff --git a/testcases/kernel/controllers/pids/pids.sh b/testcases/kernel/controllers/pids/pids.sh
index e8bafc739..26a0c6a22 100755
--- a/testcases/kernel/controllers/pids/pids.sh
+++ b/testcases/kernel/controllers/pids/pids.sh
@@ -36,66 +36,17 @@ cleanup()
{
killall -9 pids_task2 >/dev/null 2>&1
- tst_res TINFO "removing created directories"
- rmdir $testpath
- if [ "$mounted" -ne "1" ]; then
- tst_res TINFO "Umounting pids"
- umount $mount_point
- rmdir $mount_point
- fi
-}
-
-setup_cgroupv2()
-{
- mount_point=$(grep -w cgroup2 /proc/mounts | cut -f 2 | cut -d " " -f2)
- if ! grep -q pids "$mount_point"/cgroup.controllers; then
- tst_res TINFO "pids not supported on cgroup v2."
- return
- fi
-
- testpath="$mount_point/ltp_pids_$caseno"
- ROD mkdir -p "$testpath"
- task_list="cgroup.procs"
- cgroup_v="v2"
-}
-
-setup_cgroupv1()
-{
- exist=`grep -w pids /proc/cgroups | cut -f1`;
- if [ "$exist" = "" ]; then
- tst_brk TCONF NULL "pids not supported"
- fi
-
- mount_point=`grep -w pids /proc/mounts | cut -f 2 | cut -d " " -f2`
-
- if [ "$mount_point" = "" ]; then
- mounted=0
- mount_point=/dev/cgroup
- fi
-
- testpath=$mount_point/ltp_pids_$caseno
-
- if [ "$mounted" -eq "0" ]; then
- ROD mkdir -p $mount_point
- ROD mount -t cgroup -o pids none $mount_point
- fi
- ROD mkdir -p $testpath
- task_list="tasks"
- cgroup_v="v1"
+ cgroup_cleanup
}
setup()
{
- # If cgroup2 is mounted already, then let's
- # try to start with cgroup v2.
- if grep -q cgroup2 /proc/mounts; then
- setup_cgroupv2
- fi
- if [ -z "$cgroup_v" ]; then
- setup_cgroupv1
- fi
+ cgroup_require "pids"
+ cgroup_version=$(cgroup_get_version "pids")
+ testpath=$(cgroup_get_test_path "pids")
+ task_list=$(cgroup_get_task_list "pids")
- tst_res TINFO "test starts with cgroup $cgroup_v"
+ tst_res TINFO "test starts with cgroup version $cgroup_version"
}
start_pids_tasks2()
@@ -296,7 +247,7 @@ case8()
{
tst_res TINFO "set child cgroup limit smaller than its parent limit"
ROD echo $max \> $testpath/pids.max
- if [ "$cgroup_v" = "v2" ]; then
+ if [ "$cgroup_version" = "2" ]; then
ROD echo +pids \> "$testpath"/cgroup.subtree_control
fi
mkdir $testpath/child
@@ -326,7 +277,7 @@ case9()
tst_res TINFO "migrate cgroup"
lim=$((max - 1))
- if [ "$cgroup_v" = "v2" ]; then
+ if [ "$cgroup_version" = "2" ]; then
ROD echo +pids \> "$testpath"/cgroup.subtree_control
fi
for i in 1 2; do
@@ -391,5 +342,5 @@ do_test()
case$caseno
}
-. tst_test.sh
+. cgroup_lib.sh
tst_run
--
2.34.1
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [LTP] [PATCH v2 17/18] controllers: update cpuset_regression_test.sh to use newer cgroup lib
2022-07-21 20:52 [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests Luke Nowakowski-Krijger
` (15 preceding siblings ...)
2022-07-21 20:52 ` [LTP] [PATCH v3 16/18] controllers: Update pids.sh " Luke Nowakowski-Krijger
@ 2022-07-21 20:52 ` Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH v3 18/18] controllers: update cgroup_regression_test " Luke Nowakowski-Krijger
` (2 subsequent siblings)
19 siblings, 0 replies; 33+ messages in thread
From: Luke Nowakowski-Krijger @ 2022-07-21 20:52 UTC (permalink / raw)
To: ltp, rpalethorpe, liwang, pvorel, chrubis
Update the test to use the newer cgroup lib which handles mounting for
v1 and v2 controllers enabling them both to be tested and cleaning up
the mounting and cleanup process.
Signed-off-by: Luke Nowakowski-Krijger <luke.nowakowskikrijger@canonical.com>
---
v2: Change variable cgroup_v to cgroup_version
.../cpuset/cpuset_regression_test.sh | 26 +++++--------------
1 file changed, 7 insertions(+), 19 deletions(-)
diff --git a/testcases/kernel/controllers/cpuset/cpuset_regression_test.sh b/testcases/kernel/controllers/cpuset/cpuset_regression_test.sh
index 68fcf2eec..8e87d20e4 100755
--- a/testcases/kernel/controllers/cpuset/cpuset_regression_test.sh
+++ b/testcases/kernel/controllers/cpuset/cpuset_regression_test.sh
@@ -122,18 +122,13 @@ cpuset_restore()
setup()
{
- if ! is_cgroup_subsystem_available_and_enabled "cpuset"; then
- tst_brk TCONF "Either kernel does not support cpuset controller or feature not enabled"
- fi
-
- # We need to mount cpuset if it is not found.
- root_cpuset_dir=$(get_cgroup_mountpoint cpuset)
- if [ -z "$root_cpuset_dir" ]; then
- root_cpuset_dir="$LOCAL_MOUNTPOINT"
+ cgroup_require "cpuset"
+ cgroup_version=$(cgroup_get_version "cpuset")
+ root_cpuset_dir=$(cgroup_get_mountpoint "cpuset")
+ testpath=$(cgroup_get_test_path "cpuset")
+ task_list=$(cgroup_get_task_list "cpuset")
- ROD_SILENT mkdir -p ${root_cpuset_dir}
- ROD_SILENT mount -t cpuset cpuset ${root_cpuset_dir}
- fi
+ tst_res TINFO "test starts with cgroup version $cgroup_version"
if ! [ -f ${root_cpuset_dir}/${cpu_exclusive} ]; then
cpu_exclusive=cpu_exclusive
@@ -180,14 +175,7 @@ cleanup()
echo ${old_cpu_exclusive_value} > ${root_cpuset_dir}/${cpu_exclusive}
fi
- if [ -d "$LOCAL_MOUNTPOINT" ]; then
- umount ${LOCAL_MOUNTPOINT}
- if [ $? -ne 0 ]; then
- tst_res TWARN "'umount ${LOCAL_MOUNTPOINT}' failed"
- fi
-
- rmdir ${LOCAL_MOUNTPOINT}
- fi
+ cgroup_cleanup
}
test()
--
2.34.1
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [LTP] [PATCH v3 18/18] controllers: update cgroup_regression_test to use newer cgroup lib
2022-07-21 20:52 [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests Luke Nowakowski-Krijger
` (16 preceding siblings ...)
2022-07-21 20:52 ` [LTP] [PATCH v2 17/18] controllers: update cpuset_regression_test.sh " Luke Nowakowski-Krijger
@ 2022-07-21 20:52 ` Luke Nowakowski-Krijger
2022-07-26 3:33 ` Li Wang
2022-07-25 8:14 ` [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests Petr Vorel
2022-07-26 6:10 ` Li Wang
19 siblings, 1 reply; 33+ messages in thread
From: Luke Nowakowski-Krijger @ 2022-07-21 20:52 UTC (permalink / raw)
To: ltp, rpalethorpe, liwang, pvorel, chrubis
The older function in the cgroup lib 'get_cgroup_mountpoint' has been
removed, so instead replace it with its old functionaility to get
mountpoint.
Also use the newer cgroup lib require operation to mount and cleanup a
cpu controller.
Signed-off-by: Luke Nowakowski-Krijger <luke.nowakowskikrijger@canonical.com>
---
v2: Use cgroup_require "$subsys" instead of manual greping the path
v3: fixed bug where the ouput of cgroup_get_mountpoint wasnt being
captured
.../cgroup/cgroup_regression_test.sh | 31 ++++++-------------
1 file changed, 9 insertions(+), 22 deletions(-)
diff --git a/testcases/kernel/controllers/cgroup/cgroup_regression_test.sh b/testcases/kernel/controllers/cgroup/cgroup_regression_test.sh
index 69b51773c..bfa9097ec 100755
--- a/testcases/kernel/controllers/cgroup/cgroup_regression_test.sh
+++ b/testcases/kernel/controllers/cgroup/cgroup_regression_test.sh
@@ -168,17 +168,8 @@ test3()
return
fi
- cpu_subsys_path=$(get_cgroup_mountpoint "cpu")
-
- # Run the test for 30 secs
- if [ -z "$cpu_subsys_path" ]; then
- mount -t cgroup -o cpu xxx cgroup/
- if [ $? -ne 0 ]; then
- tst_res TFAIL "Failed to mount cpu subsys"
- return
- fi
- cpu_subsys_path=cgroup
- fi
+ cgroup_require "cpu"
+ cpu_subsys_path=$(cgroup_get_mountpoint "cpu")
cgroup_regression_3_1.sh $cpu_subsys_path &
pid1=$!
@@ -191,7 +182,7 @@ test3()
wait $pid2 2>/dev/null
rmdir $cpu_subsys_path/0 2> /dev/null
- tst_umount $PWD/cgroup
+ cgroup_cleanup
check_kernel_bug
}
@@ -303,21 +294,15 @@ test6()
test_7_1()
{
local subsys=$1
+ local subsys_path
# we should be careful to select a $subsys_path which is related to
# cgroup only: if cgroup debugging is enabled a 'debug' $subsys
# could be passed here as params and this will lead to ambiguity and
# errors when grepping simply for 'debug' in /proc/mounts since we'll
# find also /sys/kernel/debug. Helper takes care of this.
- local subsys_path=$(get_cgroup_mountpoint $subsys)
-
- if [ -z "$subsys_path" ]; then
- mount -t cgroup -o $subsys xxx cgroup/
- if [ $? -ne 0 ]; then
- tst_res TFAIL "failed to mount $subsys"
- return
- fi
- subsys_path=cgroup
- fi
+
+ cgroup_require "$subsys"
+ subsys_path=$(cgroup_get_mountpoint "$subsys")
mkdir $subsys_path/0
sleep 100 < $subsys_path/0 & # add refcnt to this dir
@@ -332,6 +317,8 @@ test_7_1()
wait $! 2>/dev/null
umount cgroup/
fi
+
+ cgroup_cleanup
}
test_7_2()
--
2.34.1
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply related [flat|nested] 33+ messages in thread
* Re: [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests
2022-07-21 20:52 [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests Luke Nowakowski-Krijger
` (17 preceding siblings ...)
2022-07-21 20:52 ` [LTP] [PATCH v3 18/18] controllers: update cgroup_regression_test " Luke Nowakowski-Krijger
@ 2022-07-25 8:14 ` Petr Vorel
2022-07-25 20:25 ` Luke Nowakowski-Krijger
2022-07-26 6:10 ` Li Wang
19 siblings, 1 reply; 33+ messages in thread
From: Petr Vorel @ 2022-07-25 8:14 UTC (permalink / raw)
To: Luke Nowakowski-Krijger; +Cc: ltp
Hi Luke,
could you please next time use the same version number for all patches?
Having different versions (patches have v2, v3, v5) is quite confusing.
Kind regards,
Petr
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [LTP] [PATCH v3 11/18] controllers: Update cgroup_fj_* to use newer cgroup lib and test lib
2022-07-21 20:52 ` [LTP] [PATCH v3 11/18] controllers: Update cgroup_fj_* to use newer cgroup lib and test lib Luke Nowakowski-Krijger
@ 2022-07-25 11:41 ` Richard Palethorpe
2022-07-26 6:36 ` Li Wang
0 siblings, 1 reply; 33+ messages in thread
From: Richard Palethorpe @ 2022-07-25 11:41 UTC (permalink / raw)
To: Luke Nowakowski-Krijger; +Cc: ltp
Luke Nowakowski-Krijger <luke.nowakowskikrijger@canonical.com> writes:
> +setup()
> +{
> + common_setup
This can fail and we have not set pid yet.
> + cgroup_fj_proc&
> + pid=$!
> + create_subgroup "$start_path/ltp_1"
> +}
>
> -ROD kill -9 $pid
> -wait $pid
> -ROD rmdir "$start_path/ltp_1"
> +cleanup()
> +{
> + kill -9 $pid >/dev/null 2>&1
> + wait $pid >/dev/null 2>&1
$pid is unset so this just waits forever. We need something like the
following:
if [ -n "$pid" ]; then
kill -9 $pid >/dev/null 2>&1
wait $pid >/dev/null 2>&1
fi
> + rmdir "$start_path/ltp_1" >/dev/null 2>&1
> + common_cleanup
> +}
>
--
Thank you,
Richard.
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests
2022-07-25 8:14 ` [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests Petr Vorel
@ 2022-07-25 20:25 ` Luke Nowakowski-Krijger
2022-07-26 13:00 ` Petr Vorel
0 siblings, 1 reply; 33+ messages in thread
From: Luke Nowakowski-Krijger @ 2022-07-25 20:25 UTC (permalink / raw)
To: Petr Vorel; +Cc: LTP List
[-- Attachment #1.1: Type: text/plain, Size: 462 bytes --]
Hi Petr,
On Mon, Jul 25, 2022 at 1:15 AM Petr Vorel <pvorel@suse.cz> wrote:
> Hi Luke,
>
> could you please next time use the same version number for all patches?
> Having different versions (patches have v2, v3, v5) is quite confusing.
>
> Kind regards,
> Petr
>
Yeah I agree..
It was supposed to make it easier to not have to re review stuff when the
version hasn't changed, but it gets quite confusing.
I'll use the same version number next time.
- Luke
[-- Attachment #1.2: Type: text/html, Size: 945 bytes --]
[-- Attachment #2: Type: text/plain, Size: 60 bytes --]
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [LTP] [PATCH v3 18/18] controllers: update cgroup_regression_test to use newer cgroup lib
2022-07-21 20:52 ` [LTP] [PATCH v3 18/18] controllers: update cgroup_regression_test " Luke Nowakowski-Krijger
@ 2022-07-26 3:33 ` Li Wang
0 siblings, 0 replies; 33+ messages in thread
From: Li Wang @ 2022-07-26 3:33 UTC (permalink / raw)
To: Luke Nowakowski-Krijger; +Cc: LTP List
[-- Attachment #1.1: Type: text/plain, Size: 1926 bytes --]
On Fri, Jul 22, 2022 at 4:53 AM Luke Nowakowski-Krijger <
luke.nowakowskikrijger@canonical.com> wrote:
> The older function in the cgroup lib 'get_cgroup_mountpoint' has been
> removed, so instead replace it with its old functionaility to get
> mountpoint.
>
> Also use the newer cgroup lib require operation to mount and cleanup a
> cpu controller.
>
> Signed-off-by: Luke Nowakowski-Krijger <
> luke.nowakowskikrijger@canonical.com>
> ---
> v2: Use cgroup_require "$subsys" instead of manual greping the path
> v3: fixed bug where the ouput of cgroup_get_mountpoint wasnt being
> captured
>
> .../cgroup/cgroup_regression_test.sh | 31 ++++++-------------
> 1 file changed, 9 insertions(+), 22 deletions(-)
>
> diff --git a/testcases/kernel/controllers/cgroup/cgroup_regression_test.sh
> b/testcases/kernel/controllers/cgroup/cgroup_regression_test.sh
> index 69b51773c..bfa9097ec 100755
> --- a/testcases/kernel/controllers/cgroup/cgroup_regression_test.sh
> +++ b/testcases/kernel/controllers/cgroup/cgroup_regression_test.sh
>
This patch itself looks good.
Note:
But I got one failure from test7 that was caused by randomly picking up
'rdma' ctrl to test.
tst_cgroup.c:820: TBROK: 'rdma' controller is unknown to LTP
cgroup_regression_test 7 TBROK: cgroup_require: No state was set after call
to tst_cgctl require?
# cat -n cgroup_regression_test.sh
359 test7()
360 {
...
371 subsys=`tail -n 1 /proc/cgroups | awk '{ print $1 }'`
379 }
Because on my aarch64 (Cgroup V2) test box, the last one in /proc/cgroups:
# cat /proc/cgroups
#subsys_name hierarchy num_cgroups enabled
cpuset 3 3 1
cpu 2 82 1
cpuacct 2 82 1
blkio 9 82 1
memory 12 288 1
devices 4 82 1
freezer 11 3 1
net_cls 6 3 1
perf_event 8 3 1
net_prio 6 3 1
hugetlb 10 3 1
pids 5 110 1
rdma 7 1 1
Anyway, we can fix this by picking a supported cgroup ctrl in a separate
patch.
--
Regards,
Li Wang
[-- Attachment #1.2: Type: text/html, Size: 3772 bytes --]
[-- Attachment #2: Type: text/plain, Size: 60 bytes --]
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests
2022-07-21 20:52 [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests Luke Nowakowski-Krijger
` (18 preceding siblings ...)
2022-07-25 8:14 ` [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests Petr Vorel
@ 2022-07-26 6:10 ` Li Wang
2022-07-26 8:52 ` Richard Palethorpe
19 siblings, 1 reply; 33+ messages in thread
From: Li Wang @ 2022-07-26 6:10 UTC (permalink / raw)
To: Luke Nowakowski-Krijger; +Cc: LTP List
[-- Attachment #1.1: Type: text/plain, Size: 484 bytes --]
Hi All,
I have completed my review (and test) on this patch set, generally looks
good!
With regard to the tiny issues, I think Luke you can just against the
concrete
problem to follow up and fix it. No need to resend all patches with a new
version.
I would suggest merging this patchset at present (in case of blocking
other's (or duplicate) work).
CI builds pass: https://github.com/wangli5665/ltp/runs/7460955660
Reviewed-by: Li Wang <liwang@redhat.com>
--
Regards,
Li Wang
[-- Attachment #1.2: Type: text/html, Size: 1528 bytes --]
[-- Attachment #2: Type: text/plain, Size: 60 bytes --]
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [LTP] [PATCH v3 11/18] controllers: Update cgroup_fj_* to use newer cgroup lib and test lib
2022-07-25 11:41 ` Richard Palethorpe
@ 2022-07-26 6:36 ` Li Wang
0 siblings, 0 replies; 33+ messages in thread
From: Li Wang @ 2022-07-26 6:36 UTC (permalink / raw)
To: Richard Palethorpe; +Cc: LTP List, Luke Nowakowski-Krijger
[-- Attachment #1.1: Type: text/plain, Size: 783 bytes --]
On Mon, Jul 25, 2022 at 7:44 PM Richard Palethorpe <rpalethorpe@suse.de>
wrote:
>
> Luke Nowakowski-Krijger <luke.nowakowskikrijger@canonical.com> writes:
>
> > +setup()
> > +{
> > + common_setup
>
> This can fail and we have not set pid yet.
>
> > + cgroup_fj_proc&
> > + pid=$!
> > + create_subgroup "$start_path/ltp_1"
> > +}
> >
> > -ROD kill -9 $pid
> > -wait $pid
> > -ROD rmdir "$start_path/ltp_1"
> > +cleanup()
> > +{
> > + kill -9 $pid >/dev/null 2>&1
> > + wait $pid >/dev/null 2>&1
>
> $pid is unset so this just waits forever. We need something like the
> following:
>
> if [ -n "$pid" ]; then
> kill -9 $pid >/dev/null 2>&1
> wait $pid >/dev/null 2>&1
> fi
>
>
+1 Someone who merges patch can help fix this.
--
Regards,
Li Wang
[-- Attachment #1.2: Type: text/html, Size: 1637 bytes --]
[-- Attachment #2: Type: text/plain, Size: 60 bytes --]
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests
2022-07-26 6:10 ` Li Wang
@ 2022-07-26 8:52 ` Richard Palethorpe
2022-07-26 16:04 ` Petr Vorel
0 siblings, 1 reply; 33+ messages in thread
From: Richard Palethorpe @ 2022-07-26 8:52 UTC (permalink / raw)
To: Li Wang; +Cc: LTP List, Luke Nowakowski-Krijger
Hello,
Li Wang <liwang@redhat.com> writes:
> Hi All,
>
> I have completed my review (and test) on this patch set, generally looks good!
>
> With regard to the tiny issues, I think Luke you can just against the concrete
> problem to follow up and fix it. No need to resend all patches with a new version.
>
> I would suggest merging this patchset at present (in case of blocking other's (or duplicate) work).
>
> CI builds pass: https://github.com/wangli5665/ltp/runs/7460955660
>
> Reviewed-by: Li Wang <liwang@redhat.com>
I'm also very keen to get this merged and can make the necessary
fixups. However I will give Petr Vorel chance to look at the shell
tests.
Reviewed-by: Richard Palethorpe <rpalethorpe@suse.com>
--
Thank you,
Richard.
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [LTP] [PATCH v2 09/18] testcases/lib: Implement tst_cgctl binary
2022-07-21 20:52 ` [LTP] [PATCH v2 09/18] testcases/lib: Implement tst_cgctl binary Luke Nowakowski-Krijger
@ 2022-07-26 12:48 ` Petr Vorel
0 siblings, 0 replies; 33+ messages in thread
From: Petr Vorel @ 2022-07-26 12:48 UTC (permalink / raw)
To: Luke Nowakowski-Krijger; +Cc: ltp
> Implement a binary utility that creates an interface to make calls to
> the cgroup C API.
> This will effectively allow shell scripts to make calls to the cgroup C
> api.
> Signed-off-by: Luke Nowakowski-Krijger <luke.nowakowskikrijger@canonical.com>
> ---
> v2: Add license identifier and copyright.
> Reformat with tabs instead of spaces.
> Add help format message and help function.
> Add error gotos to streamline error messaging.
> testcases/lib/Makefile | 2 +-
> testcases/lib/tst_cgctl.c | 87 +++++++++++++++++++++++++++++++++++++++
> 2 files changed, 88 insertions(+), 1 deletion(-)
> create mode 100644 testcases/lib/tst_cgctl.c
> diff --git a/testcases/lib/Makefile b/testcases/lib/Makefile
> index f2de0c832..f4f8c8524 100644
> --- a/testcases/lib/Makefile
> +++ b/testcases/lib/Makefile
> @@ -12,6 +12,6 @@ MAKE_TARGETS := tst_sleep tst_random tst_checkpoint tst_rod tst_kvcmp\
> tst_device tst_net_iface_prefix tst_net_ip_prefix tst_net_vars\
> tst_getconf tst_supported_fs tst_check_drivers tst_get_unused_port\
> tst_get_median tst_hexdump tst_get_free_pids tst_timeout_kill\
> - tst_check_kconfigs
> + tst_check_kconfigs tst_cgctl
> include $(top_srcdir)/include/mk/generic_leaf_target.mk
> diff --git a/testcases/lib/tst_cgctl.c b/testcases/lib/tst_cgctl.c
> new file mode 100644
> index 000000000..4f4fe8542
> --- /dev/null
> +++ b/testcases/lib/tst_cgctl.c
> @@ -0,0 +1,87 @@
> +// SPDX-License-Identifier: GPL-2.0-or-later
> +/*
> + * Copyright (c) 2022 Canonical Ltd.
> + */
> +
> +#include <stdio.h>
> +#include <string.h>
> +#include <stdlib.h>
> +#include <stdint.h>
> +#include <unistd.h>
> +#include "tst_cgroup.h"
> +
> +#define USAGE "Usage: tst_cgctl require [controller] [test_pid]\n\
> + \t cleanup [config (output of tst_cg_print_config)]\n\
> + \t print\n\
> + \t help\n"
nit: instead of define I'd create function usage().
I'd print always to stderr, just exit code would be different.
(we print tst_res/tst_brk output to stderr, thus it's better to use it).
Also checkpatch does not like it:
$ cd testcases/lib/ && make check-tst_cgctl
CHECK testcases/lib/tst_cgctl.c
tst_cgctl.c:13: WARNING: Avoid line continuations in quoted strings
The rest LGTM.
Reviewed-by: Petr Vorel <pvorel@suse.cz>
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests
2022-07-25 20:25 ` Luke Nowakowski-Krijger
@ 2022-07-26 13:00 ` Petr Vorel
0 siblings, 0 replies; 33+ messages in thread
From: Petr Vorel @ 2022-07-26 13:00 UTC (permalink / raw)
To: Luke Nowakowski-Krijger; +Cc: LTP List
> Hi Petr,
> On Mon, Jul 25, 2022 at 1:15 AM Petr Vorel <pvorel@suse.cz> wrote:
> > Hi Luke,
> > could you please next time use the same version number for all patches?
> > Having different versions (patches have v2, v3, v5) is quite confusing.
> > Kind regards,
> > Petr
> Yeah I agree..
> It was supposed to make it easier to not have to re review stuff when the
> version hasn't changed, but it gets quite confusing.
> I'll use the same version number next time.
Thanks!
NOTE: If you want to increase readability, just add "Changes vN+1->vN:" into cover
letter (or to patches themselves).
Kind regards,
Petr
> - Luke
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [LTP] [PATCH v5 10/18] controllers: Expand cgroup_lib shell library
2022-07-21 20:52 ` [LTP] [PATCH v5 10/18] controllers: Expand cgroup_lib shell library Luke Nowakowski-Krijger
@ 2022-07-26 13:12 ` Petr Vorel
2022-07-26 19:24 ` Luke Nowakowski-Krijger
0 siblings, 1 reply; 33+ messages in thread
From: Petr Vorel @ 2022-07-26 13:12 UTC (permalink / raw)
To: Luke Nowakowski-Krijger; +Cc: ltp
Hi Luke,
...
> +# Gets the cgroup version of the given controller
> +# USAGE: cgroup_get_version CONTROLLER
> +# RETURNS: "1" if version 1 and "2" if version 2
> +# Must call cgroup_require before calling
> +cgroup_get_version()
> {
> - local subsystem=$1
> - local mntpoint
> + local ctrl="$1"
> + local version
> - [ $# -eq 0 ] && tst_brk TBROK "get_cgroup_mountpoint: subsystem not defined"
> + [ $# -eq 0 ] && tst_brk TBROK "cgroup_get_version: controller not defined"
NOTE: this will always pass, because you pass variable in ""
(thus $1 = "" and $# = 1):
cgroup_get_task_list()
{
local ctrl="$1"
version=$(cgroup_get_version "$ctrl")
> + [ "$_cgroup_state" = "" ] && tst_brk TBROK "cgroup_get_version: No previous state found. Forgot to call cgroup_require?"
> - mntpoint=$(grep cgroup /proc/mounts | grep -w $subsystem | awk '{ print $2 }')
> - [ -z "$mntpoint" ] && return 1
> + version=$(echo "$_cgroup_state" | grep -w "^$ctrl" | awk '{ print $2 }')
> + [ "$version" = "" ] && tst_brk TBROK "cgroup_get_version: Could not find controller $ctrl"
> +
> + echo "$version"
> - echo $mntpoint
> return 0
> }
...
> +# Mounts and configures the given controller
> +# USAGE: cgroup_require CONTROLLER
> +cgroup_require()
> +{
> + local ctrl="$1"
> +
> + [ $# -eq 0 ] && tst_brk TBROK "cgroup_require: controller not defined"
> +
> + [ ! -f /proc/cgroups ] && tst_brk TCONF "Kernel does not support control groups"
> +
> + _cgroup_state=$(tst_cgctl require "$ctrl" $$)
> +
> + if [ $? -eq 32 ]; then
> + tst_brk TCONF "'tst_cgctl require' exited. Controller is probably not available?"
> + fi
> +
> + if [ $? -ne 0 ]; then
> + tst_brk TBROK "'tst_cgctl require' exited."
> + fi
FYI if cgroup_require is called from cleanup function tst_brk does not exit the
code:
tst_brk()
{
local res=$1
shift
if [ "$TST_DO_EXIT" = 1 ]; then
tst_res TWARN "$@"
return
fi
tst_res "$res" "$@"
_tst_do_exit
}
IMHO that means that $? became 0 even it was previously 32.
It's always safer to save $? into local variable if needed to store exit code
(otherwise using if, e.g. "if ! foo; then" is preferred).
NOTE: Maybe at this point it might be safer if you post next version
where you do fixes yourself. I'll try to review the rest of the shell scripts
today (C code looks correct to me).
Kind regards,
Petr
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [LTP] [PATCH v2 08/18] API/cgroup: refuse to mount blkio when io controller is mounted
2022-07-21 20:52 ` [LTP] [PATCH v2 08/18] API/cgroup: refuse to mount blkio when io controller is mounted Luke Nowakowski-Krijger
@ 2022-07-26 14:40 ` Petr Vorel
0 siblings, 0 replies; 33+ messages in thread
From: Petr Vorel @ 2022-07-26 14:40 UTC (permalink / raw)
To: Luke Nowakowski-Krijger; +Cc: ltp
Hi all,
I've merged C changes, i.e. patches up to this one.
Luke, could you please send rebased new version of the rest?
Or, if you're too busy let us know and we'll do the fixes before merge.
Kind regards,
Petr
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [LTP] [PATCH v2 05/18] tst_test_macros: Add TST_TO_STR and TST_STR macro
2022-07-21 20:52 ` [LTP] [PATCH v2 05/18] tst_test_macros: Add TST_TO_STR and TST_STR macro Luke Nowakowski-Krijger
@ 2022-07-26 14:44 ` Petr Vorel
0 siblings, 0 replies; 33+ messages in thread
From: Petr Vorel @ 2022-07-26 14:44 UTC (permalink / raw)
To: Luke Nowakowski-Krijger; +Cc: ltp
Hi all,
> include/tst_test_macros.h | 3 +++
...
> +#define TST_STR(s) #s
> +#define TST_TO_STR(s) TST_STR(s)
NOTE: I changed before merging TST_STR to TST_TO_STR_ as Cyril requested [1].
I'm sorry I forget to update git commit message.
Kind regards,
Petr
[1] https://lore.kernel.org/ltp/YnPSgLxK2jUGndIv@yuki/
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests
2022-07-26 8:52 ` Richard Palethorpe
@ 2022-07-26 16:04 ` Petr Vorel
0 siblings, 0 replies; 33+ messages in thread
From: Petr Vorel @ 2022-07-26 16:04 UTC (permalink / raw)
To: Richard Palethorpe; +Cc: LTP List, Luke Nowakowski-Krijger
Hi all,
> Hello,
> Li Wang <liwang@redhat.com> writes:
> > Hi All,
> > I have completed my review (and test) on this patch set, generally looks good!
> > With regard to the tiny issues, I think Luke you can just against the concrete
> > problem to follow up and fix it. No need to resend all patches with a new version.
> > I would suggest merging this patchset at present (in case of blocking other's (or duplicate) work).
> > CI builds pass: https://github.com/wangli5665/ltp/runs/7460955660
> > Reviewed-by: Li Wang <liwang@redhat.com>
> I'm also very keen to get this merged and can make the necessary
> fixups. However I will give Petr Vorel chance to look at the shell
> tests.
I can send tomorrow proposed fixes (diff or a patchset) so that we get the rest
merged this week.
Kind regards,
Petr
> Reviewed-by: Richard Palethorpe <rpalethorpe@suse.com>
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [LTP] [PATCH v5 10/18] controllers: Expand cgroup_lib shell library
2022-07-26 13:12 ` Petr Vorel
@ 2022-07-26 19:24 ` Luke Nowakowski-Krijger
0 siblings, 0 replies; 33+ messages in thread
From: Luke Nowakowski-Krijger @ 2022-07-26 19:24 UTC (permalink / raw)
To: Petr Vorel; +Cc: LTP List
[-- Attachment #1.1: Type: text/plain, Size: 3146 bytes --]
Hi Petr,
On Tue, Jul 26, 2022 at 6:12 AM Petr Vorel <pvorel@suse.cz> wrote:
> Hi Luke,
>
> ...
> > +# Gets the cgroup version of the given controller
> > +# USAGE: cgroup_get_version CONTROLLER
> > +# RETURNS: "1" if version 1 and "2" if version 2
> > +# Must call cgroup_require before calling
> > +cgroup_get_version()
> > {
> > - local subsystem=$1
> > - local mntpoint
> > + local ctrl="$1"
> > + local version
>
> > - [ $# -eq 0 ] && tst_brk TBROK "get_cgroup_mountpoint: subsystem
> not defined"
> > + [ $# -eq 0 ] && tst_brk TBROK "cgroup_get_version: controller not
> defined"
> NOTE: this will always pass, because you pass variable in ""
> (thus $1 = "" and $# = 1):
> cgroup_get_task_list()
> {
> local ctrl="$1"
> version=$(cgroup_get_version "$ctrl")
>
>
Yes this is true when using this function by other functions in the
library. The use case where this would fail is when someone calls it from a
test and forgets to pass a controller to it or something like that.
> > + [ "$_cgroup_state" = "" ] && tst_brk TBROK "cgroup_get_version: No
> previous state found. Forgot to call cgroup_require?"
>
> > - mntpoint=$(grep cgroup /proc/mounts | grep -w $subsystem | awk '{
> print $2 }')
> > - [ -z "$mntpoint" ] && return 1
> > + version=$(echo "$_cgroup_state" | grep -w "^$ctrl" | awk '{ print
> $2 }')
> > + [ "$version" = "" ] && tst_brk TBROK "cgroup_get_version: Could
> not find controller $ctrl"
> > +
> > + echo "$version"
>
> > - echo $mntpoint
> > return 0
> > }
> ...
>
> > +# Mounts and configures the given controller
> > +# USAGE: cgroup_require CONTROLLER
> > +cgroup_require()
> > +{
> > + local ctrl="$1"
> > +
> > + [ $# -eq 0 ] && tst_brk TBROK "cgroup_require: controller not
> defined"
> > +
> > + [ ! -f /proc/cgroups ] && tst_brk TCONF "Kernel does not support
> control groups"
> > +
> > + _cgroup_state=$(tst_cgctl require "$ctrl" $$)
> > +
> > + if [ $? -eq 32 ]; then
> > + tst_brk TCONF "'tst_cgctl require' exited. Controller is
> probably not available?"
> > + fi
> > +
> > + if [ $? -ne 0 ]; then
> > + tst_brk TBROK "'tst_cgctl require' exited."
> > + fi
> FYI if cgroup_require is called from cleanup function tst_brk does not
> exit the
> code:
>
> tst_brk()
> {
> local res=$1
> shift
>
> if [ "$TST_DO_EXIT" = 1 ]; then
> tst_res TWARN "$@"
> return
> fi
>
> tst_res "$res" "$@"
> _tst_do_exit
> }
>
> IMHO that means that $? became 0 even it was previously 32.
> It's always safer to save $? into local variable if needed to store exit
> code
> (otherwise using if, e.g. "if ! foo; then" is preferred).
>
>
That's a good point. I'll just save the return and propagate the return
code.
> NOTE: Maybe at this point it might be safer if you post next version
> where you do fixes yourself. I'll try to review the rest of the shell
> scripts
> today (C code looks correct to me).
>
>
Yeah, I will rebase and resubmit with the fixes people have mentioned.
> Kind regards,
> Petr
>
[-- Attachment #1.2: Type: text/html, Size: 4859 bytes --]
[-- Attachment #2: Type: text/plain, Size: 60 bytes --]
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 33+ messages in thread
end of thread, other threads:[~2022-07-26 19:25 UTC | newest]
Thread overview: 33+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-07-21 20:52 [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH v3 01/18] API/cgroup: Modify tst_cg_print_config for parsing and consumption Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH 02/18] API/cgroup: Add option for specific pid to tst_cg_opts Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH v2 03/18] API/cgroup: Add cgroup_find_root helper function Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH 04/18] API/cgroup: Add CTRL_NAME_MAX define Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH v2 05/18] tst_test_macros: Add TST_TO_STR and TST_STR macro Luke Nowakowski-Krijger
2022-07-26 14:44 ` Petr Vorel
2022-07-21 20:52 ` [LTP] [PATCH v5 06/18] API/cgroup: Implement tst_cg_load_config Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH v3 07/18] API/cgroup: Add more controllers to tst_cgroup Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH v2 08/18] API/cgroup: refuse to mount blkio when io controller is mounted Luke Nowakowski-Krijger
2022-07-26 14:40 ` Petr Vorel
2022-07-21 20:52 ` [LTP] [PATCH v2 09/18] testcases/lib: Implement tst_cgctl binary Luke Nowakowski-Krijger
2022-07-26 12:48 ` Petr Vorel
2022-07-21 20:52 ` [LTP] [PATCH v5 10/18] controllers: Expand cgroup_lib shell library Luke Nowakowski-Krijger
2022-07-26 13:12 ` Petr Vorel
2022-07-26 19:24 ` Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH v3 11/18] controllers: Update cgroup_fj_* to use newer cgroup lib and test lib Luke Nowakowski-Krijger
2022-07-25 11:41 ` Richard Palethorpe
2022-07-26 6:36 ` Li Wang
2022-07-21 20:52 ` [LTP] [PATCH v3 12/18] controllers: Update memcg_control_test to newer test lib and cgroup lib Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH v3 13/18] controllers: Update memcg/regression/* to new test " Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH v2 14/18] controllers: Update memcg_stress_test to use newer " Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH v2 15/18] controllers: update memcg/functional " Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH v3 16/18] controllers: Update pids.sh " Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH v2 17/18] controllers: update cpuset_regression_test.sh " Luke Nowakowski-Krijger
2022-07-21 20:52 ` [LTP] [PATCH v3 18/18] controllers: update cgroup_regression_test " Luke Nowakowski-Krijger
2022-07-26 3:33 ` Li Wang
2022-07-25 8:14 ` [LTP] [PATCH v5 00/18] Expand Cgroup lib and modify controller tests Petr Vorel
2022-07-25 20:25 ` Luke Nowakowski-Krijger
2022-07-26 13:00 ` Petr Vorel
2022-07-26 6:10 ` Li Wang
2022-07-26 8:52 ` Richard Palethorpe
2022-07-26 16:04 ` Petr Vorel
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.