> From: Artemy Kovalyov <artemyko@xxxxxxxxxxxx> > > Size of pages are held by struct ib_umem in page_size field. > > It is better to store it as an exponent, because page size by nature > is always power-of-two and used as a factor, divisor or ilog2's argument. > > The conversion of page_size to be page_shift allows to have portable > code and avoid following error while compiling on ARM: > > ERROR: "__aeabi_uldivmod" [drivers/infiniband/core/ib_core.ko] > undefined! > > CC: Selvin Xavier <selvin.xavier@xxxxxxxxxxxx> > CC: Steve Wise <swise@xxxxxxxxxxx> > CC: Lijun Ou <oulijun@xxxxxxxxxx> > CC: Shiraz Saleem <shiraz.saleem@xxxxxxxxx> > CC: Adit Ranadive <aditr@xxxxxxxxxx> > CC: Dennis Dalessandro <dennis.dalessandro@xxxxxxxxx> > CC: Ram Amrani <Ram.Amrani@xxxxxxxxxx> > Signed-off-by: Artemy Kovalyov <artemyko@xxxxxxxxxxxx> > Signed-off-by: Leon Romanovsky <leon@xxxxxxxxxx> > --- > drivers/infiniband/core/umem.c | 15 ++++++--------- > drivers/infiniband/core/umem_odp.c | 12 ++++++------ > drivers/infiniband/hw/bnxt_re/ib_verbs.c | 12 ++++++------ > drivers/infiniband/hw/cxgb3/iwch_provider.c | 4 ++-- > drivers/infiniband/hw/cxgb4/mem.c | 4 ++-- For the cxgb parts: Acked-by: Steve Wise <swise@xxxxxxxxxxxxxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
