Intend to work on the DRM_* logging macro conversion

Intend to work on the DRM_* logging macro conversion

[YJ Lin]

Hi Sean and the vkms maintainers:

I’m Leo, currently one of the mentees of the Linux Kernel Bug Fixing
Program 2024 Summer Unpaid[1]. I saw an item on the DRM TODO list
regarding "Convert logging to drm_* functions with drm_device
parameter"[2]. It also suggests reaching out to the driver maintainers
beforehand. So I’d like to understand your opinions toward making this
conversion on the vkms driver, and I’d like to start working on it if
you consider it suitable. Thank you!

(Also, do feel free to share tips for DRM and kernel development in general.)

Best,
Leo

[1] https://mentorship.lfx.linuxfoundation.org/project/9c354ea7-f08e-4d85-a93d-94d2a3cb30c8
[2] https://www.kernel.org/doc/html/latest/gpu/todo.html#convert-logging-to-drm-functions-with-drm-device-parameter

Re: Intend to work on the DRM_* logging macro conversion

[Louis Chauvet]

Le 16/09/24 - 23:46, YJ Lin a écrit :
> Hi Sean and the vkms maintainers:
> 
> I’m Leo, currently one of the mentees of the Linux Kernel Bug Fixing
> Program 2024 Summer Unpaid[1]. I saw an item on the DRM TODO list
> regarding "Convert logging to drm_* functions with drm_device
> parameter"[2]. It also suggests reaching out to the driver maintainers
> beforehand. So I’d like to understand your opinions toward making this
> conversion on the vkms driver, and I’d like to start working on it if
> you consider it suitable. Thank you!

Hi Leo,

It is very nice to work on this subject, thank you!

On the VKMS side, the current upstream driver can only create one instance 
of the device, but we are working on supporting more, so it definitly 
make sense to have per-device logging. Feel free to send patches to change 
this where it make sense.

Few series are pending on the mailing list, so feel free to point where we 
miss this too! (for example [3,4]

[3]:https://lore.kernel.org/all/20240912-google-vkms-managed-v3-0-7708d6ad262d@bootlin.com/
[4]:https://lore.kernel.org/all/20240912-b4-vkms-allocated-v1-0-29abbaa14af9@bootlin.com/

> (Also, do feel free to share tips for DRM and kernel development in general.)

Don't hesitate to send documentation patches if you had difficulties to 
understand stuff, it will profit to everyone. If you need more 
documentation about VKMS internals, a "big" patch is pending [5].

[5]:https://lore.kernel.org/all/20240905-google-clarifications-v4-1-e43c1f2e3d87@bootlin.com/

Thanks,
Louis Chauvet

> Best,
> Leo
> 
> [1] https://mentorship.lfx.linuxfoundation.org/project/9c354ea7-f08e-4d85-a93d-94d2a3cb30c8
> [2] https://www.kernel.org/doc/html/latest/gpu/todo.html#convert-logging-to-drm-functions-with-drm-device-parameter

--
Louis Chauvet, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com