ACK[1-3]/cmnt: [artful/retpoline-hints 3/3] UBUNTU: [Packaging] include the retpoline extractor in the headers

[Stefan Bader]

On 03.04.2018 18:52, Andy Whitcroft wrote:
> BugLink: http://bugs.launchpad.net/bugs/1760876
> Signed-off-by: Andy Whitcroft <apw at canonical.com>
Acked-by: Stefan Bader <stefan.bader at canonical.com>

> ---

Code looks ok but I really would prefer if there was more text to
the commits which at least roughly explains what is done and why.

-Stefan

>  debian/rules                    | 6 +++++-
>  debian/rules.d/2-binary-arch.mk | 2 ++
>  scripts/Makefile.build          | 2 +-
>  3 files changed, 8 insertions(+), 2 deletions(-)
> 
> diff --git a/debian/rules b/debian/rules
> index 61680be586f0..7519bc4a9d32 100755
> --- a/debian/rules
> +++ b/debian/rules
> @@ -119,6 +119,9 @@ clean: debian/control
>  	# Install the copyright information.
>  	cp $(DEBIAN)/copyright debian/copyright
>  
> +	# Install the retpoline extractor.
> +	cp $(DROOT)/scripts/retpoline-extract-one scripts/ubuntu-retpoline-extract-one
> +
>  	# If we have a reconstruct script use it.
>  	[ -f $(DEBIAN)/reconstruct ] && bash $(DEBIAN)/reconstruct
>  
> @@ -127,7 +130,8 @@ clean: debian/control
>  
>  distclean: clean
>  	rm -rf $(DROOT)/control debian/changelog \
> -		debian/control debian/control.stub debian/copyright
> +		debian/control debian/control.stub debian/copyright \
> +		scripts/ubuntu-retpoline-extract-one
>  
>  # Builds the image, arch headers and debug packages
>  include $(DROOT)/rules.d/2-binary-arch.mk
> diff --git a/debian/rules.d/2-binary-arch.mk b/debian/rules.d/2-binary-arch.mk
> index 6a570ab22f24..9920df16f4d4 100644
> --- a/debian/rules.d/2-binary-arch.mk
> +++ b/debian/rules.d/2-binary-arch.mk
> @@ -331,6 +331,8 @@ ifeq ($(build_arch),arm64)
>  	mkdir -p $(hdrdir)/arch/arm64/lib
>  	cp $(builddir)/build-$*/arch/arm64/kernel/ftrace-mod.o $(hdrdir)/arch/arm64/kernel
>  endif
> +	# Copy over the new retpoline extractor.
> +	cp scripts/ubuntu-retpoline-extract-one $(hdrdir)/scripts
>  	# Script to symlink everything up
>  	$(SHELL) $(DROOT)/scripts/link-headers "$(hdrdir)" "$(indeppkg)" "$*"
>  	# The build symlink
> diff --git a/scripts/Makefile.build b/scripts/Makefile.build
> index 9684c1091e62..d74c3f9f1fa8 100644
> --- a/scripts/Makefile.build
> +++ b/scripts/Makefile.build
> @@ -283,7 +283,7 @@ objtool_dep = $(objtool_obj)					\
>  			 include/config/stack/validation.h)
>  
>  ifdef CONFIG_RETPOLINE
> -cmd_ubuntu_retpoline = $(CONFIG_SHELL) $(srctree)/debian/scripts/retpoline-extract-one $(@) $(<) "$(filter -m16 %code16gcc.h,$(a_flags))";
> +cmd_ubuntu_retpoline = $(CONFIG_SHELL) $(srctree)/scripts/ubuntu-retpoline-extract-one $(@) $(<) "$(filter -m16 %code16gcc.h,$(a_flags))";
>  else
>  cmd_ubuntu_retpoline =
>  endif
> 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20180404/26553a20/attachment.sig>