On Tue, Jul 9, 2019 at 9:17 PM Quan, Evan <Evan.Quan@xxxxxxx> wrote: > > Hi Alex, > > > > There are actually two MAX_USEC_TIMEOUT related macros. > > AMD_MAX_USEC_TIMEOUT is used for SMU and AMDGPU_MAX_USEC_TIMEOUT is used widely in other parts. > > Maybe you mistreated it as AMDGPU_MAX_USEC_TIMEOUT. I was confused by them too. > > > > drivers/gpu/drm/amd/include/amd_shared.h:29:#define AMD_MAX_USEC_TIMEOUT 200000 /* 200 ms */ > > drivers/gpu/drm/amd/amdgpu/amdgpu.h:179:#define AMDGPU_MAX_USEC_TIMEOUT 100000 /* 100 ms */ > > > > I’m considering to sort these timeout settings up later. Maybe to combine them as one or give them differentiated names. > > Indeed. I was mixing them up. We should clarify those. Patch is: Acked-by: Alex Deucher <alexander.deucher@xxxxxxx> > > Regards, > > Evan > > From: Deucher, Alexander <Alexander.Deucher@xxxxxxx> > Sent: Tuesday, July 09, 2019 8:41 PM > To: Quan, Evan <Evan.Quan@xxxxxxx>; amd-gfx@xxxxxxxxxxxxxxxxxxxxx > Cc: Grodzovsky, Andrey <Andrey.Grodzovsky@xxxxxxx>; Wu, Xiaowei <Xiaowei.Wu@xxxxxxx> > Subject: Re: [PATCH] drm/amd/powerplay: increase the SMU msg response waiting time > > > > If this is just for the SMU, maybe it would be better to add a separate timeout for the SMU since this will increase the timeout for lots of things, unless we need an overall timeout increase for mode1 reset. > > > > Alex > > ________________________________ > > From: amd-gfx <amd-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx> on behalf of Evan Quan <evan.quan@xxxxxxx> > Sent: Tuesday, July 9, 2019 1:41 AM > To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx > Cc: Deucher, Alexander; Grodzovsky, Andrey; Quan, Evan; Wu, Xiaowei > Subject: [PATCH] drm/amd/powerplay: increase the SMU msg response waiting time > > > > Increase the waiting time to 1s from 200ms. This is expected > to fix some mode1 reset failures. > > Change-Id: I4c11996628cd40d559c6edea97d092f397122d78 > Signed-off-by: Evan Quan <evan.quan@xxxxxxx> > --- > drivers/gpu/drm/amd/include/amd_shared.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/include/amd_shared.h b/drivers/gpu/drm/amd/include/amd_shared.h > index 447bc4d..779c9e7 100644 > --- a/drivers/gpu/drm/amd/include/amd_shared.h > +++ b/drivers/gpu/drm/amd/include/amd_shared.h > @@ -28,7 +28,7 @@ > #endif > > > -#define AMD_MAX_USEC_TIMEOUT 200000 /* 200 ms */ > +#define AMD_MAX_USEC_TIMEOUT 1000000 /* 1000 ms */ > > /* > * Chip flags > -- > 2.7.4 > > _______________________________________________ > amd-gfx mailing list > amd-gfx@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/amd-gfx > > _______________________________________________ > amd-gfx mailing list > amd-gfx@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/amd-gfx _______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx
