From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from sog-mx-4.v43.ch3.sourceforge.com ([172.29.43.194] helo=mx.sourceforge.net) by sfs-ml-2.v29.ch3.sourceforge.com with esmtp (Exim 4.76) (envelope-from ) id 1Wk6cy-0003ST-LO for ltp-list@lists.sourceforge.net; Tue, 13 May 2014 06:55:52 +0000 Received: from mx4-phx2.redhat.com ([209.132.183.25]) by sog-mx-4.v43.ch3.sourceforge.com with esmtp (Exim 4.76) id 1Wk6cx-0006Ig-B0 for ltp-list@lists.sourceforge.net; Tue, 13 May 2014 06:55:52 +0000 Date: Tue, 13 May 2014 02:55:43 -0400 (EDT) From: Jan Stancek Message-ID: <1283427253.4147048.1399964143232.JavaMail.zimbra@redhat.com> In-Reply-To: <5371AB38.3030803@linux.vnet.ibm.com> References: <5371AB38.3030803@linux.vnet.ibm.com> MIME-Version: 1.0 Subject: Re: [LTP] [PATCH 2/2] readahead02: read in 2M chunks 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: madanvs@linux.vnet.ibm.com Cc: ltp-list@lists.sourceforge.net ----- Original Message ----- > From: "Madan" > To: ltp-list@lists.sourceforge.net > Sent: Tuesday, 13 May, 2014 7:18:48 AM > Subject: Re: [LTP] [PATCH 2/2] readahead02: read in 2M chunks > > > On Tuesday 22 April 2014 02:07 PM, Jan Stancek wrote: > > Max readahead size has been limited since commit: > > commit 6d2be915e589b58cb11418cbe1f22ff90732b6ac > > Author: Raghavendra K T > > Date: Thu Apr 3 14:48:23 2014 -0700 > > mm/readahead.c: fix readahead failure for memoryless NUMA nodes > > and limit readahead pages > > > > Update testcase to read ahead in 2M chunks, this change is > > compatible with older kernels. > > > > Signed-off-by: Jan Stancek > > --- > > testcases/kernel/syscalls/readahead/readahead02.c | 9 +++++++-- > > 1 files changed, 7 insertions(+), 2 deletions(-) > > > > diff --git a/testcases/kernel/syscalls/readahead/readahead02.c > > b/testcases/kernel/syscalls/readahead/readahead02.c > > index 8940821..5b1524e 100644 > > --- a/testcases/kernel/syscalls/readahead/readahead02.c > > +++ b/testcases/kernel/syscalls/readahead/readahead02.c > > @@ -213,8 +213,13 @@ static void read_testfile(int do_readahead, const char > > *fname, size_t fsize, > > tst_brkm(TBROK | TERRNO, cleanup, "Failed to open %s", fname); > > > > if (do_readahead) { > > - TEST(ltp_syscall(__NR_readahead, fd, (off64_t) 0, > > - (size_t) fsize)); > > + /* read ahead in chunks, 2MB is maximum since 3.15-rc1 */ > > + for (i = 0; i < fsize; i += 2*1024*1024) { > > + TEST(ltp_syscall(__NR_readahead, fd, > > + (off64_t) i, 2*1024*1024)); > > + if (TEST_RETURN != 0) > > + break; > > + } > > check_ret(0); > > *cached = get_cached_size(); > > > > IMHO, This patch is not complete please also change the test pass > criteria what i have sent the patch for. Can you explain why? If test "reads ahead" (in 2M chunks) the same amount of data, why should pass criteria change? Regards, Jan > > > ------------------------------------------------------------------------------ > "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 > ------------------------------------------------------------------------------ "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