Skip to content

debugability: workaround for SOF_TEXT_SIZE too big with smp enabled#554

Merged
lgirdwood merged 1 commit intothesofproject:masterfrom
mwierzbix:workaround-build_fail_for_smt_too_big_sof_text
Nov 8, 2018
Merged

debugability: workaround for SOF_TEXT_SIZE too big with smp enabled#554
lgirdwood merged 1 commit intothesofproject:masterfrom
mwierzbix:workaround-build_fail_for_smt_too_big_sof_text

Conversation

@mwierzbix
Copy link
Copy Markdown
Contributor

@mwierzbix mwierzbix commented Nov 8, 2018

Workaround for issue: #553

Signed-off-by: Bartosz Kokoszko bartoszx.kokoszko@linux.intel.com

@mwierzbix mwierzbix added the bug Something isn't working as expected label Nov 8, 2018
@mwierzbix mwierzbix requested a review from lgirdwood November 8, 2018 12:11
@mwierzbix
Copy link
Copy Markdown
Contributor Author

My machine is down. using @bkokoszx 's one. So that's why the sign-off is wrong, sorry about that.

@mwierzbix mwierzbix force-pushed the workaround-build_fail_for_smt_too_big_sof_text branch from 506c22e to e8bc8ed Compare November 8, 2018 12:32
@mwierzbix mwierzbix requested a review from jajanusz November 8, 2018 12:32
…issue553)

Signed-off-by: Bartosz Kokoszko <bartoszx.kokoszko@linux.intel.com>
@mwierzbix mwierzbix force-pushed the workaround-build_fail_for_smt_too_big_sof_text branch from e8bc8ed to 68bce25 Compare November 8, 2018 12:39
Copy link
Copy Markdown
Contributor

@jajanusz jajanusz left a comment

Choose a reason for hiding this comment

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

Looks like adding 800 for APL and CNL is enough, however for ICL we need 1000. Anyway it only works for local builds, need to check why it fails on CI

@mwierzbix mwierzbix self-assigned this Nov 8, 2018
@lgirdwood lgirdwood merged commit cee5f5f into thesofproject:master Nov 8, 2018
@mwierzbix mwierzbix deleted the workaround-build_fail_for_smt_too_big_sof_text branch November 8, 2018 21:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working as expected

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants