Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

imx-imx-boot-bootpart.wks.in: Increase /boot partition 64 MiB -> 256 MiB #1962

Merged
merged 1 commit into from
Oct 5, 2024

Conversation

thochstein
Copy link
Contributor

@thochstein thochstein commented Oct 2, 2024

For i.MX 9 a typical kernel is 35 MB. With gcov, that increases to 75 MB
or more. Bump the /boot partition size accordingly.

For i.MX 9 a typical kernel is 35 MB. With gcov, that increases to 75 MB
or more. Bump the /boot partition size accordingly.

Signed-off-by: Tom Hochstein <[email protected]>
@otavio
Copy link
Member

otavio commented Oct 2, 2024

Please improve the commit log or provide more information why do you need to increase the size of it.

Copy link

github-actions bot commented Oct 5, 2024

Successfully created backport PR #1965 for scarthgap.

@Livius90
Copy link
Contributor

Livius90 commented Oct 8, 2024

@thochstein @otavio
Kernel image size can be reduced if KERNEL_IMAGETYPE = "Image.gz" is used.

Other problem is that, i.MX9 uses a default ARM64 defconfig, which is an all-in-one config for kernel build. It has too many unimportant driver and modules which is never used for it.

https://github.com/Freescale/meta-freescale/blob/master/recipes-kernel/linux/linux-fslc_6.11.bb#L35C37-L35C46
https://github.com/Freescale/linux-fslc/blob/6.11.x%2Bfslc/arch/arm64/configs/defconfig

It would be good if somebody from NXP maintaniers can make a minimal defconfig for i.MX9 and for i.MX8 also.

@@ -10,11 +10,11 @@
# - ---------- -------------- --------------
# ^ ^ ^ ^ ^
# | | | | |
# 0 | 8MiB 72MiB 72MiB + rootfs + IMAGE_EXTRA_SPACE (default 10MiB)
# 0 | 8MiB 264MiB 264MiB + rootfs + IMAGE_EXTRA_SPACE (default 10MiB)
Copy link
Contributor Author

@thochstein thochstein Oct 14, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Otavio, we were surprised that the partition is now 333, not 256. The part command includes a default overhead-factor of 1.3, which explains the difference. https://docs.yoctoproject.org/dev/ref-manual/kickstart.html#command-part-or-partition. Perhaps it would be clearer to set the factor to 1.

@JackyBai
Copy link

@thochstein @otavio Kernel image size can be reduced if KERNEL_IMAGETYPE = "Image.gz" is used.

Other problem is that, i.MX9 uses a default ARM64 defconfig, which is an all-in-one config for kernel build. It has too many unimportant driver and modules which is never used for it.

https://github.com/Freescale/meta-freescale/blob/master/recipes-kernel/linux/linux-fslc_6.11.bb#L35C37-L35C46 https://github.com/Freescale/linux-fslc/blob/6.11.x%2Bfslc/arch/arm64/configs/defconfig

It would be good if somebody from NXP maintaniers can make a minimal defconfig for i.MX9 and for i.MX8 also.

There is a imx_v8_defconfig dedicated for NXP i.MX ARMv8 platform.

@Livius90
Copy link
Contributor

@JackyBai
I know that there is a imx_v8_defconfig but it has to many useless driver for an i.MX8 SoC board, it should be refactored. And i.MX9 has no any defconfig yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants