Anuj Phogat <[email protected]> writes: > The new table added in this patch matches with the table > in gfxspecs. We were programming the wrong values earlier. > > V2: Update the comment. > > Signed-off-by: Anuj Phogat <[email protected]> > Cc: Francisco Jerez <[email protected]> > Cc: "17.1" <[email protected]> > --- > src/intel/common/gen_l3_config.c | 16 ++++++++++++++++ > 1 file changed, 16 insertions(+) > > diff --git a/src/intel/common/gen_l3_config.c > b/src/intel/common/gen_l3_config.c > index ae31d08..44a4b24 100644 > --- a/src/intel/common/gen_l3_config.c > +++ b/src/intel/common/gen_l3_config.c > @@ -102,6 +102,20 @@ static const struct gen_l3_config chv_l3_configs[] = { > }; > > /** > + * BXT 2x6 validated L3 configurations. \sa ivb_l3_configs. > + */ > +static const struct gen_l3_config bxt_2x6_l3_configs[] = { > + /* SLM URB ALL DC RO IS C T */ > + {{ 0, 32, 48, 0, 0, 0, 0, 0 }}, > + {{ 0, 32, 0, 8, 40, 0, 0, 0 }}, > + {{ 0, 32, 0, 32, 16, 0, 0, 0 }}, > + {{ 16, 16, 48, 0, 0, 0, 0, 0 }}, > + {{ 16, 16, 0, 40, 8, 0, 0, 0 }}, > + {{ 16, 16, 0, 16, 32, 0, 0, 0 }}, > + {{ 0 }} > +}; > + > +/** > * Return a zero-terminated array of validated L3 configurations for the > * specified device. > */ > @@ -117,6 +131,8 @@ get_l3_configs(const struct gen_device_info *devinfo) > > case 9: > case 10: > + if (devinfo->gen >= 9 && devinfo->l3_banks == 1) > + return bxt_2x6_l3_configs;
The 'devinfo->gen >= 9' check is redundant under this case statement. With that dropped patch is: Reviewed-by: Francisco Jerez <[email protected]> > return chv_l3_configs; > > default: > -- > 2.9.4
signature.asc
Description: PGP signature
_______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
