[PATCH 2/2] drm/xe: Use 64K pages for scanout buffers for Xe2

[Matthew Auld]

On 09/07/2024 12:57, Zbigniew Kempczyński wrote:
> On Tue, Jul 09, 2024 at 12:26:15PM +0100, Matthew Auld wrote:
>> On 09/07/2024 12:12, Zbigniew Kempczyński wrote:
>>> Set Xe2 platforms to use 64K pages for display.
>>>
>>> Signed-off-by: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>
>>> Cc: Matthew Auld <matthew.auld at intel.com>
>>> Cc: Rodrigo Vivi <rodrigo.vivi at intel.com>
>>> ---
>>>    drivers/gpu/drm/xe/xe_pci.c | 1 +
>>>    1 file changed, 1 insertion(+)
>>>
>>> diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
>>> index 53b67f39c7d5..b629145d23fa 100644
>>> --- a/drivers/gpu/drm/xe/xe_pci.c
>>> +++ b/drivers/gpu/drm/xe/xe_pci.c
>>> @@ -177,6 +177,7 @@ static const struct xe_graphics_desc graphics_xe2 = {
>>>    	.name = "Xe2_LPG / Xe2_HPG",
>>>    	XE2_GFX_FEATURES,
>>> +	.vram_flags = XE_VRAM_FLAGS_DISPLAY_NEED64K,
>>
>> This change will also apply to lnl, but I don't think we have a way of
>> forcefully getting 64K pages for system memory atm. Do we just need this for
>> bmg or is it really all of xe2?
> 
> Issue we've seen was on BMG. Do you suggest to separate BMG desc entry
> to keep LNL intact?

If we go this route then yes I think best to leave lnl, if this is not 
needed there.

> 
> --
> Zbigniew
> 
>>
>>>    };
>>>    static const struct xe_media_desc media_xem = {