[Mesa-dev] [PATCH] anv: Fix L3 cache programming on Bay Trail
Jason Ekstrand
jason at jlekstrand.net
Mon Jun 19 19:06:27 UTC 2017
On Mon, Jun 19, 2017 at 11:21 AM, Francisco Jerez <currojerez at riseup.net>
wrote:
> Jonas Kulla <nyocurio at gmail.com> writes:
>
> > Valid values for URBAllocation start at 32, so substract that
> > before programming the register.
> >
> > This was missed when porting from the GL driver.
>
> Reviewed-by: Francisco Jerez <currojerez at riseup.net>
>
Reviewed-by: Jason Ekstrand <jason at jlekstrand.net>
And pushed with a CC to stable.
Welcome to mesa-dev!
--Jason
> > ---
> > src/intel/vulkan/genX_cmd_buffer.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/src/intel/vulkan/genX_cmd_buffer.c
> b/src/intel/vulkan/genX_cmd_buffer.c
> > index 59430a2..0216ea0 100644
> > --- a/src/intel/vulkan/genX_cmd_buffer.c
> > +++ b/src/intel/vulkan/genX_cmd_buffer.c
> > @@ -835,7 +835,7 @@ genX(cmd_buffer_config_l3)(struct anv_cmd_buffer
> *cmd_buffer,
> > anv_pack_struct(&l3cr2, GENX(L3CNTLREG2),
> > .SLMEnable = has_slm,
> > .URBLowBandwidth = urb_low_bw,
> > - .URBAllocation = cfg->n[GEN_L3P_URB],
> > + .URBAllocation = cfg->n[GEN_L3P_URB] - n0_urb,
> > #if !GEN_IS_HASWELL
> > .ALLAllocation = cfg->n[GEN_L3P_ALL],
> > #endif
> > --
> > 2.7.5
> >
> > _______________________________________________
> > mesa-dev mailing list
> > mesa-dev at lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170619/7633a922/attachment-0001.html>
More information about the mesa-dev
mailing list