On 5/19/25 20:22, Alex Deucher wrote: > This reverts commit b7a1a0ef12b81957584fef7b61e2d5ec049c7209. > > A user reported stuttering under heavy gfx load with this commit. > I suspect it's due to the fact that the gfx contexts are shared > between the pipes so if there is alot of load on one pipe, we could > end up stalling waiting for a context. > > On top of that, disabling the second pipe may fix the reliability > of vmid resets. > > Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/3519 > Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>
Reviewed-by: Christian König <christian.koe...@amd.com> for this one, #7 and #8 in this series. For the rest I think it would be better if you add your rb to the patches I think. Thanks a lot for looking into it, Christian. > --- > drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c > b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c > index 75ea071744eb5..14cbd1f08eb5c 100644 > --- a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c > +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c > @@ -4781,7 +4781,7 @@ static int gfx_v10_0_sw_init(struct amdgpu_ip_block > *ip_block) > case IP_VERSION(10, 3, 3): > case IP_VERSION(10, 3, 7): > adev->gfx.me.num_me = 1; > - adev->gfx.me.num_pipe_per_me = 2; > + adev->gfx.me.num_pipe_per_me = 1; > adev->gfx.me.num_queue_per_pipe = 2; > adev->gfx.mec.num_mec = 2; > adev->gfx.mec.num_pipe_per_mec = 4;
