From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755825AbdETTNU (ORCPT ); Sat, 20 May 2017 15:13:20 -0400 Received: from mout.web.de ([212.227.17.11]:53692 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751830AbdETTNR (ORCPT ); Sat, 20 May 2017 15:13:17 -0400 Subject: [PATCH 2/5] fsl_hypervisor: Delete an error message for a failed memory allocation in fsl_hv_open() From: SF Markus Elfring To: Arnd Bergmann , Jan Kara , Jesper Nilsson , Kumar Gala , Lorenzo Stoakes , Michal Hocko , Timur Tabi Cc: LKML , kernel-janitors@vger.kernel.org, Wolfram Sang References: <38a9b933-4cdc-53e7-0a4c-20f96c57648c@users.sourceforge.net> Message-ID: <464d533c-d950-b524-8945-c19647059cba@users.sourceforge.net> Date: Sat, 20 May 2017 21:12:23 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1 MIME-Version: 1.0 In-Reply-To: <38a9b933-4cdc-53e7-0a4c-20f96c57648c@users.sourceforge.net> Content-Type: text/plain; charset=utf-8 Content-Language: en-GB Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:cW8rMkCUTN3JmXXIBK1k6L3WXeDyoiUyEqR4UvG1HMrTXrBP8CL plGWEI14CXnKS5D3HpcSJAB4UmKlVAGY7XrrVuaE7T9qeka4Sj0DhgKu82rUh76YO1DyJI/ N3Rt9uERPHncyGMMgAL87ZGAPdT0cy34atsgL0MNxrV5pcN+gXe07kCZ+LmKfkxmmYxNgDj W0u6QjaeVtnp+9nXo5qbA== X-UI-Out-Filterresults: notjunk:1;V01:K0:gMgf9xhymAE=:FtSGXt11ngLOq2AzD8nGfQ AI2cI2TZa71O1jGuBoFiYIVU/xagCU6MDuKM4GhmkGfGxO3pPi+GLKkOopwEhNN/YnC+Yolv+ OkEFiTnwxSf7zyR9Ft+ALWtefTu5+x5im5nx9IQ7uJUuVm+pzLaFNb5ie5icda1W0ywy/jB4R F9LeqBsTQ8HJHGLZnHmBeLv5j3Cz2d+4qtuHn2ldhu1jE/k3VnklHncsEuWBh/UhLMK4N+4qi 6QsqzNtOoHoSuUASie/xK25Y+AlMzngioxUaJUko7vLSF9H3tkEo9/+d5RZVY4qE71Di7XswK HEgYkLZhfHLMTP5qac3hh4VBm0aO/KVHveyMnybMpL7o3zICWrP/ArSiqdXxWF5nK+4gAZyyR UnRbglGe3+rQFapNGFRHiJmWlMZMgDUc6cJD3dSyp4Vfzr6JoZvAtLuBRlpXkdQxGoeoqRsyS /A19WgGC1oE42/u2r3JzYGXrrDBB6mwrcORdQOvG8nhm5fNXeQntJazL/n0uGlOowsmNHCQlO NYgM+JhxWJUQ0enREILxifTLsoELNrXjDZw3bIQXGmmnzmj8oW9/gVSRvVvyV42CBmqEUt64g CKRAUg+YpksJZP4hHS3RHl+XE5Akx7claZikuMUunoMiExH/qE8pH2QveND941d4cb/YWTd9b 1v/8VWT3QN3SM3CyFvWie1p3gZ+1IIfUYcBRf9wzL+7JxYXZRPSXNbdbO1I1wynJEFyGew9Q9 A1KofOEFr6N6ufrZv3x+t2CUrF3YbD/+H6QLWylb5t04/95txF+llZGk1Lh14dmtyRtil64Tm kEcBe+z Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Markus Elfring Date: Sat, 20 May 2017 19:57:52 +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 --- drivers/virt/fsl_hypervisor.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/virt/fsl_hypervisor.c b/drivers/virt/fsl_hypervisor.c index d1df6c6740f1..40c5eb64f39b 100644 --- a/drivers/virt/fsl_hypervisor.c +++ b/drivers/virt/fsl_hypervisor.c @@ -663,7 +663,5 @@ static int fsl_hv_open(struct inode *inode, struct file *filp) - if (!dbq) { - pr_err("fsl-hv: out of memory\n"); + if (!dbq) return -ENOMEM; - } spin_lock_init(&dbq->lock); init_waitqueue_head(&dbq->wait); -- 2.13.0