linux-kernel - Re: Kernel build ignores missing mkimage on building uImage

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Date:	Thu, 6 Jan 2011 14:55:58 +0100
From:	Michal Marek <mmarek@...e.cz>
To:	Roland Stigge <stigge@...com.de>
Cc:	linux-kbuild@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: Kernel build ignores missing mkimage on building uImage

On Fri, Dec 17, 2010 at 05:19:17PM +0100, Roland Stigge wrote:
> Hi,
> 
> on building an uImage, I get:
> 
> $ make uImage
>   CHK     include/linux/version.h
>   CHK     include/generated/utsrelease.h
> make[1]: `include/generated/mach-types.h' is up to date.
>   CALL    scripts/checksyscalls.sh
>   CHK     include/generated/compile.h
>   Kernel: arch/arm/boot/Image is ready
>   SHIPPED arch/arm/boot/compressed/lib1funcs.S
>   AS      arch/arm/boot/compressed/lib1funcs.o
>   LD      arch/arm/boot/compressed/vmlinux
>   OBJCOPY arch/arm/boot/zImage
>   Kernel: arch/arm/boot/zImage is ready
>   UIMAGE  arch/arm/boot/uImage
> "mkimage" command not found - U-Boot images will not be built
>   Image arch/arm/boot/uImage is ready
> $
> 
> I.e. it says: "uImage is ready" even though the uImage file doesn't
> exist because mkimage is missing.
> 
> (I'm working with Linux' current mainline GIT and also see that the
> problem is also present in the mmarek repo on git.kernel.org.)
> 
> I propose the attached patch.
> 
> Thanks for considering.

The patch makes sense and afaics none of the architectures relies on
this behaviour (either they can't work without mkimage, or a
uimage-style target has to be requested by the user). Can I assume
"Signed-off-by: Roland Stigge <stigge@...com.de>" on this patch.

Thanks,
Michal
> 
> bye,
>   Roland

> diff --git a/scripts/mkuboot.sh b/scripts/mkuboot.sh
> index 2e3d3cd..446739c 100755
> --- a/scripts/mkuboot.sh
> +++ b/scripts/mkuboot.sh
> @@ -11,7 +11,7 @@ if [ -z "${MKIMAGE}" ]; then
>  	if [ -z "${MKIMAGE}" ]; then
>  		# Doesn't exist
>  		echo '"mkimage" command not found - U-Boot images will not be built' >&2
> -		exit 0;
> +		exit 1;
>  	fi
>  fi
>  

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/