Re: [RFC] drm/amd/amdgpu: get rid of else branch

2017-05-04 Thread Nikola Pajkovsky
Christian König writes: > Am 27.04.2017 um 18:17 schrieb Nikola Pajkovsky: >> This is super simple elimination of else branch and I should >> probably even use unlikely in >> >> if (ring->count_dw < count_dw) { >> >> However, amdgpu_ring_w

[PATCH] drm/amd/amdgpu: get rid of else branch

2017-05-04 Thread Nikola Pajkovsky
else branch is pointless if it's right at the end of function and use unlikely() on err path. Signed-off-by: Nikola Pajkovsky --- drivers/gpu/drm/amd/amdgpu/amdgpu.h | 45 +++-- 1 file changed, 23 insertions(+), 22 deletions(-) diff --git a/drivers/gpu/dr

[RFC] drm/amd/amdgpu: get rid of else branch

2017-04-27 Thread Nikola Pajkovsky
This is super simple elimination of else branch and I should probably even use unlikely in if (ring->count_dw < count_dw) { However, amdgpu_ring_write() has similar if condition, but does not return after DRM_ERROR and it looks suspicious. On error, we still adding v to ring and keeping c

WARNING: at drivers/gpu/drm/i915/i915_gem.c:3437 i915_gem_object_pin+0x151/0x1a0()

2012-12-11 Thread Nikola Pajkovsky
Daniel Vetter writes: > On Tue, Dec 11, 2012 at 1:31 PM, Nikola Pajkovsky > wrote: >> looks like we still have some oops in i915. i915 maintainers do you have >> any ideas what's going on? I will try to trigger that oops later today >> and provide more informati

WARNING: at drivers/gpu/drm/i915/i915_gem.c:3437 i915_gem_object_pin+0x151/0x1a0()

2012-12-11 Thread Nikola Pajkovsky
Hey folks, looks like we still have some oops in i915. i915 maintainers do you have any ideas what's going on? I will try to trigger that oops later today and provide more information. [10733.442608] WARNING: at drivers/gpu/drm/i915/i915_gem.c:3437 i915_gem_object_pin+0x151/0x1a0() [10733.442612

Re: WARNING: at drivers/gpu/drm/i915/i915_gem.c:3437 i915_gem_object_pin+0x151/0x1a0()

2012-12-11 Thread Nikola Pajkovsky
Daniel Vetter writes: > On Tue, Dec 11, 2012 at 1:31 PM, Nikola Pajkovsky wrote: >> looks like we still have some oops in i915. i915 maintainers do you have >> any ideas what's going on? I will try to trigger that oops later today >> and provide more information.

WARNING: at drivers/gpu/drm/i915/i915_gem.c:3437 i915_gem_object_pin+0x151/0x1a0()

2012-12-11 Thread Nikola Pajkovsky
Hey folks, looks like we still have some oops in i915. i915 maintainers do you have any ideas what's going on? I will try to trigger that oops later today and provide more information. [10733.442608] WARNING: at drivers/gpu/drm/i915/i915_gem.c:3437 i915_gem_object_pin+0x151/0x1a0() [10733.442612