[ptxdist] [PATCH] rules: kernel: set CROSS_STRIP to kernel toolchain

[ptxdist] [PATCH] rules: kernel: set CROSS_STRIP to kernel toolchain

[Steffen Trumtrar]

Overwrite the CROSS_STRIP environment variable.
This is needed if the kernel is build with a different toolchain
than the userspace. Stripping of kernel modules will fail otherwise,
because the kernel will try to use strip from the userspace toolchain.

Signed-off-by: Steffen Trumtrar <s.trumtrar@pengutronix.de>
---
 rules/kernel.make | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/rules/kernel.make b/rules/kernel.make
index 1e445559b490..874198c48e99 100644
--- a/rules/kernel.make
+++ b/rules/kernel.make
@@ -262,6 +262,8 @@ $(STATEDIR)/kernel.targetinstall.post: $(IMAGEDIR)/linuximage.lzo
 endif
 endif
 
+$(STATEDIR)/kernel.targetinstall.post: \
+	CROSS_STRIP=$(KERNEL_CROSS_COMPILE)strip
 $(STATEDIR)/kernel.targetinstall.post:
 	@$(call targetinfo)
 
-- 
2.20.1


_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

Re: [ptxdist] [PATCH] rules: kernel: set CROSS_STRIP to kernel toolchain

[Michael Olbrich]

On Fri, Jul 12, 2019 at 03:02:29PM +0200, Steffen Trumtrar wrote:
> Overwrite the CROSS_STRIP environment variable.
> This is needed if the kernel is build with a different toolchain
> than the userspace. Stripping of kernel modules will fail otherwise,
> because the kernel will try to use strip from the userspace toolchain.

It's not that simple. How did you test this? I'm pretty sure that this will
fail if e.g. perf is enabled.

- target-specific variables are propagated to any prerequisites. This is
  not acceptable here
- If the strip is incompatible, then I'm sure that building the userspace
  tools from the kernel with the kernel toolchain is not correct either.

This requires a more complex solution.

Michael

> Signed-off-by: Steffen Trumtrar <s.trumtrar@pengutronix.de>
> ---
>  rules/kernel.make | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/rules/kernel.make b/rules/kernel.make
> index 1e445559b490..874198c48e99 100644
> --- a/rules/kernel.make
> +++ b/rules/kernel.make
> @@ -262,6 +262,8 @@ $(STATEDIR)/kernel.targetinstall.post: $(IMAGEDIR)/linuximage.lzo
>  endif
>  endif
>  
> +$(STATEDIR)/kernel.targetinstall.post: \
> +	CROSS_STRIP=$(KERNEL_CROSS_COMPILE)strip
>  $(STATEDIR)/kernel.targetinstall.post:
>  	@$(call targetinfo)
>  
> -- 
> 2.20.1
> 
> 
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de
> 

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de