Re: [PATCH] pci: Account for virtual buses in pci_acs_path_enabled

2012-08-08 Thread Don Dutile
On 08/08/2012 11:24 AM, Alex Williamson wrote: On Tue, 2012-08-07 at 23:00 -0700, Bjorn Helgaas wrote: On Tue, Aug 7, 2012 at 2:50 PM, Don Dutile wrote: On 08/06/2012 04:47 PM, Bjorn Helgaas wrote: On Sun, Aug 5, 2012 at 11:55 PM, Alex Williamson wrote: On Sun, 2012-08-05 at 23:30

Re: [PATCH] pci: Account for virtual buses in pci_acs_path_enabled

2012-08-08 Thread Alex Williamson
On Tue, 2012-08-07 at 23:00 -0700, Bjorn Helgaas wrote: > On Tue, Aug 7, 2012 at 2:50 PM, Don Dutile wrote: > > On 08/06/2012 04:47 PM, Bjorn Helgaas wrote: > >> > >> On Sun, Aug 5, 2012 at 11:55 PM, Alex Williamson > >> wrote: > >>> > >>> On Sun, 2012-08-05 at 23:30 -0600, Bjorn Helgaas wrote:

Re: [PATCH] pci: Account for virtual buses in pci_acs_path_enabled

2012-08-08 Thread David Ahern
On 8/8/12 12:00 AM, Bjorn Helgaas wrote: On Tue, Aug 7, 2012 at 2:50 PM, Don Dutile wrote: On 08/06/2012 04:47 PM, Bjorn Helgaas wrote: On Sun, Aug 5, 2012 at 11:55 PM, Alex Williamson wrote: On Sun, 2012-08-05 at 23:30 -0600, Bjorn Helgaas wrote: On Sat, Aug 4, 2012 at 12:19 PM, Alex

Re: [PATCH] pci: Account for virtual buses in pci_acs_path_enabled

2012-08-08 Thread Bjorn Helgaas
On Tue, Aug 7, 2012 at 2:50 PM, Don Dutile wrote: > On 08/06/2012 04:47 PM, Bjorn Helgaas wrote: >> >> On Sun, Aug 5, 2012 at 11:55 PM, Alex Williamson >> wrote: >>> >>> On Sun, 2012-08-05 at 23:30 -0600, Bjorn Helgaas wrote: On Sat, Aug 4, 2012 at 12:19 PM, Alex Williamson

Re: [PATCH] pci: Account for virtual buses in pci_acs_path_enabled

2012-08-08 Thread Bjorn Helgaas
On Tue, Aug 7, 2012 at 2:50 PM, Don Dutile ddut...@redhat.com wrote: On 08/06/2012 04:47 PM, Bjorn Helgaas wrote: On Sun, Aug 5, 2012 at 11:55 PM, Alex Williamson alex.william...@redhat.com wrote: On Sun, 2012-08-05 at 23:30 -0600, Bjorn Helgaas wrote: On Sat, Aug 4, 2012 at 12:19 PM,

Re: [PATCH] pci: Account for virtual buses in pci_acs_path_enabled

2012-08-08 Thread David Ahern
On 8/8/12 12:00 AM, Bjorn Helgaas wrote: On Tue, Aug 7, 2012 at 2:50 PM, Don Dutile ddut...@redhat.com wrote: On 08/06/2012 04:47 PM, Bjorn Helgaas wrote: On Sun, Aug 5, 2012 at 11:55 PM, Alex Williamson alex.william...@redhat.com wrote: On Sun, 2012-08-05 at 23:30 -0600, Bjorn Helgaas

Re: [PATCH] pci: Account for virtual buses in pci_acs_path_enabled

2012-08-08 Thread Alex Williamson
On Tue, 2012-08-07 at 23:00 -0700, Bjorn Helgaas wrote: On Tue, Aug 7, 2012 at 2:50 PM, Don Dutile ddut...@redhat.com wrote: On 08/06/2012 04:47 PM, Bjorn Helgaas wrote: On Sun, Aug 5, 2012 at 11:55 PM, Alex Williamson alex.william...@redhat.com wrote: On Sun, 2012-08-05 at 23:30

Re: [PATCH] pci: Account for virtual buses in pci_acs_path_enabled

2012-08-08 Thread Don Dutile
On 08/08/2012 11:24 AM, Alex Williamson wrote: On Tue, 2012-08-07 at 23:00 -0700, Bjorn Helgaas wrote: On Tue, Aug 7, 2012 at 2:50 PM, Don Dutileddut...@redhat.com wrote: On 08/06/2012 04:47 PM, Bjorn Helgaas wrote: On Sun, Aug 5, 2012 at 11:55 PM, Alex Williamson alex.william...@redhat.com

Re: [PATCH] pci: Account for virtual buses in pci_acs_path_enabled

2012-08-07 Thread Don Dutile
On 08/06/2012 04:47 PM, Bjorn Helgaas wrote: On Sun, Aug 5, 2012 at 11:55 PM, Alex Williamson wrote: On Sun, 2012-08-05 at 23:30 -0600, Bjorn Helgaas wrote: On Sat, Aug 4, 2012 at 12:19 PM, Alex Williamson wrote: It's possible to have buses without an associated bridge (bus->self ==

Re: [PATCH] pci: Account for virtual buses in pci_acs_path_enabled

2012-08-07 Thread Don Dutile
On 08/06/2012 04:47 PM, Bjorn Helgaas wrote: On Sun, Aug 5, 2012 at 11:55 PM, Alex Williamson alex.william...@redhat.com wrote: On Sun, 2012-08-05 at 23:30 -0600, Bjorn Helgaas wrote: On Sat, Aug 4, 2012 at 12:19 PM, Alex Williamson alex.william...@redhat.com wrote: It's possible to have

Re: [PATCH] pci: Account for virtual buses in pci_acs_path_enabled

2012-08-06 Thread Bjorn Helgaas
On Sun, Aug 5, 2012 at 11:55 PM, Alex Williamson wrote: > On Sun, 2012-08-05 at 23:30 -0600, Bjorn Helgaas wrote: >> On Sat, Aug 4, 2012 at 12:19 PM, Alex Williamson >> wrote: >> > It's possible to have buses without an associated bridge >> > (bus->self == NULL). SR-IOV can generate such buses.

Re: [PATCH] pci: Account for virtual buses in pci_acs_path_enabled

2012-08-06 Thread Bjorn Helgaas
On Sun, Aug 5, 2012 at 11:55 PM, Alex Williamson alex.william...@redhat.com wrote: On Sun, 2012-08-05 at 23:30 -0600, Bjorn Helgaas wrote: On Sat, Aug 4, 2012 at 12:19 PM, Alex Williamson alex.william...@redhat.com wrote: It's possible to have buses without an associated bridge (bus-self ==

Re: [PATCH] pci: Account for virtual buses in pci_acs_path_enabled

2012-08-05 Thread Alex Williamson
On Sun, 2012-08-05 at 23:30 -0600, Bjorn Helgaas wrote: > On Sat, Aug 4, 2012 at 12:19 PM, Alex Williamson > wrote: > > It's possible to have buses without an associated bridge > > (bus->self == NULL). SR-IOV can generate such buses. When > > we find these, skip to the parent bus to look for

Re: [PATCH] pci: Account for virtual buses in pci_acs_path_enabled

2012-08-05 Thread Bjorn Helgaas
On Sat, Aug 4, 2012 at 12:19 PM, Alex Williamson wrote: > It's possible to have buses without an associated bridge > (bus->self == NULL). SR-IOV can generate such buses. When > we find these, skip to the parent bus to look for the next > ACS test. To make sure I understand the problem here, I

Re: [PATCH] pci: Account for virtual buses in pci_acs_path_enabled

2012-08-05 Thread David Ahern
On 8/4/12 12:19 PM, Alex Williamson wrote: It's possible to have buses without an associated bridge (bus->self == NULL). SR-IOV can generate such buses. When we find these, skip to the parent bus to look for the next ACS test. Signed-off-by: Alex Williamson --- David Ahern reported an oops

Re: [PATCH] pci: Account for virtual buses in pci_acs_path_enabled

2012-08-05 Thread David Ahern
On 8/4/12 12:19 PM, Alex Williamson wrote: It's possible to have buses without an associated bridge (bus-self == NULL). SR-IOV can generate such buses. When we find these, skip to the parent bus to look for the next ACS test. Signed-off-by: Alex Williamson alex.william...@redhat.com ---

Re: [PATCH] pci: Account for virtual buses in pci_acs_path_enabled

2012-08-05 Thread Bjorn Helgaas
On Sat, Aug 4, 2012 at 12:19 PM, Alex Williamson alex.william...@redhat.com wrote: It's possible to have buses without an associated bridge (bus-self == NULL). SR-IOV can generate such buses. When we find these, skip to the parent bus to look for the next ACS test. To make sure I understand

Re: [PATCH] pci: Account for virtual buses in pci_acs_path_enabled

2012-08-05 Thread Alex Williamson
On Sun, 2012-08-05 at 23:30 -0600, Bjorn Helgaas wrote: On Sat, Aug 4, 2012 at 12:19 PM, Alex Williamson alex.william...@redhat.com wrote: It's possible to have buses without an associated bridge (bus-self == NULL). SR-IOV can generate such buses. When we find these, skip to the parent

[PATCH] pci: Account for virtual buses in pci_acs_path_enabled

2012-08-04 Thread Alex Williamson
It's possible to have buses without an associated bridge (bus->self == NULL). SR-IOV can generate such buses. When we find these, skip to the parent bus to look for the next ACS test. Signed-off-by: Alex Williamson --- David Ahern reported an oops from iommu drivers passing NULL into this

[PATCH] pci: Account for virtual buses in pci_acs_path_enabled

2012-08-04 Thread Alex Williamson
It's possible to have buses without an associated bridge (bus-self == NULL). SR-IOV can generate such buses. When we find these, skip to the parent bus to look for the next ACS test. Signed-off-by: Alex Williamson alex.william...@redhat.com --- David Ahern reported an oops from iommu drivers