[PATCH i-g-t 1/3] tests/vmwgfx: Removing incorrect assert in mob_repeated_unref

Kamil Konieczny kamil.konieczny at linux.intel.com
Mon May 13 14:01:55 UTC 2024


Hi Zack,
On 2024-05-09 at 15:15:27 -0400, Zack Rusin wrote:
> From: Maaz Mombasawala <mombasawalam at vmware.com>
> 
> Remove an incorrect assert in test mob_repeated_unref that checked if return
> value from multiple unrefs of a mob was 0. This was a bug in the vmwgfx driver,
> since after the first unref it should return -EINVAL.
> This bug was fixed with buffer rework in the vmwgfx driver and now the unref
> ioctl behaves the correct way.
> 
> Signed-off-by: Maaz Mombasawala <mombasawalam at vmware.com>
> Reviewed-by: Martin Krastev <krastevm at vmware.com>
> Reviewed-by: Ian Forbes <iforbes at vmware.com>
> Signed-off-by: Zack Rusin <zack.rusin at broadcom.com>
> ---
>  tests/vmwgfx/vmw_ref_count.c | 6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)
> 
> diff --git a/tests/vmwgfx/vmw_ref_count.c b/tests/vmwgfx/vmw_ref_count.c
> index da4b41f89..92d49dbd0 100644
> --- a/tests/vmwgfx/vmw_ref_count.c
> +++ b/tests/vmwgfx/vmw_ref_count.c
> @@ -1,6 +1,6 @@
>  // SPDX-License-Identifier: GPL-2.0 OR MIT
>  /**********************************************************
> - * Copyright 2021-2022 VMware, Inc.
> + * Copyright 2021-2023 VMware, Inc.
-------------------- ^^^^
This should be 2024.

Regards,
Kamil

>   *
>   * Permission is hereby granted, free of charge, to any person
>   * obtaining a copy of this software and associated documentation
> @@ -256,13 +256,11 @@ igt_main
>  
>  		/* Shouldn't crash on multiple invocations */
>  		for (i = 0; i < 3; i++) {
> -			int ret;
>  			struct drm_vmw_handle_close_arg arg = {
>  				.handle = mob->handle
>  			};
> -			ret = drmCommandWrite(fd1, DRM_VMW_HANDLE_CLOSE, &arg,
> +			drmCommandWrite(fd1, DRM_VMW_HANDLE_CLOSE, &arg,
>  					      sizeof(arg));
> -			igt_assert_eq(ret, 0);
>  		}
>  		free(mob);
>  	}
> -- 
> 2.40.1
> 


More information about the igt-dev mailing list