Re: [igt-dev] [PATCH i-g-t 1/2] lib/xe_query: fix vram_memory()

[Matthew Auld <matthew.auld@intel.com>]

On 04/04/2023 23:11, Niranjana Vishwanathapura wrote:
> On Tue, Apr 04, 2023 at 01:00:41PM +0100, Matthew Auld wrote:
>> It looks like this is meant to return VRAM if supported. If we call this
>> on igpu it should return zero. There is at least one user relying on
>> this in xe_mmap@vram, where we want to skip if the mask here is zero.
>>
> 
> Below patch was sent before, but never got merged.
> https://patchwork.freedesktop.org/series/115409/
> 
> There are some minor differences. Can you pick those
> changes?

Ok, missed that... Let me check.

> 
> Also, cover letter seems to be missing for this series.
> Or, we can simple merge them into a single patch as they
> are related?

Ok, will squash.

> 
> Niranjana
> 
>> Reported-by: Gwan-gyeong Mun <gwan-gyeong.mun@intel.com>
>> Signed-off-by: Matthew Auld <matthew.auld@intel.com>
>> Cc: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
>> Cc: Matthew Brost <matthew.brost@intel.com>
>> ---
>> lib/xe/xe_query.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/lib/xe/xe_query.c b/lib/xe/xe_query.c
>> index 24a258a5e..253164d6b 100644
>> --- a/lib/xe/xe_query.c
>> +++ b/lib/xe/xe_query.c
>> @@ -380,7 +380,7 @@ uint64_t vram_memory(int fd, int gt)
>>     igt_assert(xe_dev);
>>     igt_assert(gt >= 0 && gt < xe_dev->number_gt);
>>
>> -    return native_region_for_gt(xe_dev->gts, gt);
>> +    return native_region_for_gt(xe_dev->gts, gt) & (~0x1);
>> }
>>
>> /**
>> -- 
>> 2.39.2
>>