platform/x86: ISST: Validate inputs

[PATCH 0/6] platform/x86: ISST: Validate inputs

Posted by Srinivas Pandruvada 1 month, 2 weeks ago

This series add all remaining parameter validation in addition to:
[PATCH 0/2] platform/x86: ISST: Two ioctl input validation fixes
from HyeongJun An.

These patches are dependent on this series.

Similar validation is already done in intel-speed-select tool.

Srinivas Pandruvada (6):
  platform/x86: ISST: Validate logical CPU id and clos id
  platform/x86: ISST: Validate max level for set feature
  platform/x86: ISST: Validate parameter for core power state
  platform/x86: ISST: Validate parameter for frequency and priority
  platform/x86: ISST: Use PP level enable mask
  platform/x86: ISST: Just allow 2 bits for SST feature enable

 .../intel/speed_select_if/isst_tpmi_core.c    | 55 ++++++++++++++++++-
 1 file changed, 53 insertions(+), 2 deletions(-)

-- 
2.53.0

Re: [PATCH 0/6] platform/x86: ISST: Validate inputs

Posted by Ilpo Järvinen 1 month, 1 week ago

On Tue, 11 Aug 2026 15:15:08 -0700, Srinivas Pandruvada wrote:

> This series add all remaining parameter validation in addition to:
> [PATCH 0/2] platform/x86: ISST: Two ioctl input validation fixes
> from HyeongJun An.
> 
> These patches are dependent on this series.
> 
> Similar validation is already done in intel-speed-select tool.
> 
> [...]

Thank you for your contribution, it has been applied to my local
review-ilpo-next branch. Note it will show up in the public
platform-drivers-x86/review-ilpo-next branch only once I've pushed my
local branch there, which might take a while.

FYI [if applicable to your patch], as per Linus' policy change, also
fixes are mostly routed through for-next unless the fix is for a
commit introduced in the most recent cycle or is clearly a regression
fix.

The list of commits applied:
[1/6] platform/x86: ISST: Validate logical CPU id and clos id
      commit: 437757f6acab26132e270e19110bfa79cbc78b22
[2/6] platform/x86: ISST: Validate max level for set feature
      commit: 63125701c0a525136dc0de1348dbe60ba61fed51
[3/6] platform/x86: ISST: Validate parameter for core power state
      commit: 00eec6df54d7b578d1479269e3f508b8fa3476bd
[4/6] platform/x86: ISST: Validate parameter for frequency and priority
      commit: 6363938b116e5a9f74b76bbf8d9dbb1ad2f7b0cb
[5/6] platform/x86: ISST: Use PP level enable mask
      commit: 7a0033bc39349a6b82050672a8a0928b02395b32
[6/6] platform/x86: ISST: Just allow 2 bits for SST feature enable
      commit: 7bbf743c43cc4e1145b9a578815e025d5df5813e

--
 i.