Re: [Intel-gfx] [PATCH v3] drm/i915: Rework GPU reset sequence to match driver load thaw

2014-08-26 Thread Chris Wilson
On Mon, Aug 25, 2014 at 10:18:09PM +0200, Daniel Vetter wrote: On Wed, Aug 20, 2014 at 04:56:41PM +0100, Chris Wilson wrote: On Wed, Aug 20, 2014 at 03:21:55PM +, Mcaulay, Alistair wrote: It is not the same. This is a special case when re-initialising the hw. This flag is to allow

Re: [Intel-gfx] [PATCH v3] drm/i915: Rework GPU reset sequence to match driver load thaw

2014-08-25 Thread Daniel Vetter
On Wed, Aug 20, 2014 at 04:56:41PM +0100, Chris Wilson wrote: On Wed, Aug 20, 2014 at 03:21:55PM +, Mcaulay, Alistair wrote: It is not the same. This is a special case when re-initialising the hw. This flag is to allow gem_init_hw() to complete successfully during reset. At any other

Re: [Intel-gfx] [PATCH v3] drm/i915: Rework GPU reset sequence to match driver load thaw

2014-08-25 Thread Daniel Vetter
On Thu, Aug 21, 2014 at 12:38:58PM +, Mcaulay, Alistair wrote: Hi Daniel, Is there anything else needing done before this patch can be merged? Me getting working internet mostly. Patch merged with the comment in check_wedge a bit extended. Also had to rebase, please double-check that I

Re: [Intel-gfx] [PATCH v3] drm/i915: Rework GPU reset sequence to match driver load thaw

2014-08-21 Thread Mcaulay, Alistair
Hi Daniel, Is there anything else needing done before this patch can be merged? Thanks, Alistair. -Original Message- From: Mika Kuoppala [mailto:mika.kuopp...@linux.intel.com] Sent: Tuesday, August 19, 2014 1:36 PM To: Mcaulay, Alistair; intel-gfx@lists.freedesktop.org Subject: RE:

Re: [Intel-gfx] [PATCH v3] drm/i915: Rework GPU reset sequence to match driver load thaw

2014-08-20 Thread Daniel, Thomas
-Original Message- From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf Of alistair.mcau...@intel.com Sent: Friday, August 15, 2014 6:52 PM To: intel-gfx@lists.freedesktop.org Subject: [Intel-gfx] [PATCH v3] drm/i915: Rework GPU reset sequence to match driver

Re: [Intel-gfx] [PATCH v3] drm/i915: Rework GPU reset sequence to match driver load thaw

2014-08-20 Thread Chris Wilson
: [Intel-gfx] [PATCH v3] drm/i915: Rework GPU reset sequence to match driver load thaw From: McAulay, Alistair alistair.mcau...@intel.com This patch is to address Daniels concerns over different code during reset: http://lists.freedesktop.org/archives/intel-gfx/2014-June/047758.html

Re: [Intel-gfx] [PATCH v3] drm/i915: Rework GPU reset sequence to match driver load thaw

2014-08-20 Thread Mcaulay, Alistair
Subject: [Intel-gfx] [PATCH v3] drm/i915: Rework GPU reset sequence to match driver load thaw From: McAulay, Alistair alistair.mcau...@intel.com This patch is to address Daniels concerns over different code during reset: http://lists.freedesktop.org/archives/intel-gfx/2014

Re: [Intel-gfx] [PATCH v3] drm/i915: Rework GPU reset sequence to match driver load thaw

2014-08-20 Thread Chris Wilson
On Wed, Aug 20, 2014 at 03:21:55PM +, Mcaulay, Alistair wrote: It is not the same. This is a special case when re-initialising the hw. This flag is to allow gem_init_hw() to complete successfully during reset. At any other point during reset, -EAGAIN should be returned. Indeed. You've

Re: [Intel-gfx] [PATCH v3] drm/i915: Rework GPU reset sequence to match driver load thaw

2014-08-19 Thread Mcaulay, Alistair
Hi Mika, can you please review this patch, and verify it fixes the issues in your previous review. Thanks, Alistair. -Original Message- From: Mcaulay, Alistair Sent: Friday, August 15, 2014 6:52 PM To: intel-gfx@lists.freedesktop.org Cc: Mcaulay, Alistair Subject: [PATCH v3]

Re: [Intel-gfx] [PATCH v3] drm/i915: Rework GPU reset sequence to match driver load thaw

2014-08-19 Thread Mika Kuoppala
Mcaulay, Alistair alistair.mcau...@intel.com writes: Hi Mika, can you please review this patch, and verify it fixes the issues in your previous review. Thanks, Alistair. -Original Message- From: Mcaulay, Alistair Sent: Friday, August 15, 2014 6:52 PM To:

[Intel-gfx] [PATCH v3] drm/i915: Rework GPU reset sequence to match driver load thaw

2014-08-15 Thread alistair . mcaulay
From: McAulay, Alistair alistair.mcau...@intel.com This patch is to address Daniels concerns over different code during reset: http://lists.freedesktop.org/archives/intel-gfx/2014-June/047758.html The reason for aiming as hard as possible to use the exact same code for driver load, gpu reset