[PATCH i-g-t] tests/xe_eudebug: Convert comments to be C-styled

Zbigniew Kempczyński zbigniew.kempczynski at intel.com
Wed Nov 20 11:18:24 UTC 2024


On Thu, Nov 14, 2024 at 03:41:56PM +0100, Dominik Grzegorzek wrote:
> Change two comments in xe_eudebug testcase to be properly formatted.
> This was pointed in the review, but the patch has been merge before
> correcting mentioned nits.
> 
> Fixes "05659c7f2 tests/xe_eudebug: Validate exec queue placements"
> 
> Reported-by: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>
> Signed-off-by: Dominik Grzegorzek <dominik.grzegorzek at intel.com>

Thanks:

Reviewed-by: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>

--
Zbigniew

> ---
>  tests/intel/xe_eudebug.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/intel/xe_eudebug.c b/tests/intel/xe_eudebug.c
> index 39db16457..e9765d06e 100644
> --- a/tests/intel/xe_eudebug.c
> +++ b/tests/intel/xe_eudebug.c
> @@ -474,13 +474,13 @@ static void placements_client(int fd, struct xe_eudebug_client *c)
>  		xe_eudebug_client_exec_queue_create(c, fd, &create);
>  		xe_eudebug_client_exec_queue_destroy(c, fd, &create);
>  
> -		// every other instance (logical_mask ~ 1010, 0101)
> +		/* every other instance (logical_mask ~ 1010, 0101) */
>  		create.width = 2;
>  		create.num_placements = count/2;
>  		xe_eudebug_client_exec_queue_create(c, fd, &create);
>  		xe_eudebug_client_exec_queue_destroy(c, fd, &create);
>  
> -		// logically contigous placement (logical_mask ~ 1100, 0110, 0011)
> +		/* logically contigous placement (logical_mask ~ 1100, 0110, 0011) */
>  		for (i = 0; i < count * count - 2; i++)
>  			eci[i].engine_instance = i / 2 + i % 2;
>  
> -- 
> 2.34.1
> 


More information about the igt-dev mailing list