Re: [igt-dev] [PATCH i-g-t] perf: Treat Cometlake as an alias for Coffeelake

Lionel Landwerlin
2020-01-24T17:11:03Z

Thanks.
Hopefully I'll a series soon getting rid of all this special case stuff.

Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>

On 24/01/2020 19:03, Chris Wilson wrote:
> The kernel doesn't differentiate between the Coffeelake and Cometlake
> platforms, but igt does. Add the alias for perf to use the Coffeelake
> register set for Cometlake.
>
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
> ---
> tests/perf.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/perf.c b/tests/perf.c
> index a837551cb..f1f9b7cea 100644
> --- a/tests/perf.c
> +++ b/tests/perf.c
> @@ -955,7 +955,7 @@ init_sys_info(void)
> }
> } else if (IS_GEMINILAKE(devid)) {
> test_set_uuid = "dd3fd789-e783-4204-8cd0-b671bbccb0cf";
> - } else if (IS_COFFEELAKE(devid)) {
> + } else if (IS_COFFEELAKE(devid) || IS_COMETLAKE(devid)) {
> switch (intel_gt(devid)) {
> case 1:
> test_set_uuid = "74fb4902-d3d3-4237-9e90-cbdc68d0a446";