From mboxrd@z Thu Jan 1 00:00:00 1970 From: Santosh Shilimkar Subject: Re: [PATCH 1/3] RDS: IB: Delete an error message for a failed memory allocation in rds_ib_add_one() Date: Mon, 22 May 2017 12:24:38 -0700 Message-ID: <3b02a3d1-17c9-3d8b-7edb-db640c0ea017@oracle.com> References: <44b46c51-3665-5f14-dd3b-6e6f230258a7@users.sourceforge.net> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Cc: LKML , kernel-janitors-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: SF Markus Elfring , linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, rds-devel-N0ozoZBvEnrZJqsBc5GL+g@public.gmane.org, "David S. Miller" , Sowmini Varadhan Return-path: In-Reply-To: Content-Language: en-US Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: netdev.vger.kernel.org On 5/22/2017 7:11 AM, SF Markus Elfring wrote: > From: Markus Elfring > Date: Mon, 22 May 2017 15:34:28 +0200 > > Omit an extra message for a memory allocation failure in this function. > > This issue was detected by using the Coccinelle software. > > Link: http://events.linuxfoundation.org/sites/events/files/slides/LCJ16-Refactor_Strings-WSang_0.pdf > Signed-off-by: Markus Elfring > --- > net/rds/ib.c | 5 +---- > 1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git a/net/rds/ib.c b/net/rds/ib.c > index 7a64c8db81ab..c5514d058171 100644 > --- a/net/rds/ib.c > +++ b/net/rds/ib.c > @@ -166,8 +166,5 @@ static void rds_ib_add_one(struct ib_device *device) > - if (!rds_ibdev->vector_load) { > - pr_err("RDS/IB: %s failed to allocate vector memory\n", > - __func__); > + if (!rds_ibdev->vector_load) > goto put_dev; > - } > Well the ENOMEM is not carried here so the message was usefu but its not critical so its fine to clean that up. Acked-by: Santosh Shilimkar -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html