Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756881Ab1ERKPU (ORCPT ); Wed, 18 May 2011 06:15:20 -0400 Received: from service87.mimecast.com ([94.185.240.25]:51481 "HELO service87.mimecast.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1755620Ab1ERKPT convert rfc822-to-8bit (ORCPT ); Wed, 18 May 2011 06:15:19 -0400 Subject: Re: [PATCH 2/2] trivial: Changed the printk loglevel when not able to allocate memory From: Catalin Marinas To: anish singh Cc: linux@arm.linux.org.uk, lethal@linux-sh.org, linux-fbdev@vger.kernel.org, linux-kernel@vger.kernel.org In-Reply-To: References: <1305653342.2182.15.camel@anish-desktop> Organization: ARM Limited Date: Wed, 18 May 2011 11:15:10 +0100 Message-ID: <1305713710.11468.39.camel@e102109-lin.cambridge.arm.com> Mime-Version: 1.0 X-Mailer: Evolution 2.28.1 X-OriginalArrivalTime: 18 May 2011 10:15:15.0443 (UTC) FILETIME=[7B1E1430:01CC1544] X-MC-Unique: 111051811151504201 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1338 Lines: 34 On Wed, 2011-05-18 at 04:19 +0100, anish singh wrote: > On Tue, May 17, 2011 at 11:13 PM, Catalin Marinas wrote: > On 17 May 2011 18:29, anish wrote: > > When not able to allocate memory we were using KERN_INFO as > > log level in printk so changed to KERN_ERR > > Signed-off-by: anish kumar > > > Maybe KERN_WARN? > Then shouldn't we change below case also? > 467 ret = amba_request_regions(dev, NULL); > 468 if (ret) { > 469 printk(KERN_ERR "CLCD: unable to reserve regs region\n"); > 470 goto out; > > If yes,then i will resend the patch for this also. I think the register reserving is less likely to fail because of memory allocations but more because of overlapping regions, in which case it could be a programming error. Allocating a big framebuffer is likely to fail in some memory constraint systems but I don't consider this a kernel error. That's why I suggested warning. -- Catalin -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/