On Mon, 2009-08-24 at 14:32 -0400, Mike Gahagan wrote: > When running the Memory Controller test # 4, I'm seeing this behavior below. > > TEST 4: MEMORY CONTROLLER TESTING > RUNNING SETUP..... > TEST STARTED: Please avoid using system while this test executes > /mnt/tests/kernel/distribution/ltp/20090731/ltp-full-20090731/testcases/bin/run_memctl_test.sh: > > line 327: [: 62955520: unary operator expected > TINFO Memory Resource Controller: stat check test fails in first run > /mnt/tests/kernel/distribution/ltp/20090731/ltp-full-20090731/testcases/bin/run_memctl_test.sh: > > line 364: [: 78704640: unary operator expected > TFAIL Memory Resource Controller: stat check test FAILED > *** infinite loop here, script exists memctl_test01 remains running *** > ^Cincrementing stop > > The bash error is due to the 'grep -w "active"' not matching anything so > one of the variables isn't getting set like it is supposed to. I believe > the test was written with an old version of the memory controller what > put a field called memory.stat.active in the file memory.stat. The
Correct. Somebody needs to revisit these tests, as the ABIs has changed. Thanks for pointing that out. However, your patch has the following problem: 1) Missing Signed-off-by: 2) missing header for unified diff at line 3 of patch patching file testcases/kernel/controllers/memctl/run_memctl_test.sh Please look into them when you send a patch next time ;-) Regards-- Subrata > current memory controller I am testing appears to have replaced this > field with "rss". There are fields showing anonymous and file backed > pages in memory.stat, but I don't think they are ever going to match the > memory.usage.in.bytes the test is looking for. Rss always matches and I > believe that is what we should update the check in the test to look for. > The attached patch fixes test #4 to compare memory.usage.in.bytes with rss. > ------------------------------------------------------------------------------ > Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day > trial. Simplify your report design, integration and deployment - and focus on > what you do best, core application coding. Discover what's new with > Crystal Reports now. http://p.sf.net/sfu/bobj-july > _______________________________________________ Ltp-list mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/ltp-list ------------------------------------------------------------------------------ Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day trial. Simplify your report design, integration and deployment - and focus on what you do best, core application coding. Discover what's new with Crystal Reports now. http://p.sf.net/sfu/bobj-july _______________________________________________ Ltp-list mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ltp-list
