Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1763479AbZANPgz (ORCPT ); Wed, 14 Jan 2009 10:36:55 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1755185AbZANPgn (ORCPT ); Wed, 14 Jan 2009 10:36:43 -0500 Received: from pfepa.post.tele.dk ([195.41.46.235]:59683 "EHLO pfepa.post.tele.dk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753099AbZANPgm (ORCPT ); Wed, 14 Jan 2009 10:36:42 -0500 Date: Wed, 14 Jan 2009 16:38:25 +0100 From: Sam Ravnborg To: Jaswinder Singh Rajput Cc: Geert Uytterhoeven , Jaswinder Singh Rajput , Ingo Molnar , Alan Cox , Andrew Morton , David Miller , x86 maintainers , LKML , netdev Subject: Re: [PULL -tip] fixed few make headers_check warnings Message-ID: <20090114153825.GB32082@uranus.ravnborg.org> References: <1231836016.3212.41.camel@jaswinder.satnam> <3f9a31f40901140129i1389f47frf32808f3a695a118@mail.gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <3f9a31f40901140129i1389f47frf32808f3a695a118@mail.gmail.com> User-Agent: Mutt/1.4.2.1i Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2128 Lines: 55 On Wed, Jan 14, 2009 at 02:59:54PM +0530, Jaswinder Singh Rajput wrote: > Hello Geert, > > On Wed, Jan 14, 2009 at 2:26 PM, Geert Uytterhoeven > wrote: > > On Tue, 13 Jan 2009, Jaswinder Singh Rajput wrote: > >> Please pull these patches. Earlier I also floated these patches so that I can get some feedback. > >> > >> The following changes since commit ae2659bcd0b5bf34dd4b03f1db0a2942cc09b8e6: > >> Ingo Molnar (1): > >> Merge branch 'x86/mm' > >> > >> are available in the git repository at: > >> > >> git://git.kernel.org/pub/scm/linux/kernel/git/jaswinder/linux-2.6-tiptop.git master > >> > >> Jaswinder Singh Rajput (7): > >> include of is preferred over > >> found __[us]{8,16,32,64} type without #include > >> capability.h: extern's make no sense in userspace > >> coda_psdev.h: extern's make no sense in userspace > >> in6.h: extern's make no sense in userspace > >> nubus.h: extern's make no sense in userspace > >> socket.h: extern's make no sense in userspace > > > > May I please remind you of my comment in > > http://patchwork.kernel.org/patch/2093/? > > > > Thanks for your reminder :-) > > > At least nubus.h still leaks forward declarations for kernel functions that > > lack the "extern" keyword, so they still pollute user space. > > > > Yes, you are right, I just tried to pampered weeping warning messages > which is annoying. Hi Jaswinder. I appreciate your work but I will like to question the approach. We should rather take the warnings as an indication that this file needs to be looked over and fix not only the warnings reported but rater to fix all the questionable issues on a file-by-file basis. If the patch gets involved then it should be split in two. Could I ask you to at least consider this approach. Thanks, Sam -- 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/