[tip:perf/urgent] perf arm64: Fix generate system call table failed with /tmp mounted with noexec
Commit-ID: 389373d3306553896a9e218493e5b6175c844eb0 Gitweb: https://git.kernel.org/tip/389373d3306553896a9e218493e5b6175c844eb0 Author: Hongxu Jia AuthorDate: Thu, 18 Oct 2018 16:26:13 +0800 Committer: Arnaldo Carvalho de Melo CommitDate: Fri, 19 Oct 2018 11:56:57 -0300 perf arm64: Fix generate system call table failed with /tmp mounted with noexec When /tmp is mounted with noexec, mksyscalltbl fails. [snip] |perf-1.0/tools/perf/arch/arm64/entry/syscalls//mksyscalltbl: /tmp/create-table-6VGPSt: Permission denied [snip] Add variable TMPDIR as prefix dir of the temporary file, if it is set, replace default /tmp. Signed-off-by: Hongxu Jia Cc: Alexander Shishkin Cc: Hendrik Brueckner Cc: Jiri Olsa Cc: Kim Phillips Cc: Michael Ellerman Cc: Namhyung Kim Cc: Peter Zijlstra Cc: Ravi Bangoria Cc: Sébastien Boisvert Cc: Thomas Richter Fixes: 2b5882435606 ("perf arm64: Generate system call table from asm/unistd.h") LPU-Reference: 1539851173-14959-1-git-send-email-hongxu@windriver.com Link: https://lkml.kernel.org/n/tip-1qrgq840ci0c5cy4oww95...@git.kernel.org Signed-off-by: Arnaldo Carvalho de Melo --- tools/perf/arch/arm64/entry/syscalls/mksyscalltbl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/perf/arch/arm64/entry/syscalls/mksyscalltbl b/tools/perf/arch/arm64/entry/syscalls/mksyscalltbl index 2dbb8cade048..c88fd32563eb 100755 --- a/tools/perf/arch/arm64/entry/syscalls/mksyscalltbl +++ b/tools/perf/arch/arm64/entry/syscalls/mksyscalltbl @@ -23,7 +23,7 @@ create_table_from_c() { local sc nr last_sc - create_table_exe=`mktemp /tmp/create-table-XX` + create_table_exe=`mktemp ${TMPDIR:-/tmp}/create-table-XX` {
[tip:perf/urgent] perf arm64: Fix generate system call table failed with /tmp mounted with noexec
Commit-ID: 389373d3306553896a9e218493e5b6175c844eb0 Gitweb: https://git.kernel.org/tip/389373d3306553896a9e218493e5b6175c844eb0 Author: Hongxu Jia AuthorDate: Thu, 18 Oct 2018 16:26:13 +0800 Committer: Arnaldo Carvalho de Melo CommitDate: Fri, 19 Oct 2018 11:56:57 -0300 perf arm64: Fix generate system call table failed with /tmp mounted with noexec When /tmp is mounted with noexec, mksyscalltbl fails. [snip] |perf-1.0/tools/perf/arch/arm64/entry/syscalls//mksyscalltbl: /tmp/create-table-6VGPSt: Permission denied [snip] Add variable TMPDIR as prefix dir of the temporary file, if it is set, replace default /tmp. Signed-off-by: Hongxu Jia Cc: Alexander Shishkin Cc: Hendrik Brueckner Cc: Jiri Olsa Cc: Kim Phillips Cc: Michael Ellerman Cc: Namhyung Kim Cc: Peter Zijlstra Cc: Ravi Bangoria Cc: Sébastien Boisvert Cc: Thomas Richter Fixes: 2b5882435606 ("perf arm64: Generate system call table from asm/unistd.h") LPU-Reference: 1539851173-14959-1-git-send-email-hongxu@windriver.com Link: https://lkml.kernel.org/n/tip-1qrgq840ci0c5cy4oww95...@git.kernel.org Signed-off-by: Arnaldo Carvalho de Melo --- tools/perf/arch/arm64/entry/syscalls/mksyscalltbl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/perf/arch/arm64/entry/syscalls/mksyscalltbl b/tools/perf/arch/arm64/entry/syscalls/mksyscalltbl index 2dbb8cade048..c88fd32563eb 100755 --- a/tools/perf/arch/arm64/entry/syscalls/mksyscalltbl +++ b/tools/perf/arch/arm64/entry/syscalls/mksyscalltbl @@ -23,7 +23,7 @@ create_table_from_c() { local sc nr last_sc - create_table_exe=`mktemp /tmp/create-table-XX` + create_table_exe=`mktemp ${TMPDIR:-/tmp}/create-table-XX` {
[PATCH] perf arm64 Makefile: remove harmless but unneeded extra slash
Fixes: 2b5882435606 ("perf arm64: Generate system call table from asm/unistd.h") Signed-off-by: Hongxu Jia --- tools/perf/arch/arm64/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/perf/arch/arm64/Makefile b/tools/perf/arch/arm64/Makefile index dbef716..bc2a284 100644 --- a/tools/perf/arch/arm64/Makefile +++ b/tools/perf/arch/arm64/Makefile @@ -13,7 +13,7 @@ out:= $(OUTPUT)arch/arm64/include/generated/asm header := $(out)/syscalls.c incpath := $(srctree)/tools sysdef := $(srctree)/tools/arch/arm64/include/uapi/asm/unistd.h -sysprf := $(srctree)/tools/perf/arch/arm64/entry/syscalls/ +sysprf := $(srctree)/tools/perf/arch/arm64/entry/syscalls systbl := $(sysprf)/mksyscalltbl # Create output directory if not already present -- 2.7.4
[PATCH] perf arm64 Makefile: remove harmless but unneeded extra slash
Fixes: 2b5882435606 ("perf arm64: Generate system call table from asm/unistd.h") Signed-off-by: Hongxu Jia --- tools/perf/arch/arm64/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/perf/arch/arm64/Makefile b/tools/perf/arch/arm64/Makefile index dbef716..bc2a284 100644 --- a/tools/perf/arch/arm64/Makefile +++ b/tools/perf/arch/arm64/Makefile @@ -13,7 +13,7 @@ out:= $(OUTPUT)arch/arm64/include/generated/asm header := $(out)/syscalls.c incpath := $(srctree)/tools sysdef := $(srctree)/tools/arch/arm64/include/uapi/asm/unistd.h -sysprf := $(srctree)/tools/perf/arch/arm64/entry/syscalls/ +sysprf := $(srctree)/tools/perf/arch/arm64/entry/syscalls systbl := $(sysprf)/mksyscalltbl # Create output directory if not already present -- 2.7.4
Re: [PATCH V2] perf arm64: Fix generate system call table failed with /tmp mounted with noexec
On 2018/10/19 下午11:30, Arnaldo Carvalho de Melo wrote: Exactly. So, to move forward, just send a new patch specifically for this part, i.e. the removal of the harmless but unneeded extra slash, the other part I already took care of. Got it, I will send a new patch for the removal of the harmless Thank a huge for all your guys to help me to improve commit message. //Hongxu - Arnaldo
Re: [PATCH V2] perf arm64: Fix generate system call table failed with /tmp mounted with noexec
On 2018/10/19 下午11:30, Arnaldo Carvalho de Melo wrote: Exactly. So, to move forward, just send a new patch specifically for this part, i.e. the removal of the harmless but unneeded extra slash, the other part I already took care of. Got it, I will send a new patch for the removal of the harmless Thank a huge for all your guys to help me to improve commit message. //Hongxu - Arnaldo
[PATCH V2] perf arm64: Fix generate system call table failed with /tmp mounted with noexec
When /tmp is mounted with noexec, mksyscalltbl fails. [snip] |perf-1.0/tools/perf/arch/arm64/entry/syscalls//mksyscalltbl: /tmp/create-table-6VGPSt: Permission denied [snip] Add variable TMPDIR as prefix dir of the temporary file, if it is set, replace default /tmp Remove extra slash from `syscalls//mksyscalltbl' Fixes: 2b5882435606 ("perf arm64: Generate system call table from asm/unistd.h") Signed-off-by: Hongxu Jia --- tools/perf/arch/arm64/Makefile| 2 +- tools/perf/arch/arm64/entry/syscalls/mksyscalltbl | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/perf/arch/arm64/Makefile b/tools/perf/arch/arm64/Makefile index dbef716..bc2a284 100644 --- a/tools/perf/arch/arm64/Makefile +++ b/tools/perf/arch/arm64/Makefile @@ -13,7 +13,7 @@ out:= $(OUTPUT)arch/arm64/include/generated/asm header := $(out)/syscalls.c incpath := $(srctree)/tools sysdef := $(srctree)/tools/arch/arm64/include/uapi/asm/unistd.h -sysprf := $(srctree)/tools/perf/arch/arm64/entry/syscalls/ +sysprf := $(srctree)/tools/perf/arch/arm64/entry/syscalls systbl := $(sysprf)/mksyscalltbl # Create output directory if not already present diff --git a/tools/perf/arch/arm64/entry/syscalls/mksyscalltbl b/tools/perf/arch/arm64/entry/syscalls/mksyscalltbl index 2dbb8cad..c88fd32 100755 --- a/tools/perf/arch/arm64/entry/syscalls/mksyscalltbl +++ b/tools/perf/arch/arm64/entry/syscalls/mksyscalltbl @@ -23,7 +23,7 @@ create_table_from_c() { local sc nr last_sc - create_table_exe=`mktemp /tmp/create-table-XX` + create_table_exe=`mktemp ${TMPDIR:-/tmp}/create-table-XX` { -- 2.7.4
[PATCH V2] perf arm64: Fix generate system call table failed with /tmp mounted with noexec
When /tmp is mounted with noexec, mksyscalltbl fails. [snip] |perf-1.0/tools/perf/arch/arm64/entry/syscalls//mksyscalltbl: /tmp/create-table-6VGPSt: Permission denied [snip] Add variable TMPDIR as prefix dir of the temporary file, if it is set, replace default /tmp Remove extra slash from `syscalls//mksyscalltbl' Fixes: 2b5882435606 ("perf arm64: Generate system call table from asm/unistd.h") Signed-off-by: Hongxu Jia --- tools/perf/arch/arm64/Makefile| 2 +- tools/perf/arch/arm64/entry/syscalls/mksyscalltbl | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/perf/arch/arm64/Makefile b/tools/perf/arch/arm64/Makefile index dbef716..bc2a284 100644 --- a/tools/perf/arch/arm64/Makefile +++ b/tools/perf/arch/arm64/Makefile @@ -13,7 +13,7 @@ out:= $(OUTPUT)arch/arm64/include/generated/asm header := $(out)/syscalls.c incpath := $(srctree)/tools sysdef := $(srctree)/tools/arch/arm64/include/uapi/asm/unistd.h -sysprf := $(srctree)/tools/perf/arch/arm64/entry/syscalls/ +sysprf := $(srctree)/tools/perf/arch/arm64/entry/syscalls systbl := $(sysprf)/mksyscalltbl # Create output directory if not already present diff --git a/tools/perf/arch/arm64/entry/syscalls/mksyscalltbl b/tools/perf/arch/arm64/entry/syscalls/mksyscalltbl index 2dbb8cad..c88fd32 100755 --- a/tools/perf/arch/arm64/entry/syscalls/mksyscalltbl +++ b/tools/perf/arch/arm64/entry/syscalls/mksyscalltbl @@ -23,7 +23,7 @@ create_table_from_c() { local sc nr last_sc - create_table_exe=`mktemp /tmp/create-table-XX` + create_table_exe=`mktemp ${TMPDIR:-/tmp}/create-table-XX` { -- 2.7.4