From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from out-177.mta1.migadu.com (out-177.mta1.migadu.com [95.215.58.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9526E1FBEB9 for ; Thu, 18 Sep 2025 22:47:11 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=95.215.58.177 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758235634; cv=none; b=If1t4S6ddb/WsxjJViaIW/VK9XiOuK43jdaLSAa8wbPSxGitShz6VY7OSTAJCDdldR+dDwwiigRRt8d/37HvURW7BCvtHpsQS98JQkYr/DQM1s4LqY/216+NHLnjlmLTYx5OGjPfDZ0SvW7ZOZJ4RnIl1x6A5/3e6DpFss1ksSs= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758235634; c=relaxed/simple; bh=Fb3xaBIT6ER+rev9TSsFyrE2EiE7WWvwisRDYYmGTTA=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=BsHsJ6rDV6xfkLzODuFNiVi+u757VHRPAVr4MBj8HJUfgtY9ElpWuaQsVYoEktlNjnsOpgm8Jm4jAH8u+PPmXlTgdkGTKdYX7uF1/As2bmvAeVXHmox05YvE8BtCx41zgbGpTbDUkX+p+DUn2NRyodUmovP0QG7VtS57noZhMIs= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev; spf=pass smtp.mailfrom=linux.dev; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b=QyyrU9Ry; arc=none smtp.client-ip=95.215.58.177 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.dev Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b="QyyrU9Ry" Message-ID: <457e98bf-0fa7-4ff1-affe-400ac2e0ae72@linux.dev> DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.dev; s=key1; t=1758235617; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=FzhZHxrkOAC+RPVVxC0EvN2oYPvyy96quTvFqvW0LtM=; b=QyyrU9RyxYyFbM+NmOkJQV1EVyLRTdZh8CuWtb9IstiHTPg8DWaCTLLakqiXEvFnHiQaQw T6rw8hHIeAGxrp2Ir3YOrS90UbMALUUmWpNRAFEE1sBKCSgceTZrxKct71jzfMz6bwg71R UKVl5M+t2NOFBRaqxIV8i1/zdLIpC7k= Date: Thu, 18 Sep 2025 15:46:50 -0700 Precedence: bulk X-Mailing-List: linux-rdma@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Subject: Re: [PATCH rdma-next] RDMA: Use %pe format specifier for error pointers To: Leon Romanovsky , Jason Gunthorpe Cc: Leon Romanovsky , Abhijit Gangurde , Allen Hubbe , Bart Van Assche , Chengchang Tang , Dennis Dalessandro , Junxian Huang , Kalesh AP , Konstantin Taranov , linux-rdma@vger.kernel.org, Long Li , Michael Margolin , Mustafa Ismail , Potnuri Bharat Teja , Selvin Xavier , target-devel@vger.kernel.org, Tatyana Nikolova , Yishai Hadas References: Content-Language: en-US X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. From: "yanjun.zhu" In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Migadu-Flow: FLOW_OUT On 9/18/25 10:53 AM, Leon Romanovsky wrote: > From: Leon Romanovsky > > Convert error logging throughout the RDMA subsystem to use > the %pe format specifier instead of PTR_ERR() with integer > format specifiers. We have already used this skill, and it has proven to be very useful. With it, developers no longer need to look up error codes in err.h Reviewed-by: Zhu Yanjun > > Signed-off-by: Leon Romanovsky > --- > drivers/infiniband/core/agent.c | 3 +-- > drivers/infiniband/hw/bnxt_re/ib_verbs.c | 4 ++-- > drivers/infiniband/hw/bnxt_re/main.c | 3 +-- > drivers/infiniband/hw/cxgb4/device.c | 5 ++--- > drivers/infiniband/hw/efa/efa_com.c | 4 ++-- > drivers/infiniband/hw/efa/efa_verbs.c | 6 ++++-- > drivers/infiniband/hw/hfi1/device.c | 4 ++-- > drivers/infiniband/hw/hfi1/user_sdma.c | 4 ++-- > drivers/infiniband/hw/hns/hns_roce_mr.c | 8 ++++---- > drivers/infiniband/hw/ionic/ionic_admin.c | 18 +++++++++--------- > drivers/infiniband/hw/irdma/verbs.c | 6 +++--- > drivers/infiniband/hw/mana/main.c | 5 ++--- > drivers/infiniband/hw/mana/mr.c | 6 ++++-- > drivers/infiniband/hw/mlx4/mad.c | 8 ++++---- > drivers/infiniband/hw/mlx4/qp.c | 3 ++- > drivers/infiniband/hw/mlx5/data_direct.c | 2 +- > drivers/infiniband/hw/mlx5/gsi.c | 15 +++++++++------ > drivers/infiniband/hw/mlx5/main.c | 14 ++++++++++---- > drivers/infiniband/hw/mlx5/mr.c | 3 +-- > drivers/infiniband/ulp/srpt/ib_srpt.c | 16 +++++++--------- > 20 files changed, 72 insertions(+), 65 deletions(-) >