This revision switches to a new approach by providing a helper to
initialize user FPU states for the driver, which is considerably simpler.
This approach could serve as an example for addressing similar situations
from a non-critical path.
I thought fpu_reset_fpregs() as the helper name. There is already one
with this name. Then, I realized the existing one is a bit misaligned, so
renamed it first here.
Thanks to Dave for the reviews and the suggestion.
Thanks,
Chang
The previous postings:
V2: https://lore.kernel.org/all/[email protected]
V1: https://lore.kernel.org/all/[email protected]
Chang S. Bae (3):
x86/fpu: Rename fpu_reset_fpregs() to fpu_reset_fpstate_regs()
x86/fpu: Allow FPU users to initialize FPU state
platform/x86/intel/ifs: Initialize FPU states for the scan test
arch/x86/include/asm/fpu/api.h | 2 ++
arch/x86/kernel/fpu/core.c | 17 ++++++++++++++---
drivers/platform/x86/intel/ifs/ifs.h | 1 +
drivers/platform/x86/intel/ifs/runtest.c | 7 +++++++
4 files changed, 24 insertions(+), 3 deletions(-)
base-commit: 1613e604df0cd359cf2a7fbd9be7a0bcfacfabd0
--
2.34.1
On Thu, 30 May 2024, Chang S. Bae wrote:
> This revision switches to a new approach by providing a helper to
> initialize user FPU states for the driver, which is considerably simpler.
> This approach could serve as an example for addressing similar situations
> from a non-critical path.
>
> I thought fpu_reset_fpregs() as the helper name. There is already one
> with this name. Then, I realized the existing one is a bit misaligned, so
> renamed it first here.
>
> Thanks to Dave for the reviews and the suggestion.
>
> Thanks,
> Chang
>
> The previous postings:
> V2: https://lore.kernel.org/all/[email protected]
> V1: https://lore.kernel.org/all/[email protected]
>
> Chang S. Bae (3):
> x86/fpu: Rename fpu_reset_fpregs() to fpu_reset_fpstate_regs()
> x86/fpu: Allow FPU users to initialize FPU state
> platform/x86/intel/ifs: Initialize FPU states for the scan test
>
> arch/x86/include/asm/fpu/api.h | 2 ++
> arch/x86/kernel/fpu/core.c | 17 ++++++++++++++---
> drivers/platform/x86/intel/ifs/ifs.h | 1 +
> drivers/platform/x86/intel/ifs/runtest.c | 7 +++++++
> 4 files changed, 24 insertions(+), 3 deletions(-)
Acked-by: Ilpo J?rvinen <[email protected]>
Do x86 maintainers plan to take these through their tree?
--
i.
On 6/3/24 03:11, Ilpo Järvinen wrote:
> Acked-by: Ilpo Järvinen <[email protected]>
>
> Do x86 maintainers plan to take these through their tree?
Yes, once we've gone over them very carefully.