From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH v3] drm/i915: Rework GPU reset sequence to match driver load & thaw Date: Mon, 25 Aug 2014 22:18:09 +0200 Message-ID: <20140825201809.GI15520@phenom.ffwll.local> References: <8761ht7kiq.fsf@gaia.fi.intel.com> <1408125095-10699-1-git-send-email-alistair.mcaulay@intel.com> <20140820145826.GD12830@nuc-i3427.alporthouse.com> <2F6A3166A8653C4D914E172D478C0F012E4FF8A9@IRSMSX105.ger.corp.intel.com> <20140820155641.GF12830@nuc-i3427.alporthouse.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-wi0-f170.google.com (mail-wi0-f170.google.com [209.85.212.170]) by gabe.freedesktop.org (Postfix) with ESMTP id 97AD06E21C for ; Mon, 25 Aug 2014 13:17:47 -0700 (PDT) Received: by mail-wi0-f170.google.com with SMTP id f8so4223602wiw.3 for ; Mon, 25 Aug 2014 13:17:46 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20140820155641.GF12830@nuc-i3427.alporthouse.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Chris Wilson , "Mcaulay, Alistair" , "Daniel, Thomas" , "intel-gfx@lists.freedesktop.org" List-Id: intel-gfx@lists.freedesktop.org On Wed, Aug 20, 2014 at 04:56:41PM +0100, Chris Wilson wrote: > On Wed, Aug 20, 2014 at 03:21:55PM +0000, 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 missed the point. Look closer at the reset counter and > reset ordering. We could try to mark the gpu as reset again before starting the reinit to avoid this kludge. But that has the problem that if the ring init fails we have a bit a mess in marking the gpu terminally wedged. So I think overall not prettier than what we have here ... And if I'm mistaken I guess I can put on my idiot hat and merge the fixup ;-) -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch