* [PATCH v2] selftests/thermel/intel/workload_hint: Mask unsupported types
@ 2025-08-28 20:15 Srinivas Pandruvada
2025-09-04 20:19 ` Rafael J. Wysocki
0 siblings, 1 reply; 2+ messages in thread
From: Srinivas Pandruvada @ 2025-08-28 20:15 UTC (permalink / raw)
To: rafael, daniel.lezcano, lukasz.luba
Cc: linux-pm, linux-kernel, Srinivas Pandruvada
The workload hint may contain some other hints which are not defined.
So mask out unsupported types. Currently only lower 4 bits of workload
type hints are defined.
Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
---
v2:
- Removed extra line
.../selftests/thermal/intel/workload_hint/workload_hint_test.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/tools/testing/selftests/thermal/intel/workload_hint/workload_hint_test.c b/tools/testing/selftests/thermal/intel/workload_hint/workload_hint_test.c
index ba58589a1145..ca2bd03154e4 100644
--- a/tools/testing/selftests/thermal/intel/workload_hint/workload_hint_test.c
+++ b/tools/testing/selftests/thermal/intel/workload_hint/workload_hint_test.c
@@ -144,6 +144,8 @@ int main(int argc, char **argv)
ret = sscanf(index_str, "%d", &index);
if (ret < 0)
break;
+
+ index &= 0x0f;
if (index > WORKLOAD_TYPE_MAX_INDEX)
printf("Invalid workload type index\n");
else
--
2.43.0
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH v2] selftests/thermel/intel/workload_hint: Mask unsupported types
2025-08-28 20:15 [PATCH v2] selftests/thermel/intel/workload_hint: Mask unsupported types Srinivas Pandruvada
@ 2025-09-04 20:19 ` Rafael J. Wysocki
0 siblings, 0 replies; 2+ messages in thread
From: Rafael J. Wysocki @ 2025-09-04 20:19 UTC (permalink / raw)
To: Srinivas Pandruvada
Cc: rafael, daniel.lezcano, lukasz.luba, linux-pm, linux-kernel
On Thu, Aug 28, 2025 at 10:15 PM Srinivas Pandruvada
<srinivas.pandruvada@linux.intel.com> wrote:
>
> The workload hint may contain some other hints which are not defined.
> So mask out unsupported types. Currently only lower 4 bits of workload
> type hints are defined.
>
> Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
> ---
> v2:
> - Removed extra line
>
> .../selftests/thermal/intel/workload_hint/workload_hint_test.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/tools/testing/selftests/thermal/intel/workload_hint/workload_hint_test.c b/tools/testing/selftests/thermal/intel/workload_hint/workload_hint_test.c
> index ba58589a1145..ca2bd03154e4 100644
> --- a/tools/testing/selftests/thermal/intel/workload_hint/workload_hint_test.c
> +++ b/tools/testing/selftests/thermal/intel/workload_hint/workload_hint_test.c
> @@ -144,6 +144,8 @@ int main(int argc, char **argv)
> ret = sscanf(index_str, "%d", &index);
> if (ret < 0)
> break;
> +
> + index &= 0x0f;
> if (index > WORKLOAD_TYPE_MAX_INDEX)
> printf("Invalid workload type index\n");
> else
> --
Applied, thanks!
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2025-09-04 20:19 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-08-28 20:15 [PATCH v2] selftests/thermel/intel/workload_hint: Mask unsupported types Srinivas Pandruvada
2025-09-04 20:19 ` Rafael J. Wysocki
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®