Re: [PATCH v3 1/3] classes: Add initramfs class

[Harald Seiler <hws@denx.de>]

Hi Florian,

On Mon, 2021-01-18 at 09:11 +0000, florian.bezdeka@siemens.com wrote:
> On Thu, 2021-01-14 at 11:11 +0100, Harald Seiler wrote:
> > Add a new "image" class for generating a custom initramfs.  It works
> > like this: A new minimal debian rootfs is bootstrapped and all
> > dependency packages for the new initramfs are installed.  Then, an
> > initramfs is generated from this rootfs and deployed like usual.
> > 
> > This new initramfs.bbclass "image" class should be pulled in by an
> > "initramfs image" recipe.  Said recipe then specifies all dependencies
> > of the initramfs via INITRAMFS_INSTALL and INITRAMFS_PREINSTALL (which
> > are analogous to the respective IMAGE_* variables).
> > 
> > initramfs.bbclass intentionally does _not_ expose a mechanism to change
> > /etc/initramfs-tools/initramfs.conf and /etc/initramfs-tools/modules.
> > Changes to their settings are better done via packages that deploy
> > conf-hooks to /usr/share/initramfs-tools/conf-hooks.d/ and module
> > fragment files to /usr/share/initramfs-tools/modules.d/.
> > 
> > Signed-off-by: Harald Seiler <hws@denx.de>
> > ---
> > 
[...]
> > +
> > +do_generate_initramfs() {
> > +    rootfs_do_mounts
> > +    rootfs_do_qemu
> > +
> > +    sudo -E chroot "${INITRAMFS_ROOTFS}" \
> > +        update-initramfs -u -v
> > +
> > +    if [ ! -e "${INITRAMFS_ROOTFS}/initrd.img" ]; then
> > +        die "No initramfs was found after generation!"
> > +    fi
> > +
> > +    rm -rf "${INITRAMFS_IMAGE_FILE}"
> > +    cp "${INITRAMFS_ROOTFS}/initrd.img" "${INITRAMFS_IMAGE_FILE}"
> 
> ${INITRAMFS_IMAGE_FILE} references ${DEPLOY_DIR_IMAGE}, which may not
> exist yet. So I guess we should add something like 
> 
> do_generate_initramfs[dirs] = "${DEPLOY_DIR_IMAGE}"
> 
> otherwise you have fix the dependency with some kind of "intermediate
> task" in the recipe using this class.

Yeah, I actually did this exact change in a downstream version of the
recipe but somehow forgot to add it here ... Will fix!

Harald

> > +}
> > +addtask generate_initramfs after do_rootfs before do_build
> > -- 
> > 2.29.2
> > 
> 

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-62  Fax: +49-8142-66989-80   Email: hws@denx.de