Hi Francesco, Francesco RUNDO wrote: > Hi Gowri, > > I've already fixed the same issue you reported. I've tested it successfully. > > Subrata has already merged my patch (please, find it in attachment). > > Please, check it before to submit another patch. >
I had already seen your patch and generated mine on top of Jun release (which has your fix for CROSS_COMPILE).Your patch could not help in a scenario where kernel has cgroup support but cgroupstat.h file is missing in base OS, and build failed. And hence, I am proposing this patch. Thanks, Gowri > Thanks > Regards > Francesco Rundo > > gowrishankar wrote: >> Hi Subrata, >> Please accept the patch below. >> >> Thanks, >> Gowri >> --- >> cgroups: conditionally enable building cgroup tests >> >> controllers tests gets included into default ltp build if "/proc/cgroup" >> exists. It stops the ltp build in realtime kernel environment where >> kernel >> is new and supports cgroups, but necessary file "linux/cgroupstats.h" >> may not >> exist in the base OS (like RHEL5.3). So configure command enables the >> build, >> but actual build fails, due to missing header file. >> >> Below patch proposes new symbol LTP_CHECK_CGROUPSTATS to check for >> header file >> "linux/cgroupstats.h" and include controllers in the list of tests to >> build. >> >> Tested the patch in non-RT as well as RT environment for the changes. >> >> Signed-off-by: Gowrishankar <[email protected]> >> Tested-by: Gowrishankar <[email protected]> >> --- >> >> ------------------------------------------------------------------------ >> >> Index: ltp-full-20090630/m4/ltp-cgroupstats.m4 >> =================================================================== >> --- /dev/null 1970-01-01 00:00:00.000000000 +0000 >> +++ ltp-full-20090630/m4/ltp-cgroupstats.m4 2009-07-03 13:41:55.000000000 >> +0530 >> @@ -0,0 +1,9 @@ >> +dnl >> +dnl LTP_CHECK_CGROUPSTATS >> +dnl ---------------------------- >> +dnl >> +AC_DEFUN([LTP_CHECK_CGROUPSTATS], >> +[dnl >> +AC_CHECK_HEADERS(linux/cgroupstats.h,[LTP_CHECK_CGROUPSTATS_HEADER=yes]) >> +AC_SUBST(LTP_CHECK_CGROUPSTATS_HEADER) >> +]) >> Index: ltp-full-20090630/configure.ac >> =================================================================== >> --- ltp-full-20090630.orig/configure.ac 2009-07-03 13:59:48.000000000 >> +0530 >> +++ ltp-full-20090630/configure.ac 2009-07-03 14:00:14.000000000 +0530 >> @@ -21,5 +21,6 @@ >> LTP_CHECK_SELINUX >> LTP_CHECK_CRYPTO >> LTP_CHECK_TASKSTATS >> +LTP_CHECK_CGROUPSTATS >> >> AC_OUTPUT >> Index: ltp-full-20090630/testcases/kernel/Makefile >> =================================================================== >> --- ltp-full-20090630.orig/testcases/kernel/Makefile 2009-07-03 >> 14:01:25.000000000 +0530 >> +++ ltp-full-20090630/testcases/kernel/Makefile 2009-07-03 >> 14:02:42.000000000 +0530 >> @@ -1,6 +1,12 @@ >> -SUBDIRS = power_management numa containers controllers connectors include >> fs io ipc mem pty sched security syscalls timers >> +include ../../config.mk >> + >> +SUBDIRS = power_management numa containers connectors include fs io ipc mem >> pty sched security syscalls timers >> UCLINUX_SUBDIRS = include syscalls >> >> +ifeq ($(LTP_CHECK_CGROUPSTATS_HEADER),yes) >> + SUBDIRS+= controllers >> +endif >> + >> all: >> @set -e; for i in $(SUBDIRS); do $(MAKE) -C $$i ; done >> >> >> ------------------------------------------------------------------------ >> >> ------------------------------------------------------------------------------ >> >> ------------------------------------------------------------------------ >> >> _______________________________________________ >> Ltp-list mailing list >> [email protected] >> https://lists.sourceforge.net/lists/listinfo/ltp-list >> ------------------------------------------------------------------------------ _______________________________________________ Ltp-list mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ltp-list
