From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Rafael J. Wysocki" Subject: Re: [PATCH v2 0/2] ACPI / PCI: Fix _PRT lookup for ARI enabled devices Date: Sat, 30 May 2015 01:53:30 +0200 Message-ID: <3075635.JhZ4D0jF4G@vostro.rjw.lan> References: <20150526210927.7537.55656.stgit@gimli.home> <20150529221848.GD12733@google.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7Bit Return-path: Received: from v094114.home.net.pl ([79.96.170.134]:52160 "HELO v094114.home.net.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1753944AbbE2X16 (ORCPT ); Fri, 29 May 2015 19:27:58 -0400 In-Reply-To: <20150529221848.GD12733@google.com> Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Bjorn Helgaas Cc: Alex Williamson , linux-acpi@vger.kernel.org, linux-pci@vger.kernel.org, ddutile@redhat.com, linux-kernel@vger.kernel.org, lenb@kernel.org On Friday, May 29, 2015 05:18:48 PM Bjorn Helgaas wrote: > On Tue, May 26, 2015 at 03:11:38PM -0600, Alex Williamson wrote: > > v2: don't modify entry->id.device > > > > In most cases we only use ARI with SR-IOV VFs, which do not support > > INTx and therefore never hit this problem. However, some non-SR-IOV > > implementations create multiple PFs, extending beyond the standard > > 3-bit function numbers with ARI, and do support INTx for those > > additional functions. This can happen with Solarflare SFC9120 > > adapters. The host driver typically doesn't use INTx, so we also > > haven't noticed this problem on bare metal, but when we attempt to > > assign the device to a VM using vfio-pci, we fail trying to setup > > default INTx signaling. Thanks, > > > > Alex > > > > --- > > > > Alex Williamson (2): > > PCI: Move pci_ari_enabled() to global header > > ACPI / PCI: Account for ARI in _PRT lookups > > Rafael, do you want this or should I take it? > > I put it on my pci/virtualization branch for v4.2. That's fine. Please feel free to add my ACKs to those if that helps. Rafael