[igt-dev] [PATCH i-g-t] tests/i915/i915_module_load : Use GEM_BUSY instead of SET_CACHE for sanity check

Dandamudi, Priyanka priyanka.dandamudi at intel.com
Wed Oct 19 07:26:10 UTC 2022


Add original author in the signed-off along with yours and in the author put the original author as major changes haven't been done.
After that change.
You can add Acked-by: Priyanka Dandamudi <priyanka.dandamudi at intel.com>

> -----Original Message-----
> From: igt-dev <igt-dev-bounces at lists.freedesktop.org> On Behalf Of
> vikassri07
> Sent: 14 October 2022 02:32 PM
> To: igt-dev at lists.freedesktop.org; Iddamsetty, Aravind
> <aravind.iddamsetty at intel.com>; Wilson, Chris P
> <chris.p.wilson at intel.com>
> Cc: Kumar, Janga Rahul <janga.rahul.kumar at intel.com>; Yang, Fei
> <fei.yang at intel.com>
> Subject: [igt-dev] [PATCH i-g-t] tests/i915/i915_module_load : Use
> GEM_BUSY instead of SET_CACHE for sanity check
> 
>     Use GEM_BUSY ioctl for sanity check as SET_CACHING ioctl
>     is not supported on latest platforms like MTL.
> 
>     References: VLK-36552
> 
>     Signed-off-by: vikas.srivastava <vikas.srivastava at intel.com>
>     CC: Janga Rahul Kumar <janga.rahul.kumar at intel.com>
>     Cc: Aravind Iddamsetty<aravind.iddamsetty at intel.com>
>     Cc: Fei Yang <fei.yang at intel.com>
>     Cc: Stuart Summers <stuart.summers at intel.com>
>     Cc: Chris Wilson <chris.p.wilson at intel.com>
> ---
>  tests/i915/i915_module_load.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/i915/i915_module_load.c b/tests/i915/i915_module_load.c
> index 1009a22b..eebb3167 100644
> --- a/tests/i915/i915_module_load.c
> +++ b/tests/i915/i915_module_load.c
> @@ -226,13 +226,13 @@ inject_fault(const char *module_name, const char
> *opt, int fault)
> 
>  static void gem_sanitycheck(void)
>  {
> -	struct drm_i915_gem_caching args = {};
> +	struct drm_i915_gem_busy args = {};
>  	int i915 = __drm_open_driver(DRIVER_INTEL);
>  	int expected = gem_has_lmem(i915) ? -ENODEV : -ENOENT;
>  	int err;
> 
>  	err = 0;
> -	if (ioctl(i915, DRM_IOCTL_I915_GEM_SET_CACHING, &args))
> +	if (ioctl(i915,DRM_IOCTL_I915_GEM_BUSY, &args))
>  		err = -errno;
>  	if (err == expected)
>  		store_all(i915);
> --
> 2.25.1



More information about the igt-dev mailing list