Re: [PATCH 10/12] drm/{i915,xe}: Run DRM default client setup

[Jocelyn Falempe <jfalempe@redhat.com>]

On 07/01/2025 15:42, Thomas Zimmermann wrote:
> Hi Jocelyn
> 
> 
> Am 07.01.25 um 13:10 schrieb Jocelyn Falempe:
>> On 12/12/2024 18:08, Thomas Zimmermann wrote:
>>> Rework fbdev probing to support fbdev_probe in struct drm_driver
>>> and remove the old fb_probe callback. Provide an initializer macro
>>> that sets the callback in struct drm_driver according to the kernel
>>> configuration. Call drm_client_setup_with_color_mode() to run the
>>> kernel's default client setup for DRM.
>>>
>>> This commit also prepares support for the kernel's drm_log client
>>> (or any future client) in i915. Using drm_log will also require vmap
>>> support in GEM objects.
>>
>> I've tested this series on an Alderlake laptop, and it effectively 
>> breaks the boot with drm_log on i915. (I can still see the drm_log on 
>> simpledrm, but when it switches to i915, I get a blackscreen, and the 
>> laptop hard freezes).
>> Can we wait to have the proper vmap support in GEM objects, before 
>> merging this series?
>> Or at least prevent drm_log to load on i915, until it is fixed?
> 
> Did you add vmap support as advised here? i915 will not support drm_log 
> without and this series does not add it.
> 
> A patch for vmap functions is at [1], I think.

Thanks a lot, it works much better with this patch :)
That was the missing piece to have drm_log display something on i915.

Best regards,

-- 

Jocelyn

> 
> Best regards
> Thomas
> 
> [1] https://patchwork.freedesktop.org/series/135554/
> 
> 
>>
>> Best regards,
>>
>