linux-kernel - Re: [PATCH v3 02/22] drm: convert many bridge drivers from devm_kzalloc() to devm_drm_bridge

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: <20250512160201.7d0b21d8@booty>
Date: Mon, 12 May 2025 16:02:01 +0200
From: Luca Ceresoli <luca.ceresoli@...tlin.com>
To: Maxime Ripard <mripard@...nel.org>
Cc: Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>, Thomas Zimmermann
 <tzimmermann@...e.de>, David Airlie <airlied@...il.com>, Simona Vetter
 <simona@...ll.ch>, Andrzej Hajda <andrzej.hajda@...el.com>, Neil Armstrong
 <neil.armstrong@...aro.org>, Robert Foss <rfoss@...nel.org>, Laurent
 Pinchart <Laurent.pinchart@...asonboard.com>, Jonas Karlman
 <jonas@...boo.se>, Louis Chauvet <louis.chauvet@...tlin.com>, Dmitry
 Baryshkov <lumag@...nel.org>, Thomas Petazzoni
 <thomas.petazzoni@...tlin.com>, dri-devel@...ts.freedesktop.org,
 linux-kernel@...r.kernel.org, linux-samsung-soc@...r.kernel.org
Subject: Re: [PATCH v3 02/22] drm: convert many bridge drivers from
 devm_kzalloc() to devm_drm_bridge_alloc() API

Hi Maxime,

On Sat, 10 May 2025 10:43:04 +0200
Maxime Ripard <mripard@...nel.org> wrote:

> Acked-by: Maxime Ripard <mripard@...nel.org>

Thanks for looking at this series!

Unfortunately the exynos patch is still needing a decision. Quick recap
of the story:

 1. patch was sent in v2, nobody acked/reviewed it [1]
 2. was applied by mistake to drm-misc-next [2]
 3. you pinged maintainers, no reply do far -> should be reverted [3]
 4. Louis (fomys) asked on IRC how to handle it, no reply [4]
 5. in this v3 I added a revert (patch 1), no acks/reviews on it [5]

[1] https://lore.kernel.org/lkml/20250424-drm-bridge-convert-to-alloc-api-v2-14-8f91a404d86b@bootlin.com/
[2] https://lore.kernel.org/lkml/832a9db0-cf8a-4d35-8a98-08053fbd6723@bootlin.com/
[3] https://lore.kernel.org/lkml/20250430-arrogant-marmoset-of-justice-92ced3@houat/
[4] https://dri.freedesktop.org/~cbrill/dri-log/index.php?channel=dri-devel&highlight_names=&date=2025-05-07
[5] https://lore.kernel.org/lkml/20250509-drm-bridge-convert-to-alloc-api-v3-1-b8bc1f16d7aa@bootlin.com/

What to do? Proposed plan:

 1. Louis immediately applies the revert (patch 1)
 2. I send v4 with the original patch to hopefully be reviewed/acked

Plan is OK?

I'm assuming Louis doesn't need an Acked/Reviewed-by for that, but I might
be wrong, and we both would like to avoid further mess.

This annoying issue is taking much more effort to be fixed than it took
to develop it. I'd like to get past it and think about the next steps
in bridge lifetime management.

Thanks for your understanding.

Luca

-- 
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com