Re: [ptxdist] [PATCH] image-root-squashfs: add rules for generic squashfs image

["Baeuerle, Florian" <Florian.Baeuerle@allegion.com>]

Am Montag, den 30.07.2018, 13:56 +0200 schrieb Michael Olbrich:
> On Mon, Jul 30, 2018 at 11:39:05AM +0000, Baeuerle, Florian wrote:
> > This adds rules for generating a squashfs image from the root
> > filesystem via genimage.
> > 
> > Since genimage does not seem to support squashfs-tools v3.x, this
> > package conflicts with that option.
> 
> I think we should remove support for squashfs-tools v3.x from PTXdist
> completely instead. I don't think this is needed.
> 
> Michael

Would you like that being done in this patch, or would you prefer an extra
patch for that?

> 
> > ---
> >  config/images/squashfs.config    |  8 ++++
> >  platforms/image-root-squashfs.in | 77 ++++++++++++++++++++++++++++++++
> >  rules/image-root-squashfs.make   | 41 +++++++++++++++++
> >  3 files changed, 126 insertions(+)
> >  create mode 100644 config/images/squashfs.config
> >  create mode 100644 platforms/image-root-squashfs.in
> >  create mode 100644 rules/image-root-squashfs.make
> > 
> > diff --git a/config/images/squashfs.config b/config/images/squashfs.config
> > new file mode 100644
> > index 000000000..605fcb7a5
> > --- /dev/null
> > +++ b/config/images/squashfs.config
> > @@ -0,0 +1,8 @@
> > +image @IMAGE@ {
> > +	squashfs {
> > +		extraargs = "@EXTRA_ARGS@"
> > +		compression = @COMPRESSION_MODE@
> > +		block-size = @BLOCK_SIZE@
> > +	}
> > +	name = "root"
> > +}
> > diff --git a/platforms/image-root-squashfs.in b/platforms/image-root-squashfs.in
> > new file mode 100644
> > index 000000000..d29540310
> > --- /dev/null
> > +++ b/platforms/image-root-squashfs.in
> > @@ -0,0 +1,77 @@
> > +## SECTION=image
> > +
> > +comment "root.squashfs conflicts with squashfs-3.x"
> > +	depends on HOST_SQUASHFS_TOOLS_V3X
> > +
> > +menuconfig IMAGE_ROOT_SQUASHFS
> > +	tristate
> > +	select HOST_GENIMAGE
> > +	select HOST_SQUASHFS_TOOLS
> > +	select IMAGE_ROOT_TGZ
> > +	depends on !HOST_SQUASHFS_TOOLS_V3X
> > +	prompt "Generate images/root.squashfs "
> > +	help
> > +	  Build squashfs images of the root filesystem. This image can be stored
> > +	  linearly into target's flash device at the start of the desired
> > +	  partition. You should erase the whole partition first if the image
> > +	  is smaller than partition's size. If not, garbage data in the remaining
> > +	  space could confuse the filesystem driver.
> > +
> > +if IMAGE_ROOT_SQUASHFS
> > +
> > +choice
> > +	prompt "squashfs compression mode"
> > +	help
> > +	  Select your prefered compression mode.
> > +
> > +	config IMAGE_ROOT_SQUASHFS_COMPRESSION_MODE_GZIP
> > +	bool "gzip"
> > +	help
> > +	  Select your prefered compression mode.
> > +
> > +	config IMAGE_ROOT_SQUASHFS_COMPRESSION_MODE_LZMA
> > +	bool "lzma"
> > +	help
> > +	  Select your prefered compression mode.
> > +
> > +	config IMAGE_ROOT_SQUASHFS_COMPRESSION_MODE_LZO
> > +	bool "lzo"
> > +	help
> > +	  Select your prefered compression mode.
> > +
> > +	config IMAGE_ROOT_SQUASHFS_COMPRESSION_MODE_LZ4
> > +	bool "lz4"
> > +	help
> > +	  Select your prefered compression mode.
> > +
> > +	config IMAGE_ROOT_SQUASHFS_COMPRESSION_MODE_XZ
> > +	bool "xz"
> > +	help
> > +	  Select your prefered compression mode.
> > +
> > +endchoice
> > +
> > +config IMAGE_ROOT_SQUASHFS_COMPRESSION_MODE
> > +        string
> > +        default "gzip" if IMAGE_ROOT_SQUASHFS_COMPRESSION_MODE_GZIP
> > +        default "lzma" if IMAGE_ROOT_SQUASHFS_COMPRESSION_MODE_LZMA
> > +        default "lzo" if IMAGE_ROOT_SQUASHFS_COMPRESSION_MODE_LZO
> > +        default "lz4" if IMAGE_ROOT_SQUASHFS_COMPRESSION_MODE_LZ4
> > +        default "xz" if IMAGE_ROOT_SQUASHFS_COMPRESSION_MODE_XZ
> > +
> > +config IMAGE_ROOT_SQUASHFS_BLOCK_SIZE
> > +	string
> > +	default "128k"
> > +	prompt "Block size"
> > +	help
> > +	  This allows the compression data block size to be selected, both "K" and "M"
> > +	  postfixes are supported.
> > +
> > +config IMAGE_ROOT_SQUASHFS_EXTRA_ARGS
> > +	string
> > +	default ""
> 
> This is the default. Just remove it.
> 
> > +	prompt "extra arguments passed to mksquashfs"
> > +	help
> > +	  You can add extra arguments for mksquashfs here
> > +
> > +endif
> > diff --git a/rules/image-root-squashfs.make b/rules/image-root-squashfs.make
> > new file mode 100644
> > index 000000000..ce79943f3
> > --- /dev/null
> > +++ b/rules/image-root-squashfs.make
> > @@ -0,0 +1,41 @@
> > +# -*-makefile-*-
> > +#
> > +# Copyright (C) 2018 by Florian Bäuerle <florian.baeuerle@allegion.com>
> > +#
> > +# See CREDITS for details about who has contributed to this project.
> > +#
> > +# For further information about the PTXdist project and license conditions
> > +# see the README file.
> > +#
> > +
> > +#
> > +# We provide this package
> > +#
> > +IMAGE_PACKAGES-$(PTXCONF_IMAGE_ROOT_SQUASHFS) += image-root-squashfs
> > +
> > +#
> > +# Paths and names
> > +#
> > +IMAGE_ROOT_SQUASHFS		:= image-root-squashfs
> > +IMAGE_ROOT_SQUASHFS_DIR		:= $(BUILDDIR)/$(IMAGE_ROOT_SQUASHFS)
> > +IMAGE_ROOT_SQUASHFS_IMAGE	:= $(IMAGEDIR)/root.squashfs
> > +IMAGE_ROOT_SQUASHFS_FILES	:= $(IMAGEDIR)/root.tgz
> > +IMAGE_ROOT_SQUASHFS_CONFIG	:= squashfs.config
> > +
> > +# ----------------------------------------------------------------------------
> > +# Image
> > +# ----------------------------------------------------------------------------
> > +
> > +ifdef PTXCONF_IMAGE_ROOT_SQUASHFS
> 
> This is not needed. I should remove this from the other rules. Something
> like this was necessary when we two ways to generate the same images. That
> is no longer the case.
> 
> Michael
> 
> > +IMAGE_ROOT_SQUASHFS_ENV := \
> > +	EXTRA_ARGS=$(PTXCONF_IMAGE_ROOT_SQUASHFS_EXTRA_ARGS) \
> > +	COMPRESSION_MODE=$(PTXCONF_IMAGE_ROOT_SQUASHFS_COMPRESSION_MODE) \
> > +	BLOCK_SIZE=$(PTXCONF_IMAGE_ROOT_SQUASHFS_BLOCK_SIZE)
> > +
> > +$(IMAGE_ROOT_SQUASHFS_IMAGE):
> > +	@$(call targetinfo)
> > +	@$(call image/genimage, IMAGE_ROOT_SQUASHFS)
> > +	@$(call finish)
> > +endif
> > +
> > +# vim: syntax=make
> > _______________________________________________
> > ptxdist mailing list
> > ptxdist@pengutronix.de
> 
> 
_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de