diff mbox

x86/vdso: Remove all generated files during clean

Message ID 20150219021311.24366.75477.sendpatchset@little-apple (mailing list archive)
State Not Applicable
Headers show

Commit Message

Magnus Damm Feb. 19, 2015, 2:13 a.m. UTC
From: Magnus Damm <damm+renesas@opensource.se>

Make sure the following files are removed as expected during "make clean":
 + arch/x86/vdso/vdso-image-32-int80.c
 + arch/x86/vdso/vdso-image-32-syscall.c
 + arch/x86/vdso/vdso-image-32-sysenter.c
 + arch/x86/vdso/vdso-image-64.c

Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
---

 arch/x86/vdso/Makefile |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

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

Comments

Sergei Shtylyov Feb. 19, 2015, 11:06 a.m. UTC | #1
Hello.

On 2/19/2015 5:13 AM, Magnus Damm wrote:

> From: Magnus Damm <damm+renesas@opensource.se>

> Make sure the following files are removed as expected during "make clean":
>   + arch/x86/vdso/vdso-image-32-int80.c
>   + arch/x86/vdso/vdso-image-32-syscall.c
>   + arch/x86/vdso/vdso-image-32-sysenter.c
>   + arch/x86/vdso/vdso-image-64.c

> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
> ---
>
>   arch/x86/vdso/Makefile |    4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)

> --- 0001/arch/x86/vdso/Makefile
> +++ work/arch/x86/vdso/Makefile	2015-02-19 10:50:15.975470639 +0900
> @@ -1,3 +1,4 @@
> +

    Stray new line?

>   #
>   # Building vDSO images for x86.
>   #
[...]

WBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Andy Lutomirski Feb. 20, 2015, 7:21 p.m. UTC | #2
On 02/18/2015 06:13 PM, Magnus Damm wrote:
> From: Magnus Damm <damm+renesas@opensource.se>
>
> Make sure the following files are removed as expected during "make clean":
>   + arch/x86/vdso/vdso-image-32-int80.c
>   + arch/x86/vdso/vdso-image-32-syscall.c
>   + arch/x86/vdso/vdso-image-32-sysenter.c
>   + arch/x86/vdso/vdso-image-64.c
>
> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
> ---
>
>   arch/x86/vdso/Makefile |    4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)
>
> --- 0001/arch/x86/vdso/Makefile
> +++ work/arch/x86/vdso/Makefile	2015-02-19 10:50:15.975470639 +0900
> @@ -1,3 +1,4 @@
> +
>   #
>   # Building vDSO images for x86.
>   #
> @@ -206,4 +207,5 @@ $(vdso_img_insttargets): install_%: $(ob
>   PHONY += vdso_install $(vdso_img_insttargets)
>   vdso_install: $(vdso_img_insttargets) FORCE
>
> -clean-files := vdso32-syscall* vdso32-sysenter* vdso32-int80* vdso64*
> +clean-files := vdso32-syscall* vdso32-sysenter* vdso32-int80* vdso64* \
> +               vdso-image-32* vdso-image-64*
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>

Applied, sans the extra newline.

Thanks,
Andy
--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Andy Lutomirski March 26, 2015, 12:53 a.m. UTC | #3
On Fri, Feb 20, 2015 at 11:21 AM, Andy Lutomirski <luto@amacapital.net> wrote:
> On 02/18/2015 06:13 PM, Magnus Damm wrote:
>>
>> From: Magnus Damm <damm+renesas@opensource.se>
>>
>> Make sure the following files are removed as expected during "make clean":
>>   + arch/x86/vdso/vdso-image-32-int80.c
>>   + arch/x86/vdso/vdso-image-32-syscall.c
>>   + arch/x86/vdso/vdso-image-32-sysenter.c
>>   + arch/x86/vdso/vdso-image-64.c
>>
>> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
>> ---
>>
>>   arch/x86/vdso/Makefile |    4 +++-
>>   1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> --- 0001/arch/x86/vdso/Makefile
>> +++ work/arch/x86/vdso/Makefile 2015-02-19 10:50:15.975470639 +0900
>> @@ -1,3 +1,4 @@
>> +
>>   #
>>   # Building vDSO images for x86.
>>   #
>> @@ -206,4 +207,5 @@ $(vdso_img_insttargets): install_%: $(ob
>>   PHONY += vdso_install $(vdso_img_insttargets)
>>   vdso_install: $(vdso_img_insttargets) FORCE
>>
>> -clean-files := vdso32-syscall* vdso32-sysenter* vdso32-int80* vdso64*
>> +clean-files := vdso32-syscall* vdso32-sysenter* vdso32-int80* vdso64* \
>> +               vdso-image-32* vdso-image-64*
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>
>
> Applied, sans the extra newline.

Unapplied and replaced by a different patch that handles x32 correctly.

--Andy

>
> Thanks,
> Andy
diff mbox

Patch

--- 0001/arch/x86/vdso/Makefile
+++ work/arch/x86/vdso/Makefile	2015-02-19 10:50:15.975470639 +0900
@@ -1,3 +1,4 @@ 
+
 #
 # Building vDSO images for x86.
 #
@@ -206,4 +207,5 @@  $(vdso_img_insttargets): install_%: $(ob
 PHONY += vdso_install $(vdso_img_insttargets)
 vdso_install: $(vdso_img_insttargets) FORCE
 
-clean-files := vdso32-syscall* vdso32-sysenter* vdso32-int80* vdso64*
+clean-files := vdso32-syscall* vdso32-sysenter* vdso32-int80* vdso64* \
+               vdso-image-32* vdso-image-64*