From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965013Ab1GOHHA (ORCPT ); Fri, 15 Jul 2011 03:07:00 -0400 Received: from cantor2.suse.de ([195.135.220.15]:41667 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S964786Ab1GOHHA (ORCPT ); Fri, 15 Jul 2011 03:07:00 -0400 Date: Fri, 15 Jul 2011 08:41:47 +0200 From: Greg KH To: Mike Waychison Cc: Andrew Morton , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , x86@kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/2] Kconfig: Allow disabling of CONFIG_DEVPORT Message-ID: <20110715064147.GA1021@suse.de> References: <1310675693-9204-1-git-send-email-mikew@google.com> <20110714212113.GA28639@suse.de> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Jul 14, 2011 at 03:17:24PM -0700, Mike Waychison wrote: > On Thu, Jul 14, 2011 at 2:21 PM, Greg KH wrote: > > On Thu, Jul 14, 2011 at 01:34:52PM -0700, Mike Waychison wrote: > >> It would be useful to build kernels with disabled support of /dev/port > >> for systems where we do not want to allow an administrator to access IO > >> ports directly. > >> > >> To do so however, CONFIG_DEVPORT needs to be user-selectable.  Give this > >> configuration option a name and help description. > >> > >> Google-Bug-Id: 3177114 > > > > What is that field?  It's not something that I've ever seen before... > > This is for our internal reference. Internal reference to where? > > Please don't push internal fields like this to public patches, it only > > causes confusion... > > It hasn't caused any problems so far. It helps us track things as we > pull code back downstream. Ok, but what happens if all 2000+ different companies start adding their fields to the signed-off-by: area of the kernel for their internal tracking systems? So please, don't do this, it makes no sense to the kernel community at all and we really don't want to start adding more fields to this area if we don't have to. greg k-h