Re: [PATCH] x86/speculation: Add document to describe Spectre and its mitigations

On 12/28/18 9:34 AM, Jonathan Corbet wrote:

...

Signed-off-by: Andi Kleen ak@linux.intel.com Signed-off-by: Tim Chen tim.c.chen@linux.intel.com

---

Documentation/admin-guide/spectre.rst | 502 ++++++++++++++++++++++++++++++++++ 1 file changed, 502 insertions(+) create mode 100644 Documentation/admin-guide/spectre.rst

I only saw this now, seems I wasn't copied... I'll take a deeper look, but

Thanks for taking a look. I'll make sure you are copied on the updates.

I have a couple of meta comments:

• This could arguably go in the security book rather than the admin guide. I don't really have a strong opinion on which is right at the moment, but others might.

Since l1tf.rst is already here in admin guide, that's why I keep spectre.rst here as well.

• Wherever it ends up, can you also please add it to the appropriate index.rst file so it actually gets built with the rest of the docs?

Yes, index.rst needs update too.

Thanks.

Tim