app: smc: pytest: add e2e tests for power#936
app: smc: pytest: add e2e tests for power#936jgrowdenTT wants to merge 1 commit intotenstorrent:mainfrom
Conversation
jgrowdenTT
force-pushed
the
power_test
branch
2 times, most recently
from
8d67d0c to
4d0e637
Compare
jgrowdenTT
force-pushed
the
power_test
branch
3 times, most recently
from
7e6dc5c to
cb1a9ae
Compare
| Test toggling between high and low power states and verify TDP delta. | ||
|
|
||
| Toggles between high and low power states and verifies that the TDP | ||
| difference between the two states is greater than 60W. |
There was a problem hiding this comment.
Comments say 60 W, but code says 30 W.
|
|
||
| def test_power_state_toggle(arc_chip_dut, asic_id): | ||
| """ | ||
| Validates that toggling between high and low power states results in a TDP delta > 60W |
There was a problem hiding this comment.
Same as other comment, 60 or 30 W?
There was a problem hiding this comment.
The p100 isn't saving as much power as the others.
To pass CI, I'm decreasing the limit.
I also see locally my new board (p150a) is saving less than my old board. So I've been investigating what's going on.
The test should go in; so lets start with 30, see if we can do better, then adjust limits later.
There was a problem hiding this comment.
As discussed offline -> We shouldn't be using TDP for this but should be using the total input power; this allows us to capture stuff like the MRISC power going down and up with the power command.
jgrowdenTT
force-pushed
the
power_test
branch
from
cb1a9ae to
66aeaa0
Compare
Add e2e stress test to test power ioctl. Ensure a suitable power delta is achieved and that the device remains active. Needs KMD 2.6.0, otherwise the test is skipped. Signed-off-by: James Growden <jgrowden@tenstorrent.com>
jgrowdenTT
force-pushed
the
power_test
branch
from
66aeaa0 to
0cf2a82
Compare
|
Just a reminder, no merge until KMD 2.7.0 is on our runners. |
4 participants
Add e2e stress test to test power ioctl. Ensure a suitable power delta is achieved and that the device remains active.
Needs KMD 2.6.0, otherwise the test is skipped. Locally, this passes on my p150a with KMD 2.6.0.
I would not expect this to pass on galaxy.