From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.andi.de1.cc (mail.andi.de1.cc [178.238.236.174]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 57BA7358368 for ; Wed, 29 Jul 2026 06:43:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=178.238.236.174 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785307431; cv=none; b=QVdqJ+FSnEUbyv5WxCr4buhxq0KeQj7gFly9LXxq/u50RtEFWKssDsOdoDSH9FJZ17k8FQsLvSiEUP6IugA7wyYPl/oZANCJxIQsigp7RxtYqO15wQnqMBSlnw68iGp8aI1ypFr/spt79yf3pWyhPvXY9ZDVWQLuaYjYY7333JY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785307431; c=relaxed/simple; bh=Cp0mKnGpw6oN8CQCtg5lNqFvNpdlhEo3xyunxxyp4DY=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=JijnZfB1FLcZfgkrRWlLRll0ILQdrb51zSzBDJn+hxzQQpHcoGcesVlOIUANyboGFnFSSrRXQ6GCbpxQUkt45IQ6TwGnpMIls8+9kHrk5SOAJAOkVyfeAQ0CYXhorb2N6n1mkFB/vb69uA3RnFEFwglNyFPSNmC803U3F0ZIALk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=kemnade.info; spf=pass smtp.mailfrom=kemnade.info; dkim=pass (2048-bit key) header.d=kemnade.info header.i=@kemnade.info header.b=QkSAN5H0; arc=none smtp.client-ip=178.238.236.174 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=kemnade.info Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=kemnade.info Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kemnade.info header.i=@kemnade.info header.b="QkSAN5H0" DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=kemnade.info; s=20220719; h=In-Reply-To:References:Subject:Cc:To:From: Reply-To:Content-ID:Content-Description; bh=u7mFonr551F9VYPr/lHHsnNE6ZPOkdA8vxn4kpv/EKI=; t=1785307430; x=1786517030; b=QkSAN5H07wgfi8amzrfRhy+u6kkUCiV4Y6RB67F3kMSF2GLtGaP1mX7LSpLVnP5noTqxkpSu7yO G9gdkaioGCOt7b8JOQGrO5nGpZ9UzUzgd/kBcPPdZu98jMmNxaGtir9RvYWWagCXH0LxYT8qJf+U/ PQmX0C2FLDxZeyo5VoESB1Wieto15pg1nDo7ylMk+Z/kl8+tBHYn4gwFjg9ELZ42PslsSFs/P42a3 XrJgGZ9KEaabhYHY3kmnWP/Qj+8jZ2yNvGjxVSmUItRG3BH9koP7FQWObzGCVfcNRc33ItFzz1vt0 gUpu70i79Q8xvCa/QuHee0yvOAgXtaQ8VT0Q==; Date: Tue, 28 Jul 2026 20:36:09 +0200 From: Andreas Kemnade To: Thomas Zimmermann Cc: maarten.lankhorst@linux.intel.com, mripard@kernel.org, airlied@gmail.com, simona@ffwll.ch, lumag@kernel.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH RESEND] drm/connector: check if destroy function exist Message-ID: References: <20260702191453.611070-1-andreas@kemnade.info> <44557a30-dc86-409a-b430-2b4db151843b@suse.de> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <44557a30-dc86-409a-b430-2b4db151843b@suse.de> On Fri, Jul 03, 2026 at 09:09:46AM +0200, Thomas Zimmermann wrote: > Hi > > Am 02.07.26 um 21:14 schrieb Andreas Kemnade: > > Connectors might have not resources needed to be manually freed. > > E.g. the drm_bridge_connector does not have such a callback. > > > > Fixes: c12907be57b1 ("drm/bridge-connector: switch to using drmm allocations") > > This patch uses drmm_connector_init() for the cleanup, which is a mis-design > IMHO.  Rather than calling drm_connector_cleanup(), it should call > connector->funcs->destroy, which then points to drm_connector_cleanup. This > would make the whole thing more useful and not require fixups (like this > one) elsewhere in the code. Just just my 2cts. > That just sounds like a revert to me. But maybe I misunterstand your comment > Best regards > Thomas > > > > Signed-off-by: Andreas Kemnade > > --- > > drivers/gpu/drm/drm_connector.c | 8 ++++++-- > > 1 file changed, 6 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/gpu/drm/drm_connector.c b/drivers/gpu/drm/drm_connector.c > > index 47dc53c4a738..d2dfc6e8f7d0 100644 > > --- a/drivers/gpu/drm/drm_connector.c > > +++ b/drivers/gpu/drm/drm_connector.c > > @@ -198,7 +198,9 @@ static void drm_connector_free(struct kref *kref) > > struct drm_device *dev = connector->dev; > > drm_mode_object_unregister(dev, &connector->base); > > - connector->funcs->destroy(connector); > > + > > + if (connector->funcs->destroy) > > + connector->funcs->destroy(connector); > > } > > void drm_connector_free_work_fn(struct work_struct *work) > > @@ -216,7 +218,9 @@ void drm_connector_free_work_fn(struct work_struct *work) > > llist_for_each_entry_safe(connector, n, freed, free_node) { > > drm_mode_object_unregister(dev, &connector->base); > > - connector->funcs->destroy(connector); > > + > > + if (connector->funcs->destroy) > > + connector->funcs->destroy(connector); > > } > > } > > -- > -- > Thomas Zimmermann > Graphics Driver Developer > SUSE Software Solutions Germany GmbH > Frankenstr. 146, 90461 Nürnberg, Germany, www.suse.com > GF: Jochen Jaser, Andrew McDonald, Werner Knoblich, (HRB 36809, AG Nürnberg) > > >