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

[Tim Chen]

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

>>
>> Signed-off-by: Andi Kleen <ak@xxxxxxxxxxxxxxx>
>> Signed-off-by: Tim Chen <tim.c.chen@xxxxxxxxxxxxxxx>
>> ---
>>  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