* [PATCH] selftests/thermel/intel/workload_hint: Mask unsupported types
@ 2025-08-28 19:23 Srinivas Pandruvada
2025-08-28 19:34 ` Rafael J. Wysocki
0 siblings, 1 reply; 3+ messages in thread
From: Srinivas Pandruvada @ 2025-08-28 19:23 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>
---
.../selftests/thermal/intel/workload_hint/workload_hint_test.c | 3 +++
1 file changed, 3 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..3ee018bca899 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
@@ -133,6 +133,7 @@ int main(int argc, char **argv)
} else if (ret == 0) {
printf("Poll Timeout\n");
} else {
+
if ((lseek(fd, 0L, SEEK_SET)) < 0) {
fprintf(stderr, "Failed to set pointer to beginning\n");
exit(1);
@@ -144,6 +145,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.50.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] selftests/thermel/intel/workload_hint: Mask unsupported types
2025-08-28 19:23 [PATCH] selftests/thermel/intel/workload_hint: Mask unsupported types Srinivas Pandruvada
@ 2025-08-28 19:34 ` Rafael J. Wysocki
2025-08-28 19:55 ` srinivas pandruvada
0 siblings, 1 reply; 3+ messages in thread
From: Rafael J. Wysocki @ 2025-08-28 19:34 UTC (permalink / raw)
To: Srinivas Pandruvada
Cc: rafael, daniel.lezcano, lukasz.luba, linux-pm, linux-kernel
On Thu, Aug 28, 2025 at 9:23 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>
> ---
> .../selftests/thermal/intel/workload_hint/workload_hint_test.c | 3 +++
> 1 file changed, 3 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..3ee018bca899 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
> @@ -133,6 +133,7 @@ int main(int argc, char **argv)
> } else if (ret == 0) {
> printf("Poll Timeout\n");
> } else {
> +
Is this change intentional? Looks like it is not needed.
> if ((lseek(fd, 0L, SEEK_SET)) < 0) {
> fprintf(stderr, "Failed to set pointer to beginning\n");
> exit(1);
> @@ -144,6 +145,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.50.1
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] selftests/thermel/intel/workload_hint: Mask unsupported types
2025-08-28 19:34 ` Rafael J. Wysocki
@ 2025-08-28 19:55 ` srinivas pandruvada
0 siblings, 0 replies; 3+ messages in thread
From: srinivas pandruvada @ 2025-08-28 19:55 UTC (permalink / raw)
To: Rafael J. Wysocki; +Cc: daniel.lezcano, lukasz.luba, linux-pm, linux-kernel
On Thu, 2025-08-28 at 21:34 +0200, Rafael J. Wysocki wrote:
> On Thu, Aug 28, 2025 at 9:23 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>
> > ---
> > .../selftests/thermal/intel/workload_hint/workload_hint_test.c | 3
> > +++
> > 1 file changed, 3 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..3ee018bca899 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
> > @@ -133,6 +133,7 @@ int main(int argc, char **argv)
> > } else if (ret == 0) {
> > printf("Poll Timeout\n");
> > } else {
> > +
>
> Is this change intentional? Looks like it is not needed.
Not intentional. Will send update.
Thanks,
Srinivas
>
> > if ((lseek(fd, 0L, SEEK_SET)) < 0) {
> > fprintf(stderr, "Failed to set
> > pointer to beginning\n");
> > exit(1);
> > @@ -144,6 +145,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.50.1
> >
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-08-28 19:55 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-08-28 19:23 [PATCH] selftests/thermel/intel/workload_hint: Mask unsupported types Srinivas Pandruvada
2025-08-28 19:34 ` Rafael J. Wysocki
2025-08-28 19:55 ` srinivas pandruvada
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).