Received: by 2002:a05:6a10:206:0:0:0:0 with SMTP id 6csp702357pxj; Fri, 11 Jun 2021 09:19:34 -0700 (PDT) X-Google-Smtp-Source: ABdhPJx8bM6c0iTl9gZ86zth0JUKm24ZrjSkiaEM8AIyL2ugXidTZJ+9T9CG1/0249yg4nkXpnO7 X-Received: by 2002:a17:906:a281:: with SMTP id i1mr4384868ejz.307.1623428373828; Fri, 11 Jun 2021 09:19:33 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1623428373; cv=none; d=google.com; s=arc-20160816; b=qJUhpOTOTMBrsNz4rgEjtDOJ6604qFM0WS9wKKWRzadJurkS7L1s6neduyuykiJoHt 5k9Y//xdq0NEDDvRNL3fSWtgd1X7+0TbovB3wpWC3wW3ELHb0sYQgM7l0yKZyjEWacPg k7f3a2nntm22YXYkbllOHhMiremHO90gY/i2tqcCK5QfT2MtdUUO9HpTpUOd29pRWb7j dP3mEAMk7w8pZV8HrPC2BbkF3paoTqhEfcIjWdI4Dtcu+VPpFoWu5OMNg8L/ff/bzgtY iuHCsLG2ouYB4HZKROmKCiOK1BDax7pZKe3e4TeKsVa8NBCPsTd6FaIha+xRmdc88SW/ vSCg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=yT8LLF9ty2I9/FkUf79JQP/SobFY0ekzIP+15mL22fg=; b=cwkZDRcYWefb8T0k2R08dQ5LHP7yqiFcoi+XOTmGzzQ5oxofaHidoBU7M1IP0uC3sG T5nbC081jFpy3uhtnKJtIYdHKqqqBfSRRJe9aTs9j63AaKS4mF/DIIn8Li6iExVI+RkB mmuUlGDh5ckoq1CJt8xLqKkVayJ+3f4TQzVNBpxajCh14zU/X1cSMQe7mJH2bf78azDF V5odieBuKO2N71ZUshX90HeBAtsNPTwJoGvCGZfEUaZykIRoAQ7WBlTh40FoqUKCtJ1a jgXFXVo4MdUzN4OSyLF1/qBzkpPpiUZiQyKdEgk7yj721RuuUEUpOb2Kx5GsbOgdIAMF PIwg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=toHI7dqF; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id z21si4905236eja.498.2021.06.11.09.19.10; Fri, 11 Jun 2021 09:19:33 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=toHI7dqF; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230313AbhFKQR6 (ORCPT + 99 others); Fri, 11 Jun 2021 12:17:58 -0400 Received: from mail.kernel.org ([198.145.29.99]:47260 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230281AbhFKQR6 (ORCPT ); Fri, 11 Jun 2021 12:17:58 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 94582613D9; Fri, 11 Jun 2021 16:15:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1623428160; bh=X8fq5Ssa8KruLdrdCiVfEs1JgSpE6Roo0bK+wmejB+k=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=toHI7dqFyFs3RVKe8lF5VXSfnke6XT3P1GGiJdTo3FqWDIkpD/UMJwpTTHHUR++k8 TW6DmDB2XMJ4JqU9PbTHevr/DPIp/EoUPcmedvAkTGvZbw4FnQVOdzrs2NEYcocnYy lKsNALRdXg2ITl7mJcJK3fHjp4ywSZORfvfwmJFPyomJ3R7tCU9tdHsD5gQGkJ6d+6 J82qegUsU7v8pMbKZvDrhe2ADs9S795kAKWQbQ5eYOPo8I1xNN1j5OAM9sOxPdPzBI kRj5HSOqzcd1kfXsJGXUS7jwCLDaTgP9K23UVsfvdJwuPtUK2xSTE1/IYxgMVFGgKV V+ghr6emt7k9A== From: Will Deacon To: Sai Prakash Ranjan , Jordan Crouse , Xiyu Yang , Krishna Reddy , Rob Clark , Joerg Roedel , Robin Murphy , linux-arm-kernel@lists.infradead.org, Jon Hunter , linux-kernel@vger.kernel.org, iommu@lists.linux-foundation.org Cc: catalin.marinas@arm.com, kernel-team@android.com, Will Deacon , Xin Tan , yuanxzhang@fudan.edu.cn Subject: Re: [PATCH v2] iommu/arm-smmu: Fix arm_smmu_device refcount leak in address translation Date: Fri, 11 Jun 2021 17:15:43 +0100 Message-Id: <162340795809.3699041.237536515051087882.b4-ty@kernel.org> X-Mailer: git-send-email 2.20.1 In-Reply-To: <1623293391-17261-1-git-send-email-xiyuyang19@fudan.edu.cn> References: <1623293391-17261-1-git-send-email-xiyuyang19@fudan.edu.cn> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 10 Jun 2021 10:49:20 +0800, Xiyu Yang wrote: > The reference counting issue happens in several exception handling paths > of arm_smmu_iova_to_phys_hard(). When those error scenarios occur, the > function forgets to decrease the refcount of "smmu" increased by > arm_smmu_rpm_get(), causing a refcount leak. > > Fix this issue by jumping to "out" label when those error scenarios > occur. Applied to will (for-joerg/arm-smmu/updates), thanks! [1/1] iommu/arm-smmu: Fix arm_smmu_device refcount leak in address translation https://git.kernel.org/will/c/7c8f176d6a3f Cheers, -- Will https://fixes.arm64.dev https://next.arm64.dev https://will.arm64.dev