[igt-dev] [PATCH v3 4/4] tests/xe/xe_sysfs_tile_prop: adds new test to verify per tile vram addr_range

[Upadhyay, Tejas]

> -----Original Message-----
> From: Ghimiray, Himal Prasad <himal.prasad.ghimiray at intel.com>
> Sent: Tuesday, June 27, 2023 5:53 PM
> To: igt-dev at lists.freedesktop.org
> Cc: Ghimiray, Himal Prasad <himal.prasad.ghimiray at intel.com>; Iddamsetty,
> Aravind <aravind.iddamsetty at intel.com>; Upadhyay, Tejas
> <tejas.upadhyay at intel.com>; Kumar, Janga Rahul
> <janga.rahul.kumar at intel.com>
> Subject: [PATCH v3 4/4] tests/xe/xe_sysfs_tile_prop: adds new test to verify
> per tile vram addr_range
> 
> For each tile the test reads the sysfs entry physical_vram_size_bytes and
> compares the value with vram size exposed from query ioctl.
> 
> v2:
> - Change sysfs entry name. (Tejas)
> - Change test name to xe_sysfs_tile_prop. (Rahul)
> 
> Cc: Aravind Iddamsetty <aravind.iddamsetty at intel.com>
> Cc: Upadhyay <tejas.upadhyay at intel.com>
> Cc: Janga Rahul Kumar <janga.rahul.kumar at intel.com>
> Signed-off-by: Himal Prasad Ghimiray <himal.prasad.ghimiray at intel.com>
> ---
>  tests/meson.build             |  1 +
>  tests/xe/xe_sysfs_tile_prop.c | 63 +++++++++++++++++++++++++++++++++++
>  2 files changed, 64 insertions(+)
>  create mode 100644 tests/xe/xe_sysfs_tile_prop.c
> 
> diff --git a/tests/meson.build b/tests/meson.build index 61dcc076..569f1d22
> 100644
> --- a/tests/meson.build
> +++ b/tests/meson.build
> @@ -267,6 +267,7 @@ xe_progs = [
>  	'xe_pm',
>  	'xe_prime_self_import',
>  	'xe_query',
> +	'xe_sysfs_tile_prop',
>  	'xe_vm',
>  	'xe_waitfence',
>  ]
> diff --git a/tests/xe/xe_sysfs_tile_prop.c b/tests/xe/xe_sysfs_tile_prop.c new
> file mode 100644 index 00000000..18dfeca5
> --- /dev/null
> +++ b/tests/xe/xe_sysfs_tile_prop.c
> @@ -0,0 +1,63 @@
> +// SPDX-License-Identifier: MIT
> +/*
> + * Copyright © 2023 Intel Corporation
> + */
> +
> +/**
> + * TEST: Verify physical_vram_size_bytes of each tiles
> + * SUBTEST: physical_vram_size_bytes
> + * Description:
> + *             Read sysfs entry for physical_vram_size_bytes and compare with
> + *             vram size. physical_vram_size_bytes should be more than vram
> size.
> + */
> +
> +#include <string.h>
> +#include <sys/time.h>
> +
> +#include "igt.h"
> +#include "igt_sysfs.h"
> +
> +#include "xe_drm.h"
> +#include "xe/xe_ioctl.h"
> +#include "xe/xe_query.h"
> +
> +static void test_vram_physical_vram_size_bytes(int sysfs, int tile_num,
> +u64 vram_size) {
> +	u64 physical_vram_size_bytes;
> +	char path[40];

If possible reuse some defined macro for path size. 
Reviewed-by: Tejas Upadhyay <tejas.upadhyay at intel.com>

> +
> +	igt_assert(snprintf(path, sizeof(path),
> "device/tile%d/physical_vram_size_bytes",
> +			    tile_num) < sizeof(path));
> +	igt_assert(igt_sysfs_scanf(sysfs, path, "%lx",
> &physical_vram_size_bytes) < 0);
> +	igt_assert_lt(vram_size, physical_vram_size_bytes); }
> +
> +igt_main
> +{
> +	int fd;
> +	int tile, total_tiles;
> +	static int sysfs = -1;
> +	u64 vram_size;
> +
> +	igt_fixture {
> +		fd = drm_open_driver(DRIVER_XE);
> +		xe_device_get(fd);
> +
> +		sysfs = igt_sysfs_open(fd);
> +		igt_assert(sysfs != -1);
> +	}
> +
> +	igt_subtest("physical_vram_size_bytes") {
> +		igt_require(xe_has_vram(fd));
> +		xe_for_each_tile(fd, tile, total_tiles) {
> +			vram_size = xe_vram_size(fd, tile);
> +			test_vram_physical_vram_size_bytes(sysfs, tile,
> vram_size);
> +		}
> +	}
> +
> +	igt_fixture {
> +		close(sysfs);
> +		xe_device_put(fd);
> +		close(fd);
> +	}
> +}
> --
> 2.25.1