lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 5 Nov 2014 17:05:36 -0700 From: Bjorn Helgaas <bhelgaas@...gle.com> To: Lorenzo Pieralisi <lorenzo.pieralisi@....com> Cc: linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org, linux-pci@...r.kernel.org, liviu.dudau@....com, Suravee.Suthikulpanit@....com, Will Deacon <will.deacon@....com> Subject: Re: [PATCH RFC 1/2] drivers: pci: fix window allocation order wrt bus_range filtering On Thu, Oct 23, 2014 at 04:23:06PM +0100, Lorenzo Pieralisi wrote: > The number of windows allocated for the host bridge depends on the > bus resource. Instead of first allocating the windows and then > limit the bus resource, this patch reshuffles the code so that if any > limitation is applied to the bus resource it is taken into account in > the windows allocation. > > Cc: Will Deacon <will.deacon@....com> > Signed-off-by: Lorenzo Pieralisi <lorenzo.pieralisi@....com> I applied these with Will's acks to pci/host-generic for v3.19, thanks! In my mind, posting patches to the list is implicitly a request for comments, so I usually don't pay too much attention to things explicitly marked "RFC" because I figure that's a hint that "this is work-in-progress, please comment if you see anything obviously broken." Bjorn > --- > drivers/pci/host/pci-host-generic.c | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/pci/host/pci-host-generic.c b/drivers/pci/host/pci-host-generic.c > index 3d2076f..1e1a80f 100644 > --- a/drivers/pci/host/pci-host-generic.c > +++ b/drivers/pci/host/pci-host-generic.c > @@ -276,17 +276,17 @@ static int gen_pci_parse_map_cfg_windows(struct gen_pci *pci) > return err; > } > > - pci->cfg.win = devm_kcalloc(dev, resource_size(&pci->cfg.bus_range), > - sizeof(*pci->cfg.win), GFP_KERNEL); > - if (!pci->cfg.win) > - return -ENOMEM; > - > /* Limit the bus-range to fit within reg */ > bus_max = pci->cfg.bus_range.start + > (resource_size(&pci->cfg.res) >> pci->cfg.ops->bus_shift) - 1; > pci->cfg.bus_range.end = min_t(resource_size_t, pci->cfg.bus_range.end, > bus_max); > > + pci->cfg.win = devm_kcalloc(dev, resource_size(&pci->cfg.bus_range), > + sizeof(*pci->cfg.win), GFP_KERNEL); > + if (!pci->cfg.win) > + return -ENOMEM; > + > /* Map our Configuration Space windows */ > if (!devm_request_mem_region(dev, pci->cfg.res.start, > resource_size(&pci->cfg.res), > -- > 2.1.2 > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@...r.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists