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: <qkqyfksvimaks4wb4si72shewdc7ccy4n3srpkv3jf5snphepu@zpci2dquivhj>
Date: Thu, 22 May 2025 16:08:43 +0300
From: Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>
To: Luca Ceresoli <luca.ceresoli@...tlin.com>
Cc: Maxime Ripard <mripard@...nel.org>,
        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>,
        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

On Mon, May 12, 2025 at 04:02:01PM +0200, Luca Ceresoli wrote:
> 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)

I can't help but notice that both Louis and you have bootlin.com email
addresses. Granted the lack of responses, can't you ping him internally?

>  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

-- 
With best wishes
Dmitry