arch: Disable ARCH_HAVE_FORK for arm64 as a precaution #11200
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Summary: - Temporarily disables ARCH_HAVE_FORK for arm64 to address a crash issue, as detailed in apache#9755. - This is a precautionary measure until a more permanent solution is implemented. Impact: - Temporarily limits certain functionalities on arm64, but necessary to ensure system stability. Testing: - Successfully tested on QEMU-8.1.2. - Note: please apply the changes from apache/nuttx-apps#1962. Signed-off-by: Masayuki Ishikawa <[email protected]>
masayuki2009
added a commit
to masayuki2009/incubator-nuttx-apps
that referenced
this pull request
Nov 16, 2023
Summary: - Resolves an issue where `vfork_test` was not executed following the renaming from CONFIG_ARCH_HAVE_VFORK to CONFIG_ARCH_HAVE_FORK, as detailed in apache/nuttx#9755. Impact: - Please merge apache/nuttx#11200 before merging this PR. Testing: - Successfully tested with rv-virt:smp64, spresense:smp, and sim:smp configurations. - Additional testing on aarch64 will be necessary once its related issues are resolved. Signed-off-by: Masayuki Ishikawa <[email protected]>
masayuki2009
added a commit
to masayuki2009/incubator-nuttx-apps
that referenced
this pull request
Nov 16, 2023
Summary: - Resolves an issue where `vfork_test` was not executed following the renaming from CONFIG_ARCH_HAVE_VFORK to CONFIG_ARCH_HAVE_FORK, as detailed in apache/nuttx#9755. Impact: - Please merge apache/nuttx#11200 before merging this PR. Testing: - Successfully tested with rv-virt:smp64, spresense:smp, and sim:smp. Signed-off-by: Masayuki Ishikawa <[email protected]>
xiaoxiang781216
approved these changes
Nov 16, 2023
xiaoxiang781216
pushed a commit
to apache/nuttx-apps
that referenced
this pull request
Nov 16, 2023
Summary: - Resolves an issue where `vfork_test` was not executed following the renaming from CONFIG_ARCH_HAVE_VFORK to CONFIG_ARCH_HAVE_FORK, as detailed in apache/nuttx#9755. Impact: - Please merge apache/nuttx#11200 before merging this PR. Testing: - Successfully tested with rv-virt:smp64, spresense:smp, and sim:smp. Signed-off-by: Masayuki Ishikawa <[email protected]>
Shunichi-K
pushed a commit
to SPRESENSE/nuttx-apps
that referenced
this pull request
Nov 30, 2023
Summary: - Resolves an issue where `vfork_test` was not executed following the renaming from CONFIG_ARCH_HAVE_VFORK to CONFIG_ARCH_HAVE_FORK, as detailed in apache/nuttx#9755. Impact: - Please merge apache/nuttx#11200 before merging this PR. Testing: - Successfully tested with rv-virt:smp64, spresense:smp, and sim:smp. Signed-off-by: Masayuki Ishikawa <[email protected]> (cherry picked from commit 79c7962)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
as detailed in sched/task/fork: add fork implementation #9755.
Impact
Testing