Hi Andres, You should cherry-pick below commits to mesa stable: 1. Commit eb23be1 - i965: Add and initialize l3_banks field for gen7+ 2. Commit 8521559 - i965: Fix broxton 2x6 l3 config
Please let me know if you have any problems cherry picking them. Thanks Anuj On Mon, Jun 26, 2017 at 8:21 AM, Andres Gomez <ago...@igalia.com> wrote: > Anuj, this depends on: > https://cgit.freedesktop.org/mesa/mesa/commit/src/mesa?id=eb23be1d97da290073d76c2510b8999b250f0139 > > Which didn't make it for -stable. Should we cherry-pick that too? > > On Mon, 2017-06-12 at 10:01 -0700, Anuj Phogat wrote: >> The new table added in this patch matches with the table >> in gfxspecs. We were programming the wrong values earlier. >> >> Signed-off-by: Anuj Phogat <anuj.pho...@gmail.com> >> Cc: Francisco Jerez <curroje...@riseup.net> >> Cc: "17.1" <mesa-sta...@lists.freedesktop.org> >> --- >> src/intel/common/gen_device_info.c | 1 + >> src/intel/common/gen_device_info.h | 1 + >> src/intel/common/gen_l3_config.c | 19 +++++++++++++++++++ >> 3 files changed, 21 insertions(+) >> >> diff --git a/src/intel/common/gen_device_info.c >> b/src/intel/common/gen_device_info.c >> index 75284a6..eccb464 100644 >> --- a/src/intel/common/gen_device_info.c >> +++ b/src/intel/common/gen_device_info.c >> @@ -502,6 +502,7 @@ static const struct gen_device_info gen_device_info_bxt >> = { >> >> static const struct gen_device_info gen_device_info_bxt_2x6 = { >> GEN9_LP_FEATURES_2X6, >> + .is_broxton_2x6 = 1, >> .l3_banks = 1, >> }; >> /* >> diff --git a/src/intel/common/gen_device_info.h >> b/src/intel/common/gen_device_info.h >> index 6207630..4fe1b21 100644 >> --- a/src/intel/common/gen_device_info.h >> +++ b/src/intel/common/gen_device_info.h >> @@ -41,6 +41,7 @@ struct gen_device_info >> bool is_haswell; >> bool is_cherryview; >> bool is_broxton; >> + bool is_broxton_2x6; >> bool is_kabylake; >> >> bool has_hiz_and_separate_stencil; >> diff --git a/src/intel/common/gen_l3_config.c >> b/src/intel/common/gen_l3_config.c >> index ae31d08..e17994b 100644 >> --- a/src/intel/common/gen_l3_config.c >> +++ b/src/intel/common/gen_l3_config.c >> @@ -102,6 +102,23 @@ static const struct gen_l3_config chv_l3_configs[] = { >> }; >> >> /** >> + * BXT 2x6 validated L3 configurations. \sa ivb_l3_configs. >> + * Number of ways = >> + * Allocation in KB for SKU / (Way size per bank * Number of banks). >> + * For BXT 2x6: Banks = 1, Way size per bank = 4. >> + */ >> +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 +134,8 @@ get_l3_configs(const struct gen_device_info *devinfo) >> >> case 9: >> case 10: >> + if (devinfo->is_broxton_2x6) >> + return bxt_2x6_l3_configs; >> return chv_l3_configs; >> >> default: > -- > Br, > > Andres _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
