sebastien dugue <sebastien.du...@bull.net> wrote: > > @@ -172,6 +172,7 @@ struct ib_cq *mlx4_ib_create_cq(struct ib_device > > *ibdev, int entries, int vector > > struct mlx4_uar *uar; > > int err; > > > > + entries = roundup_pow_of_two(entries + 1); > > if (entries < 1 || entries > dev->dev->caps.max_cqes) > > Is the first check still needed here then?
not really, good catch - unless we need to protect here from negative values coming e.g from user space, I'll look this up, thanks Or. Or. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html