From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932358AbWF2T7N (ORCPT ); Thu, 29 Jun 2006 15:59:13 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932366AbWF2T7M (ORCPT ); Thu, 29 Jun 2006 15:59:12 -0400 Received: from [141.84.69.5] ([141.84.69.5]:56080 "HELO mailout.stusta.mhn.de") by vger.kernel.org with SMTP id S932358AbWF2T7K (ORCPT ); Thu, 29 Jun 2006 15:59:10 -0400 Date: Thu, 29 Jun 2006 21:58:28 +0200 From: Adrian Bunk To: Andrew Morton Cc: linux-kernel@vger.kernel.org Subject: Re: [RFC: 2.6 patch] kernel/sys.c: remove unused exports Message-ID: <20060629195828.GF19712@stusta.de> References: <20060629191940.GL19712@stusta.de> <20060629123608.a2a5c5c0.akpm@osdl.org> <20060629124400.ee22dfbf.akpm@osdl.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20060629124400.ee22dfbf.akpm@osdl.org> User-Agent: Mutt/1.5.11+cvs20060403 Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Jun 29, 2006 at 12:44:00PM -0700, Andrew Morton wrote: > On Thu, 29 Jun 2006 12:36:08 -0700 > Andrew Morton wrote: > > > On Thu, 29 Jun 2006 21:19:40 +0200 > > Adrian Bunk wrote: > > > > > This patch removes the following unused exports: > > > - EXPORT_SYMBOL: > > > - in_egroup_p > > > - EXPORT_SYMBOL_GPL's: > > > - kernel_restart > > > - kernel_halt > > > > Switch 'em to EXPORT_UNUSED_SYMBOL and I'll stop dropping your patches ;) > > > > If doing this, I'd suggest it be done thusly: > > EXPORT_UNUSED_SYMBOL(in_egroup_p); /* June 2006 */ > > to aid later decision-making. I had some bad experiences with following processes you suggest the doesn't remove the symbol immediately: As you wanted me to do, I scheduled the EXPORT_SYMBOL(insert_resource) for removal on 2 May 2005 with both __deprecated_for_modules and an entry in feature-removal-schedule.txt with the target date April 2006. On 11 Apr 2006, I sent the patch to implement this scheduled removal. As of today, the latter patch is still stuck in -mm (which isn't better than having it dropped) although it's long overdue. Do you understand why I distrust your "to aid later decision-making"? Can you state publically "If there's still no in-kernel user after six months, the removal is automatically ACK'ed."? cu Adrian -- "Is there not promise of rain?" Ling Tan asked suddenly out of the darkness. There had been need of rain for many days. "Only a promise," Lao Er said. Pearl S. Buck - Dragon Seed