All of lore.kernel.org
 help / color / mirror / Atom feed
From: Li Wang via ltp <ltp@lists.linux.it>
To: Samir <samir@linux.ibm.com>
Cc: ltp@lists.linux.it
Subject: Re: [LTP] [PATCH] controllers/cgroup: Skip regression test on cgroup v2 systems
Date: Mon, 16 Mar 2026 11:52:33 +0800	[thread overview]
Message-ID: <abd-gW8X6XU0BWbe@redhat.com> (raw)
In-Reply-To: <20260315125934.1291918-1-samir@linux.ibm.com>

On Sun, Mar 15, 2026 at 01:59:33PM +0100, Samir wrote:
> The cgroup_regression_test.sh test2 attempts to mount a cgroup
> filesystem and expects a cgroup v1 hierarchy. On systems using
> cgroup v2 (unified hierarchy), the test fails with:
> 
> TFAIL: Failed to mount cgroup filesystem
> 
> because the legacy cgroup v1 mount layout is not available.
> 
> Add a check for the presence of /sys/fs/cgroup/cgroup.controllers,
> which indicates a cgroup v2 system. If detected, mark the test
> as TCONF since the test is only applicable to cgroup v1.
> 
> This prevents false failures when running the controllers suite
> on systems where cgroup v2 is enabled by default.
> 
> Signed-off-by: Samir <samir@linux.ibm.com>
> ---
>  .../kernel/controllers/cgroup/cgroup_regression_test.sh    | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/testcases/kernel/controllers/cgroup/cgroup_regression_test.sh b/testcases/kernel/controllers/cgroup/cgroup_regression_test.sh
> index 276231fe8..f6deb0d43 100755
> --- a/testcases/kernel/controllers/cgroup/cgroup_regression_test.sh
> +++ b/testcases/kernel/controllers/cgroup/cgroup_regression_test.sh
> @@ -118,6 +118,13 @@ test2()
>  	local val1
>  	local val2
>  
> +	# This test is specific to cgroup v1
> +	# Check if system is using cgroup v2
> +	if [ -f /sys/fs/cgroup/cgroup.controllers ]; then

This is typically right as the '/sys/fs/cgroup' is Cgroup default mount path,
but if a tested system does not mount there, this won't work.

Maybe we can go with the cgroup_lib.sh standard way for version check?

  cgroup_require "memory"
  cgroup_version=$(cgroup_get_version "memory")
  if [ "$cgroup_version" = "2" ]; then
          tst_res TCONF "memory controller mounted on cgroup v2 hierarchy, skipping test."
          return
  fi

-- 
Regards,
Li Wang


-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

  reply	other threads:[~2026-03-16  3:53 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-03-15 12:59 [LTP] [PATCH] controllers/cgroup: Skip regression test on cgroup v2 systems Samir
2026-03-16  3:52 ` Li Wang via ltp [this message]
2026-03-16 11:58   ` Petr Vorel
2026-03-17  7:09     ` Samir M

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=abd-gW8X6XU0BWbe@redhat.com \
    --to=ltp@lists.linux.it \
    --cc=liwang@redhat.com \
    --cc=samir@linux.ibm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.