public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
From: Han Pingtian <phan@redhat.com>
To: Garrett Cooper <yanegomi@gmail.com>
Cc: ltp-list@lists.sourceforge.net
Subject: Re: [LTP] [PATCH] Fix build of cpuset_syscall_test
Date: Mon, 21 Feb 2011 15:35:48 +0800	[thread overview]
Message-ID: <20110221073548.GD2904@hpt.nay.redhat.com> (raw)
In-Reply-To: <AANLkTimk_+8o+QQdjzweLmhB=qzzO2TGYADmEuBQtZom@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 1100 bytes --]

Sorry for the delay. Attachment is the updated patch.

Thanks.
> > Signed-off-by: Han Pingtian <phan@redhat.com>
> > ---
> >  .../cpuset/cpuset_syscall_test/Makefile            |    2 ++
> >  1 files changed, 2 insertions(+), 0 deletions(-)
> >
> > diff --git a/testcases/kernel/controllers/cpuset/cpuset_syscall_test/Makefile b/testcases/kernel/controllers/cpuset/cpuset_syscall_test/Makefile
> > index edc6265..17331fb 100644
> > --- a/testcases/kernel/controllers/cpuset/cpuset_syscall_test/Makefile
> > +++ b/testcases/kernel/controllers/cpuset/cpuset_syscall_test/Makefile
> > @@ -25,3 +25,5 @@ top_srcdir              ?= ../../../../..
> >  include $(top_srcdir)/include/mk/testcases.mk
> >  include $(abs_srcdir)/../Makefile.inc
> >  include $(top_srcdir)/include/mk/generic_leaf_target.mk
> > +
> > +LDLIBS                  += $(NUMA_LIBS)
> > --
> 
> For consistency this should be placed before generic_leaf_target.mk
> Thanks,
> -Garrett
> 
-- 
Han Pingtian
Quality Engineer
hpt @ #kernel-qe
Red Hat, Inc
Freedom ... courage ... Commitment ... ACCOUNTABILITY

[-- Attachment #2: 0001-Fix-build-of-cpuset_syscall_test.patch --]
[-- Type: text/plain, Size: 2484 bytes --]

From d5b38cc7013a7a5e9bfec198efaa1b7a026dc2c9 Mon Sep 17 00:00:00 2001
From: Han Pingtian <phan@redhat.com>
Date: Thu, 10 Feb 2011 16:44:37 +0800
Subject: [PATCH] Fix build of cpuset_syscall_test

cpuset_syscall_test now depends on numa lib:

make[1]: Entering directory `/root/ltp/testcases/kernel/controllers/cpuset/cpuset_syscall_test'
gcc -g -O2 -g -O2 -fno-strict-aliasing -pipe -Wall  -I/root/ltp/testcases/kernel/include -I../../../../../include -I../../../../../include   -L/root/ltp/testcases/kernel/controllers/cpuset/cpuset_syscall_test/../cpuset_lib -L/root/ltp/testcases/kernel/controllers/cpuset/cpuset_syscall_test/../../libcontrollers -L../../../../../lib  cpuset_syscall_test.c   -lltp -lcpu_set -lcontrollers -lltp -o cpuset_syscall_test
/root/ltp/testcases/kernel/controllers/cpuset/cpuset_syscall_test/../cpuset_lib/libcpu_set.a(libcpuset.o): In function `cpuset_addr2node':
/root/ltp/testcases/kernel/controllers/cpuset/cpuset_lib/libcpuset.c:3455: undefined reference to `get_mempolicy'
/root/ltp/testcases/kernel/controllers/cpuset/cpuset_syscall_test/../cpuset_lib/libcpu_set.a(libcpuset.o): In function `cpuset_membind':
/root/ltp/testcases/kernel/controllers/cpuset/cpuset_lib/libcpuset.c:3439: undefined reference to `set_mempolicy'
/root/ltp/testcases/kernel/controllers/cpuset/cpuset_syscall_test/../cpuset_lib/libcpu_set.a(libcpuset.o): In function `cpuset_unpin':
/root/ltp/testcases/kernel/controllers/cpuset/cpuset_lib/libcpuset.c:3752: undefined reference to `set_mempolicy'
collect2: ld returned 1 exit status
make[1]: *** [cpuset_syscall_test] Error 1
make[1]: Leaving directory `/root/ltp/testcases/kernel/controllers/cpuset/cpuset_syscall_test'
make: *** [all] Error 2

This patch fixes this problem

Signed-off-by: Han Pingtian <phan@redhat.com>
---
 .../cpuset/cpuset_syscall_test/Makefile            |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/testcases/kernel/controllers/cpuset/cpuset_syscall_test/Makefile b/testcases/kernel/controllers/cpuset/cpuset_syscall_test/Makefile
index edc6265..6565631 100644
--- a/testcases/kernel/controllers/cpuset/cpuset_syscall_test/Makefile
+++ b/testcases/kernel/controllers/cpuset/cpuset_syscall_test/Makefile
@@ -24,4 +24,8 @@ top_srcdir              ?= ../../../../..
 
 include $(top_srcdir)/include/mk/testcases.mk
 include $(abs_srcdir)/../Makefile.inc
+
+LDLIBS                  += $(NUMA_LIBS)
+
 include $(top_srcdir)/include/mk/generic_leaf_target.mk
+
-- 
1.7.1


[-- Attachment #3: Type: text/plain, Size: 387 bytes --]

------------------------------------------------------------------------------
The ultimate all-in-one performance toolkit: Intel(R) Parallel Studio XE:
Pinpoint memory and threading errors before they happen.
Find and fix more than 250 security defects in the development cycle.
Locate bottlenecks in serial and parallel code that limit performance.
http://p.sf.net/sfu/intel-dev2devfeb

[-- Attachment #4: Type: text/plain, Size: 155 bytes --]

_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

  reply	other threads:[~2011-02-21  7:32 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-02-10  8:52 [LTP] [PATCH] Fix build of cpuset_syscall_test Han Pingtian
2011-02-10 15:55 ` Garrett Cooper
2011-02-21  7:35   ` Han Pingtian [this message]
2011-04-08 12:39     ` Cyril Hrubis

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=20110221073548.GD2904@hpt.nay.redhat.com \
    --to=phan@redhat.com \
    --cc=ltp-list@lists.sourceforge.net \
    --cc=yanegomi@gmail.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox