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;
From:   Will Deacon <will@kernel.org>
To:     Johnathan Smithinovic <johnathan.smithinovic@gmx.at>,
        mingo@redhat.com, David Woodhouse <dwmw2@infradead.org>,
        tglx@linutronix.de, bp@alien8.de
Cc:     catalin.marinas@arm.com, kernel-team@android.com,
        Will Deacon <will@kernel.org>, x86@kernel.org,
        linux-kernel@vger.kernel.org,
        iommu <iommu@lists.linux-foundation.org>
Subject: Re: [PATCH] iommu/amd: Stop irq_remapping_select() matching when remapping is disabled
Date:   Tue,  5 Jan 2021 22:07:02 +0000
Message-Id: <160987465284.30865.16684216909917986687.b4-ty@kernel.org>
In-Reply-To: <04bbe8bca87f81a3cfa93ec4299e53f47e00e5b3.camel@infradead.org>
References: <ed4be9b4-24ac-7128-c522-7ef359e8185d@gmx.at> <04bbe8bca87f81a3cfa93ec4299e53f47e00e5b3.camel@infradead.org>
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
Precedence: bulk

On Tue, 05 Jan 2021 01:36:13 +0000, David Woodhouse wrote:
> The AMD IOMMU initialisation registers the IRQ remapping domain for
> each IOMMU before doing the final sanity check that every I/OAPIC is
> covered.
> 
> This means that the AMD irq_remapping_select() function gets invoked
> even when IRQ remapping has been disabled, eventually leading to a NULL
> pointer dereference in alloc_irq_table().
> 
> [...]

Applied to arm64 (for-next/iommu/fixes), thanks!

[1/1] iommu/amd: Stop irq_remapping_select() matching when remapping is disabled
      https://git.kernel.org/arm64/c/b34f10c2dc59

Cheers,
-- 
Will

https://fixes.arm64.dev
https://next.arm64.dev
https://will.arm64.dev