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

From: Jonathan Corbet
Date: Fri Sep 30 2022 - 09:33:21 EST


Bagas Sanjaya <bagasdotme@xxxxxxxxx> writes:

> The documentation above can be improved (both grammar and formatting):
>
> ---- >8 ----
>
> diff --git a/Documentation/x86/cet.rst b/Documentation/x86/cet.rst
> index 6b270a24ebc3a2..f691f7995cf088 100644
> --- a/Documentation/x86/cet.rst
> +++ b/Documentation/x86/cet.rst
> @@ -15,92 +15,101 @@ in the 64-bit kernel.
>
> 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.

Thanks,

jon