On Fri, Sep 22, 2017 at 6:31 AM Rodrigo Vivi <[email protected]> wrote:
> On Thu, Sep 21, 2017 at 11:19:49PM +0000, Oscar Mateo wrote: > > The total size of the context has decreased with the removal of the > > URB_ATOMIC section. BSpec indicates 16750 DWORDs (17 pages), plus > > one page for PPHWSP, and I'm throwing an extra page for precaution. > > I could never find this info on bspec... could you please point that to > me? Michal already pointed the doc to me! Now I know where that comes from! :) Patch Merges to dinq. Thanks > > Anyways this value matches with other HW engineers had told me a while > ago, and we now have CNL on CI, so: > > Acked-by: Rodrigo Vivi <[email protected]> > > > > > Signed-off-by: Oscar Mateo <[email protected]> > > Cc: Rodrigo Vivi <[email protected]> > > Cc: Daniele Ceraolo Spurio <[email protected]> > > Cc: Ben Widawsky <[email protected]> > > --- > > drivers/gpu/drm/i915/intel_engine_cs.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/drivers/gpu/drm/i915/intel_engine_cs.c > b/drivers/gpu/drm/i915/intel_engine_cs.c > > index 3d135c3..a3115f3 100644 > > --- a/drivers/gpu/drm/i915/intel_engine_cs.c > > +++ b/drivers/gpu/drm/i915/intel_engine_cs.c > > @@ -39,6 +39,7 @@ > > > > #define GEN8_LR_CONTEXT_RENDER_SIZE (20 * PAGE_SIZE) > > #define GEN9_LR_CONTEXT_RENDER_SIZE (22 * PAGE_SIZE) > > +#define GEN10_LR_CONTEXT_RENDER_SIZE (19 * PAGE_SIZE) > > > > #define GEN8_LR_CONTEXT_OTHER_SIZE ( 2 * PAGE_SIZE) > > > > @@ -150,6 +151,7 @@ struct engine_info { > > default: > > MISSING_CASE(INTEL_GEN(dev_priv)); > > case 10: > > + return GEN10_LR_CONTEXT_RENDER_SIZE; > > case 9: > > return GEN9_LR_CONTEXT_RENDER_SIZE; > > case 8: > > -- > > 1.9.1 > > > _______________________________________________ > Intel-gfx mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/intel-gfx >
_______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
