All of lore.kernel.org
 help / color / mirror / Atom feed
From: Subrata Modak <subrata@linux.vnet.ibm.com>
To: Francesco RUNDO <francesco.rundo@st.com>
Cc: ltp-list@lists.sourceforge.net
Subject: Re: [LTP] kernel/controllers/ testcase and CGROUPS support
Date: Tue, 16 Jun 2009 00:46:30 +0530	[thread overview]
Message-ID: <1245093390.4871.50.camel@subratamodak.linux.ibm.com> (raw)
In-Reply-To: <4A30B95F.4090903@st.com>

On Thu, 2009-06-11 at 09:59 +0200, Francesco RUNDO wrote:
> Hi Subrata,
> 
> please, replace on the LTP main the previous patch with ones attached.
> The attached patch add the right labels of ifdef.

Ok. I reverted the earlier one and applied this one. Please check the
CVS and let me know if everything is fine.

Regards--
Subrata

> 
> Sorry for that.
> 
> Thanks in advance.
> --
> Francesco Rundo
> 
> Subrata Modak wrote: 
> > On Wed, 2009-06-03 at 14:49 +0200, Francesco RUNDO wrote:
> >   
> > > This patch allows the cross-build of kernel/controllers (cgroups
> > > tests) testcase changing the policy to check the cgroups capability at
> > > build-time.
> > > Signed-off-by: Francesco Rundo <francesco.rundo@st.com>
> > >     
> > 
> > Great if that helps your CROSS_COMPILE environment. Merged.
> > 
> > Regards--
> > Subrata
> > 
> >   
> > > ---
> > > ltp-full-20090430-cross/testcases/kernel/controllers/Makefile.orig  2009-05-26 15:37:35.140000000 +0200
> > > +++
> > > ltp-full-20090430-cross/testcases/kernel/controllers/Makefile       2009-05-28 13:36:36.669998000 +0200
> > > @@ -1,10 +1,26 @@
> > > +ifdef $(CROSS_COMPILE)
> > > +CHECK_CGROUP := $(shell test -f
> > > $(TARGET_DIR)/usr/include/linux/cgroupstats.h && echo 'cgroup')
> > > +else
> > >  CHECK_CGROUP := $(shell test -f /proc/cgroups && echo 'cgroup')
> > >  CHECK_CPUCTL := $(shell grep -w cpu /proc/cgroups 2>/dev/null|cut
> > > -f1)
> > >  CHECK_MEMCTL := $(shell grep -w memory /proc/cgroups 2>/dev/null|cut
> > > -f1)
> > >  CHECK_BLOCKIOCTL := $(shell grep -w blockio /proc/cgroups
> > > 2>/dev/null|cut -f1)
> > >  CHECK_FREEZER := $(shell grep -w freezer /proc/cgroups 2>/dev/null|
> > > cut -f1)
> > >  CHECK_CPUSETCTL = $(shell grep -w cpuset /proc/cgroups 2>/dev/null|
> > > cut -f1)
> > > +endif
> > > 
> > > +ifdef $(CROSS_COMPILE)
> > > +ifeq ($(CHECK_CGROUP),cgroup)
> > > +SUBDIRS += cgroup
> > > +SUBDIRS += cpuctl
> > > +SUBDIRS += memctl
> > > +SUBDIRS += io-throttle
> > > +SUBDIRS += freezer
> > > +SUBDIRS += cpuset
> > > +else
> > > +$(info "Kernel is not compiled with control cgroup support")
> > > +endif
> > > +else
> > >  ifeq ($(CHECK_CGROUP),cgroup)
> > >  SUBDIRS += cgroup
> > >  else
> > > @@ -36,6 +52,7 @@
> > >  else
> > >  $(info "Kernel is not compiled with cpuset resource controller
> > > support")
> > >  endif
> > > +endif
> > > 
> > >  # If at least one of the controllers is available then build
> > > libcontrollers.
> > >  ifneq ($(SUBDIRS),)
> > >     
> > 
> > 
> >   
> plain text document attachment
> (ltp-full-20090430-fix-cgroups-testcase.patch)
> This patch allows the cross-build of kernel/controllers (cgroups tests) testcase changing the policy to check the cgroups capability at build-time.
> Signed-off-by: Francesco Rundo <francesco.rundo@st.com>
> --- ltp-full-20090430-cross/testcases/kernel/controllers/Makefile.orig	2009-05-26 15:37:35.140000000 +0200
> +++ ltp-full-20090430-cross/testcases/kernel/controllers/Makefile	2009-05-28 13:36:36.669998000 +0200
> @@ -1,10 +1,26 @@
> +ifdef CROSS_COMPILE
> +CHECK_CGROUP := $(shell test -f $(TARGET_DIR)/usr/include/linux/cgroupstats.h && echo 'cgroup')
> +else
>  CHECK_CGROUP := $(shell test -f /proc/cgroups && echo 'cgroup')
>  CHECK_CPUCTL := $(shell grep -w cpu /proc/cgroups 2>/dev/null|cut -f1)
>  CHECK_MEMCTL := $(shell grep -w memory /proc/cgroups 2>/dev/null|cut -f1)
>  CHECK_BLOCKIOCTL := $(shell grep -w blockio /proc/cgroups 2>/dev/null|cut -f1)
>  CHECK_FREEZER := $(shell grep -w freezer /proc/cgroups 2>/dev/null| cut -f1)
>  CHECK_CPUSETCTL = $(shell grep -w cpuset /proc/cgroups 2>/dev/null|cut -f1)
> +endif
> 
> +ifdef CROSS_COMPILE
> +ifeq ($(CHECK_CGROUP),cgroup)
> +SUBDIRS += cgroup
> +SUBDIRS += cpuctl
> +SUBDIRS += memctl
> +SUBDIRS += io-throttle
> +SUBDIRS += freezer
> +SUBDIRS += cpuset
> +else
> +$(info "Kernel is not compiled with control cgroup support")
> +endif
> +else
>  ifeq ($(CHECK_CGROUP),cgroup)
>  SUBDIRS += cgroup
>  else
> @@ -36,6 +52,7 @@
>  else
>  $(info "Kernel is not compiled with cpuset resource controller support")
>  endif
> +endif
> 
>  # If at least one of the controllers is available then build libcontrollers.
>  ifneq ($(SUBDIRS),)


------------------------------------------------------------------------------
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables unlimited
royalty-free distribution of the report engine for externally facing 
server and web deployment.
http://p.sf.net/sfu/businessobjects
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

  reply	other threads:[~2009-06-15 19:33 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-03 12:49 [LTP] kernel/controllers/ testcase and CGROUPS support Francesco RUNDO
2009-06-04  9:27 ` Subrata Modak
2009-06-04 11:29 ` Subrata Modak
2009-06-11  7:59   ` Francesco RUNDO
2009-06-15 19:16     ` Subrata Modak [this message]
2009-06-15 19:30       ` Francesco RUNDO

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1245093390.4871.50.camel@subratamodak.linux.ibm.com \
    --to=subrata@linux.vnet.ibm.com \
    --cc=francesco.rundo@st.com \
    --cc=ltp-list@lists.sourceforge.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.