-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
arch/arm64: add have fork config #14648
Conversation
I'm pretty sure ostest will crash with arm64 kernel mode. |
Yes, the current implementation of fork/vfork is only for flat/protected build, the kernel mode need be reimplemented like Linux, which is a huge task I think that. |
I mean arm64 vfork() will cause ostest crash in kernel mode, making the test unusable in arm64. |
how about change to:
before we port your riscv patch to arm64. |
Sounds good to me |
38b9dba
to
59033f8
Compare
Signed-off-by: lipengfei28 <lipengfei28@xiaomi.com>
59033f8
to
b973d2a
Compare
Sorry I have to cancel the CI Check, there's a problem with CI Test: |
1.ARCH_ARM64 select ARCH_HAVE_FORK
Update this section with information on why change is necessary,
what it exactly does and how, if new feature shows up, provide
references (dependencies, similar problems and solutions), etc.
Impact
Update this section, where applicable, on how change affects users,
build process, hardware, documentation, security, compatibility, etc.
Testing
Update this section with details on how did you verify the change,
what Host was used for build (OS, CPU, compiler, ..), what Target was
used for verification (arch, board:config, ..), etc. Providing build
and runtime logs from before and after change is highly appreciated.