On Mon, Feb 27, 2012 at 2:02 PM, Kyle McMartin <[email protected]> wrote: > static u64 ehca_get_max_hwpage_size(struct ehca_shca *shca) > { > - return 1UL << ilog2(shca->hca_cap_mr_pgsize); > + u32 pgsize = shca->hca_cap_mr_pgsize; > + return 1UL << ilog2(pgsize); > }
How about using rounddown_pow_of_two(shca->hca_cap_mr_pgsize)? Does that build? Because that seems to me to be a cleaner way of expressing this anyway... Something about that ilog2() implementation seems to trigger gcc bugs, I seem to remember some other bogus compile failure on sparc involving it as well... - R. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
