LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [PATCH 0/6] selftests: improve cross compilation support
@ 2015-04-01 23:20 Tyler Baker
  2015-04-01 23:20 ` [PATCH 1/6] selftest/memfd: enable cross compilation Tyler Baker
                   ` (6 more replies)
  0 siblings, 7 replies; 14+ messages in thread
From: Tyler Baker @ 2015-04-01 23:20 UTC (permalink / raw)
  To: Shuah Khan
  Cc: Kevin Hilman, John Stultz, Darren Hart, David Herrmann,
	Michael Ellerman, linux-kernel, linux-arm-kernel, Tyler Baker

This patch set improves cross building functionality by resovling hard coded
assumptions about the compiler used. With this patch set, all but mqueue, 
and kdbus cross build successfully as they must link against a native 
libraries. The hope is tha the above issues can be resolved in future patch 
sets. I have tested this series by building selftests for x86_64, arm and 
arm64 targets.

This motivation behind this series is to enable the kernelci.org service to
easily build, deploy, and execute selftests on many different processor
architectures without natively compiling them. With the hope of quickly
providing these test results back to the community.

This series is based on next-20150401.

Tyler Baker (6):
  selftest/memfd: enable cross compilation
  selftest/kdbus: enable cross compilation
  selftest/mount: enable cross compilation
  selftest/memfd: include default header install path
  selftest/ipc: enable cross compilation
  selftest/mqueue: enable cross compilation

 tools/testing/selftests/ipc/Makefile    | 6 +-----
 tools/testing/selftests/kdbus/Makefile  | 4 ++--
 tools/testing/selftests/memfd/Makefile  | 8 +++++---
 tools/testing/selftests/mount/Makefile  | 5 +++--
 tools/testing/selftests/mqueue/Makefile | 6 ++++--
 5 files changed, 15 insertions(+), 14 deletions(-)

-- 
2.1.4


^ permalink raw reply	[flat|nested] 14+ messages in thread

* [PATCH 1/6] selftest/memfd: enable cross compilation
  2015-04-01 23:20 [PATCH 0/6] selftests: improve cross compilation support Tyler Baker
@ 2015-04-01 23:20 ` Tyler Baker
  2015-04-01 23:20 ` [PATCH 2/6] selftest/kdbus: " Tyler Baker
                   ` (5 subsequent siblings)
  6 siblings, 0 replies; 14+ messages in thread
From: Tyler Baker @ 2015-04-01 23:20 UTC (permalink / raw)
  To: Shuah Khan
  Cc: Kevin Hilman, John Stultz, Darren Hart, David Herrmann,
	Michael Ellerman, linux-kernel, linux-arm-kernel, Tyler Baker

Use the CC variable instead of hard coding gcc. Also clean up the compiler
options by creating a CFLAGS variable.

Signed-off-by: Tyler Baker <tyler.baker@linaro.org>
---
 tools/testing/selftests/memfd/Makefile | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/tools/testing/selftests/memfd/Makefile b/tools/testing/selftests/memfd/Makefile
index 191dee9..69f08ab 100644
--- a/tools/testing/selftests/memfd/Makefile
+++ b/tools/testing/selftests/memfd/Makefile
@@ -1,17 +1,18 @@
+CC = $(CROSS_COMPILE)gcc
 CFLAGS += -D_FILE_OFFSET_BITS=64
 CFLAGS += -I../../../../include/uapi/
 CFLAGS += -I../../../../include/
 
 all:
-	gcc $(CFLAGS) memfd_test.c -o memfd_test
+	$(CC) $(CFLAGS) memfd_test.c -o memfd_test
 
 TEST_PROGS := memfd_test
 
 include ../lib.mk
 
 build_fuse:
-	gcc $(CFLAGS) fuse_mnt.c `pkg-config fuse --cflags --libs` -o fuse_mnt
-	gcc $(CFLAGS) fuse_test.c -o fuse_test
+	$(CC) $(CFLAGS) fuse_mnt.c `pkg-config fuse --cflags --libs` -o fuse_mnt
+	$(CC) $(CFLAGS) fuse_test.c -o fuse_test
 
 run_fuse: build_fuse
 	@./run_fuse_test.sh || echo "fuse_test: [FAIL]"
-- 
2.1.4


^ permalink raw reply	[flat|nested] 14+ messages in thread

* [PATCH 2/6] selftest/kdbus: enable cross compilation
  2015-04-01 23:20 [PATCH 0/6] selftests: improve cross compilation support Tyler Baker
  2015-04-01 23:20 ` [PATCH 1/6] selftest/memfd: enable cross compilation Tyler Baker
@ 2015-04-01 23:20 ` Tyler Baker
  2015-04-02 17:28   ` Shuah Khan
  2015-04-03 14:28   ` Greg KH
  2015-04-01 23:20 ` [PATCH 3/6] selftest/mount: " Tyler Baker
                   ` (4 subsequent siblings)
  6 siblings, 2 replies; 14+ messages in thread
From: Tyler Baker @ 2015-04-01 23:20 UTC (permalink / raw)
  To: Shuah Khan
  Cc: Kevin Hilman, John Stultz, Darren Hart, David Herrmann,
	Michael Ellerman, linux-kernel, linux-arm-kernel, Tyler Baker

Use the CC variable instead of hard coding gcc and include lib.mk.

Signed-off-by: Tyler Baker <tyler.baker@linaro.org>
---
 tools/testing/selftests/kdbus/Makefile | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/tools/testing/selftests/kdbus/Makefile b/tools/testing/selftests/kdbus/Makefile
index f6cfab2..de8242f 100644
--- a/tools/testing/selftests/kdbus/Makefile
+++ b/tools/testing/selftests/kdbus/Makefile
@@ -33,11 +33,13 @@ OBJS= \
 
 all: kdbus-test
 
+include ../lib.mk
+
 %.o: %.c
-	gcc $(CFLAGS) -c $< -o $@
+	$(CC) $(CFLAGS) -c $< -o $@
 
 kdbus-test: $(OBJS)
-	gcc $(CFLAGS) $^ $(LDLIBS) -o $@
+	$(CC) $(CFLAGS) $^ $(LDLIBS) -o $@
 
 run_tests:
 	./kdbus-test --tap
-- 
2.1.4


^ permalink raw reply	[flat|nested] 14+ messages in thread

* [PATCH 3/6] selftest/mount: enable cross compilation
  2015-04-01 23:20 [PATCH 0/6] selftests: improve cross compilation support Tyler Baker
  2015-04-01 23:20 ` [PATCH 1/6] selftest/memfd: enable cross compilation Tyler Baker
  2015-04-01 23:20 ` [PATCH 2/6] selftest/kdbus: " Tyler Baker
@ 2015-04-01 23:20 ` Tyler Baker
  2015-04-01 23:20 ` [PATCH 4/6] selftest/memfd: include default header install path Tyler Baker
                   ` (3 subsequent siblings)
  6 siblings, 0 replies; 14+ messages in thread
From: Tyler Baker @ 2015-04-01 23:20 UTC (permalink / raw)
  To: Shuah Khan
  Cc: Kevin Hilman, John Stultz, Darren Hart, David Herrmann,
	Michael Ellerman, linux-kernel, linux-arm-kernel, Tyler Baker

Use the CC variable instead of hard coding gcc. Also clean up the compiler
options by creating a CFLAGS variable.

Signed-off-by: Tyler Baker <tyler.baker@linaro.org>
---
 tools/testing/selftests/mount/Makefile | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/tools/testing/selftests/mount/Makefile b/tools/testing/selftests/mount/Makefile
index a5b367f..95580a9 100644
--- a/tools/testing/selftests/mount/Makefile
+++ b/tools/testing/selftests/mount/Makefile
@@ -1,9 +1,10 @@
 # Makefile for mount selftests.
-
+CFLAGS = -Wall \
+         -O2
 all: unprivileged-remount-test
 
 unprivileged-remount-test: unprivileged-remount-test.c
-	gcc -Wall -O2 unprivileged-remount-test.c -o unprivileged-remount-test
+	$(CC) $(CFLAGS) unprivileged-remount-test.c -o unprivileged-remount-test
 
 include ../lib.mk
 
-- 
2.1.4


^ permalink raw reply	[flat|nested] 14+ messages in thread

* [PATCH 4/6] selftest/memfd: include default header install path
  2015-04-01 23:20 [PATCH 0/6] selftests: improve cross compilation support Tyler Baker
                   ` (2 preceding siblings ...)
  2015-04-01 23:20 ` [PATCH 3/6] selftest/mount: " Tyler Baker
@ 2015-04-01 23:20 ` Tyler Baker
  2015-04-01 23:20 ` [PATCH 5/6] selftest/ipc: enable cross compilation Tyler Baker
                   ` (2 subsequent siblings)
  6 siblings, 0 replies; 14+ messages in thread
From: Tyler Baker @ 2015-04-01 23:20 UTC (permalink / raw)
  To: Shuah Khan
  Cc: Kevin Hilman, John Stultz, Darren Hart, David Herrmann,
	Michael Ellerman, linux-kernel, linux-arm-kernel, Tyler Baker

Include the default path for INSTALL_HDR_PATH to make it less intrusive when
cross building.

Signed-off-by: Tyler Baker <tyler.baker@linaro.org>
---
 tools/testing/selftests/memfd/Makefile | 1 +
 1 file changed, 1 insertion(+)

diff --git a/tools/testing/selftests/memfd/Makefile b/tools/testing/selftests/memfd/Makefile
index 69f08ab..3e7eb79 100644
--- a/tools/testing/selftests/memfd/Makefile
+++ b/tools/testing/selftests/memfd/Makefile
@@ -2,6 +2,7 @@ CC = $(CROSS_COMPILE)gcc
 CFLAGS += -D_FILE_OFFSET_BITS=64
 CFLAGS += -I../../../../include/uapi/
 CFLAGS += -I../../../../include/
+CFLAGS += -I../../../../usr/include/
 
 all:
 	$(CC) $(CFLAGS) memfd_test.c -o memfd_test
-- 
2.1.4


^ permalink raw reply	[flat|nested] 14+ messages in thread

* [PATCH 5/6] selftest/ipc: enable cross compilation
  2015-04-01 23:20 [PATCH 0/6] selftests: improve cross compilation support Tyler Baker
                   ` (3 preceding siblings ...)
  2015-04-01 23:20 ` [PATCH 4/6] selftest/memfd: include default header install path Tyler Baker
@ 2015-04-01 23:20 ` Tyler Baker
  2015-04-01 23:20 ` [PATCH 6/6] selftest/mqueue: " Tyler Baker
  2015-04-02 13:53 ` [PATCH 0/6] selftests: improve cross compilation support Shuah Khan
  6 siblings, 0 replies; 14+ messages in thread
From: Tyler Baker @ 2015-04-01 23:20 UTC (permalink / raw)
  To: Shuah Khan
  Cc: Kevin Hilman, John Stultz, Darren Hart, David Herrmann,
	Michael Ellerman, linux-kernel, linux-arm-kernel, Tyler Baker

Use the CC variable instead of hard coding gcc.

Signed-off-by: Tyler Baker <tyler.baker@linaro.org>
---
 tools/testing/selftests/ipc/Makefile | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/tools/testing/selftests/ipc/Makefile b/tools/testing/selftests/ipc/Makefile
index 3b6013d..25d2e70 100644
--- a/tools/testing/selftests/ipc/Makefile
+++ b/tools/testing/selftests/ipc/Makefile
@@ -12,11 +12,7 @@ endif
 CFLAGS += -I../../../../usr/include/
 
 all:
-ifeq ($(ARCH),x86)
-	gcc $(CFLAGS) msgque.c -o msgque_test
-else
-	echo "Not an x86 target, can't build msgque selftest"
-endif
+	$(CC) $(CFLAGS) msgque.c -o msgque_test
 
 TEST_PROGS := msgque_test
 
-- 
2.1.4


^ permalink raw reply	[flat|nested] 14+ messages in thread

* [PATCH 6/6] selftest/mqueue: enable cross compilation
  2015-04-01 23:20 [PATCH 0/6] selftests: improve cross compilation support Tyler Baker
                   ` (4 preceding siblings ...)
  2015-04-01 23:20 ` [PATCH 5/6] selftest/ipc: enable cross compilation Tyler Baker
@ 2015-04-01 23:20 ` Tyler Baker
  2015-04-02 13:53 ` [PATCH 0/6] selftests: improve cross compilation support Shuah Khan
  6 siblings, 0 replies; 14+ messages in thread
From: Tyler Baker @ 2015-04-01 23:20 UTC (permalink / raw)
  To: Shuah Khan
  Cc: Kevin Hilman, John Stultz, Darren Hart, David Herrmann,
	Michael Ellerman, linux-kernel, linux-arm-kernel, Tyler Baker

Use the CC variable instead of hard coding gcc. Also clean up the compiler
options by creating a CFLAGS variable.

Signed-off-by: Tyler Baker <tyler.baker@linaro.org>
---
 tools/testing/selftests/mqueue/Makefile | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/tools/testing/selftests/mqueue/Makefile b/tools/testing/selftests/mqueue/Makefile
index 6ca7261..0e3b41e 100644
--- a/tools/testing/selftests/mqueue/Makefile
+++ b/tools/testing/selftests/mqueue/Makefile
@@ -1,6 +1,8 @@
+CFLAGS = -O2
+
 all:
-	gcc -O2 mq_open_tests.c -o mq_open_tests -lrt
-	gcc -O2 -o mq_perf_tests mq_perf_tests.c -lrt -lpthread -lpopt
+	$(CC) $(CFLAGS) mq_open_tests.c -o mq_open_tests -lrt
+	$(CC) $(CFLAGS) -o mq_perf_tests mq_perf_tests.c -lrt -lpthread -lpopt
 
 include ../lib.mk
 
-- 
2.1.4


^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 0/6] selftests: improve cross compilation support
  2015-04-01 23:20 [PATCH 0/6] selftests: improve cross compilation support Tyler Baker
                   ` (5 preceding siblings ...)
  2015-04-01 23:20 ` [PATCH 6/6] selftest/mqueue: " Tyler Baker
@ 2015-04-02 13:53 ` Shuah Khan
  2015-04-02 17:19   ` Tyler Baker
  2015-04-02 17:23   ` Shuah Khan
  6 siblings, 2 replies; 14+ messages in thread
From: Shuah Khan @ 2015-04-02 13:53 UTC (permalink / raw)
  To: Tyler Baker
  Cc: Kevin Hilman, John Stultz, Darren Hart, David Herrmann,
	Michael Ellerman, linux-kernel, linux-arm-kernel, Shuah Khan

On 04/01/2015 05:20 PM, Tyler Baker wrote:
> This patch set improves cross building functionality by resovling hard coded
> assumptions about the compiler used. With this patch set, all but mqueue, 
> and kdbus cross build successfully as they must link against a native 
> libraries. The hope is tha the above issues can be resolved in future patch 
> sets. I have tested this series by building selftests for x86_64, arm and 
> arm64 targets.
> 
> This motivation behind this series is to enable the kernelci.org service to
> easily build, deploy, and execute selftests on many different processor
> architectures without natively compiling them. With the hope of quickly
> providing these test results back to the community.
> 
> This series is based on next-20150401.
> 
> Tyler Baker (6):
>   selftest/memfd: enable cross compilation
>   selftest/kdbus: enable cross compilation
>   selftest/mount: enable cross compilation
>   selftest/memfd: include default header install path
>   selftest/ipc: enable cross compilation
>   selftest/mqueue: enable cross compilation
> 
>  tools/testing/selftests/ipc/Makefile    | 6 +-----
>  tools/testing/selftests/kdbus/Makefile  | 4 ++--
>  tools/testing/selftests/memfd/Makefile  | 8 +++++---
>  tools/testing/selftests/mount/Makefile  | 5 +++--
>  tools/testing/selftests/mqueue/Makefile | 6 ++++--
>  5 files changed, 15 insertions(+), 14 deletions(-)
> 


Tyler,

Thanks for getting this work done. It is a bit late for
4.1 and I plan to get these into 4.2.

thanks,
-- Shuah

-- 
Shuah Khan
Sr. Linux Kernel Developer
Open Source Innovation Group
Samsung Research America (Silicon Valley)
shuahkh@osg.samsung.com | (970) 217-8978

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 0/6] selftests: improve cross compilation support
  2015-04-02 13:53 ` [PATCH 0/6] selftests: improve cross compilation support Shuah Khan
@ 2015-04-02 17:19   ` Tyler Baker
  2015-04-02 17:23   ` Shuah Khan
  1 sibling, 0 replies; 14+ messages in thread
From: Tyler Baker @ 2015-04-02 17:19 UTC (permalink / raw)
  To: Shuah Khan
  Cc: Kevin Hilman, John Stultz, Darren Hart, David Herrmann,
	Michael Ellerman, linux-kernel, linux-arm-kernel

Hi Shuah,

On 2 April 2015 at 06:53, Shuah Khan <shuahkh@osg.samsung.com> wrote:
> On 04/01/2015 05:20 PM, Tyler Baker wrote:
>> This patch set improves cross building functionality by resovling hard coded
>> assumptions about the compiler used. With this patch set, all but mqueue,
>> and kdbus cross build successfully as they must link against a native
>> libraries. The hope is tha the above issues can be resolved in future patch
>> sets. I have tested this series by building selftests for x86_64, arm and
>> arm64 targets.
>>
>> This motivation behind this series is to enable the kernelci.org service to
>> easily build, deploy, and execute selftests on many different processor
>> architectures without natively compiling them. With the hope of quickly
>> providing these test results back to the community.
>>
>> This series is based on next-20150401.
>>
>> Tyler Baker (6):
>>   selftest/memfd: enable cross compilation
>>   selftest/kdbus: enable cross compilation
>>   selftest/mount: enable cross compilation
>>   selftest/memfd: include default header install path
>>   selftest/ipc: enable cross compilation
>>   selftest/mqueue: enable cross compilation
>>
>>  tools/testing/selftests/ipc/Makefile    | 6 +-----
>>  tools/testing/selftests/kdbus/Makefile  | 4 ++--
>>  tools/testing/selftests/memfd/Makefile  | 8 +++++---
>>  tools/testing/selftests/mount/Makefile  | 5 +++--
>>  tools/testing/selftests/mqueue/Makefile | 6 ++++--
>>  5 files changed, 15 insertions(+), 14 deletions(-)
>>
>
>
> Tyler,
>
> Thanks for getting this work done. It is a bit late for
> 4.1 and I plan to get these into 4.2.

Thank you for your consideration, apologies for sending this series
late in the merge window. Having these patches included in 4.2 sounds
fine to me.

>
> thanks,
> -- Shuah
>
> --
> Shuah Khan
> Sr. Linux Kernel Developer
> Open Source Innovation Group
> Samsung Research America (Silicon Valley)
> shuahkh@osg.samsung.com | (970) 217-8978

Cheers,

Tyler

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 0/6] selftests: improve cross compilation support
  2015-04-02 13:53 ` [PATCH 0/6] selftests: improve cross compilation support Shuah Khan
  2015-04-02 17:19   ` Tyler Baker
@ 2015-04-02 17:23   ` Shuah Khan
  2015-04-02 18:47     ` Shuah Khan
  1 sibling, 1 reply; 14+ messages in thread
From: Shuah Khan @ 2015-04-02 17:23 UTC (permalink / raw)
  To: Tyler Baker, Greg Kroah-Hartman
  Cc: Kevin Hilman, John Stultz, Darren Hart, David Herrmann,
	Michael Ellerman, linux-kernel, linux-arm-kernel

On 04/02/2015 07:53 AM, Shuah Khan wrote:
> On 04/01/2015 05:20 PM, Tyler Baker wrote:
>> This patch set improves cross building functionality by resovling hard coded
>> assumptions about the compiler used. With this patch set, all but mqueue, 
>> and kdbus cross build successfully as they must link against a native 
>> libraries. The hope is tha the above issues can be resolved in future patch 
>> sets. I have tested this series by building selftests for x86_64, arm and 
>> arm64 targets.
>>
>> This motivation behind this series is to enable the kernelci.org service to
>> easily build, deploy, and execute selftests on many different processor
>> architectures without natively compiling them. With the hope of quickly
>> providing these test results back to the community.
>>
>> This series is based on next-20150401.
>>
>> Tyler Baker (6):
>>   selftest/memfd: enable cross compilation
>>   selftest/kdbus: enable cross compilation
>>   selftest/mount: enable cross compilation
>>   selftest/memfd: include default header install path
>>   selftest/ipc: enable cross compilation
>>   selftest/mqueue: enable cross compilation
>>
>>  tools/testing/selftests/ipc/Makefile    | 6 +-----
>>  tools/testing/selftests/kdbus/Makefile  | 4 ++--
>>  tools/testing/selftests/memfd/Makefile  | 8 +++++---
>>  tools/testing/selftests/mount/Makefile  | 5 +++--
>>  tools/testing/selftests/mqueue/Makefile | 6 ++++--
>>  5 files changed, 15 insertions(+), 14 deletions(-)
>>
> 
> 
> Tyler,
> 
> Thanks for getting this work done. It is a bit late for
> 4.1 and I plan to get these into 4.2.
> 

After taking a second look at the patches, I want to get
these into 4.1. It will be nice to enable cross compile
as well in this release.

The one patch I won't be able to take it through kselftest
tree is kdbus for logistical reasons. kdbus is going through
Greg's tree - one of his many trees :)

Greg!

Will you be able to include this kdbus patch in this series
in your pull request for kdbus. I will reply to the kdbus
patch in this series and cc you on that. If this doesn't
work, could you suggest an alternative.

thanks,
-- Shuah


-- 
Shuah Khan
Sr. Linux Kernel Developer
Open Source Innovation Group
Samsung Research America (Silicon Valley)
shuahkh@osg.samsung.com | (970) 217-8978

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 2/6] selftest/kdbus: enable cross compilation
  2015-04-01 23:20 ` [PATCH 2/6] selftest/kdbus: " Tyler Baker
@ 2015-04-02 17:28   ` Shuah Khan
  2015-04-02 22:04     ` Greg Kroah-Hartman
  2015-04-03 14:28   ` Greg KH
  1 sibling, 1 reply; 14+ messages in thread
From: Shuah Khan @ 2015-04-02 17:28 UTC (permalink / raw)
  To: Tyler Baker, Greg Kroah-Hartman
  Cc: Kevin Hilman, John Stultz, Darren Hart, David Herrmann,
	Michael Ellerman, linux-kernel, linux-arm-kernel

On 04/01/2015 05:20 PM, Tyler Baker wrote:
> Use the CC variable instead of hard coding gcc and include lib.mk.
> 
> Signed-off-by: Tyler Baker <tyler.baker@linaro.org>

Greg,

Will you be able to take this patch in with the other
kdbus sleftest patches?

Acked-by: Shuah Khan <shuahkh@osg.samsung.com>

thanks,
-- Shuah

> ---
>  tools/testing/selftests/kdbus/Makefile | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/tools/testing/selftests/kdbus/Makefile b/tools/testing/selftests/kdbus/Makefile
> index f6cfab2..de8242f 100644
> --- a/tools/testing/selftests/kdbus/Makefile
> +++ b/tools/testing/selftests/kdbus/Makefile
> @@ -33,11 +33,13 @@ OBJS= \
>  
>  all: kdbus-test
>  
> +include ../lib.mk
> +
>  %.o: %.c
> -	gcc $(CFLAGS) -c $< -o $@
> +	$(CC) $(CFLAGS) -c $< -o $@
>  
>  kdbus-test: $(OBJS)
> -	gcc $(CFLAGS) $^ $(LDLIBS) -o $@
> +	$(CC) $(CFLAGS) $^ $(LDLIBS) -o $@
>  
>  run_tests:
>  	./kdbus-test --tap
> 


-- 
Shuah Khan
Sr. Linux Kernel Developer
Open Source Innovation Group
Samsung Research America (Silicon Valley)
shuahkh@osg.samsung.com | (970) 217-8978

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 0/6] selftests: improve cross compilation support
  2015-04-02 17:23   ` Shuah Khan
@ 2015-04-02 18:47     ` Shuah Khan
  0 siblings, 0 replies; 14+ messages in thread
From: Shuah Khan @ 2015-04-02 18:47 UTC (permalink / raw)
  To: Tyler Baker, Greg Kroah-Hartman
  Cc: Kevin Hilman, John Stultz, Darren Hart, David Herrmann,
	Michael Ellerman, linux-kernel, linux-arm-kernel, Shuah Khan

On 04/02/2015 11:23 AM, Shuah Khan wrote:
> On 04/02/2015 07:53 AM, Shuah Khan wrote:
>> On 04/01/2015 05:20 PM, Tyler Baker wrote:
>>> This patch set improves cross building functionality by resovling hard coded
>>> assumptions about the compiler used. With this patch set, all but mqueue, 
>>> and kdbus cross build successfully as they must link against a native 
>>> libraries. The hope is tha the above issues can be resolved in future patch 
>>> sets. I have tested this series by building selftests for x86_64, arm and 
>>> arm64 targets.
>>>
>>> This motivation behind this series is to enable the kernelci.org service to
>>> easily build, deploy, and execute selftests on many different processor
>>> architectures without natively compiling them. With the hope of quickly
>>> providing these test results back to the community.
>>>
>>> This series is based on next-20150401.
>>>
>>> Tyler Baker (6):
>>>   selftest/memfd: enable cross compilation
>>>   selftest/kdbus: enable cross compilation
>>>   selftest/mount: enable cross compilation
>>>   selftest/memfd: include default header install path
>>>   selftest/ipc: enable cross compilation
>>>   selftest/mqueue: enable cross compilation
>>>
>>>  tools/testing/selftests/ipc/Makefile    | 6 +-----
>>>  tools/testing/selftests/kdbus/Makefile  | 4 ++--
>>>  tools/testing/selftests/memfd/Makefile  | 8 +++++---
>>>  tools/testing/selftests/mount/Makefile  | 5 +++--
>>>  tools/testing/selftests/mqueue/Makefile | 6 ++++--
>>>  5 files changed, 15 insertions(+), 14 deletions(-)
>>>
>>
>>
>> Tyler,
>>
>> Thanks for getting this work done. It is a bit late for
>> 4.1 and I plan to get these into 4.2.
>>
> 
> After taking a second look at the patches, I want to get
> these into 4.1. It will be nice to enable cross compile
> as well in this release.
> 
> The one patch I won't be able to take it through kselftest
> tree is kdbus for logistical reasons. kdbus is going through
> Greg's tree - one of his many trees :)
> 
> Greg!
> 
> Will you be able to include this kdbus patch in this series
> in your pull request for kdbus. I will reply to the kdbus
> patch in this series and cc you on that. If this doesn't
> work, could you suggest an alternative.
> 

All patches except the kdbus patch are now in linux-kselftest next
for 4.1

thanks,
-- Shuah


-- 
Shuah Khan
Sr. Linux Kernel Developer
Open Source Innovation Group
Samsung Research America (Silicon Valley)
shuahkh@osg.samsung.com | (970) 217-8978

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 2/6] selftest/kdbus: enable cross compilation
  2015-04-02 17:28   ` Shuah Khan
@ 2015-04-02 22:04     ` Greg Kroah-Hartman
  0 siblings, 0 replies; 14+ messages in thread
From: Greg Kroah-Hartman @ 2015-04-02 22:04 UTC (permalink / raw)
  To: Shuah Khan
  Cc: Tyler Baker, Kevin Hilman, John Stultz, Darren Hart,
	David Herrmann, Michael Ellerman, linux-kernel, linux-arm-kernel

On Thu, Apr 02, 2015 at 11:28:23AM -0600, Shuah Khan wrote:
> On 04/01/2015 05:20 PM, Tyler Baker wrote:
> > Use the CC variable instead of hard coding gcc and include lib.mk.
> > 
> > Signed-off-by: Tyler Baker <tyler.baker@linaro.org>
> 
> Greg,
> 
> Will you be able to take this patch in with the other
> kdbus sleftest patches?
> 
> Acked-by: Shuah Khan <shuahkh@osg.samsung.com>

Thanks, I'll pick this up through my tree.

greg k-h

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 2/6] selftest/kdbus: enable cross compilation
  2015-04-01 23:20 ` [PATCH 2/6] selftest/kdbus: " Tyler Baker
  2015-04-02 17:28   ` Shuah Khan
@ 2015-04-03 14:28   ` Greg KH
  1 sibling, 0 replies; 14+ messages in thread
From: Greg KH @ 2015-04-03 14:28 UTC (permalink / raw)
  To: Tyler Baker
  Cc: Shuah Khan, Kevin Hilman, John Stultz, Darren Hart,
	David Herrmann, Michael Ellerman, linux-kernel, linux-arm-kernel

On Wed, Apr 01, 2015 at 04:20:16PM -0700, Tyler Baker wrote:
> Use the CC variable instead of hard coding gcc and include lib.mk.
> 
> Signed-off-by: Tyler Baker <tyler.baker@linaro.org>
> Acked-by: Shuah Khan <shuahkh@osg.samsung.com>
> ---
>  tools/testing/selftests/kdbus/Makefile | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)

Applied to my kdbus branch, thanks.

greg k-h

^ permalink raw reply	[flat|nested] 14+ messages in thread

end of thread, other threads:[~2015-04-03 14:29 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-04-01 23:20 [PATCH 0/6] selftests: improve cross compilation support Tyler Baker
2015-04-01 23:20 ` [PATCH 1/6] selftest/memfd: enable cross compilation Tyler Baker
2015-04-01 23:20 ` [PATCH 2/6] selftest/kdbus: " Tyler Baker
2015-04-02 17:28   ` Shuah Khan
2015-04-02 22:04     ` Greg Kroah-Hartman
2015-04-03 14:28   ` Greg KH
2015-04-01 23:20 ` [PATCH 3/6] selftest/mount: " Tyler Baker
2015-04-01 23:20 ` [PATCH 4/6] selftest/memfd: include default header install path Tyler Baker
2015-04-01 23:20 ` [PATCH 5/6] selftest/ipc: enable cross compilation Tyler Baker
2015-04-01 23:20 ` [PATCH 6/6] selftest/mqueue: " Tyler Baker
2015-04-02 13:53 ` [PATCH 0/6] selftests: improve cross compilation support Shuah Khan
2015-04-02 17:19   ` Tyler Baker
2015-04-02 17:23   ` Shuah Khan
2015-04-02 18:47     ` Shuah Khan

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).