Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751853AbdHSSAo (ORCPT ); Sat, 19 Aug 2017 14:00:44 -0400 Received: from mout.web.de ([212.227.15.4]:64315 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751027AbdHSSAm (ORCPT ); Sat, 19 Aug 2017 14:00:42 -0400 Subject: Re: ext4: Delete an error message for a failed memory allocation in ext4_multi_mount_protect() To: Eric Sandeen , linux-ext4@vger.kernel.org Cc: Andreas Dilger , "Theodore Ts'o" , LKML , kernel-janitors@vger.kernel.org References: <5bf432ae-398d-718c-8105-3092a240a8ef@users.sourceforge.net> From: SF Markus Elfring Message-ID: Date: Sat, 19 Aug 2017 20:00:31 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-GB Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:peXDWtX9zJnnRw32w2U65/KlCglilRfXUCXXZf0CeiDr4TcuMaF /5KkDXa1jM+lTBokKheVFQmeQK4DaII7hehAay5MPJrS8QMtD7PLKqh8XsanQdJECROeXh2 2rmakKi/mCbzzgcFu9Q4MJ1SloSAJSav056BcaiiuaC4Hf/x1WFp+t8u34U3Mi4bZQnwWOh xAQe6zCOPUhwNubXnFgdg== X-UI-Out-Filterresults: notjunk:1;V01:K0:LA9D/AWmk0s=:DypB2RIIyy73+vyn3KZS/Z TnKnnsjqjzrBBo+SZlld8mOo3F6BNpPQCIKcmBJd8+jg+YZORd0d4IKBCzwTxl9WfmxYMlph0 u2KsA7REKTiRa1ajnf4Lr+2SQaQ39rMHAjdJpnKucotKbmhSlkY6reL64mXq4a00PrLmyK93t mmEy6DbZ+WPCYljANWuArHOgxFtrzdXtQf3/lk9PZZMMsYFIgWpOdMsTj1/v//4GXNVO3Qbwc WSpj9C+tGtI3uEytTAr4aKPjuy1UblYRH424dkbwWOW1UeBcOvLh+m6xm802WfmKjMzoqq2UR LxFHK88vTjm52DLF4dpVMPvkv8eTn2x5tPtAMmOkqhPR/JvWK68OA+YfY17RNnj23b8Sqjx9T 86y213boSXFcGpTT3+4vXqqMy6F0stRkDB/8lUk5IPN1pOhRZAS+NTwErShBl2j7YKxF8z6/t NvxxdZwIOlkYMr+to6L7DBGCBblhgu+pqhJCH2V2/nzY7QPxeqvwC2jyfoDtVm9Y0HD5t6cz8 vh+rpDj5J7jjajjtpt3dThCYqTAh6riHLlV6x4w33Pu9AhFTnGowNtOudL0i7lBO50Nu4Y3mY IqrTkW3PvLiLZ7AR67ucGp+r7BYR+3ycRxHC/A128hP9ZDMbgMDcoos8c61914op7gaPLfcwQ viQs5SRvDvdXgBpCKJoyPIDsb13lSKmre3O4dhPdniE1N/EshSgsAD0PlyeXIelZxu4Hcn9yq JTkKLeSSTAjh1NTwgsRQDzqqkWK9psky0N1FKX2x1YQ6jiW/s9fvuiGnoVU2o6ObtvXxt/0Ye A7Sfcdyc47kuYPQEyzRb2YjeXAakxFbis+zhVzJMxMyWC4SZ18= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 515 Lines: 17 >> Omit an extra message for a memory allocation failure in this function. > > I might be dense, but what makes this message "extra?" > > (I suppose kmalloc squawks too if it fails, Do you find the default allocation failure report sufficient? > but is Coccinelle now warning about explicit memory allocation failure warnings?) This software tool can help to find source code places for further development considerations. Would you like to clarify a corresponding search pattern a bit more? Regards, Markus