Messages in this thread | | | Date | Mon, 3 Oct 2022 12:35:54 -0700 | Subject | Re: [PATCH v2 01/39] Documentation/x86: Add CET description | From | John Hubbard <> |
| |
On 9/29/22 20:41, Bagas Sanjaya wrote: ... > 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
It's always a judgment call, as to whether to use something like ``CALL` or just plain CALL. Here, I'd like to opine that that the benefits of ``CALL`` are very small, whereas plain text in cet.rst has been made significantly worse. So the result is, "this is not worth it".
The same is true of pretty much all of the other literalizing changes below, IMHO.
Just so you have some additional input on this. I tend to spend time fussing a lot (too much, yes) over readability issues, so this jumps right out at me. :)
thanks, -- John Hubbard NVIDIA
| |