Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S261383AbVEXHOX (ORCPT ); Tue, 24 May 2005 03:14:23 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S261392AbVEXHOT (ORCPT ); Tue, 24 May 2005 03:14:19 -0400 Received: from wproxy.gmail.com ([64.233.184.193]:33354 "EHLO wproxy.gmail.com") by vger.kernel.org with ESMTP id S261383AbVEXHON convert rfc822-to-8bit (ORCPT ); Tue, 24 May 2005 03:14:13 -0400 DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=beta; d=gmail.com; h=received:message-id:date:from:reply-to:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=Y+AtHMac8HQzy5TaUfp4m0PBpzm0oAMdJiagefK39WeGGEUJ5tsNmxB8/E8QQ7ogyF+wwNow0lFWPQSmPezl80gGMfsCCrapBjh+Y3kxnJgLm65p+UG0vlwELW1k6BUnnasteKf0+Tw6+L+5o+csx3BfGv0UYJXZvu2OxvCB+hw= Message-ID: <84144f0205052400143e97796e@mail.gmail.com> Date: Tue, 24 May 2005 10:14:12 +0300 From: Pekka Enberg Reply-To: Pekka Enberg To: "ericvh@gmail.com" Subject: Re: [RFC][patch 2/7] v9fs: VFS file and directory operations (2.0-rc6) Cc: linux-kernel@vger.kernel.org, v9fs-developer@lists.sourceforge.net, viro@parcelfarce.linux.theplanet.co.uk, linux-fsdevel@vger.kernel.org, penberg@cs.helsinki.fi In-Reply-To: <200505232225.j4NMPXe1029024@ms-smtp-02-eri0.texas.rr.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7BIT Content-Disposition: inline References: <200505232225.j4NMPXe1029024@ms-smtp-02-eri0.texas.rr.com> Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 787 Lines: 26 Hi, On 5/24/05, ericvh@gmail.com wrote: > +static ssize_t > +v9fs_file_write(struct file *filp, const char __user * data, > + size_t count, loff_t * offset) > +{ > + int ret = -1; > + char *buffer; > + > + buffer = kmalloc(count, GFP_KERNEL); > + if (buffer == NULL) { > + BUG(); I think simply returning -ENOMEM is sufficient. BUG seems way too aggressive. (Found this in other places as well.) > + return -ENOMEM; > + } Pekka - 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/