All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] selftests/x86: Detect -no-pie availability
@ 2018-05-02 12:10 ` Florian Weimer
  0 siblings, 0 replies; 12+ messages in thread
From: fweimer @ 2018-05-02 12:10 UTC (permalink / raw)


Some toolchains need -no-pie to build all tests, others do not support
the -no-pie flag at all.  Therefore, add another test for the
availability of the flag.

This amends commit 3346a6a4e5ba8c040360f753b26938cec31a4bdc
("selftests: x86: sysret_ss_attrs doesn't build on a PIE build").

Signed-off-by: Florian Weimer <fweimer at redhat.com>
Acked-by: Randy Dunlap <rdunlap at infradead.org>
Tested-by: Randy Dunlap <rdunlap at infradead.org>
---
 tools/testing/selftests/x86/Makefile          |  8 +++++++-
 tools/testing/selftests/x86/trivial_program.c | 10 ++++++++++
 2 files changed, 17 insertions(+), 1 deletion(-)
 create mode 100644 tools/testing/selftests/x86/trivial_program.c

diff --git a/tools/testing/selftests/x86/Makefile b/tools/testing/selftests/x86/Makefile
index d744991c0f4f..94c89261a236 100644
--- a/tools/testing/selftests/x86/Makefile
+++ b/tools/testing/selftests/x86/Makefile
@@ -8,6 +8,7 @@ include ../lib.mk
 UNAME_M := $(shell uname -m)
 CAN_BUILD_I386 := $(shell ./check_cc.sh $(CC) trivial_32bit_program.c -m32)
 CAN_BUILD_X86_64 := $(shell ./check_cc.sh $(CC) trivial_64bit_program.c)
+CAN_BUILD_WITH_NOPIE := $(shell ./check_cc.sh $(CC) trivial_program.c -no-pie)
 
 TARGETS_C_BOTHBITS := single_step_syscall sysret_ss_attrs syscall_nt test_mremap_vdso \
 			check_initial_reg_state sigreturn iopl mpx-mini-test ioperm \
@@ -31,7 +32,12 @@ BINARIES_64 := $(TARGETS_C_64BIT_ALL:%=%_64)
 BINARIES_32 := $(patsubst %,$(OUTPUT)/%,$(BINARIES_32))
 BINARIES_64 := $(patsubst %,$(OUTPUT)/%,$(BINARIES_64))
 
-CFLAGS := -O2 -g -std=gnu99 -pthread -Wall -no-pie
+CFLAGS := -O2 -g -std=gnu99 -pthread -Wall
+
+# call32_from_64 in thunks.S uses absolute addresses.
+ifeq ($(CAN_BUILD_WITH_NOPIE),1)
+CFLAGS += -no-pie
+endif
 
 define gen-target-rule-32
 $(1) $(1)_32: $(OUTPUT)/$(1)_32
diff --git a/tools/testing/selftests/x86/trivial_program.c b/tools/testing/selftests/x86/trivial_program.c
new file mode 100644
index 000000000000..46a447163b93
--- /dev/null
+++ b/tools/testing/selftests/x86/trivial_program.c
@@ -0,0 +1,10 @@
+/* Trivial program to check that compilation with certain flags is working. */
+
+#include <stdio.h>
+
+int
+main(void)
+{
+	puts("");
+	return 0;
+}
-- 
2.14.3

--
To unsubscribe from this list: send the line "unsubscribe linux-kselftest" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply related	[flat|nested] 12+ messages in thread
* [PATCH] selftests/x86: Detect -no-pie availability
@ 2018-05-02 12:10 ` Florian Weimer
  0 siblings, 0 replies; 12+ messages in thread
From: Florian Weimer @ 2018-05-02 12:10 UTC (permalink / raw)


Some toolchains need -no-pie to build all tests, others do not support
the -no-pie flag at all.  Therefore, add another test for the
availability of the flag.

This amends commit 3346a6a4e5ba8c040360f753b26938cec31a4bdc
("selftests: x86: sysret_ss_attrs doesn't build on a PIE build").

Signed-off-by: Florian Weimer <fweimer at redhat.com>
Acked-by: Randy Dunlap <rdunlap at infradead.org>
Tested-by: Randy Dunlap <rdunlap at infradead.org>
---
 tools/testing/selftests/x86/Makefile          |  8 +++++++-
 tools/testing/selftests/x86/trivial_program.c | 10 ++++++++++
 2 files changed, 17 insertions(+), 1 deletion(-)
 create mode 100644 tools/testing/selftests/x86/trivial_program.c

diff --git a/tools/testing/selftests/x86/Makefile b/tools/testing/selftests/x86/Makefile
index d744991c0f4f..94c89261a236 100644
--- a/tools/testing/selftests/x86/Makefile
+++ b/tools/testing/selftests/x86/Makefile
@@ -8,6 +8,7 @@ include ../lib.mk
 UNAME_M := $(shell uname -m)
 CAN_BUILD_I386 := $(shell ./check_cc.sh $(CC) trivial_32bit_program.c -m32)
 CAN_BUILD_X86_64 := $(shell ./check_cc.sh $(CC) trivial_64bit_program.c)
+CAN_BUILD_WITH_NOPIE := $(shell ./check_cc.sh $(CC) trivial_program.c -no-pie)
 
 TARGETS_C_BOTHBITS := single_step_syscall sysret_ss_attrs syscall_nt test_mremap_vdso \
 			check_initial_reg_state sigreturn iopl mpx-mini-test ioperm \
@@ -31,7 +32,12 @@ BINARIES_64 := $(TARGETS_C_64BIT_ALL:%=%_64)
 BINARIES_32 := $(patsubst %,$(OUTPUT)/%,$(BINARIES_32))
 BINARIES_64 := $(patsubst %,$(OUTPUT)/%,$(BINARIES_64))
 
-CFLAGS := -O2 -g -std=gnu99 -pthread -Wall -no-pie
+CFLAGS := -O2 -g -std=gnu99 -pthread -Wall
+
+# call32_from_64 in thunks.S uses absolute addresses.
+ifeq ($(CAN_BUILD_WITH_NOPIE),1)
+CFLAGS += -no-pie
+endif
 
 define gen-target-rule-32
 $(1) $(1)_32: $(OUTPUT)/$(1)_32
diff --git a/tools/testing/selftests/x86/trivial_program.c b/tools/testing/selftests/x86/trivial_program.c
new file mode 100644
index 000000000000..46a447163b93
--- /dev/null
+++ b/tools/testing/selftests/x86/trivial_program.c
@@ -0,0 +1,10 @@
+/* Trivial program to check that compilation with certain flags is working. */
+
+#include <stdio.h>
+
+int
+main(void)
+{
+	puts("");
+	return 0;
+}
-- 
2.14.3

--
To unsubscribe from this list: send the line "unsubscribe linux-kselftest" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply related	[flat|nested] 12+ messages in thread
* [PATCH] selftests/x86: Detect -no-pie availability
@ 2018-05-02 12:10 ` Florian Weimer
  0 siblings, 0 replies; 12+ messages in thread
From: Florian Weimer @ 2018-05-02 12:10 UTC (permalink / raw)
  To: linux-x86_64, linux-kselftest, linux-arch, x86; +Cc: rdunlap, shuah

Some toolchains need -no-pie to build all tests, others do not support
the -no-pie flag at all.  Therefore, add another test for the
availability of the flag.

This amends commit 3346a6a4e5ba8c040360f753b26938cec31a4bdc
("selftests: x86: sysret_ss_attrs doesn't build on a PIE build").

Signed-off-by: Florian Weimer <fweimer@redhat.com>
Acked-by: Randy Dunlap <rdunlap@infradead.org>
Tested-by: Randy Dunlap <rdunlap@infradead.org>
---
 tools/testing/selftests/x86/Makefile          |  8 +++++++-
 tools/testing/selftests/x86/trivial_program.c | 10 ++++++++++
 2 files changed, 17 insertions(+), 1 deletion(-)
 create mode 100644 tools/testing/selftests/x86/trivial_program.c

diff --git a/tools/testing/selftests/x86/Makefile b/tools/testing/selftests/x86/Makefile
index d744991c0f4f..94c89261a236 100644
--- a/tools/testing/selftests/x86/Makefile
+++ b/tools/testing/selftests/x86/Makefile
@@ -8,6 +8,7 @@ include ../lib.mk
 UNAME_M := $(shell uname -m)
 CAN_BUILD_I386 := $(shell ./check_cc.sh $(CC) trivial_32bit_program.c -m32)
 CAN_BUILD_X86_64 := $(shell ./check_cc.sh $(CC) trivial_64bit_program.c)
+CAN_BUILD_WITH_NOPIE := $(shell ./check_cc.sh $(CC) trivial_program.c -no-pie)
 
 TARGETS_C_BOTHBITS := single_step_syscall sysret_ss_attrs syscall_nt test_mremap_vdso \
 			check_initial_reg_state sigreturn iopl mpx-mini-test ioperm \
@@ -31,7 +32,12 @@ BINARIES_64 := $(TARGETS_C_64BIT_ALL:%=%_64)
 BINARIES_32 := $(patsubst %,$(OUTPUT)/%,$(BINARIES_32))
 BINARIES_64 := $(patsubst %,$(OUTPUT)/%,$(BINARIES_64))
 
-CFLAGS := -O2 -g -std=gnu99 -pthread -Wall -no-pie
+CFLAGS := -O2 -g -std=gnu99 -pthread -Wall
+
+# call32_from_64 in thunks.S uses absolute addresses.
+ifeq ($(CAN_BUILD_WITH_NOPIE),1)
+CFLAGS += -no-pie
+endif
 
 define gen-target-rule-32
 $(1) $(1)_32: $(OUTPUT)/$(1)_32
diff --git a/tools/testing/selftests/x86/trivial_program.c b/tools/testing/selftests/x86/trivial_program.c
new file mode 100644
index 000000000000..46a447163b93
--- /dev/null
+++ b/tools/testing/selftests/x86/trivial_program.c
@@ -0,0 +1,10 @@
+/* Trivial program to check that compilation with certain flags is working. */
+
+#include <stdio.h>
+
+int
+main(void)
+{
+	puts("");
+	return 0;
+}
-- 
2.14.3

^ permalink raw reply related	[flat|nested] 12+ messages in thread
* [PATCH] selftests/x86: Detect -no-pie availability
@ 2018-05-04 19:39   ` Shuah Khan
  0 siblings, 0 replies; 12+ messages in thread
From: shuah @ 2018-05-04 19:39 UTC (permalink / raw)


On 05/02/2018 06:10 AM, Florian Weimer wrote:
> Some toolchains need -no-pie to build all tests, others do not support
> the -no-pie flag at all.  Therefore, add another test for the
> availability of the flag.
> 
> This amends commit 3346a6a4e5ba8c040360f753b26938cec31a4bdc
> ("selftests: x86: sysret_ss_attrs doesn't build on a PIE build").
> 
> Signed-off-by: Florian Weimer <fweimer at redhat.com>
> Acked-by: Randy Dunlap <rdunlap at infradead.org>
> Tested-by: Randy Dunlap <rdunlap at infradead.org>
> ---
>  tools/testing/selftests/x86/Makefile          |  8 +++++++-
>  tools/testing/selftests/x86/trivial_program.c | 10 ++++++++++
>  2 files changed, 17 insertions(+), 1 deletion(-)
>  create mode 100644 tools/testing/selftests/x86/trivial_program.c
> 
> diff --git a/tools/testing/selftests/x86/Makefile b/tools/testing/selftests/x86/Makefile
> index d744991c0f4f..94c89261a236 100644
> --- a/tools/testing/selftests/x86/Makefile
> +++ b/tools/testing/selftests/x86/Makefile
> @@ -8,6 +8,7 @@ include ../lib.mk
>  UNAME_M := $(shell uname -m)
>  CAN_BUILD_I386 := $(shell ./check_cc.sh $(CC) trivial_32bit_program.c -m32)
>  CAN_BUILD_X86_64 := $(shell ./check_cc.sh $(CC) trivial_64bit_program.c)
> +CAN_BUILD_WITH_NOPIE := $(shell ./check_cc.sh $(CC) trivial_program.c -no-pie)
>  
>  TARGETS_C_BOTHBITS := single_step_syscall sysret_ss_attrs syscall_nt test_mremap_vdso \
>  			check_initial_reg_state sigreturn iopl mpx-mini-test ioperm \
> @@ -31,7 +32,12 @@ BINARIES_64 := $(TARGETS_C_64BIT_ALL:%=%_64)
>  BINARIES_32 := $(patsubst %,$(OUTPUT)/%,$(BINARIES_32))
>  BINARIES_64 := $(patsubst %,$(OUTPUT)/%,$(BINARIES_64))
>  
> -CFLAGS := -O2 -g -std=gnu99 -pthread -Wall -no-pie
> +CFLAGS := -O2 -g -std=gnu99 -pthread -Wall
> +
> +# call32_from_64 in thunks.S uses absolute addresses.
> +ifeq ($(CAN_BUILD_WITH_NOPIE),1)
> +CFLAGS += -no-pie
> +endif
>  
>  define gen-target-rule-32
>  $(1) $(1)_32: $(OUTPUT)/$(1)_32
> diff --git a/tools/testing/selftests/x86/trivial_program.c b/tools/testing/selftests/x86/trivial_program.c
> new file mode 100644
> index 000000000000..46a447163b93
> --- /dev/null
> +++ b/tools/testing/selftests/x86/trivial_program.c
> @@ -0,0 +1,10 @@
> +/* Trivial program to check that compilation with certain flags is working. */
> +
> +#include <stdio.h>
> +
> +int
> +main(void)
> +{
> +	puts("");
> +	return 0;
> +}
> 

Thanks. I will queue this up for 4.18-rc1

thahnks,
-- Shuah
--
To unsubscribe from this list: send the line "unsubscribe linux-kselftest" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 12+ messages in thread
* [PATCH] selftests/x86: Detect -no-pie availability
@ 2018-05-04 19:39   ` Shuah Khan
  0 siblings, 0 replies; 12+ messages in thread
From: Shuah Khan @ 2018-05-04 19:39 UTC (permalink / raw)


On 05/02/2018 06:10 AM, Florian Weimer wrote:
> Some toolchains need -no-pie to build all tests, others do not support
> the -no-pie flag at all.  Therefore, add another test for the
> availability of the flag.
> 
> This amends commit 3346a6a4e5ba8c040360f753b26938cec31a4bdc
> ("selftests: x86: sysret_ss_attrs doesn't build on a PIE build").
> 
> Signed-off-by: Florian Weimer <fweimer at redhat.com>
> Acked-by: Randy Dunlap <rdunlap at infradead.org>
> Tested-by: Randy Dunlap <rdunlap at infradead.org>
> ---
>  tools/testing/selftests/x86/Makefile          |  8 +++++++-
>  tools/testing/selftests/x86/trivial_program.c | 10 ++++++++++
>  2 files changed, 17 insertions(+), 1 deletion(-)
>  create mode 100644 tools/testing/selftests/x86/trivial_program.c
> 
> diff --git a/tools/testing/selftests/x86/Makefile b/tools/testing/selftests/x86/Makefile
> index d744991c0f4f..94c89261a236 100644
> --- a/tools/testing/selftests/x86/Makefile
> +++ b/tools/testing/selftests/x86/Makefile
> @@ -8,6 +8,7 @@ include ../lib.mk
>  UNAME_M := $(shell uname -m)
>  CAN_BUILD_I386 := $(shell ./check_cc.sh $(CC) trivial_32bit_program.c -m32)
>  CAN_BUILD_X86_64 := $(shell ./check_cc.sh $(CC) trivial_64bit_program.c)
> +CAN_BUILD_WITH_NOPIE := $(shell ./check_cc.sh $(CC) trivial_program.c -no-pie)
>  
>  TARGETS_C_BOTHBITS := single_step_syscall sysret_ss_attrs syscall_nt test_mremap_vdso \
>  			check_initial_reg_state sigreturn iopl mpx-mini-test ioperm \
> @@ -31,7 +32,12 @@ BINARIES_64 := $(TARGETS_C_64BIT_ALL:%=%_64)
>  BINARIES_32 := $(patsubst %,$(OUTPUT)/%,$(BINARIES_32))
>  BINARIES_64 := $(patsubst %,$(OUTPUT)/%,$(BINARIES_64))
>  
> -CFLAGS := -O2 -g -std=gnu99 -pthread -Wall -no-pie
> +CFLAGS := -O2 -g -std=gnu99 -pthread -Wall
> +
> +# call32_from_64 in thunks.S uses absolute addresses.
> +ifeq ($(CAN_BUILD_WITH_NOPIE),1)
> +CFLAGS += -no-pie
> +endif
>  
>  define gen-target-rule-32
>  $(1) $(1)_32: $(OUTPUT)/$(1)_32
> diff --git a/tools/testing/selftests/x86/trivial_program.c b/tools/testing/selftests/x86/trivial_program.c
> new file mode 100644
> index 000000000000..46a447163b93
> --- /dev/null
> +++ b/tools/testing/selftests/x86/trivial_program.c
> @@ -0,0 +1,10 @@
> +/* Trivial program to check that compilation with certain flags is working. */
> +
> +#include <stdio.h>
> +
> +int
> +main(void)
> +{
> +	puts("");
> +	return 0;
> +}
> 

Thanks. I will queue this up for 4.18-rc1

thahnks,
-- Shuah
--
To unsubscribe from this list: send the line "unsubscribe linux-kselftest" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 12+ messages in thread
* Re: [PATCH] selftests/x86: Detect -no-pie availability
@ 2018-05-04 19:39   ` Shuah Khan
  0 siblings, 0 replies; 12+ messages in thread
From: Shuah Khan @ 2018-05-04 19:39 UTC (permalink / raw)
  To: Florian Weimer, linux-x86_64, linux-kselftest, linux-arch, x86
  Cc: rdunlap, Shuah Khan

On 05/02/2018 06:10 AM, Florian Weimer wrote:
> Some toolchains need -no-pie to build all tests, others do not support
> the -no-pie flag at all.  Therefore, add another test for the
> availability of the flag.
> 
> This amends commit 3346a6a4e5ba8c040360f753b26938cec31a4bdc
> ("selftests: x86: sysret_ss_attrs doesn't build on a PIE build").
> 
> Signed-off-by: Florian Weimer <fweimer@redhat.com>
> Acked-by: Randy Dunlap <rdunlap@infradead.org>
> Tested-by: Randy Dunlap <rdunlap@infradead.org>
> ---
>  tools/testing/selftests/x86/Makefile          |  8 +++++++-
>  tools/testing/selftests/x86/trivial_program.c | 10 ++++++++++
>  2 files changed, 17 insertions(+), 1 deletion(-)
>  create mode 100644 tools/testing/selftests/x86/trivial_program.c
> 
> diff --git a/tools/testing/selftests/x86/Makefile b/tools/testing/selftests/x86/Makefile
> index d744991c0f4f..94c89261a236 100644
> --- a/tools/testing/selftests/x86/Makefile
> +++ b/tools/testing/selftests/x86/Makefile
> @@ -8,6 +8,7 @@ include ../lib.mk
>  UNAME_M := $(shell uname -m)
>  CAN_BUILD_I386 := $(shell ./check_cc.sh $(CC) trivial_32bit_program.c -m32)
>  CAN_BUILD_X86_64 := $(shell ./check_cc.sh $(CC) trivial_64bit_program.c)
> +CAN_BUILD_WITH_NOPIE := $(shell ./check_cc.sh $(CC) trivial_program.c -no-pie)
>  
>  TARGETS_C_BOTHBITS := single_step_syscall sysret_ss_attrs syscall_nt test_mremap_vdso \
>  			check_initial_reg_state sigreturn iopl mpx-mini-test ioperm \
> @@ -31,7 +32,12 @@ BINARIES_64 := $(TARGETS_C_64BIT_ALL:%=%_64)
>  BINARIES_32 := $(patsubst %,$(OUTPUT)/%,$(BINARIES_32))
>  BINARIES_64 := $(patsubst %,$(OUTPUT)/%,$(BINARIES_64))
>  
> -CFLAGS := -O2 -g -std=gnu99 -pthread -Wall -no-pie
> +CFLAGS := -O2 -g -std=gnu99 -pthread -Wall
> +
> +# call32_from_64 in thunks.S uses absolute addresses.
> +ifeq ($(CAN_BUILD_WITH_NOPIE),1)
> +CFLAGS += -no-pie
> +endif
>  
>  define gen-target-rule-32
>  $(1) $(1)_32: $(OUTPUT)/$(1)_32
> diff --git a/tools/testing/selftests/x86/trivial_program.c b/tools/testing/selftests/x86/trivial_program.c
> new file mode 100644
> index 000000000000..46a447163b93
> --- /dev/null
> +++ b/tools/testing/selftests/x86/trivial_program.c
> @@ -0,0 +1,10 @@
> +/* Trivial program to check that compilation with certain flags is working. */
> +
> +#include <stdio.h>
> +
> +int
> +main(void)
> +{
> +	puts("");
> +	return 0;
> +}
> 

Thanks. I will queue this up for 4.18-rc1

thahnks,
-- Shuah

^ permalink raw reply	[flat|nested] 12+ messages in thread
* [PATCH] selftests/x86: Detect -no-pie availability
@ 2018-05-10 17:30     ` Shuah Khan
  0 siblings, 0 replies; 12+ messages in thread
From: shuah @ 2018-05-10 17:30 UTC (permalink / raw)


On 05/04/2018 01:39 PM, Shuah Khan wrote:
> On 05/02/2018 06:10 AM, Florian Weimer wrote:
>> Some toolchains need -no-pie to build all tests, others do not support
>> the -no-pie flag at all.  Therefore, add another test for the
>> availability of the flag.
>>
>> This amends commit 3346a6a4e5ba8c040360f753b26938cec31a4bdc
>> ("selftests: x86: sysret_ss_attrs doesn't build on a PIE build").
>>
>> Signed-off-by: Florian Weimer <fweimer at redhat.com>
>> Acked-by: Randy Dunlap <rdunlap at infradead.org>
>> Tested-by: Randy Dunlap <rdunlap at infradead.org>
>> ---
>>  tools/testing/selftests/x86/Makefile          |  8 +++++++-
>>  tools/testing/selftests/x86/trivial_program.c | 10 ++++++++++
>>  2 files changed, 17 insertions(+), 1 deletion(-)
>>  create mode 100644 tools/testing/selftests/x86/trivial_program.c
>>
>> diff --git a/tools/testing/selftests/x86/Makefile b/tools/testing/selftests/x86/Makefile
>> index d744991c0f4f..94c89261a236 100644
>> --- a/tools/testing/selftests/x86/Makefile
>> +++ b/tools/testing/selftests/x86/Makefile
>> @@ -8,6 +8,7 @@ include ../lib.mk
>>  UNAME_M := $(shell uname -m)
>>  CAN_BUILD_I386 := $(shell ./check_cc.sh $(CC) trivial_32bit_program.c -m32)
>>  CAN_BUILD_X86_64 := $(shell ./check_cc.sh $(CC) trivial_64bit_program.c)
>> +CAN_BUILD_WITH_NOPIE := $(shell ./check_cc.sh $(CC) trivial_program.c -no-pie)
>>  
>>  TARGETS_C_BOTHBITS := single_step_syscall sysret_ss_attrs syscall_nt test_mremap_vdso \
>>  			check_initial_reg_state sigreturn iopl mpx-mini-test ioperm \
>> @@ -31,7 +32,12 @@ BINARIES_64 := $(TARGETS_C_64BIT_ALL:%=%_64)
>>  BINARIES_32 := $(patsubst %,$(OUTPUT)/%,$(BINARIES_32))
>>  BINARIES_64 := $(patsubst %,$(OUTPUT)/%,$(BINARIES_64))
>>  
>> -CFLAGS := -O2 -g -std=gnu99 -pthread -Wall -no-pie
>> +CFLAGS := -O2 -g -std=gnu99 -pthread -Wall
>> +
>> +# call32_from_64 in thunks.S uses absolute addresses.
>> +ifeq ($(CAN_BUILD_WITH_NOPIE),1)
>> +CFLAGS += -no-pie
>> +endif
>>  
>>  define gen-target-rule-32
>>  $(1) $(1)_32: $(OUTPUT)/$(1)_32
>> diff --git a/tools/testing/selftests/x86/trivial_program.c b/tools/testing/selftests/x86/trivial_program.c
>> new file mode 100644
>> index 000000000000..46a447163b93
>> --- /dev/null
>> +++ b/tools/testing/selftests/x86/trivial_program.c
>> @@ -0,0 +1,10 @@
>> +/* Trivial program to check that compilation with certain flags is working. */
>> +
>> +#include <stdio.h>
>> +
>> +int
>> +main(void)
>> +{
>> +	puts("");
>> +	return 0;
>> +}
>>
> 
> Thanks. I will queue this up for 4.18-rc1
> 
> thahnks,
> -- Shuah
> 

Applied to linux-kselftest next for 4.18-rc1

thanks,
-- Shuah
--
To unsubscribe from this list: send the line "unsubscribe linux-kselftest" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 12+ messages in thread
* [PATCH] selftests/x86: Detect -no-pie availability
@ 2018-05-10 17:30     ` Shuah Khan
  0 siblings, 0 replies; 12+ messages in thread
From: Shuah Khan @ 2018-05-10 17:30 UTC (permalink / raw)


On 05/04/2018 01:39 PM, Shuah Khan wrote:
> On 05/02/2018 06:10 AM, Florian Weimer wrote:
>> Some toolchains need -no-pie to build all tests, others do not support
>> the -no-pie flag at all.  Therefore, add another test for the
>> availability of the flag.
>>
>> This amends commit 3346a6a4e5ba8c040360f753b26938cec31a4bdc
>> ("selftests: x86: sysret_ss_attrs doesn't build on a PIE build").
>>
>> Signed-off-by: Florian Weimer <fweimer at redhat.com>
>> Acked-by: Randy Dunlap <rdunlap at infradead.org>
>> Tested-by: Randy Dunlap <rdunlap at infradead.org>
>> ---
>>  tools/testing/selftests/x86/Makefile          |  8 +++++++-
>>  tools/testing/selftests/x86/trivial_program.c | 10 ++++++++++
>>  2 files changed, 17 insertions(+), 1 deletion(-)
>>  create mode 100644 tools/testing/selftests/x86/trivial_program.c
>>
>> diff --git a/tools/testing/selftests/x86/Makefile b/tools/testing/selftests/x86/Makefile
>> index d744991c0f4f..94c89261a236 100644
>> --- a/tools/testing/selftests/x86/Makefile
>> +++ b/tools/testing/selftests/x86/Makefile
>> @@ -8,6 +8,7 @@ include ../lib.mk
>>  UNAME_M := $(shell uname -m)
>>  CAN_BUILD_I386 := $(shell ./check_cc.sh $(CC) trivial_32bit_program.c -m32)
>>  CAN_BUILD_X86_64 := $(shell ./check_cc.sh $(CC) trivial_64bit_program.c)
>> +CAN_BUILD_WITH_NOPIE := $(shell ./check_cc.sh $(CC) trivial_program.c -no-pie)
>>  
>>  TARGETS_C_BOTHBITS := single_step_syscall sysret_ss_attrs syscall_nt test_mremap_vdso \
>>  			check_initial_reg_state sigreturn iopl mpx-mini-test ioperm \
>> @@ -31,7 +32,12 @@ BINARIES_64 := $(TARGETS_C_64BIT_ALL:%=%_64)
>>  BINARIES_32 := $(patsubst %,$(OUTPUT)/%,$(BINARIES_32))
>>  BINARIES_64 := $(patsubst %,$(OUTPUT)/%,$(BINARIES_64))
>>  
>> -CFLAGS := -O2 -g -std=gnu99 -pthread -Wall -no-pie
>> +CFLAGS := -O2 -g -std=gnu99 -pthread -Wall
>> +
>> +# call32_from_64 in thunks.S uses absolute addresses.
>> +ifeq ($(CAN_BUILD_WITH_NOPIE),1)
>> +CFLAGS += -no-pie
>> +endif
>>  
>>  define gen-target-rule-32
>>  $(1) $(1)_32: $(OUTPUT)/$(1)_32
>> diff --git a/tools/testing/selftests/x86/trivial_program.c b/tools/testing/selftests/x86/trivial_program.c
>> new file mode 100644
>> index 000000000000..46a447163b93
>> --- /dev/null
>> +++ b/tools/testing/selftests/x86/trivial_program.c
>> @@ -0,0 +1,10 @@
>> +/* Trivial program to check that compilation with certain flags is working. */
>> +
>> +#include <stdio.h>
>> +
>> +int
>> +main(void)
>> +{
>> +	puts("");
>> +	return 0;
>> +}
>>
> 
> Thanks. I will queue this up for 4.18-rc1
> 
> thahnks,
> -- Shuah
> 

Applied to linux-kselftest next for 4.18-rc1

thanks,
-- Shuah
--
To unsubscribe from this list: send the line "unsubscribe linux-kselftest" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 12+ messages in thread
* Re: [PATCH] selftests/x86: Detect -no-pie availability
@ 2018-05-10 17:30     ` Shuah Khan
  0 siblings, 0 replies; 12+ messages in thread
From: Shuah Khan @ 2018-05-10 17:30 UTC (permalink / raw)
  To: Florian Weimer, linux-x86_64, linux-kselftest, linux-arch, x86
  Cc: rdunlap, Shuah Khan

On 05/04/2018 01:39 PM, Shuah Khan wrote:
> On 05/02/2018 06:10 AM, Florian Weimer wrote:
>> Some toolchains need -no-pie to build all tests, others do not support
>> the -no-pie flag at all.  Therefore, add another test for the
>> availability of the flag.
>>
>> This amends commit 3346a6a4e5ba8c040360f753b26938cec31a4bdc
>> ("selftests: x86: sysret_ss_attrs doesn't build on a PIE build").
>>
>> Signed-off-by: Florian Weimer <fweimer@redhat.com>
>> Acked-by: Randy Dunlap <rdunlap@infradead.org>
>> Tested-by: Randy Dunlap <rdunlap@infradead.org>
>> ---
>>  tools/testing/selftests/x86/Makefile          |  8 +++++++-
>>  tools/testing/selftests/x86/trivial_program.c | 10 ++++++++++
>>  2 files changed, 17 insertions(+), 1 deletion(-)
>>  create mode 100644 tools/testing/selftests/x86/trivial_program.c
>>
>> diff --git a/tools/testing/selftests/x86/Makefile b/tools/testing/selftests/x86/Makefile
>> index d744991c0f4f..94c89261a236 100644
>> --- a/tools/testing/selftests/x86/Makefile
>> +++ b/tools/testing/selftests/x86/Makefile
>> @@ -8,6 +8,7 @@ include ../lib.mk
>>  UNAME_M := $(shell uname -m)
>>  CAN_BUILD_I386 := $(shell ./check_cc.sh $(CC) trivial_32bit_program.c -m32)
>>  CAN_BUILD_X86_64 := $(shell ./check_cc.sh $(CC) trivial_64bit_program.c)
>> +CAN_BUILD_WITH_NOPIE := $(shell ./check_cc.sh $(CC) trivial_program.c -no-pie)
>>  
>>  TARGETS_C_BOTHBITS := single_step_syscall sysret_ss_attrs syscall_nt test_mremap_vdso \
>>  			check_initial_reg_state sigreturn iopl mpx-mini-test ioperm \
>> @@ -31,7 +32,12 @@ BINARIES_64 := $(TARGETS_C_64BIT_ALL:%=%_64)
>>  BINARIES_32 := $(patsubst %,$(OUTPUT)/%,$(BINARIES_32))
>>  BINARIES_64 := $(patsubst %,$(OUTPUT)/%,$(BINARIES_64))
>>  
>> -CFLAGS := -O2 -g -std=gnu99 -pthread -Wall -no-pie
>> +CFLAGS := -O2 -g -std=gnu99 -pthread -Wall
>> +
>> +# call32_from_64 in thunks.S uses absolute addresses.
>> +ifeq ($(CAN_BUILD_WITH_NOPIE),1)
>> +CFLAGS += -no-pie
>> +endif
>>  
>>  define gen-target-rule-32
>>  $(1) $(1)_32: $(OUTPUT)/$(1)_32
>> diff --git a/tools/testing/selftests/x86/trivial_program.c b/tools/testing/selftests/x86/trivial_program.c
>> new file mode 100644
>> index 000000000000..46a447163b93
>> --- /dev/null
>> +++ b/tools/testing/selftests/x86/trivial_program.c
>> @@ -0,0 +1,10 @@
>> +/* Trivial program to check that compilation with certain flags is working. */
>> +
>> +#include <stdio.h>
>> +
>> +int
>> +main(void)
>> +{
>> +	puts("");
>> +	return 0;
>> +}
>>
> 
> Thanks. I will queue this up for 4.18-rc1
> 
> thahnks,
> -- Shuah
> 

Applied to linux-kselftest next for 4.18-rc1

thanks,
-- Shuah

^ permalink raw reply	[flat|nested] 12+ messages in thread
* [PATCH] selftests/x86: Detect -no-pie availability
@ 2018-02-26 14:31 ` Florian Weimer
  0 siblings, 0 replies; 12+ messages in thread
From: fweimer @ 2018-02-26 14:31 UTC (permalink / raw)


Some toolchains need -no-pie to build all tets, others do not support
the -no-pie flag at all.  Therefore, add another test for the
availability of the flag.

This amends commit 3346a6a4e5ba8c040360f753b26938cec31a4bdc
("selftests: x86: sysret_ss_attrs doesn't build on a PIE build").

Signed-off-by: Florian Weimer <fweimer at redhat.com>
---
 tools/testing/selftests/x86/Makefile          |  8 +++++++-
 tools/testing/selftests/x86/trivial_program.c | 10 ++++++++++
 2 files changed, 17 insertions(+), 1 deletion(-)
 create mode 100644 tools/testing/selftests/x86/trivial_program.c

diff --git a/tools/testing/selftests/x86/Makefile b/tools/testing/selftests/x86/Makefile
index d744991c0f4f..94c89261a236 100644
--- a/tools/testing/selftests/x86/Makefile
+++ b/tools/testing/selftests/x86/Makefile
@@ -8,6 +8,7 @@ include ../lib.mk
 UNAME_M := $(shell uname -m)
 CAN_BUILD_I386 := $(shell ./check_cc.sh $(CC) trivial_32bit_program.c -m32)
 CAN_BUILD_X86_64 := $(shell ./check_cc.sh $(CC) trivial_64bit_program.c)
+CAN_BUILD_WITH_NOPIE := $(shell ./check_cc.sh $(CC) trivial_program.c -no-pie)
 
 TARGETS_C_BOTHBITS := single_step_syscall sysret_ss_attrs syscall_nt test_mremap_vdso \
 			check_initial_reg_state sigreturn iopl mpx-mini-test ioperm \
@@ -31,7 +32,12 @@ BINARIES_64 := $(TARGETS_C_64BIT_ALL:%=%_64)
 BINARIES_32 := $(patsubst %,$(OUTPUT)/%,$(BINARIES_32))
 BINARIES_64 := $(patsubst %,$(OUTPUT)/%,$(BINARIES_64))
 
-CFLAGS := -O2 -g -std=gnu99 -pthread -Wall -no-pie
+CFLAGS := -O2 -g -std=gnu99 -pthread -Wall
+
+# call32_from_64 in thunks.S uses absolute addresses.
+ifeq ($(CAN_BUILD_WITH_NOPIE),1)
+CFLAGS += -no-pie
+endif
 
 define gen-target-rule-32
 $(1) $(1)_32: $(OUTPUT)/$(1)_32
diff --git a/tools/testing/selftests/x86/trivial_program.c b/tools/testing/selftests/x86/trivial_program.c
new file mode 100644
index 000000000000..46a447163b93
--- /dev/null
+++ b/tools/testing/selftests/x86/trivial_program.c
@@ -0,0 +1,10 @@
+/* Trivial program to check that compilation with certain flags is working. */
+
+#include <stdio.h>
+
+int
+main(void)
+{
+	puts("");
+	return 0;
+}
-- 
2.14.3

--
To unsubscribe from this list: send the line "unsubscribe linux-kselftest" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply related	[flat|nested] 12+ messages in thread
* [PATCH] selftests/x86: Detect -no-pie availability
@ 2018-02-26 14:31 ` Florian Weimer
  0 siblings, 0 replies; 12+ messages in thread
From: Florian Weimer @ 2018-02-26 14:31 UTC (permalink / raw)


Some toolchains need -no-pie to build all tets, others do not support
the -no-pie flag at all.  Therefore, add another test for the
availability of the flag.

This amends commit 3346a6a4e5ba8c040360f753b26938cec31a4bdc
("selftests: x86: sysret_ss_attrs doesn't build on a PIE build").

Signed-off-by: Florian Weimer <fweimer at redhat.com>
---
 tools/testing/selftests/x86/Makefile          |  8 +++++++-
 tools/testing/selftests/x86/trivial_program.c | 10 ++++++++++
 2 files changed, 17 insertions(+), 1 deletion(-)
 create mode 100644 tools/testing/selftests/x86/trivial_program.c

diff --git a/tools/testing/selftests/x86/Makefile b/tools/testing/selftests/x86/Makefile
index d744991c0f4f..94c89261a236 100644
--- a/tools/testing/selftests/x86/Makefile
+++ b/tools/testing/selftests/x86/Makefile
@@ -8,6 +8,7 @@ include ../lib.mk
 UNAME_M := $(shell uname -m)
 CAN_BUILD_I386 := $(shell ./check_cc.sh $(CC) trivial_32bit_program.c -m32)
 CAN_BUILD_X86_64 := $(shell ./check_cc.sh $(CC) trivial_64bit_program.c)
+CAN_BUILD_WITH_NOPIE := $(shell ./check_cc.sh $(CC) trivial_program.c -no-pie)
 
 TARGETS_C_BOTHBITS := single_step_syscall sysret_ss_attrs syscall_nt test_mremap_vdso \
 			check_initial_reg_state sigreturn iopl mpx-mini-test ioperm \
@@ -31,7 +32,12 @@ BINARIES_64 := $(TARGETS_C_64BIT_ALL:%=%_64)
 BINARIES_32 := $(patsubst %,$(OUTPUT)/%,$(BINARIES_32))
 BINARIES_64 := $(patsubst %,$(OUTPUT)/%,$(BINARIES_64))
 
-CFLAGS := -O2 -g -std=gnu99 -pthread -Wall -no-pie
+CFLAGS := -O2 -g -std=gnu99 -pthread -Wall
+
+# call32_from_64 in thunks.S uses absolute addresses.
+ifeq ($(CAN_BUILD_WITH_NOPIE),1)
+CFLAGS += -no-pie
+endif
 
 define gen-target-rule-32
 $(1) $(1)_32: $(OUTPUT)/$(1)_32
diff --git a/tools/testing/selftests/x86/trivial_program.c b/tools/testing/selftests/x86/trivial_program.c
new file mode 100644
index 000000000000..46a447163b93
--- /dev/null
+++ b/tools/testing/selftests/x86/trivial_program.c
@@ -0,0 +1,10 @@
+/* Trivial program to check that compilation with certain flags is working. */
+
+#include <stdio.h>
+
+int
+main(void)
+{
+	puts("");
+	return 0;
+}
-- 
2.14.3

--
To unsubscribe from this list: send the line "unsubscribe linux-kselftest" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply related	[flat|nested] 12+ messages in thread
* [PATCH] selftests/x86: Detect -no-pie availability
@ 2018-02-26 14:31 ` Florian Weimer
  0 siblings, 0 replies; 12+ messages in thread
From: Florian Weimer @ 2018-02-26 14:31 UTC (permalink / raw)
  To: linux-x86_64, linux-kselftest, linux-arch, x86

Some toolchains need -no-pie to build all tets, others do not support
the -no-pie flag at all.  Therefore, add another test for the
availability of the flag.

This amends commit 3346a6a4e5ba8c040360f753b26938cec31a4bdc
("selftests: x86: sysret_ss_attrs doesn't build on a PIE build").

Signed-off-by: Florian Weimer <fweimer@redhat.com>
---
 tools/testing/selftests/x86/Makefile          |  8 +++++++-
 tools/testing/selftests/x86/trivial_program.c | 10 ++++++++++
 2 files changed, 17 insertions(+), 1 deletion(-)
 create mode 100644 tools/testing/selftests/x86/trivial_program.c

diff --git a/tools/testing/selftests/x86/Makefile b/tools/testing/selftests/x86/Makefile
index d744991c0f4f..94c89261a236 100644
--- a/tools/testing/selftests/x86/Makefile
+++ b/tools/testing/selftests/x86/Makefile
@@ -8,6 +8,7 @@ include ../lib.mk
 UNAME_M := $(shell uname -m)
 CAN_BUILD_I386 := $(shell ./check_cc.sh $(CC) trivial_32bit_program.c -m32)
 CAN_BUILD_X86_64 := $(shell ./check_cc.sh $(CC) trivial_64bit_program.c)
+CAN_BUILD_WITH_NOPIE := $(shell ./check_cc.sh $(CC) trivial_program.c -no-pie)
 
 TARGETS_C_BOTHBITS := single_step_syscall sysret_ss_attrs syscall_nt test_mremap_vdso \
 			check_initial_reg_state sigreturn iopl mpx-mini-test ioperm \
@@ -31,7 +32,12 @@ BINARIES_64 := $(TARGETS_C_64BIT_ALL:%=%_64)
 BINARIES_32 := $(patsubst %,$(OUTPUT)/%,$(BINARIES_32))
 BINARIES_64 := $(patsubst %,$(OUTPUT)/%,$(BINARIES_64))
 
-CFLAGS := -O2 -g -std=gnu99 -pthread -Wall -no-pie
+CFLAGS := -O2 -g -std=gnu99 -pthread -Wall
+
+# call32_from_64 in thunks.S uses absolute addresses.
+ifeq ($(CAN_BUILD_WITH_NOPIE),1)
+CFLAGS += -no-pie
+endif
 
 define gen-target-rule-32
 $(1) $(1)_32: $(OUTPUT)/$(1)_32
diff --git a/tools/testing/selftests/x86/trivial_program.c b/tools/testing/selftests/x86/trivial_program.c
new file mode 100644
index 000000000000..46a447163b93
--- /dev/null
+++ b/tools/testing/selftests/x86/trivial_program.c
@@ -0,0 +1,10 @@
+/* Trivial program to check that compilation with certain flags is working. */
+
+#include <stdio.h>
+
+int
+main(void)
+{
+	puts("");
+	return 0;
+}
-- 
2.14.3

^ permalink raw reply related	[flat|nested] 12+ messages in thread
end of thread, other threads:[~2018-05-10 17:30 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-02 12:10 [PATCH] selftests/x86: Detect -no-pie availability fweimer
2018-05-02 12:10 ` Florian Weimer
2018-05-02 12:10 ` Florian Weimer
2018-05-04 19:39 ` shuah
2018-05-04 19:39   ` Shuah Khan
2018-05-04 19:39   ` Shuah Khan
2018-05-10 17:30   ` shuah
2018-05-10 17:30     ` Shuah Khan
2018-05-10 17:30     ` Shuah Khan
  -- strict thread matches above, loose matches on Subject: below --
2018-02-26 14:31 fweimer
2018-02-26 14:31 ` Florian Weimer
2018-02-26 14:31 ` Florian Weimer

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.