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

ARM physical heap: exclude all identity-mapped memory #2087

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

francescolavra
Copy link
Member

The memory range that is identity-mapped during boot is used, among other things, as temporary stack for both the boot processor and the secondary processors; therefore, it should not be included in the physical memory ranges, otherwise a secondary processor can corrupt memory that has been allocated for other kernel uses. This fixes unhandled exceptions that cause boot failures when running the kernel on Azure Standard_D2ps_v5 instances.
In addition, the Azure cloud_init code is being amended to avoid printing a potentially misleading error message if an unsuccessful attempt to connect to the Azure Wire Server is made before the network interface acquires a DHCP address.

When an Azure instance starts, a few seconds may elapse before its
network interface acquires a DHCP address, and during this time
interval the Azure Wire Server is unreachable. To avoid potentially
misleading log output, do not print error messages if connection to
the server fails at the first attempt.
The memory range that is identity-mapped during boot is used, among
other things, as temporary stack for both the boot processor and
the secondary processors; theefore, it should not be included in
the physical memory ranges, otherwise a secondary processor can
corrupt memory that has been allocated for other kernel uses.
This fixes unhandled exceptions that cause boot failures when
running the kernel on Azure Standard_D2ps_v5 instances.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant