From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [Intel-gfx] [PATCH] [v3] drm/i915: Fix context sizes on HSW Date: Thu, 27 Jun 2013 19:40:06 +0200 Message-ID: <20130627174006.GS18285@phenom.ffwll.local> References: <1372093293-27986-1-git-send-email-ben@bwidawsk.net> <1372222420-1403-1-git-send-email-ben@bwidawsk.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1372222420-1403-1-git-send-email-ben@bwidawsk.net> Sender: stable-owner@vger.kernel.org To: Ben Widawsky Cc: Intel GFX , stable@vger.kernel.org List-Id: intel-gfx@lists.freedesktop.org On Tue, Jun 25, 2013 at 09:53:40PM -0700, Ben Widawsky wrote: > With updates to the spec, we can actually see the context layout, and > how many dwords are allocated. That table suggests we need 70720 bytes > per HW context. Rounded up, this is 18 pages. Looking at what lives > after the current 4 pages we use, I can't see too much important (mostly > it's d3d related), but there are a couple of things which look scary. I > am hopeful this can explain some of our odd HSW failures. > > v2: Make the context only 17 pages. The power context space isn't used > ever, and execlists aren't used in our driver, making the actual total > 66944 bytes. > > v3: Add a comment to the code. (Jesse & Paulo) > > Reported-by: "Azad, Vinit" > Cc: stable@vger.kernel.org > Reviewed-by: Jesse Barnes > Signed-off-by: Ben Widawsky Queued for -next, thanks for the patch. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch