Re: [PATCH 2/3] QIB: Removing usage of pcie_set_mps()

2015-08-17 Thread Jason Gunthorpe
On Mon, Aug 17, 2015 at 07:06:10PM -0500, Bjorn Helgaas wrote: > On Mon, Aug 17, 2015 at 5:50 PM, Jiang, Dave wrote: > > On Mon, 2015-08-17 at 17:30 -0500, Bjorn Helgaas wrote: > >> [+cc Mike, linux-rdma] > >> > >> On Wed, Jul 29, 2015 at 04:18:54PM -0600, Keith Busch wrote: > >> > From: Dave Jian

Re: [PATCH 2/3] QIB: Removing usage of pcie_set_mps()

2015-08-17 Thread Bjorn Helgaas
On Mon, Aug 17, 2015 at 5:50 PM, Jiang, Dave wrote: > On Mon, 2015-08-17 at 17:30 -0500, Bjorn Helgaas wrote: >> [+cc Mike, linux-rdma] >> >> On Wed, Jul 29, 2015 at 04:18:54PM -0600, Keith Busch wrote: >> > From: Dave Jiang >> > >> > This is in perperation of un-exporting the pcie_set_mps() func

Re: [PATCH 2/3] QIB: Removing usage of pcie_set_mps()

2015-08-17 Thread Jiang, Dave
On Mon, 2015-08-17 at 17:30 -0500, Bjorn Helgaas wrote: > [+cc Mike, linux-rdma] > > On Wed, Jul 29, 2015 at 04:18:54PM -0600, Keith Busch wrote: > > From: Dave Jiang > > > > This is in perperation of un-exporting the pcie_set_mps() function > > symbol. A driver should not be changing the MPS as

Re: [PATCH 2/3] QIB: Removing usage of pcie_set_mps()

2015-08-17 Thread Bjorn Helgaas
[+cc Mike, linux-rdma] On Wed, Jul 29, 2015 at 04:18:54PM -0600, Keith Busch wrote: > From: Dave Jiang > > This is in perperation of un-exporting the pcie_set_mps() function > symbol. A driver should not be changing the MPS as that is the > responsibility of the PCI subsystem. Please explain th

[PATCH 2/3] QIB: Removing usage of pcie_set_mps()

2015-07-29 Thread Keith Busch
From: Dave Jiang This is in perperation of un-exporting the pcie_set_mps() function symbol. A driver should not be changing the MPS as that is the responsibility of the PCI subsystem. Signed-off-by: Dave Jiang --- drivers/infiniband/hw/qib/qib_pcie.c | 27 +-- 1 file