Re: [OpenWrt-Devel] [PATCH] x86: fix virutalbox squashfs images
> >> typo in title and missing "R" in name directly above. Ack > > This script is for only your typical block devices, no MTD involved. > > > Looks like you should rather fix the logic setting > > CONFIG_TARGET_IMAGE_PAD. > > This has been removed with > https://git.openwrt.org/d03ef97c1b57b2b5588115d3b7c7355399874aa3, > and this option here was rebase issue. So can I just fix the typo and everyone is okay? ___ openwrt-devel mailing list openwrt-devel@lists.openwrt.org https://lists.openwrt.org/mailman/listinfo/openwrt-devel
Re: [OpenWrt-Devel] [PATCH] x86: fix virutalbox squashfs images
Hi Daniel. W dniu 25.03.2020 o 14:18, Daniel Golle pisze: > Hi, > > On Wed, Mar 25, 2020 at 12:45:08PM +0100, m...@adrianschmutzler.de wrote: >> Hi, >> >>> -Original Message- >>> From: openwrt-devel [mailto:openwrt-devel-boun...@lists.openwrt.org] >>> On Behalf Of Paul Spooren >>> Sent: Mittwoch, 25. März 2020 08:07 >>> To: openwrt-devel@lists.openwrt.org >>> Cc: russ...@personaltelco.net >>> Subject: [OpenWrt-Devel] [PATCH] x86: fix virutalbox squashfs images >>> >>> The previous rework of x86 image creation broke the `vdi` images. ussell >> >> typo in title and missing "R" in name directly above. >> >> Best >> >> Adrian >> >>> Senior came up with this patch to fix the >>> padding. >>> >>> Tested with x86/64 with Docker (squashfs), qemustart (ext4/squashfs) and >>> virtualbox (ext4/squashfs). >>> >>> Signed-off-by: Paul Spooren >>> --- >>> target/linux/x86/image/Makefile | 10 +- >>> 1 file changed, 5 insertions(+), 5 deletions(-) >>> >>> diff --git a/target/linux/x86/image/Makefile >>> b/target/linux/x86/image/Makefile index 81a8f73efc..6d12494f54 100644 >>> --- a/target/linux/x86/image/Makefile >>> +++ b/target/linux/x86/image/Makefile >>> @@ -47,7 +47,7 @@ BOOTOPTS:=$(call qstrip,$(CONFIG_GRUB_BOOTOPTS)) >>> define Build/combined >>> $(CP) $(KDIR)/$(KERNEL_NAME) $@.boot/boot/vmlinuz >>> -$(CP) $(STAGING_DIR_ROOT)/boot/. $@.boot/boot/ >>> - PADDING="$(CONFIG_TARGET_IMAGES_PAD)" >>> SIGNATURE="$(IMG_PART_SIGNATURE)" >>> $(SCRIPT_DIR)/gen_image_generic.sh \ >>> + PADDING="1" SIGNATURE="$(IMG_PART_SIGNATURE)" > > Enabling padding unconditionally wastes space on targets which do not > need it (ie. small size NOR flash devices). This script is for only your typical block devices, no MTD involved. > Looks like you should rather fix the logic setting > CONFIG_TARGET_IMAGE_PAD. This has been removed with https://git.openwrt.org/d03ef97c1b57b2b5588115d3b7c7355399874aa3, and this option here was rebase issue. > > >>> +$(SCRIPT_DIR)/gen_image_generic.sh \ >>> $@ \ >>> $(CONFIG_TARGET_KERNEL_PARTSIZE) $@.boot \ >>> $(CONFIG_TARGET_ROOTFS_PARTSIZE) $(IMAGE_ROOTFS) \ >>> @@ -95,10 +95,10 @@ endef DEVICE_VARS += GRUB2_VARIANT define >>> Device/Default >>>ARTIFACT/image.iso := grub-config iso | iso >>> - IMAGE/combined.img := append-rootfs | pad-extra 128k | grub-config pc | >>> combined | grub-install >>> - IMAGE/combined.img.gz := append-rootfs | pad-extra 128k | grub-config >>> pc | combined | grub-install | gzip >>> - IMAGE/combined.vdi := append-rootfs | pad-extra 128k | grub-config pc | >>> combined | grub-install | qemu-image vdi >>> - IMAGE/combined.vmdk := append-rootfs | pad-extra 128k | grub-config pc >>> | combined | grub-install | qemu-image vmdk >>> + IMAGE/combined.img := grub-config pc | combined | grub-install >>> + IMAGE/combined.img.gz := grub-config pc | combined | grub-install | >>> + gzip IMAGE/combined.vdi := grub-config pc | combined | grub-install | >>> + qemu-image vdi IMAGE/combined.vmdk := grub-config pc | combined | >>> + grub-install | qemu-image vmdk >>>ifeq ($(CONFIG_TARGET_IMAGES_GZIP),y) >>> IMAGES := combined.img.gz >>>else >>> -- >>> 2.25.1 >>> >>> >>> ___ >>> openwrt-devel mailing list >>> openwrt-devel@lists.openwrt.org >>> https://lists.openwrt.org/mailman/listinfo/openwrt-devel > > > >> ___ >> openwrt-devel mailing list >> openwrt-devel@lists.openwrt.org >> https://lists.openwrt.org/mailman/listinfo/openwrt-devel > > > ___ > openwrt-devel mailing list > openwrt-devel@lists.openwrt.org > https://lists.openwrt.org/mailman/listinfo/openwrt-devel > -- TMN ___ openwrt-devel mailing list openwrt-devel@lists.openwrt.org https://lists.openwrt.org/mailman/listinfo/openwrt-devel
Re: [OpenWrt-Devel] [PATCH] x86: fix virutalbox squashfs images
Hi, On Wed, Mar 25, 2020 at 12:45:08PM +0100, m...@adrianschmutzler.de wrote: > Hi, > > > -Original Message- > > From: openwrt-devel [mailto:openwrt-devel-boun...@lists.openwrt.org] > > On Behalf Of Paul Spooren > > Sent: Mittwoch, 25. März 2020 08:07 > > To: openwrt-devel@lists.openwrt.org > > Cc: russ...@personaltelco.net > > Subject: [OpenWrt-Devel] [PATCH] x86: fix virutalbox squashfs images > > > > The previous rework of x86 image creation broke the `vdi` images. ussell > > typo in title and missing "R" in name directly above. > > Best > > Adrian > > > Senior came up with this patch to fix the > > padding. > > > > Tested with x86/64 with Docker (squashfs), qemustart (ext4/squashfs) and > > virtualbox (ext4/squashfs). > > > > Signed-off-by: Paul Spooren > > --- > > target/linux/x86/image/Makefile | 10 +- > > 1 file changed, 5 insertions(+), 5 deletions(-) > > > > diff --git a/target/linux/x86/image/Makefile > > b/target/linux/x86/image/Makefile index 81a8f73efc..6d12494f54 100644 > > --- a/target/linux/x86/image/Makefile > > +++ b/target/linux/x86/image/Makefile > > @@ -47,7 +47,7 @@ BOOTOPTS:=$(call qstrip,$(CONFIG_GRUB_BOOTOPTS)) > > define Build/combined > > $(CP) $(KDIR)/$(KERNEL_NAME) $@.boot/boot/vmlinuz > > -$(CP) $(STAGING_DIR_ROOT)/boot/. $@.boot/boot/ > > - PADDING="$(CONFIG_TARGET_IMAGES_PAD)" > > SIGNATURE="$(IMG_PART_SIGNATURE)" > > $(SCRIPT_DIR)/gen_image_generic.sh \ > > + PADDING="1" SIGNATURE="$(IMG_PART_SIGNATURE)" Enabling padding unconditionally wastes space on targets which do not need it (ie. small size NOR flash devices). Looks like you should rather fix the logic setting CONFIG_TARGET_IMAGE_PAD. > > +$(SCRIPT_DIR)/gen_image_generic.sh \ > > $@ \ > > $(CONFIG_TARGET_KERNEL_PARTSIZE) $@.boot \ > > $(CONFIG_TARGET_ROOTFS_PARTSIZE) $(IMAGE_ROOTFS) \ > > @@ -95,10 +95,10 @@ endef DEVICE_VARS += GRUB2_VARIANT define > > Device/Default > >ARTIFACT/image.iso := grub-config iso | iso > > - IMAGE/combined.img := append-rootfs | pad-extra 128k | grub-config pc | > > combined | grub-install > > - IMAGE/combined.img.gz := append-rootfs | pad-extra 128k | grub-config > > pc | combined | grub-install | gzip > > - IMAGE/combined.vdi := append-rootfs | pad-extra 128k | grub-config pc | > > combined | grub-install | qemu-image vdi > > - IMAGE/combined.vmdk := append-rootfs | pad-extra 128k | grub-config pc > > | combined | grub-install | qemu-image vmdk > > + IMAGE/combined.img := grub-config pc | combined | grub-install > > + IMAGE/combined.img.gz := grub-config pc | combined | grub-install | > > + gzip IMAGE/combined.vdi := grub-config pc | combined | grub-install | > > + qemu-image vdi IMAGE/combined.vmdk := grub-config pc | combined | > > + grub-install | qemu-image vmdk > >ifeq ($(CONFIG_TARGET_IMAGES_GZIP),y) > > IMAGES := combined.img.gz > >else > > -- > > 2.25.1 > > > > > > ___ > > openwrt-devel mailing list > > openwrt-devel@lists.openwrt.org > > https://lists.openwrt.org/mailman/listinfo/openwrt-devel > ___ > openwrt-devel mailing list > openwrt-devel@lists.openwrt.org > https://lists.openwrt.org/mailman/listinfo/openwrt-devel ___ openwrt-devel mailing list openwrt-devel@lists.openwrt.org https://lists.openwrt.org/mailman/listinfo/openwrt-devel
Re: [OpenWrt-Devel] [PATCH] x86: fix virutalbox squashfs images
Hi, > -Original Message- > From: openwrt-devel [mailto:openwrt-devel-boun...@lists.openwrt.org] > On Behalf Of Paul Spooren > Sent: Mittwoch, 25. März 2020 08:07 > To: openwrt-devel@lists.openwrt.org > Cc: russ...@personaltelco.net > Subject: [OpenWrt-Devel] [PATCH] x86: fix virutalbox squashfs images > > The previous rework of x86 image creation broke the `vdi` images. ussell typo in title and missing "R" in name directly above. Best Adrian > Senior came up with this patch to fix the > padding. > > Tested with x86/64 with Docker (squashfs), qemustart (ext4/squashfs) and > virtualbox (ext4/squashfs). > > Signed-off-by: Paul Spooren > --- > target/linux/x86/image/Makefile | 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/target/linux/x86/image/Makefile > b/target/linux/x86/image/Makefile index 81a8f73efc..6d12494f54 100644 > --- a/target/linux/x86/image/Makefile > +++ b/target/linux/x86/image/Makefile > @@ -47,7 +47,7 @@ BOOTOPTS:=$(call qstrip,$(CONFIG_GRUB_BOOTOPTS)) > define Build/combined > $(CP) $(KDIR)/$(KERNEL_NAME) $@.boot/boot/vmlinuz > -$(CP) $(STAGING_DIR_ROOT)/boot/. $@.boot/boot/ > - PADDING="$(CONFIG_TARGET_IMAGES_PAD)" > SIGNATURE="$(IMG_PART_SIGNATURE)" > $(SCRIPT_DIR)/gen_image_generic.sh \ > + PADDING="1" SIGNATURE="$(IMG_PART_SIGNATURE)" > +$(SCRIPT_DIR)/gen_image_generic.sh \ > $@ \ > $(CONFIG_TARGET_KERNEL_PARTSIZE) $@.boot \ > $(CONFIG_TARGET_ROOTFS_PARTSIZE) $(IMAGE_ROOTFS) \ > @@ -95,10 +95,10 @@ endef DEVICE_VARS += GRUB2_VARIANT define > Device/Default >ARTIFACT/image.iso := grub-config iso | iso > - IMAGE/combined.img := append-rootfs | pad-extra 128k | grub-config pc | > combined | grub-install > - IMAGE/combined.img.gz := append-rootfs | pad-extra 128k | grub-config > pc | combined | grub-install | gzip > - IMAGE/combined.vdi := append-rootfs | pad-extra 128k | grub-config pc | > combined | grub-install | qemu-image vdi > - IMAGE/combined.vmdk := append-rootfs | pad-extra 128k | grub-config pc > | combined | grub-install | qemu-image vmdk > + IMAGE/combined.img := grub-config pc | combined | grub-install > + IMAGE/combined.img.gz := grub-config pc | combined | grub-install | > + gzip IMAGE/combined.vdi := grub-config pc | combined | grub-install | > + qemu-image vdi IMAGE/combined.vmdk := grub-config pc | combined | > + grub-install | qemu-image vmdk >ifeq ($(CONFIG_TARGET_IMAGES_GZIP),y) > IMAGES := combined.img.gz >else > -- > 2.25.1 > > > ___ > openwrt-devel mailing list > openwrt-devel@lists.openwrt.org > https://lists.openwrt.org/mailman/listinfo/openwrt-devel openpgp-digital-signature.asc Description: PGP signature ___ openwrt-devel mailing list openwrt-devel@lists.openwrt.org https://lists.openwrt.org/mailman/listinfo/openwrt-devel