-
Notifications
You must be signed in to change notification settings - Fork 94
Disable saving and loading kernel's binary from cache #3505
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
base: main
Are you sure you want to change the base?
Disable saving and loading kernel's binary from cache #3505
Conversation
**Self-evaluation:** 1. Build test: [X]Passed [ ]Failed [ ]Skipped 2. Run test: [X]Passed [ ]Failed [ ]Skipped Signed-off-by: Piotr Pokorski <[email protected]>
13be106 to
d060372
Compare
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.
LGTM
| #if KERNEL_CACHE | ||
| static constexpr bool KERNEL_CACHE_ENABLED = true; | ||
| #else | ||
| static constexpr bool KERNEL_CACHE_ENABLED = false; | ||
| #endif |
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.
Maybe different naming convention for static constexprs so the are not confused with defines (all uppercase)?
|
This PR is stale because it has been open 14 days with no activity. Remove stale label or comment or this will be closed in 3 days. |
|
This PR was closed because it has been stalled for 3 days with no activity. |
|
@p-pokorski-samsung, Could you please rebase this pull request? Thank you in advance! |
Self-evaluation:
Dependency of the PR
Commits to be reviewed in this PR
{commit-1}
{commit message}
Self evaluation:
Signed-off-by: {your_name} <{your_email}>
{commit-2}
{commit message}
Self evaluation:
Signed-off-by: {your_name} <{your_email}>
Summary
Signed-off-by: {your_name} <{your_email}>