[igt-dev] [PATCH i-g-t] tests/intel/xe_debugfs : Check tile_count info

Kamil Konieczny kamil.konieczny at linux.intel.com
Fri Sep 15 18:19:04 UTC 2023


Hi Janga,

small nit in subject:

[PATCH i-g-t] tests/intel/xe_debugfs : Check tile_count info
------------------------------------^

Remove space before ':', it should be:

[PATCH i-g-t] tests/intel/xe_debugfs: Check tile_count info

On 2023-09-15 at 18:56:47 +0530, janga.rahul.kumar at intel.com wrote:
> From: Janga Rahul Kumar <janga.rahul.kumar at intel.com>
> 
> Check correctness of tile_count info.
> 
> Cc: Sai Gowtham Ch <sai.gowtham.ch at intel.com>
> Signed-off-by: Janga Rahul Kumar <janga.rahul.kumar at intel.com>
> ---
>  tests/intel/xe_debugfs.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/intel/xe_debugfs.c b/tests/intel/xe_debugfs.c
> index 88b51dbd4..2e752267c 100644
> --- a/tests/intel/xe_debugfs.c
> +++ b/tests/intel/xe_debugfs.c
> @@ -13,7 +13,7 @@
>   */
>  
>  #include "igt.h"
> -
> +#include "igt_sysfs.h"
>  #include "xe_drm.h"
>  #include "xe/xe_ioctl.h"
>  #include "xe/xe_query.h"
> @@ -117,6 +117,9 @@ test_base(int fd, struct drm_xe_query_config *config)
>  		igt_assert(igt_debugfs_search(fd, "info", reference));
>  	}
>  
> +	sprintf(reference, "tile_count %d", xe_sysfs_get_num_tiles(fd));
    ^
Use snprintf?

Regards,
Kamil

> +	igt_assert(igt_debugfs_search(fd, "info", reference));
> +
>  	igt_assert(igt_debugfs_exists(fd, "gt0", O_RDONLY));
>  	if (config->info[XE_QUERY_CONFIG_GT_COUNT] > 1)
>  		igt_assert(igt_debugfs_exists(fd, "gt1", O_RDONLY));
> -- 
> 2.25.1
> 


More information about the igt-dev mailing list