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

Marcosps ibt support #35

Merged
merged 6 commits into from
Jun 5, 2024

Conversation

giulianobelinassi
Copy link
Collaborator

No description provided.

marcosps added 5 commits June 4, 2024 17:19
Current upstream kernel relies on IBT security feature. When this is
activated we can't rely on kallsyms to access private symbols.

For now only propagate the information if IBT is enabled or anot to
places where it can be used.

Signed-off-by: Marcos Paulo de Souza <mpdesouza@suse.com>
Signed-off-by: Marcos Paulo de Souza <mpdesouza@suse.com>
This is necessary due to kernel requirements.

Signed-off-by: Marcos Paulo de Souza <mpdesouza@suse.com>
For IBT enabled kernels, private symbols need a special macro that to
allow klp-convert to create relocations to them.

Signed-off-by: Marcos Paulo de Souza <mpdesouza@suse.com>
Signed-off-by: Marcos Paulo de Souza <mpdesouza@suse.com>
testsuite/linux/ibt-1.c Outdated Show resolved Hide resolved
testsuite/linux/ibt-1.c Outdated Show resolved Hide resolved
Copy link
Collaborator

Choose a reason for hiding this comment

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

Also, this file should be moved to linux dir, instead of deleted and readded.

This commit fixes a weird case where the SymbolExternalizer attempts
to rewrite a macro that was defined in the command line.

This also fixes the first IBT testcase, as well as add new ones that
are currently failing so we mark them as XFAIL.

linux tests are now in testsuite/linux.

Signed-off-by: Giuliano Belinassi <gbelinassi@suse.de>
@marcosps marcosps mentioned this pull request Jun 5, 2024
@giulianobelinassi giulianobelinassi merged commit 8b7caed into SUSE:main Jun 5, 2024
2 checks passed
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.

2 participants