From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.0 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id A2AA8C2BBCD for ; Wed, 16 Dec 2020 09:56:44 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 5C47B2337F for ; Wed, 16 Dec 2020 09:56:44 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726115AbgLPJ4e (ORCPT ); Wed, 16 Dec 2020 04:56:34 -0500 Received: from mail.kernel.org ([198.145.29.99]:54776 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726026AbgLPJ4e (ORCPT ); Wed, 16 Dec 2020 04:56:34 -0500 Date: Wed, 16 Dec 2020 11:55:49 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1608112554; bh=c7sg4Le5nI4N13U+DswIbk6sLBXL1JJDKzhV0N+zIDk=; h=From:To:Cc:Subject:References:In-Reply-To:From; b=pfCF9nyfHWY8BcXaHloE8Fv0WLVMb31/tlReL9i0MnC7h76j9SgpjEZ6q1z1mXISQ 3f+OYaKANfiUNwwee8BvmnecXnvWBvw434gKxajvAXmpuK3i1n+wnHwuEVCXTr0QfM nd0joBTYLqAlv8XeflKSpwvn5mfnCz1MdjzVDIYR2yIxKiLkLWHBOq0tbcOrNjbBa0 72eAwQ4e9CyE2aFdln+cmLWp9v4lG4Wr7KqWo9h1eiktuTcKRo4+iyuMfyqOpKUmei LJ+AiE5CiWYXAByDffSqG2JcypN4zwWTStQbACanUB132Di/jqBaS2fcTU8H/lgMcL pibtkUGyCFiEw== From: Leon Romanovsky To: Xiao Yang Cc: linux-rdma@vger.kernel.org Subject: Re: [PATCH v2] librdmacm: Make some functions report proper errno Message-ID: <20201216095549.GC1060282@unreal> References: <20201216092252.155110-1-yangx.jy@cn.fujitsu.com> <5FD9D8B2.1020208@cn.fujitsu.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5FD9D8B2.1020208@cn.fujitsu.com> Precedence: bulk List-ID: X-Mailing-List: linux-rdma@vger.kernel.org On Wed, Dec 16, 2020 at 05:51:46PM +0800, Xiao Yang wrote: > Hi Leon, > > Thanks for your quick reply. :-) > I have done the same change on three > functions(ucma_get_device,ucma_create_cqs, rdma_create_qp_ex). > > On 2020/12/16 17:22, Xiao Yang wrote: > > Some functions reports fixed ENOMEM when getting any failure, so > > it's hard for user to know which actual error happens on them. > > > > Fixes(ucma_get_device): > > 2ffda7f29913 ("librdmacm: Only allocate verbs resources when needed") > > 191c9346f335 ("librdmacm: Reference count access to verbs context") > > Fixes(ucma_create_cqs): > > f8f1335ad8d8 ("librdmacm: make CQs optional for rdma_create_qp") > > 9e33488e8e50 ("librdmacm: fix all calls to set errno") > > Fixes(rdma_create_qp_ex): > > d2efdede11f7 ("r4019: Add support for userspace RDMA connection management abstraction (CMA)") > > 4e33a4109a62 ("librdmacm: returns errors from the library consistently") > > 995eb0c90c1a ("rdmacm: Add support for XRC QPs") > For every function, I am not sure which one is an exact commit so just > attach all related commits ids. No problem, I'll try to sort it out now. Thanks