Re: [PATCH i-g-t 2/3] tests/intel/xe_create: Stop treating the extension field as MBZ

[Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>]

On 1/31/2025 4:59 AM, Kamil Konieczny wrote:
> Hi Daniele,
> On 2025-01-30 at 14:14:56 -0800, Daniele Ceraolo Spurio wrote:
>> A new extension for the xe_create ioctl will be added to support PXP [1],
>> so in preparation for that to be merged stop expecting the field to always
>> be zero.
>>
>> [1] https://patchwork.freedesktop.org/patch/634550/
> Please add also lore.kernel.org link (could be done at merge)
> here and in next patch, Thanks!

Should I just replace this link instead of adding a new one?

Thanks,
Daniele

>
> Regards,
> Kamil
>
>> Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
>> Reviewed-by: Alan Previn <alan.previn.teres.alexis@intel.com>
>> ---
>>   tests/intel/xe_create.c | 5 -----
>>   1 file changed, 5 deletions(-)
>>
>> diff --git a/tests/intel/xe_create.c b/tests/intel/xe_create.c
>> index e3be6b774..e3ebb9fd4 100644
>> --- a/tests/intel/xe_create.c
>> +++ b/tests/intel/xe_create.c
>> @@ -243,11 +243,6 @@ static void create_invalid_mbz(int fd)
>>   	gem_close(fd, create.handle);
>>   	create.handle = 0;
>>   
>> -	/* No supported extensions yet */
>> -	create.extensions = -1;
>> -	igt_assert_eq(__ioctl_create(fd, &create), -EINVAL);
>> -	create.extensions = 0;
>> -
>>   	/* Make sure KMD rejects non-zero padding/reserved fields */
>>   	for (i = 0; i < ARRAY_SIZE(create.pad); i++) {
>>   		create.pad[i] = -1;
>> -- 
>> 2.43.0
>>