From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joel Becker Subject: Re: [patch] Fix: 'return -ENOMEM' instead of 'return ENOMEM' Date: Thu, 12 Nov 2009 11:17:58 -0800 Message-ID: <20091112191758.GA22985@mail.oracle.com> References: <4AFB2C0B.50605@gmail.com> <20091111134730.a0da9e38.akpm@linux-foundation.org> <20091112081043.GA25345@elte.hu> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: Randy Dunlap , "Stephen M. Cameron" , Mike Christie , David Airlie , James Bottomley , Jens Axboe , Evgeniy Polyakov , iss_storagedev@hp.com, Eric Dumazet , Andy Whitcroft , Dave Airlie , Hannes Eder , dri-devel@lists.sourceforge.net, Alexey Dobriyan , Mike Miller , Mark Fasheh , Karsten Keil , rostedt@goodmis.org, Karen Xie , "James E.J. Bottomley" , Hannes Reinecke , Andreas Ever To: Ingo Molnar Return-path: Content-Disposition: inline In-Reply-To: <20091112081043.GA25345@elte.hu> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: ocfs2-devel-bounces@oss.oracle.com Errors-To: ocfs2-devel-bounces@oss.oracle.com List-Id: netdev.vger.kernel.org On Thu, Nov 12, 2009 at 09:10:43AM +0100, Ingo Molnar wrote: > 5 out of 8 places look buggy - i.e. more than 60% - a checkpatch warning > would avoid real bugs here. (even ignoring the cleanliness effects of > using proper error propagation) > > Cc:-ed affected maintainers. The rightmost column are my observations. > Below is the patch fixing these. Acked-by: Joel Becker -- "Can any of you seriously say the Bill of Rights could get through Congress today? It wouldn't even get out of committee." - F. Lee Bailey Joel Becker Principal Software Developer Oracle E-mail: joel.becker@oracle.com Phone: (650) 506-8127