-
Notifications
You must be signed in to change notification settings - Fork 349
app: add os_linux_overlay.conf #10084
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
Conversation
Add an OS overlay for Linux. This overlay configures SOF to be used with SOF Linux kernel driver. The overlay modifies Kconfig configuration options to align with requirements of of the Linux driver. Similar overlays may be added later for other OS'es. Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
|
Intended to cover options that are specific to Linux SOF usage (e.g. #10028 ). Keeping as draft until I can test with a custom CI run (where the overlay is enabled). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds an OS overlay configuration file for Linux that sets up the SOF firmware to work with the Linux kernel driver by disabling context save.
- Added a new configuration file: app/os_linux_overlay.conf
- Set CONFIG_ADSP_IMR_CONTEXT_SAVE to 'n' as the Linux driver does not require firmware state retention
|
Seems to pass build tests and works with all targets, so marking as ready for review. |
|
FYI @abonislawski this will impact our signing flow for sof-bin binaries (need to build with this overlay) |
|
If the Linux version requires an overlay, then something went wrong with our default state. :) Maybe we should do the opposite? This approach forces open-source users to use extra overlays. It's not a big deal right now, but I assume this overlay will grow and we'll lose the vision of a single binary. |
|
@lgirdwood can chime in. @abonislawski I discussed before the PR and he wanted to have Linux as the overlay. The SOF LInux driver will continue to work with the non-overlay build as well, so the overlay is (at least now) a subset that disables stuff Linux driver does not need. Buy you don't need to use the overlay. You just get some extra features you cannot use with Linux driver. |
@kv2019i @abonislawski we have a scalable kernel from a single source today. This means we can optimize the binary for any OS/SKU using Kconfig i.e. some OSes may require features that other OSes don't and vice versa (same with KPIs). The need here today is driven by the different needs around context save/restore with some OSes. Long term I expect base FW will same binary for all OS/SKUs, but Kconfig differentiation will be in the SOF application/modules (but today we are not there and need to differentiate the baseFW also). |
Add an OS overlay for Linux. This overlay configures SOF to be used with SOF Linux kernel driver. The overlay modifies Kconfig configuration options to align with requirements of of the Linux driver.
Similar overlays may be added later for other OS'es.