RE: [PATCH 2/2] drm/amdgpu:256dw is enough for one ib_schedule

2016-08-25 Thread Liu, Monk
@lists.freedesktop.org Subject: Re: [PATCH 2/2] drm/amdgpu:256dw is enough for one ib_schedule Well userspace is allowed to send any number of IBs down to this if I remember correctly. So we clearly need something depending on the number of IBs or reject submissions with to many IBs earlier in the CS

Re: [PATCH 2/2] drm/amdgpu:256dw is enough for one ib_schedule

2016-08-25 Thread Christian König
buffer is only 4kb size -Original Message- From: Christian König [mailto:deathsim...@vodafone.de] Sent: Thursday, August 25, 2016 4:12 PM To: Liu, Monk ; amd-gfx@lists.freedesktop.org Subject: Re: [PATCH 2/2] drm/amdgpu:256dw is enough for one ib_schedule Well userspace is allowed to send any

Re: [PATCH 2/2] drm/amdgpu:256dw is enough for one ib_schedule

2016-08-25 Thread Christian König
Well userspace is allowed to send any number of IBs down to this if I remember correctly. So we clearly need something depending on the number of IBs or reject submissions with to many IBs earlier in the CS. Otherwise we clearly open up a possible problem where userspace can trigger a ring o

[PATCH 2/2] drm/amdgpu:256dw is enough for one ib_schedule

2016-08-24 Thread Monk Liu
Change-Id: I1ee3258276868a753e536ae2d9ae1b12e7eaf791 Signed-off-by: Monk Liu --- drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c index 4e5b2f3..b82904c 10064