Re: driver skip pci_set_master, fix it? No.

2014-04-09 Thread Bjorn Helgaas
at >>>>>> without it, >>>>>> we may also not get the PCIe error messages. >>>>> >>>>> Sure, for bridges. >>>>> >>>>> I'll get a stack trace later today, but what I suspect is happening >>

Re: driver skip pci_set_master, fix it? No.

2014-04-09 Thread Mark Lord
t; Sure, for bridges. >>>> >>>> I'll get a stack trace later today, but what I suspect is happening >>>> is that this multi-function card is being treated by the PCI layers >>>> as a "bridge" for purposes of the multiple virtual functions it imple

Re: driver skip pci_set_master, fix it? No.

2014-04-09 Thread Bjorn Helgaas
; is that this multi-function card is being treated by the PCI layers >>> as a "bridge" for purposes of the multiple virtual functions it implements. >>> >>> We will probably need to distinguish this kind of device from real bridges >>> here. >> >> Here's the c

Re: driver skip pci_set_master, fix it? No.

2014-04-09 Thread Mark Lord
; >> We will probably need to distinguish this kind of device from real bridges >> here. > > Here's the call trace, all the way back to k7_probe(), > the driver's PCI "probe" function, and beyond: > > [ 30.481454] k7: loading driver version 0.80 >

Re: driver skip pci_set_master, fix it? No.

2014-04-09 Thread Mark Lord
this multi-function card is being treated by the PCI layers > as a "bridge" for purposes of the multiple virtual functions it implements. > > We will probably need to distinguish this kind of device from real bridges > here. Here's the call trace, all the way back to k7_

Re: driver skip pci_set_master, fix it? No.

2014-04-09 Thread Mark Lord
On 14-04-08 10:51 PM, Benjamin Herrenschmidt wrote: > On Tue, 2014-04-08 at 17:18 -0400, Mark Lord wrote: >>> I assume you're talking about the one added by cf3e1feba7f9 ("PCI: >>> Workaround missing pci_set_master in pci drivers"), but as far as I >>> can tell, it only calls pci_set_master() for

Re: driver skip pci_set_master, fix it? No.

2014-04-09 Thread Mark Lord
On 14-04-08 10:51 PM, Benjamin Herrenschmidt wrote: On Tue, 2014-04-08 at 17:18 -0400, Mark Lord wrote: I assume you're talking about the one added by cf3e1feba7f9 (PCI: Workaround missing pci_set_master in pci drivers), but as far as I can tell, it only calls pci_set_master() for *bridge*

Re: driver skip pci_set_master, fix it? No.

2014-04-09 Thread Mark Lord
to k7_probe(), the driver's PCI probe function, and beyond: [ 30.481454] k7: loading driver version 0.80 [ 30.485561] pcieport :00:1c.0: driver skip pci_set_master, fix it! [ 30.485580] CPU: 2 PID: 4401 Comm: insmod Tainted: G O 3.12.14 #3 [ 30.485583] Hardware name: Supermicro

Re: driver skip pci_set_master, fix it? No.

2014-04-09 Thread Mark Lord
the call trace, all the way back to k7_probe(), the driver's PCI probe function, and beyond: [ 30.481454] k7: loading driver version 0.80 [ 30.485561] pcieport :00:1c.0: driver skip pci_set_master, fix it! [ 30.485580] CPU: 2 PID: 4401 Comm: insmod Tainted: G O 3.12.14 #3

Re: driver skip pci_set_master, fix it? No.

2014-04-09 Thread Bjorn Helgaas
to distinguish this kind of device from real bridges here. Here's the call trace, all the way back to k7_probe(), the driver's PCI probe function, and beyond: [ 30.481454] k7: loading driver version 0.80 [ 30.485561] pcieport :00:1c.0: driver skip pci_set_master, fix it! This message

Re: driver skip pci_set_master, fix it? No.

2014-04-09 Thread Mark Lord
and the tree view: pcieport :00:1c.4: driver skip pci_set_master, fix it! pcieport :00:1c.5: driver skip pci_set_master, fix it! pcieport :00:1c.0: driver skip pci_set_master, fix it! lspci -t -[:00]-+-00.0 +-01.0-[01]--+-00.0 |\-00.1

Re: driver skip pci_set_master, fix it? No.

2014-04-09 Thread Bjorn Helgaas
: loading driver version 0.80 [ 30.485561] pcieport :00:1c.0: driver skip pci_set_master, fix it! This message says we're enabling bus mastering for a PCIe Root Port, which I think is the expected behavior and shouldn't cause trouble for your device (correct me if I'm wrong). I don't know

Re: driver skip pci_set_master, fix it? No.

2014-04-08 Thread Benjamin Herrenschmidt
On Tue, 2014-04-08 at 17:18 -0400, Mark Lord wrote: > > I assume you're talking about the one added by cf3e1feba7f9 ("PCI: > > Workaround missing pci_set_master in pci drivers"), but as far as I > > can tell, it only calls pci_set_master() for *bridge* devices. What > > am I missing? Is

Re: driver skip pci_set_master, fix it? No.

2014-04-08 Thread Mark Lord
On 14-04-08 02:27 PM, Bjorn Helgaas wrote: > [+cc Ben, linux-pci] > > On Tue, Apr 8, 2014 at 10:34 AM, Mark Lord wrote: >> I am working a couple of drivers for chips that perform extensive >> bus-mastering ops. >> These including full SRIOV support, and allow assigning virtual functions to >>

Re: driver skip pci_set_master, fix it? No.

2014-04-08 Thread Bjorn Helgaas
[+cc Ben, linux-pci] On Tue, Apr 8, 2014 at 10:34 AM, Mark Lord wrote: > I am working a couple of drivers for chips that perform extensive > bus-mastering ops. > These including full SRIOV support, and allow assigning virtual functions to > virtual machines, etc. > > One thing the driver

driver skip pci_set_master, fix it? No.

2014-04-08 Thread Mark Lord
I am working a couple of drivers for chips that perform extensive bus-mastering ops. These including full SRIOV support, and allow assigning virtual functions to virtual machines, etc. One thing the driver (still in development) does for safety, is defer the call to pci_set_master() until

driver skip pci_set_master, fix it? No.

2014-04-08 Thread Mark Lord
I am working a couple of drivers for chips that perform extensive bus-mastering ops. These including full SRIOV support, and allow assigning virtual functions to virtual machines, etc. One thing the driver (still in development) does for safety, is defer the call to pci_set_master() until

Re: driver skip pci_set_master, fix it? No.

2014-04-08 Thread Bjorn Helgaas
[+cc Ben, linux-pci] On Tue, Apr 8, 2014 at 10:34 AM, Mark Lord ml...@pobox.com wrote: I am working a couple of drivers for chips that perform extensive bus-mastering ops. These including full SRIOV support, and allow assigning virtual functions to virtual machines, etc. One thing the

Re: driver skip pci_set_master, fix it? No.

2014-04-08 Thread Mark Lord
On 14-04-08 02:27 PM, Bjorn Helgaas wrote: [+cc Ben, linux-pci] On Tue, Apr 8, 2014 at 10:34 AM, Mark Lord ml...@pobox.com wrote: I am working a couple of drivers for chips that perform extensive bus-mastering ops. These including full SRIOV support, and allow assigning virtual functions

Re: driver skip pci_set_master, fix it? No.

2014-04-08 Thread Benjamin Herrenschmidt
On Tue, 2014-04-08 at 17:18 -0400, Mark Lord wrote: I assume you're talking about the one added by cf3e1feba7f9 (PCI: Workaround missing pci_set_master in pci drivers), but as far as I can tell, it only calls pci_set_master() for *bridge* devices. What am I missing? Is pci_set_master()