Received-SPF: pass (google.com: domain of linux-kernel+bounces-198967-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
From: =?UTF-8?q?Ilpo=20J=C3=A4rvinen?= <ilpo.jarvinen@linux.intel.com>
Date: Mon, 3 Jun 2024 13:11:16 +0300 (EEST)
To: "Chang S. Bae" <chang.seok.bae@intel.com>
cc: LKML <linux-kernel@vger.kernel.org>, x86@kernel.org, 
    platform-driver-x86@vger.kernel.org, tglx@linutronix.de, mingo@redhat.com, 
    bp@alien8.de, dave.hansen@linux.intel.com, 
    Hans de Goede <hdegoede@redhat.com>, tony.luck@intel.com, 
    ashok.raj@intel.com, jithu.joseph@intel.com, 
    sathyanarayanan.kuppuswamy@linux.intel.com
Subject: Re: [PATCH v3 0/3] x86/fpu: Allow the In-Field Scan driver to
 initialize FPU state
In-Reply-To: <20240530192739.172566-1-chang.seok.bae@intel.com>
Message-ID: <676bfc09-c1f4-9272-a198-3fb865868c10@linux.intel.com>
References: <20240530192739.172566-1-chang.seok.bae@intel.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/mixed; BOUNDARY="8323328-81316292-1717409305=:1529"
Content-ID: <afc06685-6e39-74d5-a470-df656471d706@linux.intel.com>

  This message is in MIME format.  The first part should be readable text,
  while the remaining parts are likely unreadable without MIME-aware tools.

--8323328-81316292-1717409305=:1529
Content-Type: text/plain; CHARSET=ISO-8859-15
Content-Transfer-Encoding: QUOTED-PRINTABLE
Content-ID: <0a16be6f-9c4b-800e-b144-cd96f45b85a3@linux.intel.com>

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.
>=20
> 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.
>=20
> Thanks to Dave for the reviews and the suggestion.
>=20
> Thanks,
> Chang
>=20
> The previous postings:
> V2: https://lore.kernel.org/all/20240507235344.249103-1-chang.seok.bae@in=
tel.com
> V1: https://lore.kernel.org/all/20240430212508.105117-1-chang.seok.bae@in=
tel.com
>=20
> 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
>=20
>  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=E4rvinen <ilpo.jarvinen@linux.intel.com>

Do x86 maintainers plan to take these through their tree?

--=20
 i.
--8323328-81316292-1717409305=:1529--