From: anup@brainfault.org (Anup Patel) To: linux-riscv@lists.infradead.org Subject: [PATCH] RISC-V: Build flat and compressed kernel images Date: Sat, 17 Nov 2018 09:02:04 +0530 [thread overview] Message-ID: <CAAhSdy3+w7xQ9=VdqvKa7HZX3FbcAbu4nJ0zXSKi9-4U0_2cNQ@mail.gmail.com> (raw) In-Reply-To: <mhng-4c0df18a-5f01-4260-b3c9-3e4b8cce829b@palmer-si-x1c4> On Sat, Nov 17, 2018 at 2:43 AM Palmer Dabbelt <palmer@sifive.com> wrote: > > On Sun, 11 Nov 2018 21:55:15 PST (-0800), anup at brainfault.org wrote: > > This patch extends Linux RISC-V build system to build and install: > > Image - Flat uncompressed kernel image > > Image.gz - Flat and GZip compressed kernel image > > > > Quiet a few bootloaders (such as Uboot, UEFI, etc) are capable of > > booting flat and compressed kernel images. In case of Uboot, booting > > Image or Image.gz is achieved using bootm command. > > > > The flat and uncompressed kernel image (i.e. Image) is very useful > > in pre-silicon developent and testing because we can create back-door > > HEX files for RAM on FPGAs from Image. > > > > Signed-off-by: Anup Patel <anup@brainfault.org> > > --- > > arch/riscv/Makefile | 15 ++++++++- > > arch/riscv/boot/.gitignore | 2 ++ > > arch/riscv/boot/Makefile | 33 ++++++++++++++++++ > > arch/riscv/boot/install.sh | 60 +++++++++++++++++++++++++++++++++ > > arch/riscv/kernel/head.S | 10 ++++++ > > arch/riscv/kernel/vmlinux.lds.S | 2 +- > > 6 files changed, 120 insertions(+), 2 deletions(-) > > create mode 100644 arch/riscv/boot/.gitignore > > create mode 100644 arch/riscv/boot/Makefile > > create mode 100644 arch/riscv/boot/install.sh > > > > diff --git a/arch/riscv/Makefile b/arch/riscv/Makefile > > index d10146197533..d117a60362eb 100644 > > --- a/arch/riscv/Makefile > > +++ b/arch/riscv/Makefile > > @@ -71,10 +71,23 @@ KBUILD_CFLAGS += $(call cc-option,-mstrict-align) > > # arch specific predefines for sparse > > CHECKFLAGS += -D__riscv -D__riscv_xlen=$(BITS) > > > > +# Default target when executing plain make > > +boot := arch/riscv/boot > > +KBUILD_IMAGE := $(boot)/Image.gz > > + > > head-y := arch/riscv/kernel/head.o > > > > core-y += arch/riscv/kernel/ arch/riscv/mm/ > > > > libs-y += arch/riscv/lib/ > > > > -all: vmlinux > > +all: Image.gz > > + > > +Image: vmlinux > > + $(Q)$(MAKE) $(build)=$(boot) $(boot)/$@ > > + > > +Image.%: Image > > + $(Q)$(MAKE) $(build)=$(boot) $(boot)/$@ > > + > > +zinstall install: > > + $(Q)$(MAKE) $(build)=$(boot) $@ > > diff --git a/arch/riscv/boot/.gitignore b/arch/riscv/boot/.gitignore > > new file mode 100644 > > index 000000000000..8dab0bb6ae66 > > --- /dev/null > > +++ b/arch/riscv/boot/.gitignore > > @@ -0,0 +1,2 @@ > > +Image > > +Image.gz > > diff --git a/arch/riscv/boot/Makefile b/arch/riscv/boot/Makefile > > new file mode 100644 > > index 000000000000..0990a9fdbe5d > > --- /dev/null > > +++ b/arch/riscv/boot/Makefile > > @@ -0,0 +1,33 @@ > > +# > > +# arch/riscv/boot/Makefile > > +# > > +# This file is included by the global makefile so that you can add your own > > +# architecture-specific flags and dependencies. > > +# > > +# This file is subject to the terms and conditions of the GNU General Public > > +# License. See the file "COPYING" in the main directory of this archive > > +# for more details. > > +# > > +# Copyright (C) 2018, Anup Patel. > > +# Author: Anup Patel <anup@brainfault.org> > > +# > > +# Based on the ia64 and arm64 boot/Makefile. > > +# > > + > > +OBJCOPYFLAGS_Image :=-O binary -R .note -R .note.gnu.build-id -R .comment -S > > + > > +targets := Image > > + > > +$(obj)/Image: vmlinux FORCE > > + $(call if_changed,objcopy) > > + > > +$(obj)/Image.gz: $(obj)/Image FORCE > > + $(call if_changed,gzip) > > + > > +install: > > + $(CONFIG_SHELL) $(srctree)/$(src)/install.sh $(KERNELRELEASE) \ > > + $(obj)/Image System.map "$(INSTALL_PATH)" > > + > > +zinstall: > > + $(CONFIG_SHELL) $(srctree)/$(src)/install.sh $(KERNELRELEASE) \ > > + $(obj)/Image.gz System.map "$(INSTALL_PATH)" > > diff --git a/arch/riscv/boot/install.sh b/arch/riscv/boot/install.sh > > new file mode 100644 > > index 000000000000..18c39159c0ff > > --- /dev/null > > +++ b/arch/riscv/boot/install.sh > > @@ -0,0 +1,60 @@ > > +#!/bin/sh > > +# > > +# arch/riscv/boot/install.sh > > +# > > +# This file is subject to the terms and conditions of the GNU General Public > > +# License. See the file "COPYING" in the main directory of this archive > > +# for more details. > > +# > > +# Copyright (C) 1995 by Linus Torvalds > > +# > > +# Adapted from code in arch/i386/boot/Makefile by H. Peter Anvin > > +# Adapted from code in arch/i386/boot/install.sh by Russell King > > +# > > +# "make install" script for the RISC-V Linux port > > +# > > +# Arguments: > > +# $1 - kernel version > > +# $2 - kernel image file > > +# $3 - kernel map file > > +# $4 - default install path (blank if root directory) > > +# > > + > > +verify () { > > + if [ ! -f "$1" ]; then > > + echo "" 1>&2 > > + echo " *** Missing file: $1" 1>&2 > > + echo ' *** You need to run "make" before "make install".' 1>&2 > > + echo "" 1>&2 > > + exit 1 > > + fi > > +} > > + > > +# Make sure the files actually exist > > +verify "$2" > > +verify "$3" > > + > > +# User may have a custom install script > > +if [ -x ~/bin/${INSTALLKERNEL} ]; then exec ~/bin/${INSTALLKERNEL} "$@"; fi > > +if [ -x /sbin/${INSTALLKERNEL} ]; then exec /sbin/${INSTALLKERNEL} "$@"; fi > > + > > +if [ "$(basename $2)" = "Image.gz" ]; then > > +# Compressed install > > + echo "Installing compressed kernel" > > + base=vmlinuz > > +else > > +# Normal install > > + echo "Installing normal kernel" > > + base=vmlinux > > +fi > > + > > +if [ -f $4/$base-$1 ]; then > > + mv $4/$base-$1 $4/$base-$1.old > > +fi > > +cat $2 > $4/$base-$1 > > + > > +# Install system map file > > +if [ -f $4/System.map-$1 ]; then > > + mv $4/System.map-$1 $4/System.map-$1.old > > +fi > > +cp $3 $4/System.map-$1 > > diff --git a/arch/riscv/kernel/head.S b/arch/riscv/kernel/head.S > > index 711190d473d4..fe884cd69abd 100644 > > --- a/arch/riscv/kernel/head.S > > +++ b/arch/riscv/kernel/head.S > > @@ -44,6 +44,16 @@ ENTRY(_start) > > amoadd.w a3, a2, (a3) > > bnez a3, .Lsecondary_start > > > > + /* Clear BSS for flat non-ELF images */ > > + la a3, __bss_start > > + la a4, __bss_stop > > + ble a4, a3, clear_bss_done > > +clear_bss: > > + REG_S zero, (a3) > > + add a3, a3, RISCV_SZPTR > > + blt a3, a4, clear_bss > > +clear_bss_done: > > + > > /* Save hart ID and DTB physical address */ > > mv s0, a0 > > mv s1, a1 > > diff --git a/arch/riscv/kernel/vmlinux.lds.S b/arch/riscv/kernel/vmlinux.lds.S > > index ece84991609c..65df1dfdc303 100644 > > --- a/arch/riscv/kernel/vmlinux.lds.S > > +++ b/arch/riscv/kernel/vmlinux.lds.S > > @@ -74,7 +74,7 @@ SECTIONS > > *(.sbss*) > > } > > > > - BSS_SECTION(0, 0, 0) > > + BSS_SECTION(PAGE_SIZE, PAGE_SIZE, 0) > > What does this do? If bootloader is loading vmlinux ELF then it will take care of zeroing BSS and SBSS section. If we have FLAT image then the head.S has to zero-out BSS and SBSS on boot HART. The secondary/non-boot HART don't need to clear BSS and SBSS. Now with BSS_SECTION(0, 0, 0), there is no alignment constraint on start of BSS and SBSS section so the BSS zeroing code in head.S has handle unaligned BSS and SBSS (i.e. not aligned to XLEN / 8 bytes). To simplify, I have made BSS and SBSS start as page aligned and put a simple assembly code in head.S to zero-out BSS and SBSS. Of course, we can also use BSS_SECTION(16, 16, 0) but I thought of avoiding magic values and made it page aligned (like few other architectures). > > > > > EXCEPTION_TABLE(0x10) > > NOTES > > Thanks! I'm going to target this for the RCs as well: it's technically a new > feature, but it seems pretty safe -- assuming the BSS_SECTION change can be > explained to me :) Thanks, Anup
WARNING: multiple messages have this Message-ID (diff)
From: Anup Patel <anup@brainfault.org> To: Palmer Dabbelt <palmer@sifive.com> Cc: Christoph Hellwig <hch@infradead.org>, Atish Patra <atish.patra@wdc.com>, linux-riscv@lists.infradead.org, Albert Ou <aou@eecs.berkeley.edu>, "linux-kernel@vger.kernel.org List" <linux-kernel@vger.kernel.org> Subject: Re: [PATCH] RISC-V: Build flat and compressed kernel images Date: Sat, 17 Nov 2018 09:02:04 +0530 [thread overview] Message-ID: <CAAhSdy3+w7xQ9=VdqvKa7HZX3FbcAbu4nJ0zXSKi9-4U0_2cNQ@mail.gmail.com> (raw) Message-ID: <20181117033204.3oeLcVm9zGUZM0lpV92Tar43KZEGRkEX5lzKkubOV20@z> (raw) In-Reply-To: <mhng-4c0df18a-5f01-4260-b3c9-3e4b8cce829b@palmer-si-x1c4> On Sat, Nov 17, 2018 at 2:43 AM Palmer Dabbelt <palmer@sifive.com> wrote: > > On Sun, 11 Nov 2018 21:55:15 PST (-0800), anup@brainfault.org wrote: > > This patch extends Linux RISC-V build system to build and install: > > Image - Flat uncompressed kernel image > > Image.gz - Flat and GZip compressed kernel image > > > > Quiet a few bootloaders (such as Uboot, UEFI, etc) are capable of > > booting flat and compressed kernel images. In case of Uboot, booting > > Image or Image.gz is achieved using bootm command. > > > > The flat and uncompressed kernel image (i.e. Image) is very useful > > in pre-silicon developent and testing because we can create back-door > > HEX files for RAM on FPGAs from Image. > > > > Signed-off-by: Anup Patel <anup@brainfault.org> > > --- > > arch/riscv/Makefile | 15 ++++++++- > > arch/riscv/boot/.gitignore | 2 ++ > > arch/riscv/boot/Makefile | 33 ++++++++++++++++++ > > arch/riscv/boot/install.sh | 60 +++++++++++++++++++++++++++++++++ > > arch/riscv/kernel/head.S | 10 ++++++ > > arch/riscv/kernel/vmlinux.lds.S | 2 +- > > 6 files changed, 120 insertions(+), 2 deletions(-) > > create mode 100644 arch/riscv/boot/.gitignore > > create mode 100644 arch/riscv/boot/Makefile > > create mode 100644 arch/riscv/boot/install.sh > > > > diff --git a/arch/riscv/Makefile b/arch/riscv/Makefile > > index d10146197533..d117a60362eb 100644 > > --- a/arch/riscv/Makefile > > +++ b/arch/riscv/Makefile > > @@ -71,10 +71,23 @@ KBUILD_CFLAGS += $(call cc-option,-mstrict-align) > > # arch specific predefines for sparse > > CHECKFLAGS += -D__riscv -D__riscv_xlen=$(BITS) > > > > +# Default target when executing plain make > > +boot := arch/riscv/boot > > +KBUILD_IMAGE := $(boot)/Image.gz > > + > > head-y := arch/riscv/kernel/head.o > > > > core-y += arch/riscv/kernel/ arch/riscv/mm/ > > > > libs-y += arch/riscv/lib/ > > > > -all: vmlinux > > +all: Image.gz > > + > > +Image: vmlinux > > + $(Q)$(MAKE) $(build)=$(boot) $(boot)/$@ > > + > > +Image.%: Image > > + $(Q)$(MAKE) $(build)=$(boot) $(boot)/$@ > > + > > +zinstall install: > > + $(Q)$(MAKE) $(build)=$(boot) $@ > > diff --git a/arch/riscv/boot/.gitignore b/arch/riscv/boot/.gitignore > > new file mode 100644 > > index 000000000000..8dab0bb6ae66 > > --- /dev/null > > +++ b/arch/riscv/boot/.gitignore > > @@ -0,0 +1,2 @@ > > +Image > > +Image.gz > > diff --git a/arch/riscv/boot/Makefile b/arch/riscv/boot/Makefile > > new file mode 100644 > > index 000000000000..0990a9fdbe5d > > --- /dev/null > > +++ b/arch/riscv/boot/Makefile > > @@ -0,0 +1,33 @@ > > +# > > +# arch/riscv/boot/Makefile > > +# > > +# This file is included by the global makefile so that you can add your own > > +# architecture-specific flags and dependencies. > > +# > > +# This file is subject to the terms and conditions of the GNU General Public > > +# License. See the file "COPYING" in the main directory of this archive > > +# for more details. > > +# > > +# Copyright (C) 2018, Anup Patel. > > +# Author: Anup Patel <anup@brainfault.org> > > +# > > +# Based on the ia64 and arm64 boot/Makefile. > > +# > > + > > +OBJCOPYFLAGS_Image :=-O binary -R .note -R .note.gnu.build-id -R .comment -S > > + > > +targets := Image > > + > > +$(obj)/Image: vmlinux FORCE > > + $(call if_changed,objcopy) > > + > > +$(obj)/Image.gz: $(obj)/Image FORCE > > + $(call if_changed,gzip) > > + > > +install: > > + $(CONFIG_SHELL) $(srctree)/$(src)/install.sh $(KERNELRELEASE) \ > > + $(obj)/Image System.map "$(INSTALL_PATH)" > > + > > +zinstall: > > + $(CONFIG_SHELL) $(srctree)/$(src)/install.sh $(KERNELRELEASE) \ > > + $(obj)/Image.gz System.map "$(INSTALL_PATH)" > > diff --git a/arch/riscv/boot/install.sh b/arch/riscv/boot/install.sh > > new file mode 100644 > > index 000000000000..18c39159c0ff > > --- /dev/null > > +++ b/arch/riscv/boot/install.sh > > @@ -0,0 +1,60 @@ > > +#!/bin/sh > > +# > > +# arch/riscv/boot/install.sh > > +# > > +# This file is subject to the terms and conditions of the GNU General Public > > +# License. See the file "COPYING" in the main directory of this archive > > +# for more details. > > +# > > +# Copyright (C) 1995 by Linus Torvalds > > +# > > +# Adapted from code in arch/i386/boot/Makefile by H. Peter Anvin > > +# Adapted from code in arch/i386/boot/install.sh by Russell King > > +# > > +# "make install" script for the RISC-V Linux port > > +# > > +# Arguments: > > +# $1 - kernel version > > +# $2 - kernel image file > > +# $3 - kernel map file > > +# $4 - default install path (blank if root directory) > > +# > > + > > +verify () { > > + if [ ! -f "$1" ]; then > > + echo "" 1>&2 > > + echo " *** Missing file: $1" 1>&2 > > + echo ' *** You need to run "make" before "make install".' 1>&2 > > + echo "" 1>&2 > > + exit 1 > > + fi > > +} > > + > > +# Make sure the files actually exist > > +verify "$2" > > +verify "$3" > > + > > +# User may have a custom install script > > +if [ -x ~/bin/${INSTALLKERNEL} ]; then exec ~/bin/${INSTALLKERNEL} "$@"; fi > > +if [ -x /sbin/${INSTALLKERNEL} ]; then exec /sbin/${INSTALLKERNEL} "$@"; fi > > + > > +if [ "$(basename $2)" = "Image.gz" ]; then > > +# Compressed install > > + echo "Installing compressed kernel" > > + base=vmlinuz > > +else > > +# Normal install > > + echo "Installing normal kernel" > > + base=vmlinux > > +fi > > + > > +if [ -f $4/$base-$1 ]; then > > + mv $4/$base-$1 $4/$base-$1.old > > +fi > > +cat $2 > $4/$base-$1 > > + > > +# Install system map file > > +if [ -f $4/System.map-$1 ]; then > > + mv $4/System.map-$1 $4/System.map-$1.old > > +fi > > +cp $3 $4/System.map-$1 > > diff --git a/arch/riscv/kernel/head.S b/arch/riscv/kernel/head.S > > index 711190d473d4..fe884cd69abd 100644 > > --- a/arch/riscv/kernel/head.S > > +++ b/arch/riscv/kernel/head.S > > @@ -44,6 +44,16 @@ ENTRY(_start) > > amoadd.w a3, a2, (a3) > > bnez a3, .Lsecondary_start > > > > + /* Clear BSS for flat non-ELF images */ > > + la a3, __bss_start > > + la a4, __bss_stop > > + ble a4, a3, clear_bss_done > > +clear_bss: > > + REG_S zero, (a3) > > + add a3, a3, RISCV_SZPTR > > + blt a3, a4, clear_bss > > +clear_bss_done: > > + > > /* Save hart ID and DTB physical address */ > > mv s0, a0 > > mv s1, a1 > > diff --git a/arch/riscv/kernel/vmlinux.lds.S b/arch/riscv/kernel/vmlinux.lds.S > > index ece84991609c..65df1dfdc303 100644 > > --- a/arch/riscv/kernel/vmlinux.lds.S > > +++ b/arch/riscv/kernel/vmlinux.lds.S > > @@ -74,7 +74,7 @@ SECTIONS > > *(.sbss*) > > } > > > > - BSS_SECTION(0, 0, 0) > > + BSS_SECTION(PAGE_SIZE, PAGE_SIZE, 0) > > What does this do? If bootloader is loading vmlinux ELF then it will take care of zeroing BSS and SBSS section. If we have FLAT image then the head.S has to zero-out BSS and SBSS on boot HART. The secondary/non-boot HART don't need to clear BSS and SBSS. Now with BSS_SECTION(0, 0, 0), there is no alignment constraint on start of BSS and SBSS section so the BSS zeroing code in head.S has handle unaligned BSS and SBSS (i.e. not aligned to XLEN / 8 bytes). To simplify, I have made BSS and SBSS start as page aligned and put a simple assembly code in head.S to zero-out BSS and SBSS. Of course, we can also use BSS_SECTION(16, 16, 0) but I thought of avoiding magic values and made it page aligned (like few other architectures). > > > > > EXCEPTION_TABLE(0x10) > > NOTES > > Thanks! I'm going to target this for the RCs as well: it's technically a new > feature, but it seems pretty safe -- assuming the BSS_SECTION change can be > explained to me :) Thanks, Anup _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv
next prev parent reply other threads:[~2018-11-17 3:32 UTC|newest] Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-11-12 5:55 [PATCH] RISC-V: Build flat and compressed kernel images Anup Patel 2018-11-12 5:55 ` Anup Patel 2018-11-16 21:13 ` Palmer Dabbelt 2018-11-16 21:13 ` Palmer Dabbelt 2018-11-17 3:32 ` Anup Patel [this message] 2018-11-17 3:32 ` Anup Patel 2018-11-19 20:10 ` Palmer Dabbelt 2018-11-19 20:10 ` Palmer Dabbelt 2018-11-21 5:06 ` Bin Meng 2018-11-21 5:06 ` Bin Meng 2018-11-21 16:28 ` Palmer Dabbelt 2018-11-21 16:28 ` Palmer Dabbelt 2018-11-22 1:43 ` Bin Meng 2018-11-22 1:43 ` Bin Meng
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='CAAhSdy3+w7xQ9=VdqvKa7HZX3FbcAbu4nJ0zXSKi9-4U0_2cNQ@mail.gmail.com' \ --to=anup@brainfault.org \ --cc=linux-riscv@lists.infradead.org \ /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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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).