On Thu, Oct 27, 2016 at 4:36 PM, Leon Romanovsky <leon@xxxxxxxxxx> wrote:
> +static void wait_for_async_commands(struct mlx5_ib_dev *dev)
> +{
> + struct mlx5_mr_cache *cache = &dev->cache;
> + struct mlx5_cache_ent *ent;
> + int total = 0;
> + int i;
> + int j;
> +
> + for (i = 0; i < MAX_MR_CACHE_ENTRIES; i++) {
> + ent = &cache->ent[i];
> + for (j = 0 ; j < 1000; j++) {
> + if (!ent->pending)
> + break;
> + msleep(50);
> + }
you had another patch on this series which change a hard coded
constant into a define, why this patch add two new hard coded
constants, so all to all, we're not making progress on that
no-hard-coded-constants front... better decide where you want to go
--
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
