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

[Marvell-armhf] Fix kernel hang due to kernel upgrade to 5.10.103 #11298

Merged
merged 1 commit into from
Jul 7, 2022

Conversation

pavannaregundi
Copy link
Contributor

Change-Id: I015856d173d50d94e30d8c555590efb70eb712ae
Signed-off-by: Pavan Naregundi pnaregundi@marvell.com

Why I did it

Kernel hang was cause due to increase in size of kernel during the upgrade to 5.10.103 and uboot env setting for fdt and initrd address was not good enough.

How I did it

Have given sufficient room for kernel, by moving fdt_addr and initrd_addr.

How to verify it

Verified by loading image on Marvell Armada 380/385.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • [x ] 202205

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

Change-Id: I015856d173d50d94e30d8c555590efb70eb712ae
Signed-off-by: Pavan Naregundi <pnaregundi@marvell.com>
@saiarcot895
Copy link
Contributor

With this change, does it mean that 28MB is reserved for the kernel?

@pavannaregundi
Copy link
Contributor Author

With this change, does it mean that 28MB is reserved for the kernel?

kernel_addr is a location of zImage loaded from disk and uncompressed kernel will be reloaded, So have chosen a safe range. 24M is diff of kernel_addr to fdt_addr.

@radha-danda
Copy link

With this change, does it mean that 28MB is reserved for the kernel?

@saiarcot895, Pavan has responded to your query. do you have any further questions? If not can you please review and merge the PR?

@carl-nokia
Copy link
Contributor

We would like to resume Open Community test suites with images build from the pipeline but need this fix - are we good to merge or is there still actions pending?

@saiarcot895 saiarcot895 merged commit c48e0ae into sonic-net:master Jul 7, 2022
yxieca pushed a commit that referenced this pull request Jul 7, 2022
…1298)

Give more room for the kernel image in memory

Change-Id: I015856d173d50d94e30d8c555590efb70eb712ae
Signed-off-by: Pavan Naregundi <pnaregundi@marvell.com>
skbarista pushed a commit to skbarista/sonic-buildimage that referenced this pull request Aug 17, 2022
…nic-net#11298)

Give more room for the kernel image in memory

Change-Id: I015856d173d50d94e30d8c555590efb70eb712ae
Signed-off-by: Pavan Naregundi <pnaregundi@marvell.com>
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.

6 participants