Re: [Intel-gfx] [PATCH] drm/i915: Skip GPU wait for scanout pin while wedged

2011-07-20 Thread Chris Wilson
On Tue, 19 Jul 2011 17:28:53 -0700, Keith Packard wrote: > Failing to pin a scanout buffer will most likely lead to a black > screen, so if the GPU is wedged, then just let the pin happen and hope > that things work out OK. > > v2: Just ignore any error from i915_gem_object_wait_rendering, as > s

[Intel-gfx] [PATCH] drm/i915: Skip GPU wait for scanout pin while wedged

2011-07-19 Thread Keith Packard
Failing to pin a scanout buffer will most likely lead to a black screen, so if the GPU is wedged, then just let the pin happen and hope that things work out OK. v2: Just ignore any error from i915_gem_object_wait_rendering, as suggested by Chris Wilson Signed-off-by: Keith Packard --- drivers/g

Re: [Intel-gfx] [PATCH] drm/i915: Skip GPU wait for scanout pin while wedged

2011-07-19 Thread Keith Packard
On Wed, 20 Jul 2011 01:03:17 +0100, Chris Wilson wrote: > This doesn't prevent us returning an error should the wait-rendering abort > due to a GPU hang occurring in the middle of the wait. Yeah, should probably check the return value and ignore the error instead. > i915_gem_object_pin_to_disp

Re: [Intel-gfx] [PATCH] drm/i915: Skip GPU wait for scanout pin while wedged

2011-07-19 Thread Chris Wilson
On Tue, 19 Jul 2011 16:26:43 -0700, Keith Packard wrote: > Failing to pin a scanout buffer will most likely lead to a black > screen, so if the GPU is wedged, then just let the pin happen and hope > that things work out OK. This doesn't prevent us returning an error should the wait-rendering abor

[Intel-gfx] [PATCH] drm/i915: Skip GPU wait for scanout pin while wedged

2011-07-19 Thread Keith Packard
Failing to pin a scanout buffer will most likely lead to a black screen, so if the GPU is wedged, then just let the pin happen and hope that things work out OK. Signed-off-by: Keith Packard --- drivers/gpu/drm/i915/i915_gem.c | 12 +--- 1 files changed, 9 insertions(+), 3 deletions(-)