On Sun, Jun 10, 2018 at 11:00:57AM +0000, Xueming(Steven) Li wrote: > Ack except one minor question below. <snip> > > - mlx5_glue->dv_query_device(attr_ctx, &attrs_out); > > - if (attrs_out.flags & MLX5DV_CONTEXT_FLAGS_MPW_ALLOWED) { > > - if (attrs_out.flags & MLX5DV_CONTEXT_FLAGS_ENHANCED_MPW) { > > + mlx5_glue->dv_query_device(ctx, &dv_attr); > > Should ctx be attr_ctx?.
Indeed, seems like I didn't validate this patch on its own after splitting it from another. Will fix in v2, thanks. -- Adrien Mazarguil 6WIND