On Wed, 15 Apr 2015 17:49:22 +0800 Minghuan Lian <Minghuan.Lian@xxxxxxxxxxxxx> wrote: [adding Jason and Thomas to the party, since they shoud be cc'd the first place] > The default page size of ITS table has been changed to 64KB, > but for some platforms the real size of allocate memory which > calculated by DEVBITS of register GITS_TYPER may smaller than > default size. In this case, psz must be decreased, otherwise, > the wrong page number will cause kernel hang. > > Signed-off-by: Minghuan Lian <Minghuan.Lian@xxxxxxxxxxxxx> > --- > drivers/irqchip/irq-gic-v3-its.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/irqchip/irq-gic-v3-its.c b/drivers/irqchip/irq-gic-v3-its.c > index 596b0a9..d0374a6 100644 > --- a/drivers/irqchip/irq-gic-v3-its.c > +++ b/drivers/irqchip/irq-gic-v3-its.c > @@ -844,6 +844,9 @@ static int its_alloc_tables(struct its_node *its) > > its->tables[i] = base; > > + while (alloc_size < psz) > + psz = psz >> 1; > + > retry_baser: > val = (virt_to_phys(base) | > (type << GITS_BASER_TYPE_SHIFT) | This doesn't make sense to me. psz is the page size that your HW supports, not necessarily something that you can actually change as you wish (some, if not most, ITSs only support one page size). Also, you now have to psz go via unsupported page sizes (like 32K and 8K), and you may end up programming junk in the registers. Good job! ;-) So making it vary to accommodate your allocation size is slightly doomed. If your allocation is smaller than your ITS page size, it is the allocation that has to be bumped up, not the page size that has to be reduced. So overall, this is a NAK. Please come up with a better fix. Thanks, M. -- Jazz is not dead. It just smells funny. -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
