Skip to content
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

config: pipeline: Update CONFIG_FRAME_WARN for i386 #958

Merged
merged 1 commit into from
Jan 1, 2025

Conversation

musamaanjum
Copy link
Contributor

@musamaanjum musamaanjum commented Dec 31, 2024

config: pipeline: Update CONFIG_FRAME_WARN for i386

CONFIG_FRAME_WARN is set to 1024 by default for i386. The allowed value
for this is 0 to 8192. Let's increase it to 2048 to avoid getting the
warning when stack size is more than default 1024. As all warnings are
being treated as errors, our allmodconfig build fails.

Example of a build failure:
https://kcidb.kernelci.org/d/build/build?var-datasource=edquppk2ghfcwc&var-origin=maestro&var-build_architecture=$__all&var-build_config_name=$__all&var-id=maestro:6772ceda423acf18d263ca6b&var-test-path=boot&from=now-100y&to=now&timezone=browser&var-test_path=&var-issue_presence=yes

Related https://github.com/kernelci/kernelci-pipeline/pull/870
Close https://github.com/kernelci/kernelci-project/issues/482
Signed-off-by: Muhammad Usama Anjum <[email protected]>

@pawiecz
Copy link
Contributor

pawiecz commented Dec 31, 2024

Similar PR was merged for a arm target previously.

For reference: #870

Copy link
Contributor

@pawiecz pawiecz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

minor: copying commit message body from #870 here (including reference to the kernelci/kernelci-project#482 - as is, i.e. with Close: tag) could be useful for tracking change history

CONFIG_FRAME_WARN is set to 1024 by default for i386. The allowed value
for this is 0 to 8192. Let's increase it to 2048 to avoid getting the
warning when stack size is more than default 1024. As all warnings are
being treated as errors, our allmodconfig build fails.

Example of a build failure:
https://kcidb.kernelci.org/d/build/build?var-datasource=edquppk2ghfcwc&var-origin=maestro&var-build_architecture=$__all&var-build_config_name=$__all&var-id=maestro:6772ceda423acf18d263ca6b&var-test-path=boot&from=now-100y&to=now&timezone=browser&var-test_path=&var-issue_presence=yes

Related kernelci#870
Close kernelci/kernelci-project#482
Signed-off-by: Muhammad Usama Anjum <[email protected]>
@musamaanjum
Copy link
Contributor Author

Updated the commit message.

@pawiecz
Copy link
Contributor

pawiecz commented Dec 31, 2024

I created Node 6773ec42ddafeb87332c75bc to verify this PR using:

kernelci-pipeline$ ./tools/kci-maintainer --checkout -u https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git -b master --latest-commit -f kbuild-clang-17-i386-allmodconfig

I'll merge this PR when the build is successfully completed

@pawiecz
Copy link
Contributor

pawiecz commented Jan 1, 2025

Processing the Node completed successfully, config includes requested changes

@pawiecz pawiecz added this pull request to the merge queue Jan 1, 2025
Merged via the queue into kernelci:main with commit 4e24dc8 Jan 1, 2025
3 checks passed
@musamaanjum
Copy link
Contributor Author

@nuclearcat I've got the failure again on 6th Jan. I assume the production hasn't been updated yet.

@nuclearcat
Copy link
Member

It got updated just today

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants