From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from sog-mx-2.v43.ch3.sourceforge.com ([172.29.43.192] helo=mx.sourceforge.net) by sfs-ml-2.v29.ch3.sourceforge.com with esmtp (Exim 4.76) (envelope-from ) id 1WejEQ-0004mu-Sw for ltp-list@lists.sourceforge.net; Mon, 28 Apr 2014 10:56:18 +0000 Received: from userp1040.oracle.com ([156.151.31.81]) by sog-mx-2.v43.ch3.sourceforge.com with esmtps (TLSv1:AES256-SHA:256) (Exim 4.76) id 1WejEQ-0004fn-4V for ltp-list@lists.sourceforge.net; Mon, 28 Apr 2014 10:56:18 +0000 Received: from acsinet22.oracle.com (acsinet22.oracle.com [141.146.126.238]) by userp1040.oracle.com (Sentrion-MTA-4.3.2/Sentrion-MTA-4.3.2) with ESMTP id s3SAuBTV016643 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Mon, 28 Apr 2014 10:56:12 GMT Received: from aserz7022.oracle.com (aserz7022.oracle.com [141.146.126.231]) by acsinet22.oracle.com (8.14.4+Sun/8.14.4) with ESMTP id s3SAuAeL026714 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO) for ; Mon, 28 Apr 2014 10:56:11 GMT Received: from abhmp0006.oracle.com (abhmp0006.oracle.com [141.146.116.12]) by aserz7022.oracle.com (8.14.4+Sun/8.14.4) with ESMTP id s3SAuAVI026708 for ; Mon, 28 Apr 2014 10:56:10 GMT Message-ID: <535E33C8.3070401@oracle.com> Date: Mon, 28 Apr 2014 14:56:08 +0400 From: Stanislav Kholmanskikh MIME-Version: 1.0 References: <1398674834-1643-1-git-send-email-wangxg.fnst@cn.fujitsu.com> <1398674834-1643-2-git-send-email-wangxg.fnst@cn.fujitsu.com> In-Reply-To: <1398674834-1643-2-git-send-email-wangxg.fnst@cn.fujitsu.com> Subject: Re: [LTP] [PATCH 2/2] kernel/mem/lib/mem.c: do the necessary cleanup work List-Id: Linux Test Project General Discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ltp-list-bounces@lists.sourceforge.net To: ltp-list@lists.sourceforge.net Hi! On 04/28/2014 12:47 PM, Xiaoguang Wang wrote: > Oom01 case will modify "/proc/sys/vm/overcommit_memory" to have tests. In most systems, > the overcommit_memory's default value is 0. When running oom01, if KSM configuration > is not enabled, "tst_brkm(TCONF, NULL, ...)" will be called, in this case, we will forget > to execute the restore operation defined in cleanup, and this will impact other test cases > in LTP, for example, fork14 in RHEL5 will succeed unexpectedly. > > Signed-off-by: Xiaoguang Wang > --- > testcases/kernel/mem/lib/mem.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/testcases/kernel/mem/lib/mem.c b/testcases/kernel/mem/lib/mem.c > index f145a2a..69a9014 100644 > --- a/testcases/kernel/mem/lib/mem.c > +++ b/testcases/kernel/mem/lib/mem.c > @@ -146,7 +146,7 @@ void testoom(int mempolicy, int lite) > oom(MLOCK, lite); > > if (access(PATH_KSM, F_OK) == -1) > - tst_brkm(TCONF, NULL, "KSM configuration is not enabled"); > + tst_brkm(TCONF, cleanup, "KSM configuration is not enabled"); In the current situation the OOM test cases will exit with TCONF if no KSM is setup on the system. But is KSM the absolute requirement for the test cases? And could we execute the KSM part of testoom() only if KSM is enabled? Thanks. > > tst_resm(TINFO, "start OOM testing for KSM pages."); > oom(KSM, lite); > ------------------------------------------------------------------------------ "Accelerate Dev Cycles with Automated Cross-Browser Testing - For FREE Instantly run your Selenium tests across 300+ browser/OS combos. Get unparalleled scalability from the best Selenium testing platform available. Simple to use. Nothing to install. Get started now for free." http://p.sf.net/sfu/SauceLabs _______________________________________________ Ltp-list mailing list Ltp-list@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/ltp-list