From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from picard.linux.it (picard.linux.it [213.254.12.146]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id AB67FC433EF for ; Mon, 24 Jan 2022 09:47:20 +0000 (UTC) Received: from picard.linux.it (localhost [IPv6:::1]) by picard.linux.it (Postfix) with ESMTP id 345FB3C8095 for ; Mon, 24 Jan 2022 10:47:18 +0100 (CET) Received: from in-3.smtp.seeweb.it (in-3.smtp.seeweb.it [217.194.8.3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by picard.linux.it (Postfix) with ESMTPS id 018D43C0F5D for ; Mon, 24 Jan 2022 10:47:07 +0100 (CET) Received: from smtp-out2.suse.de (smtp-out2.suse.de [195.135.220.29]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by in-3.smtp.seeweb.it (Postfix) with ESMTPS id 3814D1A00929 for ; Mon, 24 Jan 2022 10:47:06 +0100 (CET) Received: from relay2.suse.de (relay2.suse.de [149.44.160.134]) by smtp-out2.suse.de (Postfix) with ESMTP id D4B6C1F38B; Mon, 24 Jan 2022 09:47:05 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_rsa; t=1643017625; h=from:from:reply-to:reply-to:date:date:message-id:message-id:to:to: cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=kzWmxxCCxBLgr+RmKBlTQw1OgLOq9h4UjqlY2gEarCI=; b=rQDiq8wv2K/hAZKH0fhEGJrNyv07MDJH6gej+kHRNQmyRjd27njpUgs2YroZLXmN1yzbG9 /ahttpnQs7ICX/nOBabppfnvlCfO3P7aHAjOfsTqNh/R7bc0KCu67j6LErFYhYVWyo/Vv5 EbDWQIuhSBtyKtoGUy6mnnaDGT+iyVk= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_ed25519; t=1643017625; h=from:from:reply-to:reply-to:date:date:message-id:message-id:to:to: cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=kzWmxxCCxBLgr+RmKBlTQw1OgLOq9h4UjqlY2gEarCI=; b=O8g/zAD4u+qOp8vqsgYVB5XdWr2SuVflUrSPaHrSB1LzAb1dLfGD6eoVrEaySNfoyqR1z9 1Jl/352SDiGtcqCg== Received: from g78 (unknown [10.163.24.90]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by relay2.suse.de (Postfix) with ESMTPS id 91B6FA3B8B; Mon, 24 Jan 2022 09:47:05 +0000 (UTC) References: User-agent: mu4e 1.6.10; emacs 27.2 From: Richard Palethorpe To: Luke Nowakowski-Krijger Date: Mon, 24 Jan 2022 09:40:20 +0000 In-reply-to: Message-ID: <87sftdzcrr.fsf@suse.de> MIME-Version: 1.0 X-Virus-Scanned: clamav-milter 0.102.4 at in-3.smtp.seeweb.it X-Virus-Status: Clean Subject: Re: [LTP] [PATCH 00/16] Expand Cgroup lib and modify controller tests X-BeenThere: ltp@lists.linux.it X-Mailman-Version: 2.1.29 Precedence: list List-Id: Linux Test Project List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: rpalethorpe@suse.de Cc: ltp@lists.linux.it Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ltp-bounces+ltp=archiver.kernel.org@lists.linux.it Sender: "ltp" Hello Luke, Luke Nowakowski-Krijger writes: > 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. Thanks I am really happy to see this! It would be nice to see the tests rewritten in C, but if this works, then perhaps we can get many more tests working on both V1 and V2 quite quickly. Including perhaps some kernel selftests (IIRC there are some in shell as well). > > Luke Nowakowski-Krijger (16): > API/cgroup: Modify tst_cgroup_print_config for easier parsing and > consumption > API/cgroup: Add option for specific pid to tst_cgroup_opts > API/cgroup: Add cgroup_find_root helper function > API/cgroup: Implement tst_cgroup_load_config() > API/cgroup: Add more controllers to tst_cgroup > API/cgroup: Change to TWARN when v2 controllers change > 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 | 7 +- > lib/tst_cgroup.c | 314 +++++++++++++++++- > .../cgroup/cgroup_regression_test.sh | 17 +- > .../controllers/cgroup_fj/cgroup_fj_common.sh | 105 ++---- > .../cgroup_fj/cgroup_fj_function.sh | 169 ++++++---- > .../controllers/cgroup_fj/cgroup_fj_proc.c | 24 +- > .../controllers/cgroup_fj/cgroup_fj_stress.sh | 168 +++++----- > testcases/kernel/controllers/cgroup_lib.sh | 128 +++++-- > .../cpuset/cpuset_regression_test.sh | 26 +- > .../controllers/memcg/control/mem_process.c | 28 +- > .../memcg/control/memcg_control_test.sh | 150 +++------ > .../memcg/functional/memcg_force_empty.sh | 2 +- > .../controllers/memcg/functional/memcg_lib.sh | 54 +-- > .../memcg/regression/memcg_regression_test.sh | 202 +++++------ > .../memcg/regression/memcg_test_1.c | 40 +-- > .../memcg/regression/memcg_test_2.c | 24 +- > .../memcg/regression/memcg_test_3.c | 35 +- > .../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 | 65 +--- > testcases/lib/Makefile | 2 +- > testcases/lib/tst_cgctl.c | 69 ++++ > 23 files changed, 966 insertions(+), 769 deletions(-) > create mode 100644 testcases/lib/tst_cgctl.c -- Thank you, Richard. -- Mailing list info: https://lists.linux.it/listinfo/ltp