Hi!
> 
> Looks good.
> 
> I will try to apply the patchset and run the tests tomorrow.

I've tested the patset (all 8 patches) and it works fine.

I've got a failure on cpuset_memory_pressure_testset.sh:

cpuset10    5  TFAIL  :  root group's memory_pressure didn't have memory 
pressure rate.
/opt/ltp/testcases/bin/cpuset_memory_pressure_testset.sh: line 195:
15280 Killed                  ./cpuset_memory_pressure $usemem
cpuset10    6  TFAIL  :  root group's memory_pressure didn't have memory 
pressure rate.

however the changes you did to the test are more or less cosmetic.

Now I would like to see a bit more verbose git log for
"cpuset_memory_spread_test: number of nodes and sync" patch (You should
write commit logs verbose enough so that peroson who wasn't involved in
the fix would understand how the changes fix the prolem).

And with the fixes to the reorganization patch and then I'm fine with
the whole series.

-- 
Cyril Hrubis
chru...@suse.cz

------------------------------------------------------------------------------
October Webinars: Code for Performance
Free Intel webinars can help you accelerate application performance.
Explore tips for MPI, OpenMP, advanced profiling, and more. Get the most from 
the latest Intel processors and coprocessors. See abstracts and register >
http://pubads.g.doubleclick.net/gampad/clk?id=60134791&iu=/4140/ostg.clktrk
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to