From: Eric Sandeen Subject: Re: [PATCH] allocate struct ext4_allocation_context from a kmem cache to save stack space Date: Thu, 07 Feb 2008 19:06:49 -0600 Message-ID: <47ABAB29.2060300@redhat.com> References: <47A9E8CA.2070404@redhat.com> <1202429513.3840.12.camel@localhost.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: ext4 development To: cmm@us.ibm.com Return-path: Received: from mx1.redhat.com ([66.187.233.31]:44787 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758516AbYBHBIv (ORCPT ); Thu, 7 Feb 2008 20:08:51 -0500 In-Reply-To: <1202429513.3840.12.camel@localhost.localdomain> Sender: linux-ext4-owner@vger.kernel.org List-ID: Mingming Cao wrote: > Do you intend to remove the #ifdef CONFIG_PROC_FS, or it's a accident? I > think we need keep that to allow ext4 build without procfs configured. > > Other than this, the patch looks fine to me.:) oh, it kind of snuck in. It actually should still build, as remove_proc_entry is a no-op function w/o the config option. Feel free to leave it in if you like though. -Eric