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/linux-x230: remove debug symbols #565

Merged
merged 1 commit into from
May 22, 2019

Conversation

merge
Copy link
Contributor

@merge merge commented May 20, 2019

Remove debug symbols and dynamic debug support from Linux. This reduces
our resulting bzImage by 100K.

This should help when size becomes even more limited, see #562

@merge
Copy link
Contributor Author

merge commented May 20, 2019

for the record, I flashed that and did a test run.

@paulmenzel
Copy link
Contributor

paulmenzel commented May 20, 2019 via email

Remove debug symbols and dynamic debug support from Linux. This reduces
our resulting bzImage by 100K.

This should help when size becomes even more limited, see linuxboot#562
@merge
Copy link
Contributor Author

merge commented May 20, 2019

thanks for asking, I was wrong and updated the commit message. It's 100K: 3031K -> 2936K. circa.

@merge
Copy link
Contributor Author

merge commented May 20, 2019

With this reduction I was able to do a test-build using coreboot's vboot measured boot. (not without this reduction - we really are at our size limit currently).

The usable cbfs size (fmap see #562) for our vboot-case is not yet defined. This should just be a beginning and we should become even smaller after this!

@tlaurion
Copy link
Collaborator

#517 is the biggest culprit

@merge
Copy link
Contributor Author

merge commented May 21, 2019

#517 is the biggest culprit

I see. it is good to have another promising candidate to work on. Unless we need to have debug symbold by default, it doesn't make this change less useful though, right? I'd say let's reduce our size where possible without losing any functionality.

@tlaurion tlaurion self-assigned this May 22, 2019
@tlaurion tlaurion merged commit cccb846 into linuxboot:master May 22, 2019
@merge merge deleted the linux_remove_debug branch May 23, 2019 07:49
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