All of lore.kernel.org
 help / color / mirror / Atom feed
From: Petr Vorel <pvorel@suse.cz>
To: Sebastian Chlad <sebastianchlad@gmail.com>
Cc: Sebastian Chlad <sebastian.chlad@suse.com>, ltp@lists.linux.it
Subject: Re: [LTP] [PATCH v2] controllers: Remove obsolete testplan.txt
Date: Fri, 14 Nov 2025 11:19:08 +0100	[thread overview]
Message-ID: <20251114101908.GB52218@pevik> (raw)
In-Reply-To: <20251113184216.7768-1-sebastian.chlad@suse.com>

Hi Sebastian,

> The testplan.txt file contains outdated information and is unlikely to be
> maintained going forward. The tests are already self-contained and clearly
> organized by controller type.

> Removing this file avoids confusion and reduces maintenance overhead.

> -testplan.txt
> -------------
> -A brief description of the plan for resource management testing.
> -
>  test_controllers.sh
>  -------------------
>  This is the main script file that starts the test. It first checks if the
> diff --git a/testcases/kernel/controllers/testplan.txt b/testcases/kernel/controllers/testplan.txt
> deleted file mode 100644
> index 7fbca2a05..000000000
> --- a/testcases/kernel/controllers/testplan.txt
> +++ /dev/null
> @@ -1,15 +0,0 @@
> -THE RESOURCE MANAGEMENT (CONTROLLERS) TEST PLAN
> -
> -This directory "controllers" is created to include all test cases related to
> -the resource controllers in linux. The testplan at present includes testing
> -of cpu controller, memory controller and cpuset controller.
> -There are test cases to test cpu, memory and cpuset controller. In future new
> -testcases will be included to test these controllers further.
> -Each new controller will have it's own directory to contain all its test
> -cases.
> -A brief description of all the testcases is given in the corresponding
> -controllers directory.
> -
> -For more information on resource controllers please refer to
> -cgroups/cgroups.txt, cgroups/memory.txt and cgroups/cpusets.txt in kernel
> -source code documentation.

+1. This file is indeed useless.

Reviewed-by: Petr Vorel <pvorel@suse.cz>

Proof how it is outdated is that it does not reflect the doc move in kernel:

0d942766453f ("cgroup: rename Documentation/cgroups/ to Documentation/cgroup-legacy/") # v4.5-rc1
da82c92f1150 ("docs: cgroup-v1: add it to the admin-guide book") # v5.3-rc1

Kind regards,
Petr

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

  reply	other threads:[~2025-11-14 10:19 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-11-13 18:42 [LTP] [PATCH v2] controllers: Remove obsolete testplan.txt Sebastian Chlad
2025-11-14 10:19 ` Petr Vorel [this message]
2025-11-18 15:06   ` Petr Vorel

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=20251114101908.GB52218@pevik \
    --to=pvorel@suse.cz \
    --cc=ltp@lists.linux.it \
    --cc=sebastian.chlad@suse.com \
    --cc=sebastianchlad@gmail.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.