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 X-Spam-Level: X-Spam-Status: No, score=-12.7 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,INCLUDES_PATCH,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_2 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id A4F8BC433ED for ; Wed, 28 Apr 2021 07:20:53 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 71C0F61139 for ; Wed, 28 Apr 2021 07:20:51 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231600AbhD1HVe (ORCPT ); Wed, 28 Apr 2021 03:21:34 -0400 Received: from mail.kernel.org ([198.145.29.99]:37128 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229478AbhD1HVe (ORCPT ); Wed, 28 Apr 2021 03:21:34 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 0E426600D4; Wed, 28 Apr 2021 07:20:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1619594450; bh=hj7LHzq35NS14XVvUmoBorUTi57+snt04nyX+mFSnxM=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=JRnEBUtcIFPUnNIwZD51/4+WmD6FX6YRXkQQ8RVyiUS+NIg2OHbUp1RRJ5nszgfB9 XImoUbge/fhE+v+5u3hi8D7kiWDojgsTHZlyCytq2BPG6XgE1IdjFIiRGM/ByhQAqR z79MgsmPLRt0eQAy7r9fAYMX6zhWWZV7mZ8+TStWdRNbe/LkGVRYk+89qiJMlCeAs5 ibSQmKbn+Hi1nTbW013iPP8vBKrxisB//QKYS5i7bisTCjZk8JIVli/GlaM9hHF5NK xeGbBB746UfNJayI0oIeQAqpcIgpZqfJEXSVu/mwa3SJ6VAycE6beNoChlJxqS++Cv 5E2MD4FMXJkCw== Date: Wed, 28 Apr 2021 09:20:43 +0200 From: Mauro Carvalho Chehab To: Dmitry Osipenko Cc: linuxarm@huawei.com, mauro.chehab@huawei.com, Greg Kroah-Hartman , Jonathan Hunter , Mauro Carvalho Chehab , Thierry Reding , devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, linux-tegra@vger.kernel.org Subject: Re: [PATCH v3 25/79] staging: media: vde: use pm_runtime_resume_and_get() Message-ID: <20210428092043.380c9d4c@coco.lan> In-Reply-To: <2e8bdea5-b2a7-df95-9d93-7c1f2df6158d@gmail.com> References: <2e8bdea5-b2a7-df95-9d93-7c1f2df6158d@gmail.com> X-Mailer: Claws Mail 3.17.8 (GTK+ 2.24.33; x86_64-redhat-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-tegra@vger.kernel.org Em Tue, 27 Apr 2021 14:47:01 +0300 Dmitry Osipenko escreveu: > 27.04.2021 13:26, Mauro Carvalho Chehab =D0=BF=D0=B8=D1=88=D0=B5=D1=82: > > @@ -1088,8 +1090,9 @@ static int tegra_vde_remove(struct platform_devic= e *pdev) > > { > > struct tegra_vde *vde =3D platform_get_drvdata(pdev); > > struct device *dev =3D &pdev->dev; > > + int ret; > > =20 > > - pm_runtime_get_sync(dev); > > + ret =3D pm_runtime_resume_and_get(dev); =20 >=20 > Should be cleaner to return error directly here, IMO. I double-checked how drivers/base/platform.c deals with non-zero returns at the .remove method: static int platform_remove(struct device *_dev) { struct platform_driver *drv =3D to_platform_driver(_dev->driver); struct platform_device *dev =3D to_platform_device(_dev); =09 if (drv->remove) { int ret =3D drv->remove(dev); =09 if (ret) dev_warn(_dev, "remove callback returned a non-zer= o value. This will be ignored.\n"); } dev_pm_domain_detach(_dev, true); =09 return 0; } Basically, it will print a message but will ignore whatever happens afterwards. So, if the driver is changed to return an error there, it will leak resources. Thanks, Mauro 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 X-Spam-Level: X-Spam-Status: No, score=-10.3 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, USER_AGENT_SANE_2 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 07308C433ED for ; Wed, 28 Apr 2021 07:20:57 +0000 (UTC) Received: from smtp2.osuosl.org (smtp2.osuosl.org [140.211.166.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 976F461139 for ; Wed, 28 Apr 2021 07:20:54 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 976F461139 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=driverdev-devel-bounces@linuxdriverproject.org Received: from localhost (localhost [127.0.0.1]) by smtp2.osuosl.org (Postfix) with ESMTP id 505FE4020F; Wed, 28 Apr 2021 07:20:54 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from smtp2.osuosl.org ([127.0.0.1]) by localhost (smtp2.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id vgnog_-FFaB8; Wed, 28 Apr 2021 07:20:53 +0000 (UTC) Received: from ash.osuosl.org (ash.osuosl.org [140.211.166.34]) by smtp2.osuosl.org (Postfix) with ESMTP id 5CA394027A; Wed, 28 Apr 2021 07:20:53 +0000 (UTC) Received: from smtp2.osuosl.org (smtp2.osuosl.org [140.211.166.133]) by ash.osuosl.org (Postfix) with ESMTP id 5CE911BF31B for ; Wed, 28 Apr 2021 07:20:51 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp2.osuosl.org (Postfix) with ESMTP id 5941A4027A for ; Wed, 28 Apr 2021 07:20:51 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from smtp2.osuosl.org ([127.0.0.1]) by localhost (smtp2.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id CpfkZ2YuRaBd for ; Wed, 28 Apr 2021 07:20:50 +0000 (UTC) X-Greylist: domain auto-whitelisted by SQLgrey-1.8.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp2.osuosl.org (Postfix) with ESMTPS id CA48C4020F for ; Wed, 28 Apr 2021 07:20:50 +0000 (UTC) Received: by mail.kernel.org (Postfix) with ESMTPSA id 0E426600D4; Wed, 28 Apr 2021 07:20:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1619594450; bh=hj7LHzq35NS14XVvUmoBorUTi57+snt04nyX+mFSnxM=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=JRnEBUtcIFPUnNIwZD51/4+WmD6FX6YRXkQQ8RVyiUS+NIg2OHbUp1RRJ5nszgfB9 XImoUbge/fhE+v+5u3hi8D7kiWDojgsTHZlyCytq2BPG6XgE1IdjFIiRGM/ByhQAqR z79MgsmPLRt0eQAy7r9fAYMX6zhWWZV7mZ8+TStWdRNbe/LkGVRYk+89qiJMlCeAs5 ibSQmKbn+Hi1nTbW013iPP8vBKrxisB//QKYS5i7bisTCjZk8JIVli/GlaM9hHF5NK xeGbBB746UfNJayI0oIeQAqpcIgpZqfJEXSVu/mwa3SJ6VAycE6beNoChlJxqS++Cv 5E2MD4FMXJkCw== Date: Wed, 28 Apr 2021 09:20:43 +0200 From: Mauro Carvalho Chehab To: Dmitry Osipenko Subject: Re: [PATCH v3 25/79] staging: media: vde: use pm_runtime_resume_and_get() Message-ID: <20210428092043.380c9d4c@coco.lan> In-Reply-To: <2e8bdea5-b2a7-df95-9d93-7c1f2df6158d@gmail.com> References: <2e8bdea5-b2a7-df95-9d93-7c1f2df6158d@gmail.com> X-Mailer: Claws Mail 3.17.8 (GTK+ 2.24.33; x86_64-redhat-linux-gnu) MIME-Version: 1.0 X-BeenThere: driverdev-devel@linuxdriverproject.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Linux Driver Project Developer List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devel@driverdev.osuosl.org, Greg Kroah-Hartman , linuxarm@huawei.com, Jonathan Hunter , linux-tegra@vger.kernel.org, Thierry Reding , mauro.chehab@huawei.com, Mauro Carvalho Chehab , linux-kernel@vger.kernel.org, linux-media@vger.kernel.org Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Errors-To: driverdev-devel-bounces@linuxdriverproject.org Sender: "devel" RW0gVHVlLCAyNyBBcHIgMjAyMSAxNDo0NzowMSArMDMwMApEbWl0cnkgT3NpcGVua28gPGRpZ2V0 eEBnbWFpbC5jb20+IGVzY3JldmV1OgoKPiAyNy4wNC4yMDIxIDEzOjI2LCBNYXVybyBDYXJ2YWxo byBDaGVoYWIg0L/QuNGI0LXRgjoKPiA+IEBAIC0xMDg4LDggKzEwOTAsOSBAQCBzdGF0aWMgaW50 IHRlZ3JhX3ZkZV9yZW1vdmUoc3RydWN0IHBsYXRmb3JtX2RldmljZSAqcGRldikKPiA+ICB7Cj4g PiAgCXN0cnVjdCB0ZWdyYV92ZGUgKnZkZSA9IHBsYXRmb3JtX2dldF9kcnZkYXRhKHBkZXYpOwo+ ID4gIAlzdHJ1Y3QgZGV2aWNlICpkZXYgPSAmcGRldi0+ZGV2Owo+ID4gKwlpbnQgcmV0Owo+ID4g IAo+ID4gLQlwbV9ydW50aW1lX2dldF9zeW5jKGRldik7Cj4gPiArCXJldCA9IHBtX3J1bnRpbWVf cmVzdW1lX2FuZF9nZXQoZGV2KTsgIAo+IAo+IFNob3VsZCBiZSBjbGVhbmVyIHRvIHJldHVybiBl cnJvciBkaXJlY3RseSBoZXJlLCBJTU8uCgpJIGRvdWJsZS1jaGVja2VkIGhvdyBkcml2ZXJzL2Jh c2UvcGxhdGZvcm0uYyBkZWFscyB3aXRoIG5vbi16ZXJvCnJldHVybnMgYXQgdGhlIC5yZW1vdmUg bWV0aG9kOgoKCXN0YXRpYyBpbnQgcGxhdGZvcm1fcmVtb3ZlKHN0cnVjdCBkZXZpY2UgKl9kZXYp Cgl7CgkgICAgICAgIHN0cnVjdCBwbGF0Zm9ybV9kcml2ZXIgKmRydiA9IHRvX3BsYXRmb3JtX2Ry aXZlcihfZGV2LT5kcml2ZXIpOwoJICAgICAgICBzdHJ1Y3QgcGxhdGZvcm1fZGV2aWNlICpkZXYg PSB0b19wbGF0Zm9ybV9kZXZpY2UoX2Rldik7CgkKCSAgICAgICAgaWYgKGRydi0+cmVtb3ZlKSB7 CgkgICAgICAgICAgICAgICAgaW50IHJldCA9IGRydi0+cmVtb3ZlKGRldik7CgkKCSAgICAgICAg ICAgICAgICBpZiAocmV0KQoJICAgICAgICAgICAgICAgICAgICAgICAgZGV2X3dhcm4oX2Rldiwg InJlbW92ZSBjYWxsYmFjayByZXR1cm5lZCBhIG5vbi16ZXJvIHZhbHVlLiBUaGlzIHdpbGwgYmUg aWdub3JlZC5cbiIpOwoJICAgICAgICB9CgkgICAgICAgIGRldl9wbV9kb21haW5fZGV0YWNoKF9k ZXYsIHRydWUpOwoJCgkgICAgICAgIHJldHVybiAwOwoJfQoKQmFzaWNhbGx5LCBpdCB3aWxsIHBy aW50IGEgbWVzc2FnZSBidXQgd2lsbCBpZ25vcmUgd2hhdGV2ZXIgaGFwcGVucwphZnRlcndhcmRz LgoKU28sIGlmIHRoZSBkcml2ZXIgaXMgY2hhbmdlZCB0byByZXR1cm4gYW4gZXJyb3IgdGhlcmUs IGl0IHdpbGwgbGVhawpyZXNvdXJjZXMuCgpUaGFua3MsCk1hdXJvCl9fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmRldmVsIG1haWxpbmcgbGlzdApkZXZlbEBs aW51eGRyaXZlcnByb2plY3Qub3JnCmh0dHA6Ly9kcml2ZXJkZXYubGludXhkcml2ZXJwcm9qZWN0 Lm9yZy9tYWlsbWFuL2xpc3RpbmZvL2RyaXZlcmRldi1kZXZlbAo=