Skip to content

Conversation

@ekurdybx
Copy link
Contributor

@ekurdybx ekurdybx commented Nov 12, 2025

  1. Added option to run check-suspend-resume test with sleepgraph, to measure resume time.
  2. Updated test-socwatch to measure residency time.

@ekurdybx ekurdybx force-pushed the residency-time-test branch from d0acf87 to 3c0746c Compare November 12, 2025 14:34
@sofci
Copy link
Collaborator

sofci commented Nov 12, 2025

Can one of the admins verify this patch?

reply test this please to run this test once

@ekurdybx ekurdybx force-pushed the residency-time-test branch 6 times, most recently from 5339424 to 44ac594 Compare November 12, 2025 17:31
@ekurdybx ekurdybx marked this pull request as ready for review November 12, 2025 17:38
@ekurdybx ekurdybx requested review from a team, golowanow, lgirdwood and marc-hb as code owners November 12, 2025 17:38
OPT_NAME['p']='processid' OPT_DESC['p']='Fail immediately if this process dies'
OPT_HAS_ARG['p']=1 OPT_VAL['p']=''

OPT_NAME['s']='run-sleepgraph' OPT_DESC['s']='run with sleepgraph'
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
OPT_NAME['s']='run-sleepgraph' OPT_DESC['s']='run with sleepgraph'
OPT_NAME['s']='run-sleepgraph' OPT_DESC['s']='run with sleepgraph (http://github.com/intel/pm-graph.git)'

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@ekurdybx ekurdybx force-pushed the residency-time-test branch from 44ac594 to b3559be Compare November 13, 2025 08:53
@ekurdybx ekurdybx force-pushed the residency-time-test branch 2 times, most recently from bda355b to ea3af5e Compare November 13, 2025 10:22
@ekurdybx ekurdybx marked this pull request as draft November 13, 2025 14:35
@ekurdybx ekurdybx force-pushed the residency-time-test branch 5 times, most recently from 053a152 to d030eb0 Compare November 14, 2025 10:16
@ekurdybx ekurdybx marked this pull request as ready for review November 14, 2025 10:16
@ekurdybx ekurdybx force-pushed the residency-time-test branch from d030eb0 to 9069216 Compare November 14, 2025 10:28
Copy link
Collaborator

@marc-hb marc-hb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

BTW I just had a chat with someone familiar with https://github.com/intel/S0ixSelftestTool You should give it a try.

Compared to SoCWatch,
Cons:

  • Reports only public data
  • Less up to date

Pros:

  • Smarter and user friendlier. Tries to focus on relevant information based on the current state and current issues.
  • Linux-specific: reports a few Linux-specific datapoints that does SoCWatch does not.

So: not mutually exclusive.

cc: @

@ekurdybx ekurdybx requested a review from PLangowski December 1, 2025 10:48
@ekurdybx ekurdybx force-pushed the residency-time-test branch 2 times, most recently from cbc88f4 to 0c3aeff Compare December 1, 2025 11:09
Copy link
Contributor

@KamilxPaszkiet KamilxPaszkiet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@ekurdybx ekurdybx force-pushed the residency-time-test branch 9 times, most recently from dde17ab to 8e0a9f4 Compare December 2, 2025 14:46
@ekurdybx ekurdybx force-pushed the residency-time-test branch 4 times, most recently from 4812c7c to d66bbaf Compare December 4, 2025 14:22
Add option to run check-suspend-resume test with sleepgraph, to measure
resume time. Updated test-socwatch to measure residency time.

Signed-off-by: Emilia Kurdybelska <emiliax.kurdybelska@intel.com>
@ekurdybx ekurdybx force-pushed the residency-time-test branch from d66bbaf to 9e3567c Compare December 4, 2025 14:59
@majunkier majunkier merged commit 7ed2f47 into thesofproject:main Dec 4, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants