From mboxrd@z Thu Jan 1 00:00:00 1970 From: Damien Lespiau Subject: Re: [PATCH 17/49] drm/i915/bdw: A bit more advanced context init/fini Date: Tue, 1 Apr 2014 14:51:27 +0100 Message-ID: <20140401135127.GD16291@strange.amr.corp.intel.com> References: <1395943218-7708-1-git-send-email-oscar.mateo@intel.com> <1395943218-7708-18-git-send-email-oscar.mateo@intel.com> <20140401003804.GA26578@strange.amr.corp.intel.com> <92648605EABDA246B775AAB04C95A7A3012BE598@IRSMSX103.ger.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by gabe.freedesktop.org (Postfix) with ESMTP id 449376E6F8 for ; Tue, 1 Apr 2014 06:52:07 -0700 (PDT) Content-Disposition: inline In-Reply-To: <92648605EABDA246B775AAB04C95A7A3012BE598@IRSMSX103.ger.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: "Mateo Lozano, Oscar" Cc: "intel-gfx@lists.freedesktop.org" , Ben Widawsky , "Widawsky, Benjamin" List-Id: intel-gfx@lists.freedesktop.org On Tue, Apr 01, 2014 at 02:47:19PM +0100, Mateo Lozano, Oscar wrote: > > > --- a/drivers/gpu/drm/i915/i915_lrc.c > > > +++ b/drivers/gpu/drm/i915/i915_lrc.c > > > @@ -41,7 +41,45 @@ > > > #include > > > #include "i915_drv.h" > > > > > > +#define GEN8_LR_CONTEXT_SIZE (21 * PAGE_SIZE) > > = > > I'm a bit puzzled by that number: > > - I found a sentence saying: "the Context Image for the rendering > > engine consists of 20 4K pages", which seems that it includes the > > HWS page (on the same page it says context layout =3D HWS Page + > > register state context). > > - When looking at the register state context for the render engine: > > 18096 dwords -> 18 pages, so in total it'd be 19 pages (need to add > > the HWS Page) > > - Clearly I must be missing something :) > > - That's only for the render engine, other engines have a much smaller > > context, smaller enough that it's worth looking at their exact size. > > - It'd be nice to work out the real size from the *CXT_*SIZE > > registers. > = > Hmmmm... I=B4ll try to get the real context sizes from the registers and > compare. At least for RCS, VCS and BCS since there doesn=B4t seem to be > a register for VECS? Couldn't find it either. I guess we'll need to ask the help of a friend. Or the 50/50 joker maybe. -- = Damien