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.

Sign up for GitHub

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

Jump to bottom

debugability: macros breaking multicore builds (sof section .text not fitting in region sof_text) #553

Closed
mwierzbix opened this issue Nov 8, 2018 · 8 comments
Closed

debugability: macros breaking multicore builds (sof section .text not fitting in region sof_text) #553

mwierzbix opened this issue Nov 8, 2018 · 8 comments
Assignees
@mwierzbix
Labels
APL Applies to Apollolake platform bug Something isn't working as expected CNL Applies to Cannonlake platform ICL Applies to IceLake platform P1 Blocker bugs or important features SUE Applies to Sue Creek platform

Comments

@mwierzbix
Copy link
Contributor

Building multicore (for instance with --with-arch=xtensa-smp ), we get following error mesasge
sof section .text' will not fit in region sof_text'
@mwierzbix
Copy link
Contributor Author

It would maybe be good if Travis had both single- and mult- core builds?

@mwierzbix mwierzbix added the bug Something isn't working as expected label Nov 8, 2018
@mwierzbix mwierzbix self-assigned this Nov 8, 2018
@lgirdwood
Copy link
Member

@mwierzbix already given this task to @xiulipan , but this makes it official.

@mwierzbix mwierzbix added APL Applies to Apollolake platform CNL Applies to Cannonlake platform P1 Blocker bugs or important features SUE Applies to Sue Creek platform ICL Applies to IceLake platform labels Nov 8, 2018
@mwierzbix
Copy link
Contributor Author

I was just about to PR a workaround, and with both @mmaka1 and @akloniex to investigate.

@lgirdwood
Copy link
Member

I guess you will be able to fix first since it's evening time now for @xiulipan

@mwierzbix
Copy link
Contributor Author

mwierzbix commented Nov 8, 2018 •
edited
Loading

Temporary fix would be to add 0x800 to SOF_TEXT_SIZE to allow other developers to work while we work this issue out.

And if we don't manage, then maybe @xiulipan will :)

@lgirdwood
Copy link
Member

Sorry, I was meaning have @xiulipan fix Travis CI for SMP builds, not for memory sizes

@mwierzbix
Copy link
Contributor Author

Oh, well then he would discover that they fail because of this :P

@mwierzbix mwierzbix mentioned this issue Nov 8, 2018
Merged
@bkokoszx bkokoszx mentioned this issue Nov 8, 2018
Merged
@mwierzbix mwierzbix mentioned this issue Nov 8, 2018
Merged
@mwierzbix mwierzbix changed the title debugability: macros breaking multicore builds debugability: macros breaking multicore builds (sof section .text not fitting in region sof_text) Nov 9, 2018
@mwierzbix
Copy link
Contributor Author

Status update: workaround is already merged to master, @akloniex has a fix PRed.

@mwierzbix mwierzbix added SUE Applies to Sue Creek platform fixed and removed SUE Applies to Sue Creek platform labels Nov 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mwierzbix mwierzbix

Labels
APL Applies to Apollolake platform bug Something isn't working as expected CNL Applies to Cannonlake platform ICL Applies to IceLake platform P1 Blocker bugs or important features SUE Applies to Sue Creek platform
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lgirdwood @mwierzbix