From: Sam Ravnborg <sam@ravnborg.org>
To: Andy Lutomirski <luto@amacapital.net>
Cc: Josh Boyer <jwboyer@fedoraproject.org>,
"H. Peter Anvin" <hpa@zytor.com>, Michal Marek <mmarek@suse.cz>,
"linux-kbuild@vger.kernel.org" <linux-kbuild@vger.kernel.org>,
"Linux-Kernel@Vger. Kernel. Org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 2/2] x86,vdso: Fix vdso_install
Date: Wed, 11 Jun 2014 21:16:09 +0200 [thread overview]
Message-ID: <20140611191609.GA16069@ravnborg.org> (raw)
In-Reply-To: <CALCETrXnGqrPh94zL91um+xC3Hdy1SQmBc6_pOkNBGEmX_2rVw@mail.gmail.com>
> >
> > Can we please fix this in a way where we do not need to add stuff to core kbuild.
> > If the original approach was used then make took care of the looping
> > and the foreach part was not needed.
Following is my first quick approach.
It reuses the old - a little complex logic to do the looping and copying.
Thsi is only tested on my i386 box / build.
Sam
diff --git a/arch/x86/vdso/Makefile b/arch/x86/vdso/Makefile
index 9769df0..39fb152 100644
--- a/arch/x86/vdso/Makefile
+++ b/arch/x86/vdso/Makefile
@@ -9,11 +9,6 @@ VDSOX32-$(CONFIG_X86_X32_ABI) := y
VDSO32-$(CONFIG_X86_32) := y
VDSO32-$(CONFIG_COMPAT) := y
-vdso-install-$(VDSO64-y) += vdso.so
-vdso-install-$(VDSOX32-y) += vdsox32.so
-vdso-install-$(VDSO32-y) += $(vdso32-images)
-
-
# files to link into the vdso
vobjs-y := vdso-note.o vclock_gettime.o vgetcpu.o
@@ -179,12 +174,12 @@ GCOV_PROFILE := n
# Install the unstripped copy of vdso*.so listed in $(vdso-install-y).
#
quiet_cmd_vdso_install = INSTALL $@
- cmd_vdso_install = cp $(obj)/$@.dbg $(MODLIB)/vdso/$@
-$(vdso-install-y): %.so: $(obj)/%.so.dbg FORCE
+ cmd_vdso_install = cp $(obj)/$@ $(MODLIB)/vdso/$(@:.dbg=)
+$(vdso_img_sodbg): %.so.dbg: $(obj)/%.so.dbg FORCE
@mkdir -p $(MODLIB)/vdso
$(call cmd,vdso_install)
-PHONY += vdso_install $(vdso-install-y)
-vdso_install: $(vdso-install-y)
+PHONY += vdso_install $(vdso_img_sodbg)
+vdso_install: $(vdso_img_sodbg)
clean-files := vdso32-syscall* vdso32-sysenter* vdso32-int80*
next prev parent reply other threads:[~2014-06-11 19:16 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-11 14:14 vdso_install target broken post-3.15 Josh Boyer
2014-06-11 15:16 ` Josh Boyer
2014-06-11 15:18 ` H. Peter Anvin
2014-06-11 15:20 ` Josh Boyer
2014-06-11 15:21 ` Andy Lutomirski
2014-06-11 15:24 ` H. Peter Anvin
2014-06-11 15:20 ` Andy Lutomirski
2014-06-11 15:24 ` H. Peter Anvin
2014-06-11 16:20 ` [PATCH 0/2] x86,vdso,kbuild: Fix vdso_install Andy Lutomirski
2014-06-11 16:20 ` [PATCH 1/2] kbuild: Add recipe-cmd, an @-less cmd variant Andy Lutomirski
2014-06-11 16:20 ` [PATCH 2/2] x86,vdso: Fix vdso_install Andy Lutomirski
2014-06-11 17:23 ` Josh Boyer
2014-06-11 17:27 ` H. Peter Anvin
2014-06-11 17:33 ` Josh Boyer
2014-06-11 17:42 ` Andy Lutomirski
2014-06-11 17:45 ` Josh Boyer
2014-06-11 18:45 ` Andy Lutomirski
2014-06-11 18:51 ` Sam Ravnborg
2014-06-11 19:03 ` Andy Lutomirski
2014-06-11 19:16 ` Sam Ravnborg [this message]
2014-06-11 19:18 ` Andy Lutomirski
2014-06-11 16:24 ` [PATCH 0/2] x86,vdso,kbuild: " Josh Boyer
2014-06-11 16:36 ` Sam Ravnborg
2014-06-11 16:41 ` Josh Boyer
2014-06-11 18:44 ` Sam Ravnborg
2014-06-11 19:01 ` Andy Lutomirski
2014-06-17 22:54 ` vdso_install target broken post-3.15 Andy Lutomirski
2014-06-18 3:45 ` Andy Lutomirski
2014-06-18 3:48 ` H. Peter Anvin
2014-06-18 4:22 ` Andy Lutomirski
2014-06-18 13:09 ` Josh Boyer
2014-06-18 15:12 ` Andy Lutomirski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20140611191609.GA16069@ravnborg.org \
--to=sam@ravnborg.org \
--cc=hpa@zytor.com \
--cc=jwboyer@fedoraproject.org \
--cc=linux-kbuild@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=luto@amacapital.net \
--cc=mmarek@suse.cz \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox