* [PATCH 1/2] kbuild: dummy-tools: Add elfedit.
@ 2021-04-08 21:31 Michal Suchanek
2021-04-08 21:31 ` [PATCH 2/2] MAINTAINERS: add pattern for dummy-tools Michal Suchanek
2021-04-10 18:12 ` [PATCH 1/2] kbuild: dummy-tools: Add elfedit Masahiro Yamada
0 siblings, 2 replies; 9+ messages in thread
From: Michal Suchanek @ 2021-04-08 21:31 UTC (permalink / raw)
To: linux-kbuild; +Cc: Michal Suchanek, Masahiro Yamada, Michal Marek, linux-kernel
elfedit is used in Makefile
Makefile:GCC_TOOLCHAIN_DIR := $(dir $(shell which $(CROSS_COMPILE)elfedit))
which causes this error getting printed
which: no elfedit in (./scripts/dummy-tools)
Add elfedit to dummy-tools to avoid this error.
Signed-off-by: Michal Suchanek <msuchanek@suse.de>
---
scripts/dummy-tools/elfedit | 1 +
1 file changed, 1 insertion(+)
create mode 120000 scripts/dummy-tools/elfedit
diff --git a/scripts/dummy-tools/elfedit b/scripts/dummy-tools/elfedit
new file mode 120000
index 000000000000..c0648b38dd42
--- /dev/null
+++ b/scripts/dummy-tools/elfedit
@@ -0,0 +1 @@
+ld
\ No newline at end of file
--
2.26.2
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 2/2] MAINTAINERS: add pattern for dummy-tools.
2021-04-08 21:31 [PATCH 1/2] kbuild: dummy-tools: Add elfedit Michal Suchanek
@ 2021-04-08 21:31 ` Michal Suchanek
2021-04-11 12:38 ` Masahiro Yamada
2021-04-10 18:12 ` [PATCH 1/2] kbuild: dummy-tools: Add elfedit Masahiro Yamada
1 sibling, 1 reply; 9+ messages in thread
From: Michal Suchanek @ 2021-04-08 21:31 UTC (permalink / raw)
To: linux-kbuild; +Cc: Michal Suchanek, Masahiro Yamada, Michal Marek, linux-kernel
scripts/get_maintainer.pl does not find a maintainer for new files
otherwise.
Signed-off-by: Michal Suchanek <msuchanek@suse.de>
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index c80ad735b384..ce631ec44e1a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -9690,6 +9690,7 @@ F: scripts/*vmlinux*
F: scripts/Kbuild*
F: scripts/Makefile*
F: scripts/basic/
+F: scripts/dummy-tools/
F: scripts/mk*
F: scripts/mod/
F: scripts/package/
--
2.26.2
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] kbuild: dummy-tools: Add elfedit.
2021-04-08 21:31 [PATCH 1/2] kbuild: dummy-tools: Add elfedit Michal Suchanek
2021-04-08 21:31 ` [PATCH 2/2] MAINTAINERS: add pattern for dummy-tools Michal Suchanek
@ 2021-04-10 18:12 ` Masahiro Yamada
2021-04-11 10:18 ` Michal Suchánek
1 sibling, 1 reply; 9+ messages in thread
From: Masahiro Yamada @ 2021-04-10 18:12 UTC (permalink / raw)
To: Michal Suchanek
Cc: Linux Kbuild mailing list, Michal Marek, Linux Kernel Mailing List
On Fri, Apr 9, 2021 at 6:31 AM Michal Suchanek <msuchanek@suse.de> wrote:
>
> elfedit is used in Makefile
>
> Makefile:GCC_TOOLCHAIN_DIR := $(dir $(shell which $(CROSS_COMPILE)elfedit))
>
> which causes this error getting printed
>
> which: no elfedit in (./scripts/dummy-tools)
I am OK with this patch, but how did you reproduce it?
>
> Add elfedit to dummy-tools to avoid this error.
>
> Signed-off-by: Michal Suchanek <msuchanek@suse.de>
> ---
> scripts/dummy-tools/elfedit | 1 +
> 1 file changed, 1 insertion(+)
> create mode 120000 scripts/dummy-tools/elfedit
>
> diff --git a/scripts/dummy-tools/elfedit b/scripts/dummy-tools/elfedit
> new file mode 120000
> index 000000000000..c0648b38dd42
> --- /dev/null
> +++ b/scripts/dummy-tools/elfedit
> @@ -0,0 +1 @@
> +ld
> \ No newline at end of file
> --
> 2.26.2
>
--
Best Regards
Masahiro Yamada
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] kbuild: dummy-tools: Add elfedit.
2021-04-10 18:12 ` [PATCH 1/2] kbuild: dummy-tools: Add elfedit Masahiro Yamada
@ 2021-04-11 10:18 ` Michal Suchánek
2021-04-11 11:37 ` Masahiro Yamada
0 siblings, 1 reply; 9+ messages in thread
From: Michal Suchánek @ 2021-04-11 10:18 UTC (permalink / raw)
To: Masahiro Yamada
Cc: Linux Kbuild mailing list, Michal Marek, Linux Kernel Mailing List
On Sun, Apr 11, 2021 at 03:12:40AM +0900, Masahiro Yamada wrote:
> On Fri, Apr 9, 2021 at 6:31 AM Michal Suchanek <msuchanek@suse.de> wrote:
> >
> > elfedit is used in Makefile
> >
> > Makefile:GCC_TOOLCHAIN_DIR := $(dir $(shell which $(CROSS_COMPILE)elfedit))
> >
> > which causes this error getting printed
> >
> > which: no elfedit in (./scripts/dummy-tools)
>
>
> I am OK with this patch, but how did you reproduce it?
make ARCH=arm CROSS_COMPILE=scripts/dummy-tools/ allmodconfig
it possibly depends on the config you already have, too.
Thanks
Michal
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] kbuild: dummy-tools: Add elfedit.
2021-04-11 10:18 ` Michal Suchánek
@ 2021-04-11 11:37 ` Masahiro Yamada
2021-04-11 20:03 ` Michal Suchánek
2021-04-12 12:46 ` Michal Suchánek
0 siblings, 2 replies; 9+ messages in thread
From: Masahiro Yamada @ 2021-04-11 11:37 UTC (permalink / raw)
To: Michal Suchánek
Cc: Linux Kbuild mailing list, Michal Marek, Linux Kernel Mailing List
On Sun, Apr 11, 2021 at 7:18 PM Michal Suchánek <msuchanek@suse.de> wrote:
>
> On Sun, Apr 11, 2021 at 03:12:40AM +0900, Masahiro Yamada wrote:
> > On Fri, Apr 9, 2021 at 6:31 AM Michal Suchanek <msuchanek@suse.de> wrote:
> > >
> > > elfedit is used in Makefile
> > >
> > > Makefile:GCC_TOOLCHAIN_DIR := $(dir $(shell which $(CROSS_COMPILE)elfedit))
> > >
> > > which causes this error getting printed
> > >
> > > which: no elfedit in (./scripts/dummy-tools)
> >
> >
> > I am OK with this patch, but how did you reproduce it?
>
> make ARCH=arm CROSS_COMPILE=scripts/dummy-tools/ allmodconfig
>
> it possibly depends on the config you already have, too.
>
> Thanks
>
> Michal
Maybey, are you working on linux-next?
[1]
$ git checkout add74f8473^
$ make ARCH=arm CROSS_COMPILE=scripts/dummy-tools/ allmodconfig
[2]
$ git checkout add74f8473
$ make ARCH=arm CROSS_COMPILE=scripts/dummy-tools/ allmodconfig
If [1] is OK, but [2] is NG,
commit add74f8473 is the root cause.
At least, I do not think this would happen
in the mainline kernel.
--
Best Regards
Masahiro Yamada
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2/2] MAINTAINERS: add pattern for dummy-tools.
2021-04-08 21:31 ` [PATCH 2/2] MAINTAINERS: add pattern for dummy-tools Michal Suchanek
@ 2021-04-11 12:38 ` Masahiro Yamada
0 siblings, 0 replies; 9+ messages in thread
From: Masahiro Yamada @ 2021-04-11 12:38 UTC (permalink / raw)
To: Michal Suchanek
Cc: Linux Kbuild mailing list, Michal Marek, Linux Kernel Mailing List
On Fri, Apr 9, 2021 at 6:31 AM Michal Suchanek <msuchanek@suse.de> wrote:
>
> scripts/get_maintainer.pl does not find a maintainer for new files
> otherwise.
>
> Signed-off-by: Michal Suchanek <msuchanek@suse.de>
> ---
Applied to linux-kbuild. Thanks.
> MAINTAINERS | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index c80ad735b384..ce631ec44e1a 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -9690,6 +9690,7 @@ F: scripts/*vmlinux*
> F: scripts/Kbuild*
> F: scripts/Makefile*
> F: scripts/basic/
> +F: scripts/dummy-tools/
> F: scripts/mk*
> F: scripts/mod/
> F: scripts/package/
> --
> 2.26.2
>
--
Best Regards
Masahiro Yamada
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] kbuild: dummy-tools: Add elfedit.
2021-04-11 11:37 ` Masahiro Yamada
@ 2021-04-11 20:03 ` Michal Suchánek
2021-04-16 19:22 ` Miguel Ojeda
2021-04-12 12:46 ` Michal Suchánek
1 sibling, 1 reply; 9+ messages in thread
From: Michal Suchánek @ 2021-04-11 20:03 UTC (permalink / raw)
To: Masahiro Yamada
Cc: Linux Kbuild mailing list, Michal Marek, Linux Kernel Mailing List
On Sun, Apr 11, 2021 at 08:37:03PM +0900, Masahiro Yamada wrote:
> On Sun, Apr 11, 2021 at 7:18 PM Michal Suchánek <msuchanek@suse.de> wrote:
> >
> > On Sun, Apr 11, 2021 at 03:12:40AM +0900, Masahiro Yamada wrote:
> > > On Fri, Apr 9, 2021 at 6:31 AM Michal Suchanek <msuchanek@suse.de> wrote:
> > > >
> > > > elfedit is used in Makefile
> > > >
> > > > Makefile:GCC_TOOLCHAIN_DIR := $(dir $(shell which $(CROSS_COMPILE)elfedit))
> > > >
> > > > which causes this error getting printed
> > > >
> > > > which: no elfedit in (./scripts/dummy-tools)
> > >
> > >
> > > I am OK with this patch, but how did you reproduce it?
> >
> > make ARCH=arm CROSS_COMPILE=scripts/dummy-tools/ allmodconfig
> >
> > it possibly depends on the config you already have, too.
> >
> > Thanks
> >
> > Michal
>
>
> Maybey, are you working on linux-next?
Yes, I do have the rust support.
>
>
> [1]
> $ git checkout add74f8473^
> $ make ARCH=arm CROSS_COMPILE=scripts/dummy-tools/ allmodconfig
>
> [2]
> $ git checkout add74f8473
> $ make ARCH=arm CROSS_COMPILE=scripts/dummy-tools/ allmodconfig
>
>
>
> If [1] is OK, but [2] is NG,
> commit add74f8473 is the root cause.
Sounds like it. Before that the elfedit test was conditional on clang.
I can try to do the bisect but I think the reason it broke is quite
clear.
Thanks
Michal
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] kbuild: dummy-tools: Add elfedit.
2021-04-11 11:37 ` Masahiro Yamada
2021-04-11 20:03 ` Michal Suchánek
@ 2021-04-12 12:46 ` Michal Suchánek
1 sibling, 0 replies; 9+ messages in thread
From: Michal Suchánek @ 2021-04-12 12:46 UTC (permalink / raw)
To: Masahiro Yamada
Cc: Linux Kbuild mailing list, Michal Marek, Linux Kernel Mailing List
On Sun, Apr 11, 2021 at 08:37:03PM +0900, Masahiro Yamada wrote:
> On Sun, Apr 11, 2021 at 7:18 PM Michal Suchánek <msuchanek@suse.de> wrote:
> >
> > On Sun, Apr 11, 2021 at 03:12:40AM +0900, Masahiro Yamada wrote:
> > > On Fri, Apr 9, 2021 at 6:31 AM Michal Suchanek <msuchanek@suse.de> wrote:
> > > >
> > > > elfedit is used in Makefile
> > > >
> > > > Makefile:GCC_TOOLCHAIN_DIR := $(dir $(shell which $(CROSS_COMPILE)elfedit))
> > > >
> > > > which causes this error getting printed
> > > >
> > > > which: no elfedit in (./scripts/dummy-tools)
> > >
> > >
> > > I am OK with this patch, but how did you reproduce it?
> >
> > make ARCH=arm CROSS_COMPILE=scripts/dummy-tools/ allmodconfig
> >
> > it possibly depends on the config you already have, too.
> >
> > Thanks
> >
> > Michal
>
>
> Maybey, are you working on linux-next?
>
>
> [1]
> $ git checkout add74f8473^
> $ make ARCH=arm CROSS_COMPILE=scripts/dummy-tools/ allmodconfig
>
> [2]
> $ git checkout add74f8473
> $ make ARCH=arm CROSS_COMPILE=scripts/dummy-tools/ allmodconfig
Indeed, the rust support is the cause of the issue:
add74f8473c5ca7d8947c760ff355df991a259bb (HEAD) Rust support
0d02ec6b3136c73c09e7859f0d0e4e2c4c07b49b (tag: v5.12-rc4, s390/master,
s390/linux/master) Linux 5.12-rc4
While v5.12-rc4 is OK add74f8473 produces the error.
There is additional issue that I noticed: dummy tools does not work
seamlessly with O=
$ make CROSS_COMPILE=$(pwd)/scripts/dummy-tools/ O=/scratch/ppc64/ V=1 ARCH=powerpc allmodconfig > /dev/null
which: no elfedit in (/home/michal/linux-2.6/scripts/dummy-tools)
$ make CROSS_COMPILE=scripts/dummy-tools/ O=/scratch/ppc64/ V=1 ARCH=powerpc allmodconfig > /dev/null
which: no elfedit in (./scripts/dummy-tools)
scripts/Kconfig.include:39: compiler 'scripts/dummy-tools/gcc' not found
make[2]: *** [/home/michal/linux-2.6/scripts/kconfig/Makefile:63:
allmodconfig] Error 1
make[1]: *** [/home/michal/linux-2.6/Makefile:636: allmodconfig] Error 2
make: *** [Makefile:222: __sub-make] Error 2
Perhaps linking dummy-tools into the target directory would be a good
idea?
Thanks
Michal
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] kbuild: dummy-tools: Add elfedit.
2021-04-11 20:03 ` Michal Suchánek
@ 2021-04-16 19:22 ` Miguel Ojeda
0 siblings, 0 replies; 9+ messages in thread
From: Miguel Ojeda @ 2021-04-16 19:22 UTC (permalink / raw)
To: Michal Suchánek
Cc: Masahiro Yamada, Linux Kbuild mailing list, Michal Marek,
Linux Kernel Mailing List
On Sun, Apr 11, 2021 at 11:27 PM Michal Suchánek <msuchanek@suse.de> wrote:
>
> Yes, I do have the rust support.
I applied your patch in rust-next and it is in the latest linux-next,
let me know if that helped.
Masahiro, if you want to apply it on your side, just let me know!
Cheers,
Miguel
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2021-04-16 19:22 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-08 21:31 [PATCH 1/2] kbuild: dummy-tools: Add elfedit Michal Suchanek
2021-04-08 21:31 ` [PATCH 2/2] MAINTAINERS: add pattern for dummy-tools Michal Suchanek
2021-04-11 12:38 ` Masahiro Yamada
2021-04-10 18:12 ` [PATCH 1/2] kbuild: dummy-tools: Add elfedit Masahiro Yamada
2021-04-11 10:18 ` Michal Suchánek
2021-04-11 11:37 ` Masahiro Yamada
2021-04-11 20:03 ` Michal Suchánek
2021-04-16 19:22 ` Miguel Ojeda
2021-04-12 12:46 ` Michal Suchánek
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).