Thread (222 messages) 222 messages, 21 authors, 2022-11-03

Re: [PATCH v2 01/39] Documentation/x86: Add CET description

From: Bagas Sanjaya <hidden>
Date: 2022-10-04 02:18:21
Also in: linux-api, linux-arch, linux-mm, lkml

On 10/3/22 23:56, Edgecombe, Rick P wrote:
On Fri, 2022-09-30 at 20:41 +0700, Bagas Sanjaya wrote:
quoted
On 9/30/22 20:33, Jonathan Corbet wrote:
quoted
quoted
  CET introduces Shadow Stack and Indirect Branch Tracking.
Shadow stack is
  a secondary stack allocated from memory and cannot be directly
modified by
-applications. When executing a CALL instruction, the processor
pushes the
+applications. When executing a ``CALL`` instruction, the
processor pushes the
Just to be clear, not everybody is fond of sprinkling lots of
``literal
text`` throughout the documentation in this way.  Heavy use of it
will
certainly clutter the plain-text file and can be a net negative
overall.
Actually there is a trade-off between semantic correctness and plain-
text
clarity. With regards to inline code samples (like identifiers), I
fall
into the former camp. But when I'm reviewing patches for which the
surrounding documentation go latter camp (leave code samples alone
without
markup), I can adapt to that style as long as it causes no warnings
whatsover.
Thanks. Unless anyone has any objections, I think I'll take all these
changes, except for the literal-izing of the instructions. They are not
really being used as code samples in this case.

Bagas, can you reply with your sign-off and I'll just apply it?
OK, here goes...

Signed-off-by: Bagas Sanjaya <redacted>

-- 
An old man doll... just what I always wanted! - Clara
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help