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