On Fri, Apr 18, 2014 at 02:19:50PM +0200, Thomas Petazzoni wrote: > From: Willy Tarreau <[email protected]> > > mvebu_pcie_handle_membase_change() and > mvebu_pcie_handle_iobase_change() do not correctly compute the window > size. PCI uses an inclusive start/end address pair, which requires a > +1 when converting to size. > > This only worked because a bug in the mbus driver allowed it to > silently accept and round up bogus sizes. > > Fix this by adding one to the computed size. > > Fixes: 45361a4fe4464180815157654aabbd2afb4848ad ('PCIe driver for Marvell > Armada 370/XP systems') > Cc: <[email protected]> # v3.11+ > Signed-off-by: Willy Tarreau <[email protected]> > Reviewed-By: Jason Gunthorpe <[email protected]> > Signed-off-by: Thomas Petazzoni <[email protected]> > Tested-by: Neil Greatorex <[email protected]>
Acked-by: Bjorn Helgaas <[email protected]> If I were merging via my tree, I would tweak the subject capitalization so "git log --oneline drivers/pci/host/pci-mvebu.c" looks consistent. > --- > drivers/pci/host/pci-mvebu.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/pci/host/pci-mvebu.c b/drivers/pci/host/pci-mvebu.c > index d3d1cfd..4829921 100644 > --- a/drivers/pci/host/pci-mvebu.c > +++ b/drivers/pci/host/pci-mvebu.c > @@ -331,7 +331,7 @@ static void mvebu_pcie_handle_iobase_change(struct > mvebu_pcie_port *port) > port->iowin_base = port->pcie->io.start + iobase; > port->iowin_size = ((0xFFF | ((port->bridge.iolimit & 0xF0) << 8) | > (port->bridge.iolimitupper << 16)) - > - iobase); > + iobase) + 1; > > mvebu_mbus_add_window_remap_by_id(port->io_target, port->io_attr, > port->iowin_base, port->iowin_size, > @@ -364,7 +364,7 @@ static void mvebu_pcie_handle_membase_change(struct > mvebu_pcie_port *port) > port->memwin_base = ((port->bridge.membase & 0xFFF0) << 16); > port->memwin_size = > (((port->bridge.memlimit & 0xFFF0) << 16) | 0xFFFFF) - > - port->memwin_base; > + port->memwin_base + 1; > > mvebu_mbus_add_window_by_id(port->mem_target, port->mem_attr, > port->memwin_base, port->memwin_size); > -- > 1.9.2 > -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
