Re: [PATCH rdma-core 4/6] libhns: Submit bad wr into user

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, May 11, 2018 at 11:33:28AM +0800, Lijun Ou wrote:
> When generated bad work reqeust, it needs to
> report to user. This patch mainly fixes it.
> 
> Signed-off-by: Lijun Ou <oulijun@xxxxxxxxxx>
> ---
>  providers/hns/hns_roce_u_hw_v2.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/providers/hns/hns_roce_u_hw_v2.c b/providers/hns/hns_roce_u_hw_v2.c
> index 06830ac..42bd5dd 100644
> --- a/providers/hns/hns_roce_u_hw_v2.c
> +++ b/providers/hns/hns_roce_u_hw_v2.c
> @@ -695,7 +695,7 @@ static int hns_roce_u_v2_post_send(struct ibv_qp *ibvqp, struct ibv_send_wr *wr,
>  				*bad_wr = wr;
>  				printf("data len=%d, send_flags = 0x%x!\r\n",
>  					rc_sq_wqe->msg_len, wr->send_flags);
> -				return ret;
> +				goto out;

No prints in drivers, please send a patch removing all of the printfs
and fprintfs

You can have opt-in debug scheme, like mlx5, that would be some good
code to move into util/

Thanks,
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



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux