From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH 08/16] drm/i915: remove unused restore_gtt_mappings optimization during suspend Date: Tue, 21 Oct 2014 16:50:12 +0300 Message-ID: <20141021135012.GJ4284@intel.com> References: <1410362229-814-1-git-send-email-imre.deak@intel.com> <1410362229-814-9-git-send-email-imre.deak@intel.com> <20140911074903.GF28332@nuc-i3427.alporthouse.com> <1410436775.4879.8.camel@intelbox> <20140911071527.66d2b2ba@jbarnes-desktop> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by gabe.freedesktop.org (Postfix) with ESMTP id 06B986E202 for ; Tue, 21 Oct 2014 14:46:38 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20140911071527.66d2b2ba@jbarnes-desktop> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Jesse Barnes Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Thu, Sep 11, 2014 at 07:15:27AM -0700, Jesse Barnes wrote: > On Thu, 11 Sep 2014 14:59:35 +0300 > Imre Deak wrote: > = > > On Thu, 2014-09-11 at 08:49 +0100, Chris Wilson wrote: > > > On Wed, Sep 10, 2014 at 06:17:01PM +0300, Imre Deak wrote: > > > > Since correctness wins over optimal code and since the optimization > > > = > > > Optimal code is also correct ;-) s/optimal/just plain broken/ > > = > > Yes, bad wording. To clarify, since the optimization is now always off > > anyway (and it's also broken), I would hope that we could remove it for > > now and concentrate on fixing the existing s/r issues. Once we find that > > things are stable enough we could add back this optimization. > = > Arg, I guess we didn't test after moving to the opregion test? Or maybe > it was working when it landed for S3 and not for S4? Or broke sometime > after it landed? > = > Anyway this is a really valuable optimization for resume time on some > platforms, and really we shouldn't have other agents clobbering our GTT > on resume, so I'd really like to fix/re-add it asap. If/when we add this back I would suggest that we also add a sanity check that can be enabled with drm.debug which would verify the GTT mapping are what we expect. That way at least most developer would have it enabled and we'd catch problems earlier, and also bug reporters would be forced to enable it when we ask for dmesg w/ debugs. -- = Ville Syrj=E4l=E4 Intel OTC