[Buildroot] [PATCH v2 1/2] configs/odroidc2: increase rootfs size to 124M

[Buildroot] [PATCH v2 1/2] configs/odroidc2: increase rootfs size to 124M

[Dagg Stompler]

as result of e2fsprogs version bump, the image size required was
increased.
so to fix it and make sure there is enough size, set BR2_TARGET_ROOTFS_EXT2_SIZE
to 124M

Signed-off-by: Dagg Stompler <daggs@gmx.com>
---

 v1 -> v2:
    reduce size from 256 to 124 (Yann E. MORIN)

 configs/odroidc2_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/configs/odroidc2_defconfig b/configs/odroidc2_defconfig
index 5f97fc3dd8..70e539d6f9 100644
--- a/configs/odroidc2_defconfig
+++ b/configs/odroidc2_defconfig
@@ -10,6 +10,7 @@ BR2_TARGET_GENERIC_GETTY_PORT="ttyAML0"
 BR2_TARGET_GENERIC_GETTY_BAUDRATE_115200=y
 BR2_TARGET_ROOTFS_EXT2=y
 BR2_TARGET_ROOTFS_EXT2_4=y
+BR2_TARGET_ROOTFS_EXT2_SIZE="124M"
 BR2_ROOTFS_OVERLAY="board/hardkernel/odroidc2/rootfs_overlay"
 BR2_ROOTFS_POST_IMAGE_SCRIPT="board/hardkernel/odroidc2/post-image.sh"

--
2.33.0

_______________________________________________
buildroot mailing list
buildroot@lists.buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

[Buildroot] [PATCH 2/2] genimae/odroidc2: increase image size to 128M

[Dagg Stompler]

as a result of BR2_TARGET_ROOTFS_EXT2_SIZE's increase to 124M, increase
the image size to 128M

Thanks to Yann E. MORIN <yann.morin.1998@free.fr> for pointing it out

Signed-off-by: Dagg Stompler <daggs@gmx.com>
---
 board/hardkernel/odroidc2/genimage.cfg | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/board/hardkernel/odroidc2/genimage.cfg b/board/hardkernel/odroidc2/genimage.cfg
index f49c1deaf3..4ef229024a 100644
--- a/board/hardkernel/odroidc2/genimage.cfg
+++ b/board/hardkernel/odroidc2/genimage.cfg
@@ -6,7 +6,7 @@ image boot.vfat {
 			"boot.scr"
 		}
 	}
-	size = 64M
+	size = 128M
 }

 image sdcard.img {
--
2.33.0

_______________________________________________
buildroot mailing list
buildroot@lists.buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH v2 1/2] configs/odroidc2: increase rootfs size to 124M

[Yann E. MORIN]

Dagg,

On 2021-09-11 19:25 +0300, Dagg Stompler spake thusly:
> as result of e2fsprogs version bump, the image size required was
> increased.
> so to fix it and make sure there is enough size, set BR2_TARGET_ROOTFS_EXT2_SIZE
> to 124M

So, I was wondering what the reason for the size increase was, so I
spawned a build in three conditions:

  - my server's Ubuntu 16.04
  - my laptop's Ubuntu 21.04
  - a docker Bullseye on my server (so, minus the kernel, a stock Debian
    Bullseye)

In all three cases, the build succeeded with current master. The sice
of the target/ directory is roughly 40MiB in all three cases, which is
still far enoiugh from 60MiB that it should not cause any issue.

So, I'm still not sure that the reason is "Debian Bullseye", or that the
solution is "increase the filesystem size".

Could you please try by aplying this patch to master, and rerun the
build, and share the error message, please:

    diff --git a/fs/ext2/ext2.mk b/fs/ext2/ext2.mk
    index 74c99bd7b1..83c967c9df 100644
    --- a/fs/ext2/ext2.mk
    +++ b/fs/ext2/ext2.mk
    @@ -16,7 +16,7 @@ EXT2_MKFS_OPTS = $(call qstrip,$(BR2_TARGET_ROOTFS_EXT2_MKFS_OPTIONS))
     EXT2_LABEL = $(subst ",,$(BR2_TARGET_ROOTFS_EXT2_LABEL))
     #" Syntax highlighting... :-/ )
     
    -EXT2_OPTS = \
    +EXT2_OPTS = -v \
     	-d $(TARGET_DIR) \
     	-r $(BR2_TARGET_ROOTFS_EXT2_REV) \
     	-N $(BR2_TARGET_ROOTFS_EXT2_INODES) \

> Signed-off-by: Dagg Stompler <daggs@gmx.com>
> ---
> 
>  v1 -> v2:
>     reduce size from 256 to 124 (Yann E. MORIN)
> 
>  configs/odroidc2_defconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/configs/odroidc2_defconfig b/configs/odroidc2_defconfig
> index 5f97fc3dd8..70e539d6f9 100644
> --- a/configs/odroidc2_defconfig
> +++ b/configs/odroidc2_defconfig
> @@ -10,6 +10,7 @@ BR2_TARGET_GENERIC_GETTY_PORT="ttyAML0"
>  BR2_TARGET_GENERIC_GETTY_BAUDRATE_115200=y
>  BR2_TARGET_ROOTFS_EXT2=y
>  BR2_TARGET_ROOTFS_EXT2_4=y
> +BR2_TARGET_ROOTFS_EXT2_SIZE="124M"

If only this patch is applied, then the build *will* fail, becasue the
size in genimage.cfg is still not fixed. Both sizes must be changed in
the same patch.

Regards,
Yann E. MORIN.

>  BR2_ROOTFS_OVERLAY="board/hardkernel/odroidc2/rootfs_overlay"
>  BR2_ROOTFS_POST_IMAGE_SCRIPT="board/hardkernel/odroidc2/post-image.sh"
> 
> --
> 2.33.0
> 
> _______________________________________________
> buildroot mailing list
> buildroot@lists.buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@lists.buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH 2/2] genimae/odroidc2: increase image size to 128M

[Yann E. MORIN]

Dagg, All,

On 2021-09-11 19:25 +0300, Dagg Stompler spake thusly:
> as a result of BR2_TARGET_ROOTFS_EXT2_SIZE's increase to 124M, increase
> the image size to 128M
> 
> Thanks to Yann E. MORIN <yann.morin.1998@free.fr> for pointing it out
> 
> Signed-off-by: Dagg Stompler <daggs@gmx.com>
> ---
>  board/hardkernel/odroidc2/genimage.cfg | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/board/hardkernel/odroidc2/genimage.cfg b/board/hardkernel/odroidc2/genimage.cfg
> index f49c1deaf3..4ef229024a 100644
> --- a/board/hardkernel/odroidc2/genimage.cfg
> +++ b/board/hardkernel/odroidc2/genimage.cfg
> @@ -6,7 +6,7 @@ image boot.vfat {
>  			"boot.scr"
>  		}
>  	}
> -	size = 64M
> +	size = 128M

Arg, I was completely off on that one; this size only applies to the
vfat partition, not to the whole SDcard image. My deepest apologies
about this...

Regards,
Yann E. MORIN.

>  }
> 
>  image sdcard.img {
> --
> 2.33.0
> 
> _______________________________________________
> buildroot mailing list
> buildroot@lists.buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@lists.buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH v2 1/2] configs/odroidc2: increase rootfs size to 124M

[daggs]

Greetings Yan,

> Sent: Saturday, September 11, 2021 at 10:44 PM
> From: "Yann E. MORIN" <yann.morin.1998@free.fr>
> To: "Dagg Stompler" <daggs@gmx.com>
> Cc: buildroot@buildroot.org
> Subject: Re: [Buildroot] [PATCH v2 1/2] configs/odroidc2: increase rootfs size to 124M
>
> Dagg,
>
> On 2021-09-11 19:25 +0300, Dagg Stompler spake thusly:
> > as result of e2fsprogs version bump, the image size required was
> > increased.
> > so to fix it and make sure there is enough size, set BR2_TARGET_ROOTFS_EXT2_SIZE
> > to 124M
>
> So, I was wondering what the reason for the size increase was, so I
> spawned a build in three conditions:
>
>   - my server's Ubuntu 16.04
>   - my laptop's Ubuntu 21.04
>   - a docker Bullseye on my server (so, minus the kernel, a stock Debian
>     Bullseye)
>
> In all three cases, the build succeeded with current master. The sice
> of the target/ directory is roughly 40MiB in all three cases, which is
> still far enoiugh from 60MiB that it should not cause any issue.
>
> So, I'm still not sure that the reason is "Debian Bullseye", or that the
> solution is "increase the filesystem size".
>
> Could you please try by aplying this patch to master, and rerun the
> build, and share the error message, please:
>
>     diff --git a/fs/ext2/ext2.mk b/fs/ext2/ext2.mk
>     index 74c99bd7b1..83c967c9df 100644
>     --- a/fs/ext2/ext2.mk
>     +++ b/fs/ext2/ext2.mk
>     @@ -16,7 +16,7 @@ EXT2_MKFS_OPTS = $(call qstrip,$(BR2_TARGET_ROOTFS_EXT2_MKFS_OPTIONS))
>      EXT2_LABEL = $(subst ",,$(BR2_TARGET_ROOTFS_EXT2_LABEL))
>      #" Syntax highlighting... :-/ )
>
>     -EXT2_OPTS = \
>     +EXT2_OPTS = -v \
>      	-d $(TARGET_DIR) \
>      	-r $(BR2_TARGET_ROOTFS_EXT2_REV) \
>      	-N $(BR2_TARGET_ROOTFS_EXT2_INODES) \
>
it doesn't reproduces anymore with latest master

Dagg.

_______________________________________________
buildroot mailing list
buildroot@lists.buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot