On Wed, Jun 20, 2018 at 07:28:22PM +0300, Yishai Hadas wrote:
> @@ -80,7 +80,16 @@ static int copy_to_scat(struct mlx5_wqe_data_seg *scat, void *buf, int *size,
>
> for (i = 0; i < max; ++i) {
> copy = min_t(long, *size, be32toh(scat->byte_count));
> - memcpy((void *)(unsigned long)be64toh(scat->addr), buf, copy);
> +
> + /* When NULL MR is used can't copy to target,
> + * expected to be NULL.
> + */
> + if (unlikely(scat->lkey == ctx->dump_fill_mkey_be))
> + ;
> + else
What is that? Better as:
if (likely(scat->lkey != ctx->dump_fill_mkey_be))
memcpy((void *)(unsigned long)be64toh(scat->addr),
buf, copy);
Jason
--
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
