Date:   Wed, 17 Nov 2021 07:15:07 -0700
From:   Alex Williamson <alex.williamson@redhat.com>
To:     Christoph Hellwig <hch@infradead.org>
Cc:     Yifeng Li <tomli@tomli.me>, Bjorn Helgaas <bhelgaas@google.com>,
        linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org,
        kvm@vger.kernel.org
Subject: Re: [PATCH v2] PCI: Add func1 DMA quirk for Marvell 88SE9125 SATA
 controller
Message-ID: <20211117071507.775e12b8.alex.williamson@redhat.com>
In-Reply-To: <YZTVdOlEbMb0tv59@infradead.org>
References: <YZPA+gSsGWI6+xBP@work>
        <YZTVdOlEbMb0tv59@infradead.org>
Organization: Red Hat
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Precedence: bulk

On Wed, 17 Nov 2021 02:12:04 -0800
Christoph Hellwig <hch@infradead.org> wrote:

> On Tue, Nov 16, 2021 at 02:32:26PM +0000, Yifeng Li wrote:
> > Like other SATA controller chips in the Marvell 88SE91xx series, the
> > Marvell 88SE9125 has the same DMA requester ID hardware bug that prevents
> > it from working under IOMMU. This patch adds its device ID 0x9125 to the
> > Function 1 DMA alias quirk list.  
> 
> Btw, do we need to prevent vfio assignment for all devices with this
> quirk?

No, the alias is taken into account with grouping and IOMMU
programming, it should work with vfio.  Thanks,

Alex