Page MenuHomePhabricator

[x86][seses] Add documentation for SESES
Needs ReviewPublic

Authored by zbrid on Tue, Mar 10, 10:07 AM.

Details

Summary

This patch includes documentation about the pass Speculative Execution
Side Effect Suppression. The document describes Load Value Injection,
the mitigation, and all the experiments ran on various LFENCE
placements.

One pager on Load Value Injection:
https://software.intel.com/security-software-guidance/software-guidance/load-value-injection

Deep dive on Load Value Injection:
https://software.intel.com/security-software-guidance/insights/deep-dive-load-value-injection

Impl patch: https://reviews.llvm.org/D75939

Diff Detail

Event Timeline

zbrid created this revision.Tue, Mar 10, 10:07 AM
Herald added a project: Restricted Project. · View Herald TranscriptTue, Mar 10, 10:07 AM
craig.topper added inline comments.Wed, Mar 11, 3:32 PM
llvm/docs/SpeculativeExecutionSideEffectSuppression.md
545

These commit ids are only valid for you local copy right? And would change any time you rebase.

craig.topper added inline comments.Wed, Mar 11, 3:40 PM
llvm/docs/SpeculativeExecutionSideEffectSuppression.md
545

Or is this the baseline revision you applied your patches against?

zbrid updated this revision to Diff 250065.Thu, Mar 12, 2:57 PM
zbrid marked 3 inline comments as done.

Clarify what the Git commit IDs in the benchmark section refers to

zbrid added a comment.Thu, Mar 12, 4:55 PM

Updated based on comments. Thanks for the feedback!

llvm/docs/SpeculativeExecutionSideEffectSuppression.md
545

This the baseline revision I applied my patches against when I ran the benchmarks. I'll update the documentation to clarify that.