From: Ben Widawsky <[email protected]> V2 (Anuj): Squash the changes in one patch rebase on master. Address the review comments made by Francisco Jerez. Do the URB allocation per slice (not per bank).
Signed-off-by: Ben Widawsky <[email protected]> Signed-off-by: Anuj Phogat <[email protected]> Cc: Francisco Jerez <[email protected]> --- V1 was sent out with the heading: "i965/cnl: Properly handle l3 configuration" --- src/intel/common/gen_l3_config.c | 22 +++++++++++++++++++++- 1 file changed, 21 insertions(+), 1 deletion(-) diff --git a/src/intel/common/gen_l3_config.c b/src/intel/common/gen_l3_config.c index 9a7771a..052c21b 100644 --- a/src/intel/common/gen_l3_config.c +++ b/src/intel/common/gen_l3_config.c @@ -116,6 +116,24 @@ static const struct gen_l3_config bxt_2x6_l3_configs[] = { }; /** + * On CNL, the previous "IS" "C" and "T" fields do not exist in a + * programmable way. As a result we have fewer allocation parameters. + */ +static const struct gen_l3_config cnl_l3_configs[] = { + /* SLM URB Rest DC RO */ + {{ 0, 64, 64, 0, 0 }}, + {{ 0, 64, 0, 16, 48 }}, + {{ 0, 48, 0, 16, 64 }}, + {{ 0, 32, 0, 0, 96 }}, + {{ 0, 32, 96, 0, 0 }}, + {{ 0, 32, 0, 16, 80 }}, + {{ 32, 16, 80, 0, 0 }}, + {{ 32, 16, 0, 64, 16 }}, + {{ 32, 0, 96, 0, 0 }}, + {{ 0 }} +}; + +/** * Return a zero-terminated array of validated L3 configurations for the * specified device. */ @@ -130,11 +148,13 @@ get_l3_configs(const struct gen_device_info *devinfo) return (devinfo->is_cherryview ? chv_l3_configs : bdw_l3_configs); case 9: - case 10: if (devinfo->gen >= 9 && devinfo->l3_banks == 1) return bxt_2x6_l3_configs; return chv_l3_configs; + case 10: + return cnl_l3_configs; + default: unreachable("Not implemented"); } -- 2.9.4 _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
