Did you contact tools team about this? Since the smc messages PPSMC_MSG_PmStatusLogStart and PPSMC_MSG_PmStatusLogSample are exclusively used by tools, driver just shared it. And AGT gets different result as driver if you change it back to 20ms. To keep consistent result as AGT, please verify it through full test cases. Regards, Eric On 2018-04-04 04:25 AM, Rex Zhu wrote: > we don't have limit of [50ms, 4sec] sampling period. > only confused the average power and max power. > so don't need to increase delay time. > > This reverts commit 462d8dcc9fec0d89f1ff6a1f93f1d4f670878c71. > > Signed-off-by: Rex Zhu <Rex.Zhu at amd.com> > --- > drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c b/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c > index aaa9f5b..40f2f87 100644 > --- a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c > +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c > @@ -3368,8 +3368,7 @@ static int smu7_get_gpu_power(struct pp_hwmgr *hwmgr, > "Failed to start pm status log!", > return -1); > > - /* Sampling period from 50ms to 4sec */ > - msleep_interruptible(200); > + msleep_interruptible(20); > > PP_ASSERT_WITH_CODE(!smum_send_msg_to_smc(hwmgr, > PPSMC_MSG_PmStatusLogSample),
