From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj1-f51.google.com (mail-pj1-f51.google.com [209.85.216.51]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8C95541C72 for ; Sat, 8 Aug 2026 06:37:19 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.51 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786171042; cv=none; b=HwVCdkaD6tyOtIvrWjZoH1JR0fUAUdFUtNIZ57/VWbz0TF2MvC0Vo5yexJDW90MUZI3KueagTFfkFNnQETLcyoTwijBY7edjvIkF75Utv+RTMj+tZ2ssXDEofZWfmop43ifMlYiIMKZW6rJnJZj2umf9ce+tNmgFa7ts3Rjb9Z0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786171042; c=relaxed/simple; bh=3ebgjBAUsHDL5aSLBAEDQetZe1n1aC6xwo33fb9arHY=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=Rqtll1PEp9381MlS8hpiL88B+D5KS6IBp13TyfhEuffLtMABk6r989f8wsVD15kVJvPWbIEZMCwOSzrdlN3oBWDwElUibxj4nruAq8q4ipHQ+tvPEbYmglLzSjjX0ljZW1f2M+40gswXq+1LVNxZ6olu1XdMt9n9/V0scYgFDvs= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=O8Gy6oOG; arc=none smtp.client-ip=209.85.216.51 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="O8Gy6oOG" Received: by mail-pj1-f51.google.com with SMTP id 98e67ed59e1d1-38e07ebd263so167883a91.1 for ; Fri, 07 Aug 2026 23:37:19 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1786171039; x=1786775839; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to:content-type; bh=Q983uYDsq4Xe9yOLC9LdYCCWYqVj2QLw2dxzgl89ls8=; b=O8Gy6oOGgY1SHFJWgAB7Pgoup9HuUKPdg/W/0Nf00/+iBsnhKWxFobbEIcuXD5pppX HfNSbYBV8OzGXleqnXfQFExLCW9eeYhfQG0B2zd3mKasTIaP3zBL/4OqIUPCRpkvAPGF knUtgdMWUvE49IxtID8lKemm50dlSEOiPRqZ2b93vvrlIFO9DEdJ0MLi5sYciH2vStGN aRuvB4L/n6rN7USzWognpo9ey0Dt+IfOGOqwW6RK2xfLtD1MtIw4mQb9SISYtNPSbPDH CZtwf+qtjZCn/aszvd3RQoPIGyhACbJaUROBDjrb7q1JozfcvKZKiXToSv0GVaTlXJFs 2BWQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1786171039; x=1786775839; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=Q983uYDsq4Xe9yOLC9LdYCCWYqVj2QLw2dxzgl89ls8=; b=pxnpW8WhW4QZ7LEF0b9AQ9e40vMRX0OGDWYIO17+pNvmxFw8sNBJmRuAtdTlaSzrjP c4oQPOEcBBY1zg7rAKbH3JY6E9oaN4u0YbFVH1D7YRU9zXxKtyhlDXDXNdNcfXaR/spT YcryYRoA13A/hYtgNGdwAFLlcy/eKqYrCOUcJqoEnv5Q9ZyHG/BcPn61v8isXy1MqOuc HTwCsTsGXc1DehzoB6t5D0sFlCeVNefQqFubSbYpF1rfYUVb5rY89xSeP/jXt1wWkFWh Jc/9xaQYTMK94NYNBBssn3EoWqN2hF55McDWFD2SRKaJaSKK5iQSdSJKDPz8YBOBhIsw kb3g== X-Forwarded-Encrypted: i=1; AHgh+RrvbgCjH8AloYSKprq/78F6/DZeohuIW7nLC5zSzhGi2lI/AiUHJj/1R7qSnKztqnc77kv97jAocL8ghp4=@vger.kernel.org X-Gm-Message-State: AOJu0YzTCjm5InzrNWgF4P77B3vJ5wWRFZnpLAfK8DEYkI4z//okuhrI gB90sCj8d3tF0VMq8FDyoGU4r1STIzj2weNdZ+ZXbA63/Rd9ou8/nA6i X-Gm-Gg: AR+sD12prLAOZLvGMacGz8qmWAcZrekoBjdiGGNrtbCJt//muiAZfiuiEA0nd9F9tuG kHwG7QaYlsyIQv6t9+aooDu5zdM0yH/MfcuqBrH6DeDjxUdIFKI4qWQZp4nI9/tx13ipbB5AE+i bIG8SdPJe0iS116FdAha1e7IdtibIhePPJoBzynE/l18/h6xPxDyWcDiSIlkpg8SvA2AfbgJShz ts+yMCdElpgujFQYTIW4cZOmY6IpFRqL2Q6TgM4l/VFn+ItyuqXd/QWl+X3ph5mbB/xwptOPnCh mnNk0XCFSJiBGOGsQ8fa0cfoTQOK0GT2wrTBJF7b0gEQi/DviZt2ORBv9bC+Cqtj+4lLEpjlIDk HtwwddSw3J5rgGwtHO6T62tUTLzXbl/pWlcY3Vohi5VjFRrbNfnoIDZttV5aoSeo/DcIz+V+rpA fq4Df7NUqlHjcn99c4aYMHvwsw77SLj3EHHBAajVypXMs= X-Received: by 2002:a17:90b:3b4a:b0:38f:23d6:b4ad with SMTP id 98e67ed59e1d1-3909d8172edmr13381640a91.8.1786171038869; Fri, 07 Aug 2026 23:37:18 -0700 (PDT) Received: from lgs.. ([2001:250:5800:1000::f280]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-3925fd95c40sm5015659a91.8.2026.08.07.23.37.13 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 07 Aug 2026 23:37:18 -0700 (PDT) From: Guangshuo Li To: Olivia Mackall , Herbert Xu , Nicolas Ferre , Alexandre Belloni , Claudiu Beznea , =?UTF-8?q?Uwe=20Kleine-K=C3=B6nig=20=28The=20Capable=20Hub=29?= , Guangshuo Li , Takashi Sakamoto , Bjorn Helgaas , Danilo Krummrich , linux-crypto@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Cc: stable@vger.kernel.org Subject: [PATCH] hwrng: atmel: fix usage_count leak when autosuspend_delay is negative Date: Sat, 8 Aug 2026 14:34:58 +0800 Message-ID: <20260808063458.2609136-1-lgs201920130244@gmail.com> X-Mailer: git-send-email 2.43.0 Precedence: bulk X-Mailing-List: linux-crypto@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit atmel_trng_probe() calls pm_runtime_use_autosuspend(), but neither the probe failure path nor atmel_trng_remove() calls the matching pm_runtime_dont_use_autosuspend() before disabling runtime PM. If the autosuspend delay is set to a negative value while autosuspend is enabled, the runtime PM core increments usage_count to prevent runtime suspend. Without calling pm_runtime_dont_use_autosuspend() during teardown, this reference is not dropped and usage_count remains unbalanced. Add the missing pm_runtime_dont_use_autosuspend() calls to both the probe failure and remove paths before disabling runtime PM. This issue was found by manual code inspection. Fixes: c4f51eab6ce0 ("hwrng: atmel - add runtime pm support") Cc: stable@vger.kernel.org Signed-off-by: Guangshuo Li --- drivers/char/hw_random/atmel-rng.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/char/hw_random/atmel-rng.c b/drivers/char/hw_random/atmel-rng.c index 4ebbc44fecf0..6265d9a8d140 100644 --- a/drivers/char/hw_random/atmel-rng.c +++ b/drivers/char/hw_random/atmel-rng.c @@ -150,6 +150,7 @@ static int atmel_trng_probe(struct platform_device *pdev) ret = devm_hwrng_register(&pdev->dev, &trng->rng); if (ret) { + pm_runtime_dont_use_autosuspend(&pdev->dev); pm_runtime_disable(&pdev->dev); pm_runtime_set_suspended(&pdev->dev); #ifndef CONFIG_PM @@ -165,6 +166,7 @@ static void atmel_trng_remove(struct platform_device *pdev) struct atmel_trng *trng = platform_get_drvdata(pdev); atmel_trng_cleanup(trng); + pm_runtime_dont_use_autosuspend(&pdev->dev); pm_runtime_disable(&pdev->dev); pm_runtime_set_suspended(&pdev->dev); } -- 2.43.0