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=-8.7 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 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 4FB0BC07E96 for ; Tue, 6 Jul 2021 06:40:38 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 292F66198B for ; Tue, 6 Jul 2021 06:40:38 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230132AbhGFGnP (ORCPT ); Tue, 6 Jul 2021 02:43:15 -0400 Received: from mailout2.w1.samsung.com ([210.118.77.12]:12655 "EHLO mailout2.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230086AbhGFGnO (ORCPT ); Tue, 6 Jul 2021 02:43:14 -0400 Received: from eucas1p2.samsung.com (unknown [182.198.249.207]) by mailout2.w1.samsung.com (KnoxPortal) with ESMTP id 20210706064035euoutp02be4d28a47dfa211cc10d0a670e0e00b8~PIGs_wT-23203532035euoutp02g for ; Tue, 6 Jul 2021 06:40:35 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 mailout2.w1.samsung.com 20210706064035euoutp02be4d28a47dfa211cc10d0a670e0e00b8~PIGs_wT-23203532035euoutp02g DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=samsung.com; s=mail20170921; t=1625553635; bh=/FABSd7h+/aNCApA9rE7hWI30dZ59mhJinjTD6hB7CA=; h=Subject:To:Cc:From:Date:In-Reply-To:References:From; b=mw/uNbJ7p1edrGnszgtpngkES8iSYpe1MsV4USxGvyC5iyXC3oyl8gwSVs5jj7COh beM8bTl9NU4Dg2FOKjeXFpA6f+gHXk10MHAMzrApgofRf2wdeZRc+BBFZWYSOCHJ/O bUtVT3kd//PBjmZhP/sFWAwRdVd2kdS1L1OBS1qU= Received: from eusmges3new.samsung.com (unknown [203.254.199.245]) by eucas1p2.samsung.com (KnoxPortal) with ESMTP id 20210706064035eucas1p24281895a290f144da55d106c266d640d~PIGssoy6J0361803618eucas1p2m; Tue, 6 Jul 2021 06:40:35 +0000 (GMT) Received: from eucas1p1.samsung.com ( [182.198.249.206]) by eusmges3new.samsung.com (EUCPMTA) with SMTP id F3.DA.56448.3EAF3E06; Tue, 6 Jul 2021 07:40:35 +0100 (BST) Received: from eusmtrp1.samsung.com (unknown [182.198.249.138]) by eucas1p1.samsung.com (KnoxPortal) with ESMTPA id 20210706064034eucas1p1e4bd9281a5badb012c1a65f0dbf8169a~PIGsOCGgD1739017390eucas1p1d; Tue, 6 Jul 2021 06:40:34 +0000 (GMT) Received: from eusmgms2.samsung.com (unknown [182.198.249.180]) by eusmtrp1.samsung.com (KnoxPortal) with ESMTP id 20210706064034eusmtrp1afc12f722a94dcc8e36e702650db4e17~PIGsNL00w1015410154eusmtrp1t; Tue, 6 Jul 2021 06:40:34 +0000 (GMT) X-AuditID: cbfec7f5-d3bff7000002dc80-ee-60e3fae37e8d Received: from eusmtip2.samsung.com ( [203.254.199.222]) by eusmgms2.samsung.com (EUCPMTA) with SMTP id 07.B6.20981.2EAF3E06; Tue, 6 Jul 2021 07:40:34 +0100 (BST) Received: from [106.210.134.192] (unknown [106.210.134.192]) by eusmtip2.samsung.com (KnoxPortal) with ESMTPA id 20210706064034eusmtip28242c78fef63937cdba889b3821b449b~PIGrnZo003174931749eusmtip2H; Tue, 6 Jul 2021 06:40:33 +0000 (GMT) Subject: Re: [PATCH] iommu: qcom: Revert "iommu/arm: Cleanup resources in case of probe error path" To: Amey Narkhede Cc: iommu@lists.linux-foundation.org, linux-arm-kernel@lists.infradead.org, linux-arm-msm@vger.kernel.org, Rob Clark , Will Deacon , Robin Murphy , Joerg Roedel , Krishna Reddy From: Marek Szyprowski Message-ID: <71106cbb-ff6d-02cb-7743-dfbd27bdaddf@samsung.com> Date: Tue, 6 Jul 2021 08:40:33 +0200 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101 Thunderbird/78.11.0 MIME-Version: 1.0 In-Reply-To: <20210705143054.nxy4ii6t6npj22ov@archlinux> Content-Transfer-Encoding: 7bit Content-Language: en-US X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFtrEKsWRmVeSWpSXmKPExsWy7djPc7qPfz1OMJiyjs3i0vZlrBYL9ltb dM7ewG6x6fE1VouJ+8+yWzxf+IPZ4uCHJ6wWfXuDLFrumDpwejw5OI/JY828NYweO2fdZffY tKqTzWPzknqPyTeWM3r0Nr9j8/i8SS6AI4rLJiU1J7MstUjfLoErY/PZY+wFW9kq7s5Yw9jA 2M/axcjJISFgIrH6fRtTFyMXh5DACkaJXxcvMUM4Xxgl7q2bBuV8ZpRY0rGCBaZl98JrUInl jBI/pp1mBkkICXxklGhbJgliCwukSjSsngS2Q0RAR+LRxoVsIA3MAtOZJA4/XQ2WYBMwlOh6 2wWU4ODgFbCTWPvXGsRkEVCR+PS5HqRCVCBZ4v28GWDVvAKCEidnPgG7gVPAQuLltonsIDaz gLzE9rdzmCFscYlbT+aDvSMh8J9D4sqHZVB/ukjsWvmeCcIWlnh1fAs7hC0jcXpyDwtEQzOj xMNza9khnB5GictNMxghqqwl7pz7BXYos4CmxPpd+hBhR4lXf3eChSUE+CRuvBWEOIJPYtK2 6cwQYV6JjjYhiGo1iVnH18GtPXjhEvMERqVZSF6bheSdWUjemYWwdwEjyypG8dTS4tz01GLj vNRyveLE3OLSvHS95PzcTYzAZHX63/GvOxhXvPqod4iRiYPxEKMEB7OSCK/IlEcJQrwpiZVV qUX58UWlOanFhxilOViUxHl3bV0TLySQnliSmp2aWpBaBJNl4uCUamAKN9fc01Vq+Y4j6J9p /TeFqE//hHK+rcmWc1R/Ip/x9ENA3Z1vcRGzM/ZLLkmssnz6pun9gjhWXlmluW0zP3xwsk43 DvO9MsHuqvDaPvefH2/6rZJ+89Jp3pmSXM3b9+8WNP3cYsWs2755qefsLWXCvhNWljt53r2q m/6m8gb35OUvIi4d03E3vFKczv0gzzqqnNEtb/q06s8mM08d6jo7w7TwQNPHb78zL760ERBi KQ3/0XBsEi8rp3F16vGPWY5HmsXfTP+Xo2BXcrV5m4TbxK/Tlhlf3G/ANa/7eseEnFcLLyxs F1ryU+TSk2MT5u3WFF0rOSNMtOBgd81eF+nFDDXvpjc+bX/TPTfQNmWDEktxRqKhFnNRcSIA OZINCcUDAAA= X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFjrKIsWRmVeSWpSXmKPExsVy+t/xe7qPfj1OMLj6Vcni0vZlrBYL9ltb dM7ewG6x6fE1VouJ+8+yWzxf+IPZ4uCHJ6wWfXuDLFrumDpwejw5OI/JY828NYweO2fdZffY tKqTzWPzknqPyTeWM3r0Nr9j8/i8SS6AI0rPpii/tCRVISO/uMRWKdrQwkjP0NJCz8jEUs/Q 2DzWyshUSd/OJiU1J7MstUjfLkEvY/PZY+wFW9kq7s5Yw9jA2M/axcjJISFgIrF74TXmLkYu DiGBpYwSX1/NYIFIyEicnNYAVSQs8edaFxtE0XtGiZ5F65lBEsICqRJHn54EaxAR0JF4tHEh WBGzwHQmieuvP7JAdOxnlDg8+zQTSBWbgKFE11uQURwcvAJ2Emv/WoOYLAIqEp8+14NUiAok S/xc384GYvMKCEqcnPkEbD6ngIXEy20T2UFsZgEziXmbHzJD2PIS29/OgbLFJW49mc80gVFo FpL2WUhaZiFpmYWkZQEjyypGkdTS4tz03GIjveLE3OLSvHS95PzcTYzA+Nx27OeWHYwrX33U O8TIxMF4iFGCg1lJhFdkyqMEId6UxMqq1KL8+KLSnNTiQ4ymQO9MZJYSTc4HJoi8knhDMwNT QxMzSwNTSzNjJXFekyNr4oUE0hNLUrNTUwtSi2D6mDg4pRqYOjf23PuwY9+sRKEJ3mtrBORC mfU7jqn0Tf0qynes/Icxd/iEI51u90ySlR4yf/9v4C0S5TixP+LaOouie8uS7u5Z+P6pksYi 6z6uQ1sd/0REXDddEHOiRO5+ap5g2jHT344P3YN/FyxP5T3nqT1F0el4cObCtNP3JCbpvZfX +dyYLtAstezDZRb+IHcr9yuCUzqXeG854HL+5UnnM/GhbMuWx7ZKzDvwT4dVy9Fx8aldl4S7 jjPVVNck313pdPy1mUYHzxUGrTOiHzK/xiTsZ9fom7nsXmTw2SviS6sd+O/mLt14wfWfgN3h Vbf8OyVUZrSm8U/86XL9//Gdb87UloZPuv7LxPirq7Oei8opJZbijERDLeai4kQAt9mzc1gD AAA= X-CMS-MailID: 20210706064034eucas1p1e4bd9281a5badb012c1a65f0dbf8169a X-Msg-Generator: CA Content-Type: text/plain; charset="utf-8" X-RootMTR: 20210705065703eucas1p2e89258a2fc286896b755047e06f514cb X-EPHeader: CA CMS-TYPE: 201P X-CMS-RootMailID: 20210705065703eucas1p2e89258a2fc286896b755047e06f514cb References: <20210705065657.30356-1-m.szyprowski@samsung.com> <20210705143054.nxy4ii6t6npj22ov@archlinux> Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org On 05.07.2021 16:30, Amey Narkhede wrote: > On 21/07/05 08:56AM, Marek Szyprowski wrote: >> QCOM IOMMU driver calls bus_set_iommu() for every IOMMU device controller, >> what fails for the second and latter IOMMU devices. This is intended and >> must be not fatal to the driver registration process. Also the cleanup >> path should take care of the runtime PM state, what is missing in the >> current patch. Revert relevant changes to the QCOM IOMMU driver until >> a proper fix is prepared. >> > Apologies for the broken patch I don't have any arm machine to test the > patches. Is this bug unique to qcom iommu? Frankly, I have no idea. Just grep for bus_set_iommu() and check if the caller might be executed more than once. Best regards -- Marek Szyprowski, PhD Samsung R&D Institute Poland