linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] sparc: vdso: clean-up vdso Makefile
@ 2018-09-12  3:39 Masahiro Yamada
  2018-10-08  5:57 ` David Miller
  0 siblings, 1 reply; 2+ messages in thread
From: Masahiro Yamada @ 2018-09-12  3:39 UTC (permalink / raw)
  To: David S . Miller, sparclinux
  Cc: Masahiro Yamada, Shannon Nelson, linux-kernel, Nick Alcock,
	Nagarathnam Muthusamy, Greg Kroah-Hartman

arch/sparc/vdso/Makefile is a replica of arch/x86/entry/vdso/Makefile.

Clean-up the Makefile in the same way as I did for x86:

 - Remove unnecessary export
 - Put the generated linker script to $(obj)/ instead of $(src)/
 - Simplify cmd_vdso2c

The corresponding x86 commits are:

 - 61615faf0a89 ("x86/build/vdso: Remove unnecessary export in Makefile")
 - 1742ed2088cc ("x86/build/vdso: Put generated linker scripts to $(obj)/")
 - c5fcdbf15523 ("x86/build/vdso: Simplify 'cmd_vdso2c'")

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
---

 arch/sparc/vdso/Makefile | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/arch/sparc/vdso/Makefile b/arch/sparc/vdso/Makefile
index dd0b5a9..dc85570 100644
--- a/arch/sparc/vdso/Makefile
+++ b/arch/sparc/vdso/Makefile
@@ -31,23 +31,21 @@ obj-y += $(vdso_img_objs)
 targets += $(vdso_img_cfiles)
 targets += $(vdso_img_sodbg) $(vdso_img-y:%=vdso%.so)
 
-export CPPFLAGS_vdso.lds += -P -C
+CPPFLAGS_vdso.lds += -P -C
 
 VDSO_LDFLAGS_vdso.lds = -m64 -Wl,-soname=linux-vdso.so.1 \
 			-Wl,--no-undefined \
 			-Wl,-z,max-page-size=8192 -Wl,-z,common-page-size=8192 \
 			$(DISABLE_LTO)
 
-$(obj)/vdso64.so.dbg: $(src)/vdso.lds $(vobjs) FORCE
+$(obj)/vdso64.so.dbg: $(obj)/vdso.lds $(vobjs) FORCE
 	$(call if_changed,vdso)
 
 HOST_EXTRACFLAGS += -I$(srctree)/tools/include
 hostprogs-y			+= vdso2c
 
 quiet_cmd_vdso2c = VDSO2C  $@
-define cmd_vdso2c
-	$(obj)/vdso2c $< $(<:%.dbg=%) $@
-endef
+      cmd_vdso2c = $(obj)/vdso2c $< $(<:%.dbg=%) $@
 
 $(obj)/vdso-image-%.c: $(obj)/vdso%.so.dbg $(obj)/vdso%.so $(obj)/vdso2c FORCE
 	$(call if_changed,vdso2c)
-- 
2.7.4


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

* Re: [PATCH] sparc: vdso: clean-up vdso Makefile
  2018-09-12  3:39 [PATCH] sparc: vdso: clean-up vdso Makefile Masahiro Yamada
@ 2018-10-08  5:57 ` David Miller
  0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2018-10-08  5:57 UTC (permalink / raw)
  To: yamada.masahiro
  Cc: sparclinux, shannon.nelson, linux-kernel, nick.alcock,
	nagarathnam.muthusamy, gregkh

From: Masahiro Yamada <yamada.masahiro@socionext.com>
Date: Wed, 12 Sep 2018 12:39:13 +0900

> arch/sparc/vdso/Makefile is a replica of arch/x86/entry/vdso/Makefile.
> 
> Clean-up the Makefile in the same way as I did for x86:
> 
>  - Remove unnecessary export
>  - Put the generated linker script to $(obj)/ instead of $(src)/
>  - Simplify cmd_vdso2c
> 
> The corresponding x86 commits are:
> 
>  - 61615faf0a89 ("x86/build/vdso: Remove unnecessary export in Makefile")
>  - 1742ed2088cc ("x86/build/vdso: Put generated linker scripts to $(obj)/")
>  - c5fcdbf15523 ("x86/build/vdso: Simplify 'cmd_vdso2c'")
> 
> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>

Applied.

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

end of thread, other threads:[~2018-10-08  5:57 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-09-12  3:39 [PATCH] sparc: vdso: clean-up vdso Makefile Masahiro Yamada
2018-10-08  5:57 ` David Miller

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).