lkml.org 
[lkml]   [2021]   [Jul]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [RFC PATCH v1 0/2] Introduce XSAVE/XRSTOR self-test
From
Date
On 7/26/21 8:34 PM, Pengfei Xu wrote:
> The XSAVE feature set supports the saving and restoring of state components
> such as FPU, which is used for process context switching.

This sentence is really awkward. It reads at first as saying that the
FPU is used for context switching. Can you rephrase.

> In order to ensure that XSAVE works correctly, add XSAVE basic test for
> XSAVE architecture functionality.

This sentence needs to be start on the same line as the previous one,
*or* be in a new paragraph. Please rewrap it.

> This patch set tests XSAVE/XRSTOR instructions on x86 platforms and verify if
> the XSAVE/XRSTOR works correctly during signal handling.

This reads to me like you are going to test the XSAVE/XRSTOR
instructions *in* a signal handler, instead of testing the XSAVE/XRSTOR
instructions that the kernel uses at signal entry/exit.

Also, the kernel does *NOT* *USE* XSAVE/XRSTOR in many cases to
save/restore signal state. The changelog could be read as implying that
it does.

> Cases such as signal handling, process creation, other xstate(except FPU)
> tests for XSAVE check, etc. will be added to the Linux kernel self-test.
> If appropriate, it is even planned to add the [1] mentioned XSAVE issues
> reproduce and some XSAVE anomaly tests to the kernel self-test.

This is not clear whether it is talking about *this* series int he
future tense (will be added) or whether it is talking about future *work*.

Maybe something like this:

This series introduces only the most basic XSAVE tests. In the
future, the intention is to continue expanding the scope of
these selftests to include more kernel XSAVE-related
functionality and XSAVE-managed features like AMX and shadow
stacks.

\
 
 \ /
  Last update: 2021-07-27 19:47    [W:0.076 / U:0.292 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site