From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 6906FC54E58 for ; Thu, 21 Mar 2024 06:46:13 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Content-ID:In-Reply-To: References:Message-ID:Date:Subject:To:From:Reply-To:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=H4JXYQOB7AK0aKbAIsoTPAkjG0dqqc9ciJ8aTgjcnBk=; b=ph7ZLA87K7Shcc 8c3SMcGNCckaam5NivPKfP/v/a7MMFhce0heDZq7OW1HXaEe3eX6OY/C2RJM6tv97P8bU9aigX5jS IShrqkC4H2raOyGaIUcfozkdyhrWTwgAd8SqjyMvEPK3aRitUoLl/JDMEb3wRjPaer90kWK2C6062 8sYL5W9m3AF5Eh9Zd0ITWwWdYmV7Y/HCg/gqhKFReQhbTjhofbRruSVRf3Io5mbDIDQvkYaeg0yWW /Qhduz3i0nRo2PbHsJchZZ6ztZztZ9A7gX/GyaFK/bkUqX0JTK5GKMXJFOCW8Q9dR659HDH/z1kPl r70qIM59TUhMNtkZUQXQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rnCBa-000000024fV-1T17; Thu, 21 Mar 2024 06:45:58 +0000 Received: from esa.microchip.iphmx.com ([68.232.153.233]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1rnCBX-000000024el-0eEd for linux-arm-kernel@lists.infradead.org; Thu, 21 Mar 2024 06:45:56 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1711003554; x=1742539554; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-id:content-transfer-encoding: mime-version; bh=WfYSakJOVs/8tzjUc408hiqvnR3wCt2o3ZObDLMti94=; b=RGF8mNEmz2zefMj9KOA513nYk+XYIKPu4v4/qAYMeDgH8eDQXz4scGUC cAW0hoQc0LlHumutAPUSZwC5DySGgf389YJCep0/ikQEsQV3jxuoOjFPX GwRk45H5mv53fP/ge09EuYgI/fLacPkavFOlJdvTTzGyiZL7yc5VXGr68 Vw9dZC82SCUpHTLyBs3Kfz2h9LhxMATZ2mJXrXCXgGqJT2+hNBAh6FM7n S4NwGA42sJyTOrO/v285rR8dI+UnJCEsSyKkW28ZutBOZcWt+CMteMUsI xgBLitvihK7CLj6ZkDbiwkMvRssfdnAaiLvYveDRmKUGkWyi41buYjlBE Q==; X-CSE-ConnectionGUID: c1TEra19TIWbQjUU2vOQ5A== X-CSE-MsgGUID: nicGgi8AQIuKHFsZXSgOPw== X-IronPort-AV: E=Sophos;i="6.07,142,1708412400"; d="scan'208";a="19717943" X-Amp-Result: SKIPPED(no attachment in message) Received: from unknown (HELO email.microchip.com) ([170.129.1.10]) by esa1.microchip.iphmx.com with ESMTP/TLS/ECDHE-RSA-AES128-GCM-SHA256; 20 Mar 2024 23:45:51 -0700 Received: from chn-vm-ex04.mchp-main.com (10.10.85.152) by chn-vm-ex03.mchp-main.com (10.10.85.151) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35; Wed, 20 Mar 2024 23:45:33 -0700 Received: from NAM11-CO1-obe.outbound.protection.outlook.com (10.10.215.250) by email.microchip.com (10.10.87.151) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35 via Frontend Transport; Wed, 20 Mar 2024 23:45:33 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=BOAiZW2sVET826Y2kn3qH/5h4yFz8IaTwZzAjhU8WVrYtgqmBxf8Q99GKSMLswOAdunwH5msjN4/AhRVU23uDEweCYHsKkqhDWwY42jfXM6QGYY0XY65s5edv8QoYuPJGlyx9esHt9wEhShVf1dGL1APL4U9eBuLr8j1XLrb9Obc5PxfpOU750+TRJ4QWcHuslY+Nye0VETHTuugazPUWiByfRe5rF39rRfd2e7qXF4Z9rEhn0VVc2sR8oAMAaZLQp+Sw4Igoo9xszuo2IoL+w4DAqZj9H/E9NbdS/EaZs3x2nRTe3HeqO6+pWeZQknhpAXoja4bq3Gqe+dUX3AkGg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=WfYSakJOVs/8tzjUc408hiqvnR3wCt2o3ZObDLMti94=; b=ig82Udz3A1JMr08JbuDmS+tMc/34j29/NgB6p8h1G8TD9XuDTfdAkVoCX1dDQ8sLZErsC+01HCrxQbFq+PiFMoZm7WPLTvhDnjkEv5QE66pUmXY84Cw5vRCne6F3oZzVOofC1pMBwt8b6NwlN6qxBufbYKyAV1VTAsktloRYagcFcq4RJ0WzMZoU0L6JSeDkhF+mGOJJP8DQHVqnLkpizGzrVPS0pW+8mdnC945yDY9wQZsIfbJjUpy/XtG5MRcD9kjCG0634XZBEWWw7NzHLQUB/VI5eeT13M3AfIS1dUWXgmnZWNjdd5mPHfvZc2WeSmSnU3X3U9r5QSUYZ8tQFA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=microchip.com; dmarc=pass action=none header.from=microchip.com; dkim=pass header.d=microchip.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=microchip.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=WfYSakJOVs/8tzjUc408hiqvnR3wCt2o3ZObDLMti94=; b=OJXk3YXD7F0NA1g852LuVr2IodvDQlTETtxlwU2kvQErysNl6iWkuRGvoItOst2hNIAdAM/oTnIjNt9t3x2YUtsM349rhuS0GbKWwfhYcdiLlLoBZpPie/mLNJKBfG6AQJGCTZv35UQM9rcrvcbO3T8FhUtV0lEdJoEVT+U29WwzIXzVWYe1BgFMmBRXXnIpJGHCRaAm2Q+N46TNrdcWFYbw0MRHoCLNcIjcsz8OYe9NR3pNcHKs9m8sujkkj/Wj8k3HY9/500KOV/NcIaLLCr0UYHPdwkxu0cxvxzlMd2bIPQ0h9ouTv9tlpPk9COxFNSY2uWb3bG8gaJ5P6feOkA== Received: from PH0PR11MB5192.namprd11.prod.outlook.com (2603:10b6:510:3b::9) by DS0PR11MB7997.namprd11.prod.outlook.com (2603:10b6:8:125::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7409.10; Thu, 21 Mar 2024 06:45:30 +0000 Received: from PH0PR11MB5192.namprd11.prod.outlook.com ([fe80::8cb4:86ac:461a:d683]) by PH0PR11MB5192.namprd11.prod.outlook.com ([fe80::8cb4:86ac:461a:d683%4]) with mapi id 15.20.7409.022; Thu, 21 Mar 2024 06:45:30 +0000 From: To: Subject: Re: [PATCH] drm/atmel-hlcdc: Release CRTC commit when destroying plane state Thread-Topic: [PATCH] drm/atmel-hlcdc: Release CRTC commit when destroying plane state Thread-Index: AQHacAPxwRwz4b6YaUagRU9HeiVOubEtfTaAgAZUkYCABPXqgIAJDwEA Date: Thu, 21 Mar 2024 06:45:30 +0000 Message-ID: References: <20240315122527.344534-1-pl.dourneau@klervi.com> In-Reply-To: <20240315122527.344534-1-pl.dourneau@klervi.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: user-agent: Mozilla Thunderbird authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=microchip.com; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: PH0PR11MB5192:EE_|DS0PR11MB7997:EE_ x-ms-office365-filtering-correlation-id: f3297edc-f9e8-48c3-6da5-08dc4972802c x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: m642joqjpKGrcuVgTg/raivvyvliCxE7U2eC5hKmaKVAlz4G1YIDSZIiibUIGQ8Luc8UByRLmW9MWOA08g0DBYJVty17f0hcg7nIAmbTnJga7egIhWoRfPNSOYErnfeWm1rs5lnV9p8MwrF3rmb6j4zLL4llEU8boTt6jNMHiP3RTHNoAz4RGvLA+lCALfE9koXKJZ+Vjh5CRa6IA291jZTdWMWrSNzCBvwZNT/7FKzGPZBaP65MKhxhXKSke3NFaxPc8yJBGQaaY5wUqRKEO09c6FtM6BtcbVoox9XGSBYZvLt+NUSFCVXccTMR/d/Pw7h4PzXUGKdVeNBSnZE+b9Wr9f2iKRD2EMjXE1vHTOTGcdxxj8oFXxHONx/ft1P0jRb9lFbE5rxJ9R+eD2w/QP2W4INITbtFh61lK+RchF+YV81dQyuPP/WXjmd5GQ4MOhFuEQ3kw81uvzgZ/Uxt/lxdkq8atowBEPPt8TnySIF8Yzze2nlP5f7jhqOe+m3K+t5ORdC+ILX2suhMNzEL85ushgqDFokexY9B3L6m1UhoAJQXEPIAwt2aOVh2Q6JkkqKr2qeKvmv+nvluscvscqhpm5H9XBGVAd7w8j+sGXzruXpc1Bh+zMfAvWp4XuPYLtDvK28ARvp/zMhKmc1rqgvykcoHIX4iKF2WsMBKpac= x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:PH0PR11MB5192.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(1800799015)(7416005)(366007)(376005)(38070700009);DIR:OUT;SFP:1101; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?R2dlTWVUL0pUNGdMblRlRENXOVIva3hhVUNJaWZvV1JiTXZ4eGRHRXJBU09O?= =?utf-8?B?elVHbExmVTVwc1l6Y0l3a2N6cTlsYjRKVEtMSjdLaTg2S2I2VnNXamdrNnha?= =?utf-8?B?OS9lYU81a1BGZDJjUk1mdWNRWEhPLzE1R0NzWWxYVVhnWG15aWxmRkVDakdL?= =?utf-8?B?WmFwV3lFeG5ub25GQitSVmJxaXAzYXhmSnhmaVNkNVhLQzd2Ti9rc3dYSnlV?= =?utf-8?B?N2k4U1pDSGRQNHJJSVpiQmJDVm1Rc3VJaXZVMThKbDd6Qzlid3RERTRJMEdx?= =?utf-8?B?VWRVN2EvdnVWNExYblYvQ3UxYjJlZnB4RFY1Q2R2bmVJcTQyU3VyRlVXcGdp?= =?utf-8?B?T29pYjVCbWpBdktTdEZMckNxbWZvOWlobzlDTm1VV2tjdVNuK2ZkQkpYZFFo?= =?utf-8?B?L3JhSlpvLzZyNnI0WG9ubDk4aXVZYkRkWmNwOUN3NUp0UUp4eGVJWHNmUXE3?= =?utf-8?B?RGdmaG9uRm94SjFMcWRSbGMvc1dhNGlaUG5VV3JQUndpc05WeHpIL3BLcis1?= =?utf-8?B?MmlOcmxpajBYN3NMRFVZRmJaRzBTNDRlb3ZDRVZ5OXBJUTB6amVHOFF2MWpp?= =?utf-8?B?N3RuNGF1RHdlUHBrMFNWb28zbThzUXN5MGRnb05EcDZBZjFLUmtEVzE5bTB0?= =?utf-8?B?NkV0WDVqZ2VHVzZOMXZOQnFGU0RScGcrcWpzZldObHlJSWU1ajVDd0xIMTRt?= =?utf-8?B?d0sxQ2J0ZjA5Y0s1Y0s2d2M0aWY2TFl6YTM4eGFHSnM2SWpjZmRpTFlIdkZi?= =?utf-8?B?TlBEdmRON1ZBVDJLbWcvbTQyVXRMMVBaNHRWSldvUXRmemhqb2FiNmFtTWdF?= =?utf-8?B?NXA0Qy9CUndueVVWdThLMEdrOFNJUURTQnBaYzZyQTJUc2lySTN1cDdpZ3U3?= =?utf-8?B?SEVFZTd3U2R4bFN4SlZML3FSRmVsc29GVkFRd3ZFU25tWWVpNHNEL2lPK2RO?= =?utf-8?B?NUhDRTBLRlA3TVJUQnJYN0pGM0ZEVlluWml3VlBCbDU3S2QxdTVoMEkwSk9l?= =?utf-8?B?N0dtZ3BRSDZMcGdLRDRsYS9wMDFDZ2hkWTFCVENTZjhqRWNVTGI2bXQ5SGFR?= =?utf-8?B?dWdPVlNnUkJRbzcxbFJwVnVrM0pwMVR3WG45UDkzVFVpbWRhQzBKdnBFNHIw?= =?utf-8?B?YlA4WTA2OTlJbnEzZENRaDdPcVJUdXNEb3lyK2hjaVFydUNjZFJSWnpkYzhP?= =?utf-8?B?MEFNU1A2YWoyaEgwbEgwc0dJWUNXUEVrYmEySGlSaHRTY3dzSDM1Y0xDMUVH?= =?utf-8?B?Qi9SWk16THFlQlZ4UitYbFhnbHczbU9vTkpNQ3pta0UvcFhvSFVjSlVGOHlo?= =?utf-8?B?Z3VtejgzZzlVeURQSXFieU1mRUhzREdtcGg4QTQxRGhhWTJCYUszM0orY1BL?= =?utf-8?B?N0pPbEpOZU5GR2c3QVBzYWszQ3R3U2g3R1ZWZU5aZ1lLV3FxamlSbU03TnVW?= =?utf-8?B?NHdCcWJEM0RKOUFnY3FvdTRlcEgyTUVrR0xhTjJqZnZiU3cxUnluODVVMktr?= =?utf-8?B?SFhaQnkrZ1J3Z3F0cnpmZExUT3BHM0ZhazEvRm1JWmNjdXVobE9JZGdZZ1BJ?= =?utf-8?B?MHZ0ZzRkQVpQV1U4RHlYaEtRYmtMdUdDRWV3K3hicXhTZlBxTDdXWTcxQTd0?= =?utf-8?B?THFONy9CaXNIQWx3Z25UbUFBQUtBL0sxcmlwcjREUkMxZnpSQzUya0FKQU16?= =?utf-8?B?aHptcWN3TnAvK0hLVjA2UitNWEtYNVIwZzltSVFnYVpSakw4UitQWndFS09J?= =?utf-8?B?b0huT21FVHJEUmpjTHM3VEdkbjNHQ29PK3pnZU5WSzA3b1hzcjVrcW5pQjhu?= =?utf-8?B?MWlJSzhuVlBka0J2YUM2eDV3MUZrb21PeXJFWDJHQU9sVS9yODMreGVCbXVt?= =?utf-8?B?eWdJKzhWTnFMQmI1bTJUQkszbFdOektWRzBLUVA3REJZVjdkcFg0V2ZwM2tN?= =?utf-8?B?MFdVWDdVTGQyaWpFcnRyRDh0c3YwYW50YXhFWHplNFlCbDZPQzUzUFBVVHpD?= =?utf-8?B?a1VYUUNRdUQvNDliZWpOMnN2ZmpudWx1KzFmRjlCOUdJYjZFbndpc3ZJUTJw?= =?utf-8?B?NjhQSWxyVTdCWkpkK2FBdGVrelNoV1ExS3VtdXc1QzBRZ2x0bUVYS1JZY1FG?= =?utf-8?B?WVFjQks2aC9yWHp2YmVuUWV4ZEFESC9SZW1qckNiVUUvYjd3QkNDdzFaaDFn?= =?utf-8?B?bmc9PQ==?= Content-ID: <8A2CA5AE981FEF41950535DF8754A62E@namprd11.prod.outlook.com> MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: PH0PR11MB5192.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: f3297edc-f9e8-48c3-6da5-08dc4972802c X-MS-Exchange-CrossTenant-originalarrivaltime: 21 Mar 2024 06:45:30.5090 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 3f4057f3-b418-4d4e-ba84-d55b4e897d88 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: kEcU8fl7I0jLN7IzSqn+GKapzAUy/GdzAj4IjhXQAkhbl1BbSSmT8JC+cQrfsRKlRf9yphSTicSqzGI4kdZnO6Hv1WVzA2u5k1757bIwn9E= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS0PR11MB7997 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240320_234555_297854_443F7574 X-CRM114-Status: GOOD ( 17.17 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Manikandan.M@microchip.com, alexandre.belloni@bootlin.com, linux-kernel@vger.kernel.org, daniel@ffwll.ch, bbrezillon@kernel.org, sam@ravnborg.org, maarten.lankhorst@linux.intel.com, mripard@kernel.org, claudiu.beznea@tuxon.dev, n.georges@klervi.com, dri-devel@lists.freedesktop.org, tzimmermann@suse.de, b.alcaina@klervi.com, airlied@gmail.com, a.lahache@klervi.com, linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 3/15/24 13:25, Pierre-Louis Dourneau wrote: > [You don't often get email from pl.dourneau@klervi.com. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ] > > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > On 3/12/24, Pierre-Louis Dourneau wrote: >> On 3/8/24, Ludovic.Desroches@microchip.com wrote: >>> This patch fixes the memory leak but introduces a crash on my side when >>> exiting a graphics app using the Microchip graphics library. >> >> We've tried to reproduce your crash with 6.1.22-linux4microchip-2023.04, >> to no avail. We'll try to upgrade to 6.1.55-linux4microchip-2023.10 (your >> version) and test again. > > I was able to test a few more recent kernel versions[0] with the patch > applied. None yielded any crash, be it running Microchip's EGT samples[1] > or libdrm's modetest. Although, what I did manage to reproduce was a > refcount underflow similar to the one you had: > > # modetest -M atmel-hlcdc -s 32:#0 -P 33@47:800x400@XR24 -a > setting mode 1024x600-65.48Hz on connectors 32, crtc 47 > testing 800x400@XR24 on plane 33, crtc 47 > [ 75.736699] ------------[ cut here ]------------ > [ 75.741353] WARNING: CPU: 0 PID: 200 at lib/refcount.c:28 refcount_warn_saturate+0xf0/0x15c > [ 75.749731] refcount_t: underflow; use-after-free. > [ 75.754574] CPU: 0 PID: 200 Comm: modetest Not tainted 6.1.55-linux4microchip-2023.10 #4 > [ 75.762915] Hardware name: Microchip SAM9X60 > [ 75.767198] unwind_backtrace from show_stack+0x10/0x18 > [ 75.772423] show_stack from dump_stack_lvl+0x28/0x34 > [ 75.777479] dump_stack_lvl from __warn+0x8c/0xc0 > [ 75.782187] __warn from warn_slowpath_fmt+0x74/0xa8 > [ 75.787158] warn_slowpath_fmt from refcount_warn_saturate+0xf0/0x15c > [ 75.793611] refcount_warn_saturate from __drm_atomic_helper_plane_destroy_state+0xd0/0xd4 > [ 75.801894] __drm_atomic_helper_plane_destroy_state from atmel_hlcdc_plane_atomic_destroy_state+0x38/0x48 > [ 75.811573] atmel_hlcdc_plane_atomic_destroy_state from drm_atomic_state_default_clear+0x1c4/0x2fc > [ 75.820642] drm_atomic_state_default_clear from __drm_atomic_state_free+0x7c/0xb0 > [ 75.828228] __drm_atomic_state_free from drm_mode_atomic_ioctl+0x868/0xb88 > [ 75.835204] drm_mode_atomic_ioctl from drm_ioctl+0x200/0x3c4 > [ 75.840960] drm_ioctl from sys_ioctl+0x240/0xb48 > [ 75.845669] sys_ioctl from ret_fast_syscall+0x0/0x44 > [ 75.850725] Exception stack(0xc8c91fa8 to 0xc8c91ff0) > [ 75.855794] 1fa0: 004b0ec0 00000003 00000003 c03864bc becd7bc8 becd7b98 > [ 75.863992] 1fc0: 004b0ec0 00000003 becd7bc8 00000036 00000003 00000003 b6f22f80 00000400 > [ 75.872183] 1fe0: b6f21e74 becd7a68 b6f07494 b6f61cc8 > [ 75.877289] ---[ end trace 0000000000000000 ]--- > Atomic Commit failed [1] > > Same but without using the atomic uAPI (`-a` option removed): > > # modetest -M atmel-hlcdc -s 32:#0 -P 33@47:800x400@XR24 > setting mode 1024x600-65.48Hz on connectors 32, crtc 47 > testing 800x400@XR24 overlay plane 33 > failed to enable plane: Invalid argument > > [ 98.542958] ------------[ cut here ]------------ > [ 98.547547] WARNING: CPU: 0 PID: 28 at lib/refcount.c:28 refcount_warn_saturate+0xf0/0x15c > [ 98.555902] refcount_t: underflow; use-after-free. > [ 98.560698] CPU: 0 PID: 28 Comm: kworker/0:7 Tainted: G W 6.1.55-linux4microchip-2023.10 #6 > [ 98.570695] Hardware name: Microchip SAM9X60 > [ 98.574972] Workqueue: events drm_mode_rmfb_work_fn > [ 98.579859] unwind_backtrace from show_stack+0x10/0x18 > [ 98.587615] show_stack from dump_stack_lvl+0x28/0x34 > [ 98.595201] dump_stack_lvl from __warn+0x8c/0xc0 > [ 98.602438] __warn from warn_slowpath_fmt+0x74/0xa8 > [ 98.609937] warn_slowpath_fmt from refcount_warn_saturate+0xf0/0x15c > [ 98.618919] refcount_warn_saturate from __drm_atomic_helper_plane_destroy_state+0xd0/0xd4 > [ 98.629740] __drm_atomic_helper_plane_destroy_state from atmel_hlcdc_plane_atomic_destroy_state+0x38/0x48 > [ 98.641947] atmel_hlcdc_plane_atomic_destroy_state from drm_atomic_state_default_clear+0x1c4/0x2fc > [ 98.653545] drm_atomic_state_default_clear from __drm_atomic_state_free+0x7c/0xb0 > [ 98.663660] __drm_atomic_state_free from drm_framebuffer_remove+0x48c/0x540 > [ 98.673252] drm_framebuffer_remove from drm_mode_rmfb_work_fn+0x68/0x84 > [ 98.682495] drm_mode_rmfb_work_fn from process_one_work+0x1b4/0x3f4 > [ 98.691390] process_one_work from worker_thread+0x214/0x4e8 > [ 98.699587] worker_thread from kthread+0xb4/0xd8 > [ 98.706824] kthread from ret_from_fork+0x14/0x28 > [ 98.714060] Exception stack(0xc88adfb0 to 0xc88adff8) > [ 98.719125] dfa0: 00000000 00000000 00000000 00000000 > [ 98.727327] dfc0: 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 > [ 98.735520] dfe0: 00000000 00000000 00000000 00000000 00000013 00000000 > [ 98.742219] ---[ end trace 0000000000000000 ]--- > > The first one is not deterministic, you have to try a few times to trigger > it. The second one is a hit every time. > > Same commands on a kernel without the patch don't report any underflow. > Note the commit in the first command also fails on a kernel without the > patch, which I guess is expected as plane 33 is the primary plane and I'm > trying set dimensions that do not match the size of the display. The commit > succeeds when invoking with the correct dimensions, but then I can't make > it produce an underflow. Same with the second command. > > It seems to only trigger once per boot. Running the commands again does not > yield another underflow. > > Looking at the disassembly, this is an underflow of the drm_crtc_commit's > refcount this time. In the warning you had, it was on a framebuffer object. > > Anyway, I'll go back to the drawing board, study more closely the resource > release part of the driver. Thanks for having brought up the issues with > the patch. Hello, Sorry for the late reply, I was off previous weeks. Late me know if you need more information or testing on my side. From the top of my head, the crash was systematic on my side and I think I get it with any EGT application. I use an EGT app to check if the memory leak is still here. It continuously change the text of a button causing many calls of drm_atomic_helper_setup_commit. After a while, we can see that the virtual memory is constant while the system memory decreases. At least, your patch sounds to fix this memory leak. Regards, Ludovic > > [0]: Namely, linux4microchip-2023.10 (6.1.55), v6.8, and drm-next-2023-03-13 > [1]: https://github.com/linux4sam/egt/tree/master/examples > > Pierre-Louis _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel