linux-kernel - Re: [PATCH 00/10] drm: Remove usage of deprecated DRM

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <8735992hyi.fsf@intel.com>
Date:   Wed, 21 Dec 2022 12:01:57 +0200
From:   Jani Nikula <jani.nikula@...ux.intel.com>
To:     Thomas Zimmermann <tzimmermann@...e.de>,
        Siddh Raman Pant <code@...dh.me>,
        Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
        Maxime Ripard <mripard@...nel.org>,
        David Airlie <airlied@...il.com>,
        Daniel Vetter <daniel@...ll.ch>
Cc:     linux-kernel <linux-kernel@...r.kernel.org>,
        dri-devel <dri-devel@...ts.freedesktop.org>
Subject: Re: [PATCH 00/10] drm: Remove usage of deprecated DRM_* macros

On Wed, 21 Dec 2022, Thomas Zimmermann <tzimmermann@...e.de> wrote:
> Hi
>
> Am 20.12.22 um 21:16 schrieb Siddh Raman Pant:
>> This patchset aims to remove usages of deprecated DRM_* macros from the
>> files residing in drivers/gpu/drm root.
>> 
>> In process, I found out that NULL as first argument of drm_dbg_* wasn't
>> working, but it was listed as the alternative in deprecation comment,
>> so I fixed that before removing usages of DRM_DEBUG_* macros.
>> 
>> This patchset should be applied in order as changes might be dependent.
>> 
>> Please review and let me know if any errors are there, and hopefully
>> this gets accepted.
>
> Thanks for the patchset. Overall this looks fine.
>
> But the use of pr_() functions is not optimal in my opinion. I know that 
> I recommended using it, but at a closer look, I think we might want to 
> reconsider that. I wouldn't like losing the '[drm]' tag, but adding them 
> to each pr_() call isn't a good idea either.
>
> I think it would be nicer to replace DRM_INFO with drm_info(NULL, ). 
> Depending on the device being NULL, the helpers would select dev-based 
> messages or printk-based messages. It would then work like drm_dbg_(). 
> But it needs some additional changes.
>
> Can you first move all pr_() related changes into a new patchset? The 
> trivial conversion were the device is known and not NULL can remain in 
> this patchset and be merged soon. We can afterwards have a separate 
> discussion for the changes that currently involve pr_().

Same thoughts, agreed.

BR,
Jani.


>
> Best regards
> Thomas
>
>> 
>> Siddh Raman Pant (10):
>>    drm: Remove usage of deprecated DRM_INFO
>>    drm: Remove usage of deprecated DRM_NOTE
>>    drm: Remove usage of deprecated DRM_ERROR
>>    drm/print: Fix support for NULL as first argument of drm_dbg_*
>>    drm: Remove usage of deprecated DRM_DEBUG
>>    drm: Remove usage of deprecated DRM_DEBUG_DRIVER
>>    drm: Remove usage of deprecated DRM_DEBUG_KMS
>>    drm: Remove usage of deprecated DRM_DEBUG_PRIME
>>    drm/drm_blend: Remove usage of deprecated DRM_DEBUG_ATOMIC
>>    drm/drm_lease: Remove usage of deprecated DRM_DEBUG_LEASE
>> 
>>   drivers/gpu/drm/drm_agpsupport.c        |   4 +-
>>   drivers/gpu/drm/drm_blend.c             |  13 ++-
>>   drivers/gpu/drm/drm_bridge.c            |   8 +-
>>   drivers/gpu/drm/drm_bufs.c              | 122 ++++++++++++------------
>>   drivers/gpu/drm/drm_client_modeset.c    | 118 +++++++++++++----------
>>   drivers/gpu/drm/drm_color_mgmt.c        |   4 +-
>>   drivers/gpu/drm/drm_connector.c         |  28 +++---
>>   drivers/gpu/drm/drm_context.c           |  18 ++--
>>   drivers/gpu/drm/drm_crtc.c              |  36 ++++---
>>   drivers/gpu/drm/drm_crtc_helper.c       |  62 ++++++------
>>   drivers/gpu/drm/drm_debugfs_crc.c       |   8 +-
>>   drivers/gpu/drm/drm_displayid.c         |   6 +-
>>   drivers/gpu/drm/drm_dma.c               |  10 +-
>>   drivers/gpu/drm/drm_drv.c               |  28 +++---
>>   drivers/gpu/drm/drm_edid.c              |  17 ++--
>>   drivers/gpu/drm/drm_file.c              |  18 ++--
>>   drivers/gpu/drm/drm_flip_work.c         |   2 +-
>>   drivers/gpu/drm/drm_framebuffer.c       |   3 +-
>>   drivers/gpu/drm/drm_gem.c               |   7 +-
>>   drivers/gpu/drm/drm_gem_dma_helper.c    |   6 +-
>>   drivers/gpu/drm/drm_gem_shmem_helper.c  |   6 +-
>>   drivers/gpu/drm/drm_hashtab.c           |  10 +-
>>   drivers/gpu/drm/drm_ioc32.c             |  13 +--
>>   drivers/gpu/drm/drm_ioctl.c             |  24 ++---
>>   drivers/gpu/drm/drm_irq.c               |   4 +-
>>   drivers/gpu/drm/drm_kms_helper_common.c |   2 +-
>>   drivers/gpu/drm/drm_lease.c             |  68 ++++++-------
>>   drivers/gpu/drm/drm_legacy_misc.c       |   4 +-
>>   drivers/gpu/drm/drm_lock.c              |  36 +++----
>>   drivers/gpu/drm/drm_mipi_dbi.c          |  19 ++--
>>   drivers/gpu/drm/drm_mm.c                |   8 +-
>>   drivers/gpu/drm/drm_mode_config.c       |   2 +-
>>   drivers/gpu/drm/drm_mode_object.c       |   6 +-
>>   drivers/gpu/drm/drm_modes.c             |  10 +-
>>   drivers/gpu/drm/drm_modeset_helper.c    |   2 +-
>>   drivers/gpu/drm/drm_pci.c               |  14 +--
>>   drivers/gpu/drm/drm_plane.c             |  46 ++++-----
>>   drivers/gpu/drm/drm_probe_helper.c      |  39 ++++----
>>   drivers/gpu/drm/drm_rect.c              |   4 +-
>>   drivers/gpu/drm/drm_scatter.c           |  19 ++--
>>   drivers/gpu/drm/drm_syncobj.c           |   2 +-
>>   drivers/gpu/drm/drm_sysfs.c             |  22 ++---
>>   drivers/gpu/drm/drm_vm.c                |  45 +++++----
>>   include/drm/drm_print.h                 |  91 ++++++++++++++----
>>   44 files changed, 549 insertions(+), 465 deletions(-)
>> 

-- 
Jani Nikula, Intel Open Source Graphics Center