From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 19/19] drm/i915: only enable HWSTAM interrupts on postinstall on ILK+ Date: Thu, 3 Apr 2014 11:31:35 +0200 Message-ID: <20140403093135.GA7225@phenom.ffwll.local> References: <1396377447-2177-1-git-send-email-przanoni@gmail.com> <1396377447-2177-20-git-send-email-przanoni@gmail.com> <20140402112145.GA23428@strange.amr.corp.intel.com> <20140402112351.GA5602@nuc-i3427.alporthouse.com> <20140402112714.GB5602@nuc-i3427.alporthouse.com> <20140402152806.GC5602@nuc-i3427.alporthouse.com> <20140402160834.GD30534@strange.amr.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-wi0-f173.google.com (mail-wi0-f173.google.com [209.85.212.173]) by gabe.freedesktop.org (Postfix) with ESMTP id AE19E6E9E6 for ; Thu, 3 Apr 2014 02:31:39 -0700 (PDT) Received: by mail-wi0-f173.google.com with SMTP id z2so7001269wiv.6 for ; Thu, 03 Apr 2014 02:31:38 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20140402160834.GD30534@strange.amr.corp.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Damien Lespiau Cc: Paulo Zanoni , Intel Graphics Development List-Id: intel-gfx@lists.freedesktop.org On Wed, Apr 02, 2014 at 05:08:34PM +0100, Damien Lespiau wrote: > On Wed, Apr 02, 2014 at 04:28:06PM +0100, Chris Wilson wrote: > > On Wed, Apr 02, 2014 at 11:14:58AM -0300, Paulo Zanoni wrote: > > > 2014-04-02 8:27 GMT-03:00 Chris Wilson : > > > > On Wed, Apr 02, 2014 at 12:23:51PM +0100, Chris Wilson wrote: > > > >> On Wed, Apr 02, 2014 at 12:21:45PM +0100, Damien Lespiau wrote: > > > >> > On Tue, Apr 01, 2014 at 03:37:27PM -0300, Paulo Zanoni wrote: > > > >> > > From: Paulo Zanoni > > > >> > > > > > >> > > We should only enable interrupts at postinstall. > > > >> > > > > > >> > > And now on ILK/SNB/IVB/HSW the irq_preinstall and irq_postinstall > > > >> > > functions leave the hardware in the same state. > > > >> > > > > > >> > > Signed-off-by: Paulo Zanoni > > > >> > > > > >> > Orthogonal note to this patch, I'm wondering why we enable any interrupt > > > >> > reporting in the HW status page, I don't see anywhere we read that > > > >> > information back? Any idea? > > > >> > > > >> Back in the SNB days, interrupts broke without this piece of magic. > > > > > > > > To be more precise, iirc, it was a step towards getting coherent > > > > breadcrumb writes into the HWS. As it turns out, there were more steps > > > > required. Considering that it can now probably be dropped again? Any > > > > takers? > > > > > > I noticed the HWSTAM code looks weird, but I don't really know much > > > about why it's there, so I decided to not add any regressions. I wrote > > > this patch a long time ago, and gave up on upstreaming it, but in case > > > anybody wants, I can send it to the list: > > > http://cgit.freedesktop.org/~pzanoni/linux/commit/?h=c8-wip&id=7ebe245a2c55379ddc7b36f1fb440215c23f1570 > > > . Look at the commit message, it may be an interesting starting point > > > if anybody wants to dig on the issue... > > > > Yup, that is very interesting. That may be the minimal piece of HWSTAM > > magic we need, and be an interim step to removing HWSTAM entirely. > > Would you mind resending: > - a patch with Chris' comment above, that's alone is useful for the > next guy looking at the code, > - a patch with the change (Maybe reusing GT_RENDER_USER_INTERRUPT as > HWSTAM uses the same definitions as the other interrupts registers)? > > I'm sure we can get some reviews on the first, hopefully some testing on > the second. Seconded. Documenting what we're doing is always good ;-) -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch