JianyuWang0623 opened a new pull request, #20170:
URL: https://github.com/apache/nuttx/pull/20170
## Summary
* Switch `qemu-i486` (`nsh`, `vga_fb`) from `nsh_main` to nxinit's
`init_main`, matching the migration already done for `sim`, `rv-virt` and
`qemu-armv8a`.
* Two commits: an `arch/x86` CPP fix (adds `-P`) that the board switch
depends on to parse `init.rc` correctly, followed by the board
defconfig/Makefile/init.rc change itself. See each commit message for the
causal chain.
## Impact
* New feature added / existing feature changed? YES (init entry point
switched from nsh to nxinit on 2 configs).
* Impact on user? NO.
* Impact on build? YES (x86 CPP invocation gains `-P`).
* Impact on hardware (arch/board/driver)? YES (`arch/x86`,
`boards/x86/qemu/qemu-i486`).
* Impact on documentation? NO (mirrors an existing, already-documented
pattern).
* Impact on security? NO.
* Impact on compatibility? NO.
* Anything else? The CPP fix is a prerequisite for the board commit to
boot correctly; it is otherwise independent and can be reverted on its own.
## Testing
I confirm that changes are verified on local setup and works as intended:
* Build Host(s): Linux, gcc-13 + gcc-13-multilib (host gcc -m32, no cross
toolchain required).
* Target(s): x86, `qemu-i486:nsh` and `qemu-i486:vga_fb`, run under
`qemu-system-i386`.
Testing logs after change:
```
nsh (-cpu 486 -m 2):
nsh> ps
TID PID PPID PRI POLICY TYPE NPX STATE EVENT SIGMASK
STACK COMMAND
0 0 0 0 FIFO Kthread - Ready
0000000000000000 0002024 Idle_Task
2 2 0 100 FIFO Task - Waiting Semaphore
0000000000000000 0002004 init_main
3 3 2 100 FIFO Task - Running
0000000000000000 0002012 sh
nsh> mount
/etc type romfs
/proc type procfs
nsh> free
total used free maxused maxfree nused nfree name
572784 9680 563104 10048 563104 52 1 Umem
vga_fb (-cpu 486 -m 1024 -vga std -serial stdio -display none): same
init_main/sh parent-child relationship, /etc romfs mounted, `fb`
framebuffer self-test completes ("Test finished").
```
## PR verification Self-Check
* [x] This PR introduces only one functional change.
* [x] I have updated all required description fields above.
* [x] My PR adheres to Contributing
[Guidelines](https://github.com/apache/nuttx/blob/master/CONTRIBUTING.md) and
[Documentation](https://nuttx.apache.org/docs/latest/contributing/index.html)
(git commit title and message, coding standard, etc).
* [x] My PR is still work in progress (not ready for review).
* [ ] My PR is ready for review and can be safely merged into a codebase.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
