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

guids.S: Include <cet.h> when CET is enabled #149

Merged
merged 1 commit into from
Apr 12, 2020

Conversation

hjl-tools
Copy link
Contributor

Intel Control-flow Enforcement Technology (CET):

https://software.intel.com/en-us/articles/intel-sdm

contains shadow stack (SHSTK) and indirect branch tracking (IBT). When
CET is enabled, ELF object files must be marked with .note.gnu.property
section. CET enabled GCC provides a header file, <cet.h>, which can be
included in assembly files to generate the CET marker automatically.

This fixes: #148

Signed-off-by: H.J. Lu hjl.tools@gmail.com

Intel Control-flow Enforcement Technology (CET):

https://software.intel.com/en-us/articles/intel-sdm

contains shadow stack (SHSTK) and indirect branch tracking (IBT). When
CET is enabled, ELF object files must be marked with .note.gnu.property
section. CET enabled GCC provides a header file, <cet.h>, which can be
included in assembly files to generate the CET marker automatically.

Signed-off-by: H.J. Lu <hjl.tools@gmail.com>
@codonell
Copy link

codonell commented Apr 9, 2020

This patch looks good to me. The object generated from guids.S only contains data. The goal of including cet.h is to enable the markup of this object and thus allow the final objects to contain the right markup. The goal for downstream distributions is to correctly markup all objects so we can have a clean system for booting with CET turned on.

@vathpela vathpela merged commit 478b37b into rhboot:master Apr 12, 2020
@hjl-tools hjl-tools deleted the hjl/cet/master branch April 12, 2020 18:52
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.

Add Intel CET support
3 participants