Hi! > Add cpu_hotplug tests to scenario_groups/default. > > Rename files in include and tools directories. > > Modify Makefile. > > Some cleanup of cpuhotplug0*.sh: > Obtain some variables' value from parameters. > Use functions in test.sh, eg. tst_brkm. > Use 'TST_CLEANUP' shell variable. > > Fix error in the procedure of offline/online cpu. > > Modify 'sar' command in cpuhotplug06.sh. > > Currently cpuhotplug01.sh is disabled, because some irqs' > smp_affinity can't be set. > > Signed-off-by: Xing Gu <gux.f...@cn.fujitsu.com> > --- > runtest/cpuhotplug | 14 +- > scenario_groups/default | 1 + > testcases/kernel/hotplug/cpu_hotplug/ChangeLog | 6 +- > testcases/kernel/hotplug/cpu_hotplug/Makefile | 10 +- > testcases/kernel/hotplug/cpu_hotplug/README | 4 +- > .../kernel/hotplug/cpu_hotplug/functional/Makefile | 25 +++ > .../hotplug/cpu_hotplug/functional/cpuhotplug01.sh | 136 +++++++------- > .../hotplug/cpu_hotplug/functional/cpuhotplug02.sh | 84 ++++++--- > .../hotplug/cpu_hotplug/functional/cpuhotplug03.sh | 116 +++++++----- > .../hotplug/cpu_hotplug/functional/cpuhotplug04.sh | 99 ++++++---- > .../hotplug/cpu_hotplug/functional/cpuhotplug05.sh | 118 +++++++----- > .../hotplug/cpu_hotplug/functional/cpuhotplug06.sh | 85 +++++---- > .../hotplug/cpu_hotplug/functional/cpuhotplug07.sh | 90 +++++---- > .../kernel/hotplug/cpu_hotplug/include/Makefile | 25 +++ > .../kernel/hotplug/cpu_hotplug/include/hotplug.fns | 202 -------------------- > .../kernel/hotplug/cpu_hotplug/include/hotplug.sh | 203 > +++++++++++++++++++++ > .../hotplug/cpu_hotplug/include/testsuite.fns | 170 ----------------- > .../hotplug/cpu_hotplug/include/testsuite.sh | 171 +++++++++++++++++
The hotplug.sh and testsuite.sh files should be prefixed with the cpuhotplug_ too, otherwise they are likely to collide when files are installed into the LTP bin directory. And it's usally better to split the patches to part that moves files and part that does changes which is easier to review. But given that there seems to be only one line change it may be okay. Do you know what has been changed there? They look pretty much the same to me. Othewise it looks good. -- Cyril Hrubis chru...@suse.cz ------------------------------------------------------------------------------ Open source business process management suite built on Java and Eclipse Turn processes into business applications with Bonita BPM Community Edition Quickly connect people, data, and systems into organized workflows Winner of BOSSIE, CODIE, OW2 and Gartner awards http://p.sf.net/sfu/Bonitasoft _______________________________________________ Ltp-list mailing list Ltp-list@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/ltp-list