[Buildroot] [PATCH] configs/pcengines_apu2_defconfig: linux needs host libelf

[Buildroot] [PATCH] configs/pcengines_apu2_defconfig: linux needs host libelf

[Giulio Benetti]

Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
---
 configs/pcengines_apu2_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/configs/pcengines_apu2_defconfig b/configs/pcengines_apu2_defconfig
index 0792c93c19..a85808554c 100644
--- a/configs/pcengines_apu2_defconfig
+++ b/configs/pcengines_apu2_defconfig
@@ -8,6 +8,7 @@ BR2_LINUX_KERNEL_CUSTOM_VERSION=y
 BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="5.16"
 BR2_LINUX_KERNEL_USE_CUSTOM_CONFIG=y
 BR2_LINUX_KERNEL_CUSTOM_CONFIG_FILE="board/pcengines/apu2/linux.config"
+BR2_LINUX_KERNEL_NEEDS_HOST_LIBELF=y
 
 # Default hostname and TTY
 BR2_TARGET_GENERIC_HOSTNAME="apu2"
-- 
2.25.1

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

[Buildroot] [PATCH] configs/roc_pc_rk3399_defconfig: expand rootfs size to 120M

[Giulio Benetti]

Default 60M for rootfs are not enough and generate a build failure, so
let's expand rootfs size to 120M.

Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
---
 configs/roc_pc_rk3399_defconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/configs/roc_pc_rk3399_defconfig b/configs/roc_pc_rk3399_defconfig
index dfbc5e4f47..ae7cf19a80 100644
--- a/configs/roc_pc_rk3399_defconfig
+++ b/configs/roc_pc_rk3399_defconfig
@@ -45,7 +45,7 @@ BR2_TARGET_GENERIC_HOSTNAME="roc-rk3399-pc"
 BR2_TARGET_GENERIC_ISSUE="Welcome to ROC-RK3399-PC!"
 BR2_TARGET_ROOTFS_EXT2=y
 BR2_TARGET_ROOTFS_EXT2_4=y
-BR2_TARGET_ROOTFS_EXT2_SIZE="64M"
+BR2_TARGET_ROOTFS_EXT2_SIZE="120M"
 BR2_PACKAGE_HOST_DOSFSTOOLS=y
 BR2_PACKAGE_HOST_GENIMAGE=y
 BR2_PACKAGE_HOST_MTOOLS=y
-- 
2.25.1

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

Re: [Buildroot] [PATCH] configs/roc_pc_rk3399_defconfig: expand rootfs size to 120M

[Arnout Vandecappelle]

On 02/02/2022 18:10, Giulio Benetti wrote:
> Default 60M for rootfs are not enough and generate a build failure, so
> let's expand rootfs size to 120M.

  Did you find out _why_ 64M wasn't enough for just the kernel and busybox? Is 
it because of all the stupid modules in the default arm64 defconfig? But most 
other arm64 defconfigs don't seem to need this...

> 
> Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
> ---
>   configs/roc_pc_rk3399_defconfig | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/configs/roc_pc_rk3399_defconfig b/configs/roc_pc_rk3399_defconfig
> index dfbc5e4f47..ae7cf19a80 100644
> --- a/configs/roc_pc_rk3399_defconfig
> +++ b/configs/roc_pc_rk3399_defconfig
> @@ -45,7 +45,7 @@ BR2_TARGET_GENERIC_HOSTNAME="roc-rk3399-pc"
>   BR2_TARGET_GENERIC_ISSUE="Welcome to ROC-RK3399-PC!"
>   BR2_TARGET_ROOTFS_EXT2=y
>   BR2_TARGET_ROOTFS_EXT2_4=y
> -BR2_TARGET_ROOTFS_EXT2_SIZE="64M"
> +BR2_TARGET_ROOTFS_EXT2_SIZE="120M"
>   BR2_PACKAGE_HOST_DOSFSTOOLS=y
>   BR2_PACKAGE_HOST_GENIMAGE=y
>   BR2_PACKAGE_HOST_MTOOLS=y

  Unrelated by this patch, but the image doesn't have a FAT partition so 
dosfstols and mtools aren't needed.

  Regards,
  Arnout

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

Re: [Buildroot] [PATCH] configs/roc_pc_rk3399_defconfig: expand rootfs size to 120M

[Giulio Benetti]

Hi Arnout,

On 13/02/22 12:35, Arnout Vandecappelle wrote:
> 
> 
> On 02/02/2022 18:10, Giulio Benetti wrote:
>> Default 60M for rootfs are not enough and generate a build failure, so
>> let's expand rootfs size to 120M.
> 
>    Did you find out _why_ 64M wasn't enough for just the kernel and busybox? Is
> it because of all the stupid modules in the default arm64 defconfig? But most
> other arm64 defconfigs don't seem to need this...

It is because of modules(30M) and linux "Image", it's not compressed, so 
it's 25M. So only for Linux+modules => 55M, plus the few other 
stuff(Busybox+libc etc.)1M

Also, nothing changed since the beginning, so I don't know how could 
this happened:
# git log --oneline configs/roc_pc_rk3399_defconfig
f51cbc33d9 configs/roc_pc_rk3399: fix U-Boot dependencies
80ae1aedcb configs/roc_rk3399_pc: new defconfig
# git log --oneline roc-rk3399-pc/
80ae1aedcb configs/roc_rk3399_pc: new defconfig

U-Boot dependencies are host dependencies, so it doesn't impact.

# git log --oneline fs/ext2:
5ece6be60b fs/ext2: fir (again) namespace for variables
db7d786140 fs/ext2: fix namespace for variables
5631c38024 fs/ext2: pass a default string to filesystem label
...
here board has been added

So it doesn't seem a ext2 problem too.
I can't understand how it could be building in the beginning.

We have 2 solutions:
- expand rootfs
- use a compressed linux Image.gz(25M => 9M), but need target testing

Honestly I don't think systems with SoCs like RK3399 have problems with 
56M more, so I would go with expanding rootfs to 120M and if maintainer 
in Cc wants to shrink Linux and target testing it is welcome to send a 
patch.

What do you think?

Best regards
-- 
Giulio Benetti
Benetti Engineering sas

>>
>> Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
>> ---
>>    configs/roc_pc_rk3399_defconfig | 2 +-
>>    1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/configs/roc_pc_rk3399_defconfig b/configs/roc_pc_rk3399_defconfig
>> index dfbc5e4f47..ae7cf19a80 100644
>> --- a/configs/roc_pc_rk3399_defconfig
>> +++ b/configs/roc_pc_rk3399_defconfig
>> @@ -45,7 +45,7 @@ BR2_TARGET_GENERIC_HOSTNAME="roc-rk3399-pc"
>>    BR2_TARGET_GENERIC_ISSUE="Welcome to ROC-RK3399-PC!"
>>    BR2_TARGET_ROOTFS_EXT2=y
>>    BR2_TARGET_ROOTFS_EXT2_4=y
>> -BR2_TARGET_ROOTFS_EXT2_SIZE="64M"
>> +BR2_TARGET_ROOTFS_EXT2_SIZE="120M"
>>    BR2_PACKAGE_HOST_DOSFSTOOLS=y
>>    BR2_PACKAGE_HOST_GENIMAGE=y
>>    BR2_PACKAGE_HOST_MTOOLS=y
> 
>    Unrelated by this patch, but the image doesn't have a FAT partition so
> dosfstols and mtools aren't needed.
> 
>    Regards,
>    Arnout
> 

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

Re: [Buildroot] [PATCH] configs/pcengines_apu2_defconfig: linux needs host libelf

[Arnout Vandecappelle]

On 02/02/2022 18:10, Giulio Benetti wrote:
> Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>

  Applied to master, thanks.

  Regards,
  Arnout

> ---
>   configs/pcengines_apu2_defconfig | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/configs/pcengines_apu2_defconfig b/configs/pcengines_apu2_defconfig
> index 0792c93c19..a85808554c 100644
> --- a/configs/pcengines_apu2_defconfig
> +++ b/configs/pcengines_apu2_defconfig
> @@ -8,6 +8,7 @@ BR2_LINUX_KERNEL_CUSTOM_VERSION=y
>   BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="5.16"
>   BR2_LINUX_KERNEL_USE_CUSTOM_CONFIG=y
>   BR2_LINUX_KERNEL_CUSTOM_CONFIG_FILE="board/pcengines/apu2/linux.config"
> +BR2_LINUX_KERNEL_NEEDS_HOST_LIBELF=y
>   
>   # Default hostname and TTY
>   BR2_TARGET_GENERIC_HOSTNAME="apu2"
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH] configs/roc_pc_rk3399_defconfig: expand rootfs size to 120M

[Arnout Vandecappelle]

On 14/02/2022 01:03, Giulio Benetti wrote:
> Hi Arnout,
> 
> On 13/02/22 12:35, Arnout Vandecappelle wrote:
>>
>>
>> On 02/02/2022 18:10, Giulio Benetti wrote:
>>> Default 60M for rootfs are not enough and generate a build failure, so
>>> let's expand rootfs size to 120M.
>>
>>    Did you find out _why_ 64M wasn't enough for just the kernel and busybox? Is
>> it because of all the stupid modules in the default arm64 defconfig? But most
>> other arm64 defconfigs don't seem to need this...
> 
> It is because of modules(30M) and linux "Image", it's not compressed, so it's 
> 25M. So only for Linux+modules => 55M, plus the few other stuff(Busybox+libc 
> etc.)1M

  Yeah those giant aarch64 default configs are mightily annoying...

  I'm thinking, perhaps we should just bite the bullet and move the default ext2 
size to 120M...

  Peter, Yann, what do you think?

  Regards,
  Arnout

> 
> Also, nothing changed since the beginning, so I don't know how could this happened:
> # git log --oneline configs/roc_pc_rk3399_defconfig
> f51cbc33d9 configs/roc_pc_rk3399: fix U-Boot dependencies
> 80ae1aedcb configs/roc_rk3399_pc: new defconfig
> # git log --oneline roc-rk3399-pc/
> 80ae1aedcb configs/roc_rk3399_pc: new defconfig
> 
> U-Boot dependencies are host dependencies, so it doesn't impact.
> 
> # git log --oneline fs/ext2:
> 5ece6be60b fs/ext2: fir (again) namespace for variables
> db7d786140 fs/ext2: fix namespace for variables
> 5631c38024 fs/ext2: pass a default string to filesystem label
> ...
> here board has been added
> 
> So it doesn't seem a ext2 problem too.
> I can't understand how it could be building in the beginning.
> 
> We have 2 solutions:
> - expand rootfs
> - use a compressed linux Image.gz(25M => 9M), but need target testing
> 
> Honestly I don't think systems with SoCs like RK3399 have problems with 56M 
> more, so I would go with expanding rootfs to 120M and if maintainer in Cc wants 
> to shrink Linux and target testing it is welcome to send a patch.
> 
> What do you think?
> 
> Best regards
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH] configs/roc_pc_rk3399_defconfig: expand rootfs size to 120M

[Yann E. MORIN]

Arnout, All,

On 2022-02-14 21:13 +0100, Arnout Vandecappelle spake thusly:
> On 14/02/2022 01:03, Giulio Benetti wrote:
[--SNIP--]
> >It is because of modules(30M) and linux "Image", it's not compressed, so
> >it's 25M. So only for Linux+modules => 55M, plus the few other
> >stuff(Busybox+libc etc.)1M
>  Yeah those giant aarch64 default configs are mightily annoying...
>  I'm thinking, perhaps we should just bite the bullet and move the default
> ext2 size to 120M...
>  Peter, Yann, what do you think?

ACK.

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  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@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH] configs/roc_pc_rk3399_defconfig: expand rootfs size to 120M

[Arnout Vandecappelle]

On 02/02/2022 18:10, Giulio Benetti wrote:
> Default 60M for rootfs are not enough and generate a build failure, so
> let's expand rootfs size to 120M.
> 
> Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
> ---
>   configs/roc_pc_rk3399_defconfig | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/configs/roc_pc_rk3399_defconfig b/configs/roc_pc_rk3399_defconfig
> index dfbc5e4f47..ae7cf19a80 100644
> --- a/configs/roc_pc_rk3399_defconfig
> +++ b/configs/roc_pc_rk3399_defconfig
> @@ -45,7 +45,7 @@ BR2_TARGET_GENERIC_HOSTNAME="roc-rk3399-pc"
>   BR2_TARGET_GENERIC_ISSUE="Welcome to ROC-RK3399-PC!"
>   BR2_TARGET_ROOTFS_EXT2=y
>   BR2_TARGET_ROOTFS_EXT2_4=y
> -BR2_TARGET_ROOTFS_EXT2_SIZE="64M"
> +BR2_TARGET_ROOTFS_EXT2_SIZE="120M"

  Even though I proposed to move the default to 120M, that's for next anyway, 
and this defconfig should be fixed on master, so applied to master, thanks.

  Regards,
  Arnout

>   BR2_PACKAGE_HOST_DOSFSTOOLS=y
>   BR2_PACKAGE_HOST_GENIMAGE=y
>   BR2_PACKAGE_HOST_MTOOLS=y
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH] configs/roc_pc_rk3399_defconfig: expand rootfs size to 120M

[Peter Korsgaard]

>>>>> "Arnout" == Arnout Vandecappelle <arnout@mind.be> writes:

 >> It is because of modules(30M) and linux "Image", it's not
 >> compressed, so it's 25M. So only for Linux+modules => 55M, plus the
 >> few other stuff(Busybox+libc etc.)1M

 >  Yeah those giant aarch64 default configs are mightily annoying...

 >  I'm thinking, perhaps we should just bite the bullet and move the
 > default ext2 size to 120M...

 >  Peter, Yann, what do you think?

Fine by me.

-- 
Bye, Peter Korsgaard
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot