[LTP] [PATCH v2] controllers: Remove obsolete testplan.txt

[LTP] [PATCH v2] controllers: Remove obsolete testplan.txt

[Sebastian Chlad]

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.

Signed-off-by: Sebastian Chlad <sebastian.chlad@suse.com>
---
 testcases/kernel/controllers/README       |  5 -----
 testcases/kernel/controllers/testplan.txt | 15 ---------------
 2 files changed, 20 deletions(-)
 delete mode 100644 testcases/kernel/controllers/testplan.txt

diff --git a/testcases/kernel/controllers/README b/testcases/kernel/controllers/README
index b1b00eb6a..2c9650773 100644
--- a/testcases/kernel/controllers/README
+++ b/testcases/kernel/controllers/README
@@ -1,5 +1,4 @@
 The complete dir tree is for testcases for resource management testing of linux kernel.
-For the test plan please refer to the file testplan.txt
 
 --------------
 ***WARNING:***
@@ -14,10 +13,6 @@ similar work as above.
 
 FILES DESCRIPTION:
 
-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.
-- 
2.51.0


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

Re: [LTP] [PATCH v2] controllers: Remove obsolete testplan.txt

[Petr Vorel]

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

Re: [LTP] [PATCH v2] controllers: Remove obsolete testplan.txt

[Petr Vorel]

Hi Sebastian,

merged, thank you!

Kind regards,
Petr

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